melver added inline comments.

================
Comment at: llvm/include/llvm/IR/Attributes.td:90
+/// Do not instrument function with sanitizers.
+def DisableSanitizerInstrumentation: 
EnumAttr<"disable_sanitizer_instrumentation", [FnAttr]>;
+
----------------
There's this long-tail of changes required for adding new keywords to the IR. 
Have a look at https://reviews.llvm.org/D102772 -- things that I see currently 
missing are various tests etc.


================
Comment at: llvm/lib/Bitcode/Writer/BitcodeWriter.cpp:631
     return bitc::ATTR_KIND_COLD;
+  case Attribute::DisableSanitizerInstrumentation:
+    return bitc::ATTR_DISABLE_SANITIZER_INSTRUMENTATION;
----------------
There's also BitcodeReader, which needs something similar.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108029

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

Reply via email to