[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