[clang] [CIR] Fix mlir::ValueRange init from ArrayRef warning (PR #146577)

via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 1 10:43:05 PDT 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clangir

Author: Amr Hesham (AmrDeveloper)

<details>
<summary>Changes</summary>

Fix initalizing ValueRange with ArrayRef `ValueRange(ArrayRef<Value>(std::forward<Arg>(arg))) {}` warning

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


1 Files Affected:

- (modified) clang/include/clang/CIR/Dialect/IR/CIROps.td (+1-1) 


``````````diff
diff --git a/clang/include/clang/CIR/Dialect/IR/CIROps.td b/clang/include/clang/CIR/Dialect/IR/CIROps.td
index 4daff74cbae5a..4d3ebfb93615d 100644
--- a/clang/include/clang/CIR/Dialect/IR/CIROps.td
+++ b/clang/include/clang/CIR/Dialect/IR/CIROps.td
@@ -489,7 +489,7 @@ def ReturnOp : CIR_Op<"return", [ParentOneOf<["FuncOp", "ScopeOp", "IfOp",
 
   // Allow building a ReturnOp with no return operand.
   let builders = [
-    OpBuilder<(ins), [{ build($_builder, $_state, std::nullopt); }]>
+    OpBuilder<(ins), [{ build($_builder, $_state, {}); }]>
   ];
 
   // Provide extra utility definitions on the c++ operation class definition.

``````````

</details>


https://github.com/llvm/llvm-project/pull/146577


More information about the cfe-commits mailing list