Skip to content

Commit 395093e

Browse files
authored
[msan] Switch to -msan-handle-icmp-exact my default (#113200)
Fixes #111212. This grows .text by 5.3% on CTMark, (or 2.6% large internal binary) Perf regressed by 1.6%. We will try to improve in follow up patches. It worth to pay some performance regression to fix correctness to avoid stuff like #111212.
1 parent 54cf62d commit 395093e

File tree

3 files changed

+513
-43
lines changed

3 files changed

+513
-43
lines changed

llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,7 @@ static cl::opt<bool>
276276
static cl::opt<bool>
277277
ClHandleICmpExact("msan-handle-icmp-exact",
278278
cl::desc("exact handling of relational integer ICmp"),
279-
cl::Hidden, cl::init(false));
279+
cl::Hidden, cl::init(true));
280280

281281
static cl::opt<bool> ClHandleLifetimeIntrinsics(
282282
"msan-handle-lifetime-intrinsics",

0 commit comments

Comments
 (0)