Re: [Mesa-dev] [PATCH 1/3] ac/nir: only use the new raw/struct image atomic intrinsics with LLVM 9+

On 4/22/19 7:19 PM, Marek Olšák wrote: Is "XXX" in the comment really necessary? No, but I already pushed the series. Marek On Thu, Apr 18, 2019 at 3:20 AM Samuel Pitoiset mailto:samuel.pitoi...@gmail.com>> wrote: They are buggy with LLVM 8 because they weren't marked as source

Re: [Mesa-dev] [PATCH 1/3] ac/nir: only use the new raw/struct image atomic intrinsics with LLVM 9+

Is "XXX" in the comment really necessary? Marek On Thu, Apr 18, 2019 at 3:20 AM Samuel Pitoiset wrote: > They are buggy with LLVM 8 because they weren't marked as source > of divergence, see r358579. > > Fixes: dd0172e865f ("radv: Use structured intrinsics instead of indexing > workaround for

[Mesa-dev] [PATCH 1/3] ac/nir: only use the new raw/struct image atomic intrinsics with LLVM 9+

They are buggy with LLVM 8 because they weren't marked as source of divergence, see r358579. Fixes: dd0172e865f ("radv: Use structured intrinsics instead of indexing workaround for GFX9.")" Signed-off-by: Samuel Pitoiset --- src/amd/common/ac_nir_to_llvm.c | 5 - 1 file changed, 4