[flang-commits] [flang] [mlir][Flang][NFC] Replace use of `vector.insertelement/extractelement` (PR #143272)
Diego Caballero via flang-commits
flang-commits at lists.llvm.org
Sat Jun 7 07:40:23 PDT 2025
https://github.com/dcaballe created https://github.com/llvm/llvm-project/pull/143272
This PR is part of the last step to remove `vector.extractelement` and `vector.insertelement` ops (RFC: https://discourse.llvm.org/t/rfc-psa-remove-vector-extractelement-and-vector-insertelement-ops-in-favor-of-vector-extract-and-vector-insert-ops).
It replaces `vector.insertelement` and `vector.extractelement` with `vector.insert` and `vector.extract` in Flang. It looks like no lit tests are impacted?
>From 13243bc88932e0eae5aa60860663f06b49bc902b Mon Sep 17 00:00:00 2001
From: Diego Caballero <dcaballero at nvidia.com>
Date: Sat, 7 Jun 2025 13:58:50 +0000
Subject: [PATCH] [mlir][Flang][NFC] Replace use of
`vector.insertelement/extractelement`
This PR is part of the last step to remove `vector.extractelement` and `vector.insertelement` ops
(RFC: https://discourse.llvm.org/t/rfc-psa-remove-vector-extractelement-and-vector-insertelement-ops-in-favor-of-vector-extract-and-vector-insert-ops)
It replaces `vector.insertelement` and `vector.extractelement` with `vector.insert` and
`vector.extract` in Flang. It looks like no lit tests are impacted.
---
flang/lib/Optimizer/Builder/PPCIntrinsicCall.cpp | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/flang/lib/Optimizer/Builder/PPCIntrinsicCall.cpp b/flang/lib/Optimizer/Builder/PPCIntrinsicCall.cpp
index fcc91752552c3..6090f008adb1c 100644
--- a/flang/lib/Optimizer/Builder/PPCIntrinsicCall.cpp
+++ b/flang/lib/Optimizer/Builder/PPCIntrinsicCall.cpp
@@ -1685,7 +1685,7 @@ PPCIntrinsicLibrary::genVecExtract(mlir::Type resultType,
if (!isNativeVecElemOrderOnLE())
uremOp = convertVectorElementOrder(builder, loc, vecTyInfo, uremOp);
- return builder.create<mlir::vector::ExtractElementOp>(loc, varg0, uremOp);
+ return builder.create<mlir::vector::ExtractOp>(loc, varg0, uremOp);
}
// VEC_INSERT
@@ -1706,8 +1706,8 @@ PPCIntrinsicLibrary::genVecInsert(mlir::Type resultType,
if (!isNativeVecElemOrderOnLE())
uremOp = convertVectorElementOrder(builder, loc, vecTyInfo, uremOp);
- auto res{builder.create<mlir::vector::InsertElementOp>(loc, argBases[0],
- varg1, uremOp)};
+ auto res{
+ builder.create<mlir::vector::InsertOp>(loc, argBases[0], varg1, uremOp)};
return builder.create<fir::ConvertOp>(loc, vecTyInfo.toFirVectorType(), res);
}
More information about the flang-commits
mailing list