================
@@ -94,15 +94,37 @@ void test_cvt_scale_pk(global half8 *outh8, global bfloat8 
*outy8, uint2 src2,
   *outf16 = __builtin_amdgcn_cvt_scale_pk16_f32_bf6(src3, scale, 16); // 
expected-error {{argument value 16 is outside the valid range [0, 15]}}
 }
 
-void test_amdgcn_load_monitor(global int* b32gaddr, global v2i* b64gaddr, 
global v4i* b128gaddr, int *b32faddr, v2i* b64faddr, v4i *b128faddr,
-                              global int* b32out, global v2i* b64out, global 
v4i* b128out, int cpol)
+void test_amdgcn_load_monitor_ao_constant(global int* b32gaddr, global v2i* 
b64gaddr, global v4i* b128gaddr, int *b32faddr, v2i* b64faddr, v4i *b128faddr,
+                              global int* b32out, global v2i* b64out, global 
v4i* b128out, int ao)
 {
-  *b32out  = __builtin_amdgcn_global_load_monitor_b32(b32gaddr, cpol); // 
expected-error {{'__builtin_amdgcn_global_load_monitor_b32' must be a constant 
integer}}
-  *b64out  = __builtin_amdgcn_global_load_monitor_b64(b64gaddr, cpol); // 
expected-error {{'__builtin_amdgcn_global_load_monitor_b64' must be a constant 
integer}}
-  *b128out = __builtin_amdgcn_global_load_monitor_b128(b128gaddr, cpol); // 
expected-error {{'__builtin_amdgcn_global_load_monitor_b128' must be a constant 
integer}}
-  *b32out  = __builtin_amdgcn_flat_load_monitor_b32(b32faddr, cpol); // 
expected-error {{'__builtin_amdgcn_flat_load_monitor_b32' must be a constant 
integer}}
-  *b64out  = __builtin_amdgcn_flat_load_monitor_b64(b64faddr, cpol); // 
expected-error {{'__builtin_amdgcn_flat_load_monitor_b64' must be a constant 
integer}}
-  *b128out = __builtin_amdgcn_flat_load_monitor_b128(b128faddr, cpol); // 
expected-error {{'__builtin_amdgcn_flat_load_monitor_b128' must be a constant 
integer}}
+  *b32out  = __builtin_amdgcn_global_load_monitor_b32(b32gaddr, ao, ""); // 
expected-error {{'__builtin_amdgcn_global_load_monitor_b32' must be a constant 
integer}}
----------------
arsenm wrote:

We've been moving away from using strings as the builtin interface for the 
syncscope. This should use the enums 

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

Reply via email to