================ @@ -1042,8 +1042,8 @@ Instruction *InstCombinerImpl::visitLoadInst(LoadInst &LI) { } // None of the following transforms are legal for volatile/ordered atomic - // loads. Most of them do apply for unordered atomics. - if (!LI.isUnordered()) return nullptr; + // loads and sanitizers. Most of them do apply for unordered atomics. + if (mustSuppressSpeculation(LI)) return nullptr; ---------------- nikic wrote:
Can we move this check to where it is relevant, i.e. the select fold in particular? Also, should this check be inside isSafeToLoadUnconditionally maybe? https://github.com/llvm/llvm-project/pull/100773 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits