[PATCH] D76153: [SimplifyCFG] try branch-to-branch simplification sooner

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 16 12:35:05 PDT 2020


efriedma added a comment.

> But then again, we run this pass multiple times in the normal opt pipeline, so we must be layering cost-based speculations on top of one another...unless I'm missing something that prevents that.

The SimplifyCFG pass (lib/Transforms/Scalar/SimplifyCFGPass.cpp) iterates until it can't make any more changes; we shouldn't be depending on any data recorded off to the side.  I'm pretty sure the issue is really some sort of transform ordering issue; I'm mostly interested in knowing which specific transforms are involved.


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

https://reviews.llvm.org/D76153





More information about the llvm-commits mailing list