[PATCH] D16941: [NVPTX] Mark nvvm synchronizing intrinsics as convergent.

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 5 18:25:43 PST 2016


hfinkel added a comment.

In http://reviews.llvm.org/D16941#345531, @hfinkel wrote:

> In http://reviews.llvm.org/D16941#345516, @jingyue wrote:
>
> >
>


...

> While you're there ;) - you might look at making our handling of noduplicate more consistent as well. I just noticed, for example, that loop unswitching checks for convergent but not for noduplicate.


Per your comment, ignore this ;) -- I missed the Metrics.notDuplicatable check.


http://reviews.llvm.org/D16941





More information about the llvm-commits mailing list