================
@@ -45,6 +45,10 @@ __attribute__((amdgpu_num_vgpr(64))) // 
expected-no-diagnostics
 __global__ void num_vgpr_64() {
 // CHECK: define{{.*}} amdgpu_kernel void @_Z11num_vgpr_64v() 
[[NUM_VGPR_64:#[0-9]+]]
 }
+__attribute__((amdgpu_kernarg_preload_count(2))) // expected-no-diagnostics
----------------
AaronBallman wrote:

I see this pattern is used elsewhere in the test but FWIW, `// 
expected-no-diagnostics` is confusing because there's no `-verify` on the `RUN` 
line and that's the usual comment to check that. That we repeat it all over the 
test file is a bit odd, the comment doesn't really help clarify anything.

(Might be worth stripping all those comments as an NFC change.)

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

Reply via email to