[PATCH] D142161: [LegacyDivergenceAnalysis] Add NewPM support
Anshil Gandhi via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 19 21:24:42 PST 2023
gandhi21299 added inline comments.
================
Comment at: llvm/lib/Analysis/LegacyDivergenceAnalysis.cpp:421
+ auto *TTI = AM.getCachedResult<TargetIRAnalysis>(F);
+ if (!TTI || !TTI->hasBranchDivergence())
+ return PreservedAnalyses::all();
----------------
arsenm wrote:
> No TTI probably shouldn’t be allowed
L421 already checks for that, doesn't it?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142161/new/
https://reviews.llvm.org/D142161
More information about the llvm-commits
mailing list