[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