[PATCH] D56470: [NewPM] Second attempt at porting ASan

[Leonard Chan via Phabricator via llvm-commits]

leonardchan added inline comments.


================
Comment at: clang/lib/CodeGen/BackendUtil.cpp:922
 
+void EmitAssemblyHelper::addSanitizersAtO0(ModulePassManager &MPM) {
+  if (LangOpts.Sanitize.has(SanitizerKind::Address)) {
----------------
philip.pfaffe wrote:
> Could this be a free function?
Yes, although the sanitizer depends on a few things provided by the `EmitAssemblyHelper`, so those will have to be passed as arguments.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56470