[PATCH] D151241: [AMDGPUCodegenPrepare] Add NewPM Support

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 24 11:12:55 PDT 2023


aeubanks added a comment.

In D151241#4369287 <https://reviews.llvm.org/D151241#4369287>, @gandhi21299 wrote:

> Preserve TargetLibraryAnalysis and AssumptionAnalysis in all cases. If no changes are made by the pass, preserve all analyses. Otherwise preserve the two only.

there is no need to preserve those two, they are always preserved (see the `invalidate` method) even if you don't preserve all analyses


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151241



More information about the llvm-commits mailing list