[llvm] [LoopPeel] LCSSA form is destroyed by LoopPeel, preserve it (PR #78696)

Vedant Paranjape via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 19 21:31:38 PST 2024


vedantparanjape-amd wrote:

> > > > Now we are unaware if such a transformation is expected of simplifyLoopIV and if it is also expected to maintain LCSSA.
> > > 
> > > 
> > > simplifyLoopIV is supposed to maintain LCSSA form.
> > 
> > 
> > So, should we update the simplifyLoopIV or just call formLCSSAform after simplifyLoopIV ?
> 
> We should fix simplifyLoopIV. Note that it already contains various replacementPreservesLCSSAForm checks already.

I think you mean to say simplifyLoopAfterUnroll has various replacementPreservesLCSSAForm checks, not the SimplifyLoopIV.

https://github.com/llvm/llvm-project/pull/78696


More information about the llvm-commits mailing list