[PATCH] D92887: [LoopUnroll] Introduce llvm.noalias.decl and use if for duplicating noalias metadata as needed
    Jeroen Dobbelaere via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Dec  9 09:34:11 PST 2020
    
    
  
jeroen.dobbelaere added a comment.
In D92887#2442047 <https://reviews.llvm.org/D92887#2442047>, @nikic wrote:
> I'd suggest to split this up into (at least) three parts:
>
> - LangRef patch (not present yet here) and the addition of the intrinsic.
> - Teaching everything relevant to look past the intrinsic and emitting it in inliner.
> - Making use of it in loop unrolling.
>
> This will make it easier to identify regressions caused by the extra intrinsic blocking some transform, because of missing special handling or similar.
Yes, that makes sense. I hope to have an update tomorrow.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D92887/new/
https://reviews.llvm.org/D92887
    
    
More information about the llvm-commits
mailing list