[PATCH] D120976: [AMDGPU] Add llvm.amdgcn.s.setprio intrinsic

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/test/CodeGen/AMDGPU/llvm.amdgcn.s.setprio.ll:19
+; GCN-NEXT:    s_setprio 0xffff
+; GCN-NEXT:    s_setprio 0x10000
+; GCN-NEXT:    s_setprio -1
----------------
kerbowa wrote:
> arsenm wrote:
> > This is printing an unencodable value, should have sign extended or something
> Do you mean the test should have a sign extend or that it should not select with an unencodable value?
Ideally the frontend would diagnose, but I would expect discarding the extra bits on select


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120976