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

[Philip Pfaffe via Phabricator via llvm-commits]

philip.pfaffe accepted this revision.
philip.pfaffe added a comment.
Herald added a subscriber: jdoerfert.

Tiny final nit, otherwise looks great!



================
Comment at: clang/lib/CodeGen/BackendUtil.cpp:922
 
+void EmitAssemblyHelper::addSanitizersAtO0(ModulePassManager &MPM) {
+  if (LangOpts.Sanitize.has(SanitizerKind::Address)) {
----------------
Could this be a free function?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56470