Skip to content

Commit 2385111

Browse files
committed
Revert "Fix false positive on icmp ptr with samesign flag (AliveToolkit#1151)"
This reverts commit da0d731.
1 parent 38769b8 commit 2385111

File tree

2 files changed

+1
-47
lines changed

2 files changed

+1
-47
lines changed

ir/instr.cpp

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -2771,19 +2771,7 @@ StateValue ICmp::toSMT(State &s) const {
27712771
auto scalar = [&](const StateValue &a, const StateValue &b) -> StateValue {
27722772
auto fn2 = [&](Cond c) { return fn(a.value, b.value, c); };
27732773
auto v = cond != Any ? fn2(cond) : build_icmp_chain(cond_var(), fn2);
2774-
expr np = true;
2775-
if (flags & SameSign) {
2776-
if (isPtrCmp()) {
2777-
auto &m = s.getMemory();
2778-
Pointer lhs(m, a.value);
2779-
Pointer rhs(m, b.value);
2780-
m.observesAddr(lhs);
2781-
m.observesAddr(rhs);
2782-
np = lhs.getAddress().sign() == rhs.getAddress().sign();
2783-
} else {
2784-
np = a.value.sign() == b.value.sign();
2785-
}
2786-
}
2774+
auto np = flags & SameSign ? a.value.sign() == b.value.sign() : true;
27872775
return { v.toBVBool(), a.non_poison && b.non_poison && np };
27882776
};
27892777

tests/alive-tv/samesign-ptr.srctgt.ll

Lines changed: 0 additions & 34 deletions
This file was deleted.

0 commit comments

Comments
 (0)