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

Philip Pfaffe via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 10 12:22:45 PDT 2019


philip.pfaffe accepted this revision.
philip.pfaffe added a comment.
This revision is now accepted and ready to land.

Nit aside, looks good!



================
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);
----------------
There are some naming clashes here. In the other sanitizers these functions are called `sanitizeFunction` and `initializeModule`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61709





More information about the cfe-commits mailing list