[llvm] [AMDGPU] Defaults for missing dimensions in SYCL required wg size (PR #68872)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 23 23:09:41 PDT 2023


================
@@ -317,10 +317,38 @@ static bool processUse(CallInst *CI, bool IsV5OrAbove) {
   return MadeChange;
 }
 
+// SYCL allows required work-group size attribute to be partially specified
----------------
arsenm wrote:

First of all, why would this happen? I would expect all producers of this to agree on one consistent format (which we should also add to the LangRef).

Second, a random backend pass shouldn't be going out of its way to canonicalize metadata.

https://github.com/llvm/llvm-project/pull/68872


More information about the llvm-commits mailing list