eugenis accepted this revision. eugenis added a comment. This revision is now accepted and ready to land.
LGTM with 1 comment ================ Comment at: llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp:3573 + case LibFunc_atomic_load: + if (!isa<CallInst>(CB)) { + llvm::errs() << "MSAN -- cannot instrument invoke of libatomic load." ---------------- Probably need the same check for atomic_store. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85559/new/ https://reviews.llvm.org/D85559 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits