[PATCH] D19431: [LoopDist] Add llvm.loop.distribute.enable loop metadata

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 26 15:17:50 PDT 2016


hfinkel added a comment.

In http://reviews.llvm.org/D19431#412892, @anemet wrote:

> In http://reviews.llvm.org/D19431#412884, @hfinkel wrote:
>
> > Isn't DistributeSCEVCheckThreshold just such a limit?
>
>
> Ah good point, I was overly focused on memchecks ;).   Let me update the patch.
>
> Are you OK with the overall reasoning that we should have dedicate loop distribution pragma/metadata?


Yes, I think this makes sense.

> Thanks for review.

> 

> Adam





http://reviews.llvm.org/D19431





More information about the llvm-commits mailing list