Skip to content

Commit 2da9723

Browse files
committed
Remove redundant if(enable_pointer_check)
goto_checkt::pointer_rel_check returns early when enable_pointer_check is false, there is no need to test this again.
1 parent 751c986 commit 2da9723

File tree

1 file changed

+8
-11
lines changed

1 file changed

+8
-11
lines changed

src/analyses/goto_check.cpp

Lines changed: 8 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1128,18 +1128,15 @@ void goto_checkt::pointer_rel_check(
11281128
{
11291129
// add same-object subgoal
11301130

1131-
if(enable_pointer_check)
1132-
{
1133-
exprt same_object=::same_object(expr.op0(), expr.op1());
1131+
exprt same_object = ::same_object(expr.op0(), expr.op1());
11341132

1135-
add_guarded_property(
1136-
same_object,
1137-
"same object violation",
1138-
"pointer",
1139-
expr.find_source_location(),
1140-
expr,
1141-
guard);
1142-
}
1133+
add_guarded_property(
1134+
same_object,
1135+
"same object violation",
1136+
"pointer",
1137+
expr.find_source_location(),
1138+
expr,
1139+
guard);
11431140
}
11441141
}
11451142

0 commit comments

Comments
 (0)