[PATCH] D52739: [PassManager/Sanitizer] Port of AddresSanitizer pass from legacy to new PassManager

[Leonard Chan via Phabricator via llvm-commits]

leonardchan added inline comments.


================
Comment at: include/llvm/IR/AddressSanitizerPass.h:23
+
+class AddressSanitizerWrapper;
+class AddressSanitizerModuleWrapper;
----------------
fedor.sergeev wrote:
> I dont believe you need any extra wrappers in addition to AddressSanitizer*Pass ones.
> 
> Just use class AddressSanitizer/AddresSanitizerModule here.
Done, but this would require moving `AddressSanitizer`, `AddressSanitizerModule`, and `FunctionStackPoisoner` outside of their anonymous namespace.


Repository:
  rL LLVM

https://reviews.llvm.org/D52739