[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