[PATCH] D96257: [AMDGPU] Rename amdgcn_wwm to amdgcn_strictwwm

Piotr Sobczak via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 10 00:56:35 PST 2021


piotr added inline comments.


================
Comment at: llvm/include/llvm/IR/IntrinsicsAMDGPU.td:1603
 // undefined value.
+def int_amdgcn_strictwwm : Intrinsic<[llvm_any_ty],
+  [LLVMMatchType<0>], [IntrNoMem, IntrSpeculatable,
----------------
arsenm wrote:
> Would somewhat prefer a _/. between strict and wwm
I don't have a strong opinion myself. I can add "_" if that is easier to parse (and leave soft_wqm as it is). 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96257



More information about the llvm-commits mailing list