[PATCH] D77232: [mlir][GPUToSPIRV] Make spv.interface_var_abi attribute on arguments either be unspecified on all arguments to use default ABI, or be present on all arguments.

Mon Apr 20 11:54:00 PDT 2020

antiagainst requested changes to this revision.
antiagainst added inline comments.
This revision now requires changes to proceed.
Herald added a subscriber: frgossen.

================
Comment at: mlir/test/Conversion/GPUToSPIRV/if.mlir:40
+    gpu.func @kernel_nested_selection(%arg3 : memref<10xf32>,
+                                      %arg4 : memref<10xf32>,
+                                      %arg5 : i1, %arg6 : i1)
----------------
Let's revert changes to these tests. They should pass as-is. Applies to following unaffected tests.

================
Comment at: mlir/test/Conversion/GPUToSPIRV/simple.mlir:40
+    // CHECK-SAME: spv.entry_point_abi = {local_size = dense<[32, 4, 1]> : vector<3xi32>}
+    gpu.func @basic_module_structure_preset_ABI(
+      %arg0 : f32
----------------
I think we can just test the gpu.func itself without the wrapping gpu.module and the corresponding launch op? This way it's clearer regarding the intended use case of attaching these attributes: it gives us a way to use and control the CodeGen side as a stand-alone component. :)

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77232/new/

https://reviews.llvm.org/D77232