vitalybuka added inline comments.
================ Comment at: llvm/lib/Transforms/Instrumentation/SanitizerCoverage.cpp:922 + auto Store = IRB.CreateStore(ConstantInt::getTrue(Int1Ty), FlagPtr); + Store->setAtomic(AtomicOrdering::NotAtomic); + Store->setAlignment(llvm::MaybeAlign(FunctionBoolArray->getAlign())); ---------------- Store->setAtomic(AtomicOrdering::NotAtomic); should be defaul ================ Comment at: llvm/lib/Transforms/Instrumentation/SanitizerCoverage.cpp:923 + Store->setAtomic(AtomicOrdering::NotAtomic); + Store->setAlignment(llvm::MaybeAlign(FunctionBoolArray->getAlign())); + SetNoSanitizeMetadata(Store); ---------------- isn't alignment is always 1 which is already default? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77244/new/ https://reviews.llvm.org/D77244 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
