nickdesaulniers added inline comments.

================
Comment at: clang/lib/CodeGen/SanitizerMetadata.cpp:105-107
 void SanitizerMetadata::disableSanitizerForInstruction(llvm::Instruction *I) {
-  I->setMetadata(llvm::LLVMContext::MD_nosanitize,
-                 llvm::MDNode::get(CGM.getLLVMContext(), std::nullopt));
+  I->setNoSanitizeMetadata();
 }
----------------
How about deleting this definition and updating the two call sites in:
1. clang/lib/CodeGen/ItaniumCXXABI.cpp
2. clang/lib/CodeGen/CodeGenFunction.cpp


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D150632/new/

https://reviews.llvm.org/D150632

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to