Skip to content

Conversation

@jpienaar
Copy link
Member

No description provided.

@llvmbot llvmbot added the mlir label May 14, 2025
@llvmbot
Copy link
Member

llvmbot commented May 14, 2025

@llvm/pr-subscribers-mlir

Author: Jacques Pienaar (jpienaar)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/139847.diff

1 Files Affected:

  • (modified) mlir/lib/TableGen/Pattern.cpp (+1-1)
diff --git a/mlir/lib/TableGen/Pattern.cpp b/mlir/lib/TableGen/Pattern.cpp
index ab605391faf6a..e36a7f814331b 100644
--- a/mlir/lib/TableGen/Pattern.cpp
+++ b/mlir/lib/TableGen/Pattern.cpp
@@ -305,7 +305,7 @@ std::string SymbolInfoMap::SymbolInfo::getValueAndRangeUse(
     auto *operand = cast<NamedTypeConstraint *>(op->getArg(getArgIndex()));
     if (operand->isOptional()) {
       auto repl =
-          formatv(fmt, formatv("({0}.empty() ? Value() : *{0}.begin())", name));
+          formatv(fmt, formatv("({0}.empty() ? ::mlir::Value() : *{0}.begin())", name));
       LLVM_DEBUG(dbgs() << repl << " (OptionalOperand)\n");
       return std::string(repl);
     }

@github-actions
Copy link

github-actions bot commented May 14, 2025

✅ With the latest revision this PR passed the C/C++ code formatter.

@jpienaar jpienaar merged commit fbeab2c into llvm:main May 14, 2025
6 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants