Skip to content

Commit dec7dd8

Browse files
el-evmmha
andcommitted
Address comments
Co-authored-by: Morris Hafner <[email protected]>
1 parent 0b5b910 commit dec7dd8

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

clang/lib/CIR/Lowering/DirectToLLVM/LowerToLLVM.cpp

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2931,23 +2931,23 @@ mlir::LogicalResult CIRToLLVMInlineAsmOpLowering::matchAndRewrite(
29312931

29322932
SmallVector<mlir::Value> llvmOperands;
29332933
SmallVector<mlir::Value> cirOperands;
2934-
for (auto [llvmOp, cirOp] :
2935-
llvm::zip(adaptor.getAsmOperands(), op.getAsmOperands())) {
2936-
llvmOperands.append(llvmOp.begin(), llvmOp.end());
2937-
cirOperands.append(cirOp.begin(), cirOp.end());
2934+
for (auto const&[llvmOp, cirOp] :
2935+
zip(adaptor.getAsmOperands(), op.getAsmOperands())) {
2936+
append_range(llvmOperands, llvmOp);
2937+
append_range(cirOperands, cirOp);
29382938
}
29392939

29402940
// so far we infer the llvm dialect element type attr from
29412941
// CIR operand type.
2942-
for (auto [i, cirOpAttr] : llvm::enumerate(op.getOperandAttrs())) {
2942+
for (auto const&[cirOpAttr, cirOp] : zip(op.getOperandAttrs(), cirOperands)) {
29432943
if (!cirOpAttr) {
29442944
opAttrs.push_back(mlir::Attribute());
29452945
continue;
29462946
}
29472947

2948-
SmallVector<mlir::NamedAttribute> attrs;
2948+
llvm::SmallVector<mlir::NamedAttribute, 1> attrs;
29492949
cir::PointerType typ =
2950-
mlir::cast<cir::PointerType>(cirOperands[i].getType());
2950+
mlir::cast<cir::PointerType>(cirOp.getType());
29512951
mlir::TypeAttr typAttr = mlir::TypeAttr::get(convertTypeForMemory(
29522952
*getTypeConverter(), dataLayout, typ.getPointee()));
29532953

0 commit comments

Comments
 (0)