diff --git a/clang/lib/CIR/Dialect/IR/CIRDialect.cpp b/clang/lib/CIR/Dialect/IR/CIRDialect.cpp index d6d7c9cb96c35..91c3050c46806 100644 --- a/clang/lib/CIR/Dialect/IR/CIRDialect.cpp +++ b/clang/lib/CIR/Dialect/IR/CIRDialect.cpp @@ -1187,7 +1187,8 @@ void cir::IfOp::getSuccessorRegions(mlir::RegionBranchPoint point, } mlir::ValueRange cir::IfOp::getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getOperation()->getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getOperation()->getResults()) + : ValueRange(); } void cir::IfOp::build(OpBuilder &builder, OperationState &result, Value cond, @@ -1231,7 +1232,8 @@ void cir::ScopeOp::getSuccessorRegions( } mlir::ValueRange cir::ScopeOp::getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getOperation()->getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getOperation()->getResults()) + : ValueRange(); } void cir::ScopeOp::build( @@ -1398,7 +1400,8 @@ void cir::CaseOp::getSuccessorRegions( } mlir::ValueRange cir::CaseOp::getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getOperation()->getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getOperation()->getResults()) + : ValueRange(); } void cir::CaseOp::build(OpBuilder &builder, OperationState &result, @@ -1429,7 +1432,8 @@ void cir::SwitchOp::getSuccessorRegions( } mlir::ValueRange cir::SwitchOp::getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getOperation()->getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getOperation()->getResults()) + : ValueRange(); } void cir::SwitchOp::build(OpBuilder &builder, OperationState &result, @@ -1661,7 +1665,8 @@ void cir::GlobalOp::getSuccessorRegions( } mlir::ValueRange cir::GlobalOp::getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getOperation()->getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getOperation()->getResults()) + : ValueRange(); } static void printGlobalOpTypeAndInitialValue(OpAsmPrinter &p, cir::GlobalOp op, @@ -2339,7 +2344,8 @@ void cir::TernaryOp::getSuccessorRegions( } mlir::ValueRange cir::TernaryOp::getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getOperation()->getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getOperation()->getResults()) + : ValueRange(); } void cir::TernaryOp::build( @@ -3576,7 +3582,8 @@ void cir::TryOp::getSuccessorRegions( } mlir::ValueRange cir::TryOp::getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getOperation()->getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getOperation()->getResults()) + : ValueRange(); } static void diff --git a/mlir/lib/Dialect/EmitC/IR/EmitC.cpp b/mlir/lib/Dialect/EmitC/IR/EmitC.cpp index d14cc2c645f5a..1d4b748b2a88a 100644 --- a/mlir/lib/Dialect/EmitC/IR/EmitC.cpp +++ b/mlir/lib/Dialect/EmitC/IR/EmitC.cpp @@ -893,7 +893,8 @@ void IfOp::getSuccessorRegions(RegionBranchPoint point, } ValueRange IfOp::getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getOperation()->getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getOperation()->getResults()) + : ValueRange(); } void IfOp::getEntrySuccessorRegions(ArrayRef operands, diff --git a/mlir/lib/Dialect/GPU/IR/GPUDialect.cpp b/mlir/lib/Dialect/GPU/IR/GPUDialect.cpp index 11d22a2024a2e..033a94e3f8fce 100644 --- a/mlir/lib/Dialect/GPU/IR/GPUDialect.cpp +++ b/mlir/lib/Dialect/GPU/IR/GPUDialect.cpp @@ -2407,7 +2407,7 @@ void WarpExecuteOnLane0Op::getSuccessorRegions( } ValueRange WarpExecuteOnLane0Op::getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getResults()) : ValueRange(); } void WarpExecuteOnLane0Op::build(OpBuilder &builder, OperationState &result, TypeRange resultTypes, Value laneId, diff --git a/mlir/lib/Dialect/MemRef/IR/MemRefOps.cpp b/mlir/lib/Dialect/MemRef/IR/MemRefOps.cpp index 97020bc49ba04..e0f7a8b452a1d 100644 --- a/mlir/lib/Dialect/MemRef/IR/MemRefOps.cpp +++ b/mlir/lib/Dialect/MemRef/IR/MemRefOps.cpp @@ -413,7 +413,7 @@ void AllocaScopeOp::getSuccessorRegions( } ValueRange AllocaScopeOp::getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getResults()) : ValueRange(); } /// Given an operation, return whether this op is guaranteed to diff --git a/mlir/lib/Dialect/OpenACC/IR/OpenACC.cpp b/mlir/lib/Dialect/OpenACC/IR/OpenACC.cpp index a49085f1a92cd..abfcad88c2437 100644 --- a/mlir/lib/Dialect/OpenACC/IR/OpenACC.cpp +++ b/mlir/lib/Dialect/OpenACC/IR/OpenACC.cpp @@ -428,7 +428,7 @@ getSingleRegionOpSuccessorRegions(Operation *op, Region ®ion, static ValueRange getSingleRegionSuccessorInputs(Operation *op, RegionSuccessor successor) { - return successor.isParent() ? op->getResults() : ValueRange(); + return successor.isParent() ? ValueRange(op->getResults()) : ValueRange(); } void KernelsOp::getSuccessorRegions(RegionBranchPoint point, diff --git a/mlir/lib/Dialect/SCF/IR/SCF.cpp b/mlir/lib/Dialect/SCF/IR/SCF.cpp index 294c8412064ff..5b6e9304de505 100644 --- a/mlir/lib/Dialect/SCF/IR/SCF.cpp +++ b/mlir/lib/Dialect/SCF/IR/SCF.cpp @@ -311,7 +311,8 @@ void ExecuteRegionOp::getSuccessorRegions( } ValueRange ExecuteRegionOp::getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getOperation()->getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getOperation()->getResults()) + : ValueRange(); } //===----------------------------------------------------------------------===// @@ -2137,7 +2138,8 @@ void IfOp::getSuccessorRegions(RegionBranchPoint point, } ValueRange IfOp::getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getOperation()->getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getOperation()->getResults()) + : ValueRange(); } void IfOp::getEntrySuccessorRegions(ArrayRef operands, @@ -3875,7 +3877,8 @@ void IndexSwitchOp::getSuccessorRegions( } ValueRange IndexSwitchOp::getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getOperation()->getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getOperation()->getResults()) + : ValueRange(); } void IndexSwitchOp::getEntrySuccessorRegions( diff --git a/mlir/lib/Dialect/Shape/IR/Shape.cpp b/mlir/lib/Dialect/Shape/IR/Shape.cpp index 79b820654d888..95546b4a4deb7 100644 --- a/mlir/lib/Dialect/Shape/IR/Shape.cpp +++ b/mlir/lib/Dialect/Shape/IR/Shape.cpp @@ -354,7 +354,7 @@ void AssumingOp::getSuccessorRegions( } ValueRange AssumingOp::getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getResults()) : ValueRange(); } void AssumingOp::inlineRegionIntoParent(AssumingOp &op, diff --git a/mlir/lib/Dialect/Transform/TuneExtension/TuneExtensionOps.cpp b/mlir/lib/Dialect/Transform/TuneExtension/TuneExtensionOps.cpp index 21d2b13d18405..8932ddfb988d7 100644 --- a/mlir/lib/Dialect/Transform/TuneExtension/TuneExtensionOps.cpp +++ b/mlir/lib/Dialect/Transform/TuneExtension/TuneExtensionOps.cpp @@ -133,7 +133,8 @@ void transform::tune::AlternativesOp::getSuccessorRegions( ValueRange transform::tune::AlternativesOp::getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getOperation()->getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getOperation()->getResults()) + : ValueRange(); } void transform::tune::AlternativesOp::getRegionInvocationBounds( diff --git a/mlir/test/lib/Dialect/Test/TestOpDefs.cpp b/mlir/test/lib/Dialect/Test/TestOpDefs.cpp index c0150ed5ced4b..7a8c3703498fb 100644 --- a/mlir/test/lib/Dialect/Test/TestOpDefs.cpp +++ b/mlir/test/lib/Dialect/Test/TestOpDefs.cpp @@ -788,7 +788,7 @@ void AnyCondOp::getSuccessorRegions(RegionBranchPoint point, } ValueRange AnyCondOp::getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getResults()) : ValueRange(); } void AnyCondOp::getRegionInvocationBounds( diff --git a/mlir/unittests/Interfaces/ControlFlowInterfacesTest.cpp b/mlir/unittests/Interfaces/ControlFlowInterfacesTest.cpp index 0272905c34f58..3682f073186df 100644 --- a/mlir/unittests/Interfaces/ControlFlowInterfacesTest.cpp +++ b/mlir/unittests/Interfaces/ControlFlowInterfacesTest.cpp @@ -73,7 +73,8 @@ struct LoopRegionsOp } ValueRange getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getOperation()->getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getOperation()->getResults()) + : ValueRange(); } using RegionBranchOpInterface::Trait::getSuccessorRegions; @@ -101,7 +102,8 @@ struct DoubleLoopRegionsOp } ValueRange getSuccessorInputs(RegionSuccessor successor) { - return successor.isParent() ? getOperation()->getResults() : ValueRange(); + return successor.isParent() ? ValueRange(getOperation()->getResults()) + : ValueRange(); } using RegionBranchOpInterface::Trait<