[PATCH] D61709: [NewPM] Port HWASan and Kernel HWASan

Leonard Chan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 14 14:15:19 PDT 2019


leonardchan added inline comments.


================
Comment at: llvm/lib/Transforms/Instrumentation/HWAddressSanitizer.cpp:179
 
-  bool runOnFunction(Function &F) override;
-  bool doInitialization(Module &M) override;
+  bool instrumentFunction(Function &F);
+  void initializeWithModule(Module &M);
----------------
philip.pfaffe wrote:
> There are some naming clashes here. In the other sanitizers these functions are called `sanitizeFunction` and `initializeModule`.
Renamed


Repository:
  rC Clang

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

https://reviews.llvm.org/D61709





More information about the llvm-commits mailing list