[PATCH] D82342: Preserve GlobalsAA analysis result in LowerConstantIntrinsics

[Florian Hahn via Phabricator via llvm-commits]

fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.

LGTM. LowerConstantIntrinsics should not modify globals and can preserve GlobalsAA.  This is also in line with what we currently do in many other passes and not preserving GlobalsAA can lead to serious performance regressions.

I can commit this patch for you in a day or so, to give other people a chance to chime in as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82342