[PATCH] D89774: [NPM] port -unify-loop-exits to NPM

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 20 09:34:10 PDT 2020


aeubanks added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/UnifyLoopExits.cpp:241
+    return PreservedAnalyses::all();
+  return getLoopPassPreservedAnalyses();
+}
----------------
the legacy pass doesn't call `getLoopAnalysisUsage(AU)`, are you sure that this preserves all loop analyses? Or is it just `LoopAnalysis`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89774



More information about the llvm-commits mailing list