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

[Fedor Sergeev via Phabricator via llvm-commits]

fedor.sergeev added a comment.

In https://reviews.llvm.org/D54337#1294096, @philip.pfaffe wrote:

> Making this a module pass will cost us all the nice chaining and cache locality we get from a function pass. 
>  That will make the already slow instrumentation even more expensive. Do you see a way to keep this as a function pass?


I assume the main problem with this being function pass in NewPM  is the need for initialization, which is too slow when being done per-function.

There are ways around this issue, perhaps the most logical one would be to implement analysis for this GlobalMetadata thing...


Repository:
  rL LLVM

https://reviews.llvm.org/D54337