[Mlir-commits] [mlir] [mlir][gpu] Make launch_func op use SymbolUserOpInterface (PR #173277)

[Muhammad Abdul]

================
@@ -137,14 +137,14 @@ module attributes {gpu.container_module} {
 // -----
 
 module attributes {gpu.container_module} {
-  module @kernels {
+  gpu.module @kernels_container {
     gpu.func @kernel_1(%arg1 : !llvm.ptr) kernel {
       gpu.return
     }
   }
 
   func.func @launch_func_missing_kernel_attr(%sz : index, %arg : !llvm.ptr) {
-    // expected-error at +1 {{kernel module 'kernels' is undefined}}
+    // expected-error at +1 {{kernel container 'kernels' is undefined}}
----------------
0xzre wrote:

Wouldn't this make the test pass by lookup error instead of missing kernel attr as defined on name?

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