[PATCH] D90104: [LoopUnroll] Duplicate noalias metadata

[Jeroen Dobbelaere via Phabricator via llvm-commits]

jeroen.dobbelaere added a comment.

It would make sense to add comments in the code to explain _why_ this is needed and why it is safe to do.
note: the transformation should be correct, as the unrolled version might now 'alias' with usages of the original scopes that are outside the loop.

- aka, this will be a pessimization for some use cases (those where the cloning should not be needed)
- for reference, D68510 <https://reviews.llvm.org/D68510> is the patch in the 'full restrict patchset' (D68484 <https://reviews.llvm.org/D68484>) that adds similar, but more complete suport.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D90104/new/

https://reviews.llvm.org/D90104