================
@@ -3655,11 +3655,6 @@ Error AMDGPUKernelTy::launchImpl(GenericDeviceTy 
&GenericDevice,
                                  KernelArgsTy &KernelArgs,
                                  KernelLaunchParamsTy LaunchParams,
                                  AsyncInfoWrapperTy &AsyncInfoWrapper) const {
-  if (ArgsSize != LaunchParams.Size &&
----------------
kevinsala wrote:

The first condition will be true if there are implicit kernel arguments. The 
second condition can't be checked anymore because the `LanuchParams.Size` does 
not account for the tail padding of user arguments. Thus, `ArgsSize` can 
actually be larger than `LaunchParams.Size + getImplicitArgsSize()` because the 
padding between these last two is not accounted.

https://github.com/llvm/llvm-project/pull/156229
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to