[Mlir-commits] [mlir] [MLIR] Improve translation of DISubrange. (PR #93689)
Christian Ulmann
llvmlistbot at llvm.org
Wed May 29 09:24:36 PDT 2024
================
@@ -625,10 +625,10 @@ def LLVM_DINamespaceAttr : LLVM_Attr<"DINamespace", "di_namespace",
def LLVM_DISubrangeAttr : LLVM_Attr<"DISubrange", "di_subrange", /*traits=*/[],
"DINodeAttr"> {
let parameters = (ins
- OptionalParameter<"IntegerAttr">:$count,
- OptionalParameter<"IntegerAttr">:$lowerBound,
- OptionalParameter<"IntegerAttr">:$upperBound,
- OptionalParameter<"IntegerAttr">:$stride
+ OptionalParameter<"::mlir::Attribute">:$count,
----------------
Dinistro wrote:
I do not recall this, but I guess attribute parameters cannot be constraint through table gen, or can they? Ideally, we would explicitly list which types are legal here.
https://github.com/llvm/llvm-project/pull/93689
More information about the Mlir-commits
mailing list