================
@@ -99,6 +99,7 @@ class ROCDLDialectLLVMIRTranslationInterface
       if (!llvmFunc->hasFnAttribute("amdgpu-flat-work-group-size")) {
         llvmFunc->addFnAttr("amdgpu-flat-work-group-size", "1,256");
       }
+      llvmFunc->addFnAttr("amdgpu-implicitarg-num-bytes", "256");
----------------
saiislam wrote:

This PR depends on 79038 (the other review) and they both need to land 
together. When I created stacked PRs, it seems that Github brought both the 
commits together in this PR.

You may use [this 
](https://github.com/llvm/llvm-project/pull/79039/commits/a66ac33975381b1acdae0c177842d2f711ad9ab9)
 to see changes of this review only.

https://github.com/llvm/llvm-project/pull/79039
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to