[PATCH] D58964: [AMDGPU] Allow MIMG with no uses in adjustWritemask in isel

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: test/CodeGen/AMDGPU/img-nouse-adjust.ll:14-15
+  %0 = tail call <4 x float> @llvm.amdgcn.image.load.2d.v4f32.i32(i32 15, i32 undef, i32 undef, <8 x i32> undef, i32 0, i32 0) #3
+  call void @llvm.trap()
+  unreachable
+}
----------------
dstuttard wrote:
> arsenm wrote:
> > The trap should be removed and the unreachable replaced with return
> The problem doesn't happen if you remove the trap.
A fake side effect asm call would probably work and be less likely to break for unrelated reasons


Repository:
  rL LLVM

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

https://reviews.llvm.org/D58964