[PATCH] D120976: [AMDGPU] Add llvm.amdgcn.s.setprio intrinsic
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 7 09:27:41 PST 2022
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
More information about the llvm-commits
mailing list