[PATCH] D89854: [GVN LoadPRE] Add an option to disable splitting backedge

Serguei Katkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 21 21:21:11 PDT 2020


skatkov added a comment.

In D89854#2344384 <https://reviews.llvm.org/D89854#2344384>, @fhahn wrote:

> This seems reasonable. Do you plan to use this option in the default pipeline/do you think there are any benefits for the regular pipeline?

Hi Florian, to be honest I have no idea of the impact on default pipeline. I know that GVN Load PRE negatively impacts peeling, loop predication, so the passes expecting that latch has a conditional branch.
So I do not plan to update the default pipeline but if there is a volunteer to play with default pipeline, all tools will be there.


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

https://reviews.llvm.org/D89854



More information about the llvm-commits mailing list