[Mlir-commits] [mlir] [MLIR][LLVM] Fix #llvm.constant_range crashing in storage uniquer (PR #135772)
River Riddle
llvmlistbot at llvm.org
Tue Apr 15 10:08:42 PDT 2025
================
@@ -1094,9 +1094,8 @@ def LLVM_TBAATagArrayAttr
// ConstantRangeAttr
//===----------------------------------------------------------------------===//
def LLVM_ConstantRangeAttr : LLVM_Attr<"ConstantRange", "constant_range"> {
- let parameters = (ins
- "::llvm::APInt":$lower,
- "::llvm::APInt":$upper
+ let parameters = (ins APIntParameter<"">:$lower,
+ APIntParameter<"">:$upper
----------------
River707 wrote:
One conceptual alternative to that I suppose, would be to detect APInt and use APIntParameter equivalent logic instead. Either way, would be nice to remove a footgun.
https://github.com/llvm/llvm-project/pull/135772
More information about the Mlir-commits
mailing list