[PATCH] D54337: [ASan] Make AddressSanitizer a ModulePass

[Philip Pfaffe via Phabricator via llvm-commits]

philip.pfaffe added a comment.

It depends on what you mean by task. There is no clean way for depending on //changes// between IR units. I.e. you can't express the requirement that two passes have to run in a specific order. But that's okay, implementing such requirements would incure insane complexity. Fortunately, if I recall correctly Asan oesn't need that! Correct me if I'm wrong.

But depending on //information// about other IR units is possible, that's why we have analyses. For asan this means: leave asan a function pass, but pull things pre-computed for the module into a module level analysis.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D54337