[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