[PATCH] D158843: [ConstraintElim] Move just before loop simplification pipeline.
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 22 06:35:18 PDT 2023
fhahn added a comment.
In D158843#4649884 <https://reviews.llvm.org/D158843#4649884>, @nikic wrote:
> LGTM
>
> Could you please add a few words to the patch description on how this improves optimization of the phase ordering test? It's not super obvious looking at just the diff.
Thanks! I extended the description, it's not obvious and a side effect of SimplifyCFG adjusting the CFG so it allows us to add additional conditions. This seems to lead to a few improvements overall which is nice, but the main benefit is to slightly reduce the compile-time when using SCEV in the pass for D152730 <https://reviews.llvm.org/D152730>
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D158843/new/
https://reviews.llvm.org/D158843
More information about the llvm-commits
mailing list