[PATCH] D77455: [mlir][ODS] Add support for variadic regions.

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 4 12:13:32 PDT 2020


mehdi_amini added inline comments.


================
Comment at: mlir/test/Dialect/LLVMIR/global.mlir:83
 // expected-error @+1 {{'addr_space' failed to satisfy constraint: 32-bit signless integer attribute whose value is non-negative}}
-"llvm.mlir.global"() {sym_name = "foo", type = !llvm.i64, value = 42 : i64, addr_space = 1.0 : f32, linkage = 0} : () -> ()
+"llvm.mlir.global"() ({}) {sym_name = "foo", type = !llvm.i64, value = 42 : i64, addr_space = 1.0 : f32, linkage = 0} : () -> ()
 
----------------
Is this a requirement because of the verifier change?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77455





More information about the llvm-commits mailing list