fixup! Simplify byte_extract(byte_update(...)) without overlap

This fixes a bits/bytes confusing, resulting in missed optimisations of
byte_extract(byte_update(...)) nestings when the byte_extract was at
an offset higher than the update. Found while debugging
model-checking/kani#1958.

src/util/simplify_expr.cpp

@@ -1727,7 +1727,8 @@ simplify_exprt::simplify_byte_extract(const byte_extract_exprt &expr)
         const auto update_size = pointer_offset_bits(bu.value().type(), ns);
         if(
           update_size.has_value() &&
-          *offset >= *update_offset * bu.get_bits_per_byte() + *update_size)
+          *offset * expr.get_bits_per_byte() >=
+            *update_offset * bu.get_bits_per_byte() + *update_size)
         {
           auto tmp = expr;
           tmp.op() = bu.op();