[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