[PATCH] D57748: AMDGPU: Add inverse ballot intrinsic

Connor Abbott via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 5 06:16:39 PST 2019


cwabbott added a comment.

In D57748#1385046 <https://reviews.llvm.org/D57748#1385046>, @arsenm wrote:

> The user can't actually guarantee the argument is uniform, since transforms are allowed to touch the argument. We're accumulating quite a few places that assume this though


Yeah, that's true. We're already currently disabling some problematic transforms in Mesa because of that. We can probably get a proper solution for that when we add thread group semantics to the LangRef (c.f. the llvm-dev thread), but we shouldn't need to wait on that to get this in.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57748





More information about the llvm-commits mailing list