Skip to content

Commit a01c7da

Browse files
chelinilorenzo chelini
authored andcommitted
Drop compound literals
There is not need to use compound literals in C++. In this case we can use brace initialization and avoid compiler warnings.
1 parent 97c30b1 commit a01c7da

File tree

1 file changed

+8
-12
lines changed

1 file changed

+8
-12
lines changed

tools/mlir-clang/Lib/CGStmt.cc

Lines changed: 8 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -600,9 +600,8 @@ ValueCategory MLIRScanner::VisitDoStmt(clang::DoStmt *fors) {
600600
auto i1Ty = builder.getIntegerType(1);
601601
auto type = mlir::MemRefType::get({}, i1Ty, {}, 0);
602602
auto truev = builder.create<ConstantIntOp>(loc, true, 1);
603-
loops.push_back(
604-
(LoopContext){builder.create<mlir::memref::AllocaOp>(loc, type),
605-
builder.create<mlir::memref::AllocaOp>(loc, type)});
603+
loops.push_back({builder.create<mlir::memref::AllocaOp>(loc, type),
604+
builder.create<mlir::memref::AllocaOp>(loc, type)});
606605
builder.create<mlir::memref::StoreOp>(loc, truev, loops.back().noBreak);
607606

608607
auto toadd = builder.getInsertionBlock()->getParent();
@@ -662,9 +661,8 @@ ValueCategory MLIRScanner::VisitWhileStmt(clang::WhileStmt *fors) {
662661
auto i1Ty = builder.getIntegerType(1);
663662
auto type = mlir::MemRefType::get({}, i1Ty, {}, 0);
664663
auto truev = builder.create<ConstantIntOp>(loc, true, 1);
665-
loops.push_back(
666-
(LoopContext){builder.create<mlir::memref::AllocaOp>(loc, type),
667-
builder.create<mlir::memref::AllocaOp>(loc, type)});
664+
loops.push_back({builder.create<mlir::memref::AllocaOp>(loc, type),
665+
builder.create<mlir::memref::AllocaOp>(loc, type)});
668666
builder.create<mlir::memref::StoreOp>(loc, truev, loops.back().noBreak);
669667

670668
auto toadd = builder.getInsertionBlock()->getParent();
@@ -814,9 +812,8 @@ ValueCategory MLIRScanner::VisitSwitchStmt(clang::SwitchStmt *stmt) {
814812
auto i1Ty = builder.getIntegerType(1);
815813
auto type = mlir::MemRefType::get({}, i1Ty, {}, 0);
816814
auto truev = builder.create<ConstantIntOp>(loc, true, 1);
817-
loops.push_back(
818-
(LoopContext){builder.create<mlir::memref::AllocaOp>(loc, type),
819-
builder.create<mlir::memref::AllocaOp>(loc, type)});
815+
loops.push_back({builder.create<mlir::memref::AllocaOp>(loc, type),
816+
builder.create<mlir::memref::AllocaOp>(loc, type)});
820817
builder.create<mlir::memref::StoreOp>(loc, truev, loops.back().noBreak);
821818
builder.create<mlir::memref::StoreOp>(loc, truev,
822819
loops.back().keepRunning);
@@ -838,9 +835,8 @@ ValueCategory MLIRScanner::VisitSwitchStmt(clang::SwitchStmt *stmt) {
838835
auto i1Ty = builder.getIntegerType(1);
839836
auto type = mlir::MemRefType::get({}, i1Ty, {}, 0);
840837
auto truev = builder.create<ConstantIntOp>(loc, true, 1);
841-
loops.push_back(
842-
(LoopContext){builder.create<mlir::memref::AllocaOp>(loc, type),
843-
builder.create<mlir::memref::AllocaOp>(loc, type)});
838+
loops.push_back({builder.create<mlir::memref::AllocaOp>(loc, type),
839+
builder.create<mlir::memref::AllocaOp>(loc, type)});
844840
builder.create<mlir::memref::StoreOp>(loc, truev, loops.back().noBreak);
845841
builder.create<mlir::memref::StoreOp>(loc, truev,
846842
loops.back().keepRunning);

0 commit comments

Comments
 (0)