[PATCH] D138216: [AMDGPU] Intrinsic to expose s_wait_event for export ready

David Stuttard via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 21 01:34:37 PST 2022


dstuttard marked 2 inline comments as done.
dstuttard added inline comments.


================
Comment at: llvm/include/llvm/IR/IntrinsicsAMDGPU.td:2071
+def int_amdgcn_wait_event_export_ready :
+  Intrinsic<[], [], [IntrNoMem, IntrHasSideEffects, IntrWillReturn]
+>;
----------------
arsenm wrote:
> Why is the immediate missing from the intrinsic? Should this also have the ClangBuiltin name declared here?
Added the ClangBuiltin
Updated the name to include the "s" since that's 2 people who've asked now.
The intrinsic doesn't map to s_wait_event, just to the s_wait_event export_ready - that's why it doesn't take the immediate argument.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138216



More information about the llvm-commits mailing list