Simplifier: do not create extractbits with pointer or floatbv type

tautschnig · tautschnig · commit a7e7d838ef3c · 2025-07-23T21:41:36.000Z
This is a fix-up for 525c23d: we must not remove typecasts towards pointers (or floatbv) when the operand is extractbits as our back-ends do not support extractbits with pointer type and casts towards floatbv are non-trivial.
diff --git a/src/util/simplify_expr.cpp b/src/util/simplify_expr.cpp
@@ -1342,6 +1342,7 @@ simplify_exprt::simplify_typecast(const typecast_exprt &expr)
   else if(auto extractbits = expr_try_dynamic_cast<extractbits_exprt>(operand))
   {
     if(
+      expr_type_id != ID_floatbv && expr_type_id != ID_pointer &&
       can_cast_type<bitvector_typet>(expr_type) &&
       can_cast_type<bitvector_typet>(operand.type()) &&
       to_bitvector_type(expr_type).get_width() ==

Original file line number	Diff line number	Diff line change
`@@ -1342,6 +1342,7 @@ simplify_exprt::simplify_typecast(const typecast_exprt &expr)`
`1342`	`1342`	`else if(auto extractbits = expr_try_dynamic_cast<extractbits_exprt>(operand))`
`1343`	`1343`	`{`
`1344`	`1344`	`if(`
	`1345`	`+ expr_type_id != ID_floatbv && expr_type_id != ID_pointer &&`
`1345`	`1346`	`can_cast_type<bitvector_typet>(expr_type) &&`
`1346`	`1347`	`can_cast_type<bitvector_typet>(operand.type()) &&`
`1347`	`1348`	`to_bitvector_type(expr_type).get_width() ==`