fmayer added inline comments.
================
Comment at: llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp:1499-1502
+ if (ClUseStackSafety && findAllocaForValue(Ptr)) {
+ if (SSI && SSI->stackAccessIsSafe(*Inst)) {
+ return true;
+ }
----------------
Doesn't need nested if.
I would order this as ClUseStackSafety && SSI && findAllocaForValue(Ptr) && ...
(cheapest first).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D112098/new/
https://reviews.llvm.org/D112098
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
