[PATCH] D126455: [FuncSpec] Make the Function Specializer part of the IPSCCP pass.

Alexandros Lamprineas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 2 10:48:51 PDT 2022


labrinea marked 7 inline comments as done.
labrinea added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/SCCP.cpp:50
   PreservedAnalyses PA;
   PA.preserve<DominatorTreeAnalysis>();
   PA.preserve<PostDominatorTreeAnalysis>();
----------------
chill wrote:
> Now that we added `LoopAnalysis` we may well preserve it too. (I should have included it in the patch which introduced the `LoopAnalysis` here)
I tried this but the compiler crashes. Probably because SCCP deletes dead basic blocks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126455



More information about the llvm-commits mailing list