[llvm] [SPIR-V] Fix minor build warnings (PR #109425)
Nathan Gauër via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 20 06:55:36 PDT 2024
https://github.com/Keenuts created https://github.com/llvm/llvm-project/pull/109425
`warning: 'InsertPosition' is deprecated: Use BasicBlock::iterators for insertion instead [-Wdeprecated-declarations]`
Calling `Instruction->getIterator()` removes the warning.
>From b79e46318f9965d34bb9e978d2a69f1f7f260a42 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Nathan=20Gau=C3=ABr?= <brioche at google.com>
Date: Fri, 20 Sep 2024 15:53:36 +0200
Subject: [PATCH] [SPIR-V] Fix minor build warnings
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
`warning: 'InsertPosition' is deprecated: Use BasicBlock::iterators for
insertion instead [-Wdeprecated-declarations]`
Calling `Instruction->getIterator()` removes the warning.
Signed-off-by: Nathan Gauër <brioche at google.com>
---
llvm/lib/Target/SPIRV/SPIRVRegularizer.cpp | 6 +++---
llvm/lib/Target/SPIRV/SPIRVStripConvergentIntrinsics.cpp | 2 +-
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/llvm/lib/Target/SPIRV/SPIRVRegularizer.cpp b/llvm/lib/Target/SPIRV/SPIRVRegularizer.cpp
index 322e051a87db1a..3b37ee93816934 100644
--- a/llvm/lib/Target/SPIRV/SPIRVRegularizer.cpp
+++ b/llvm/lib/Target/SPIRV/SPIRVRegularizer.cpp
@@ -127,7 +127,7 @@ void SPIRVRegularizer::runLowerConstExpr(Function &F) {
ReplList.push_back(Inst);
Repl = InsertElementInst::Create(
(Repl ? Repl : PoisonValue::get(Vec->getType())), V,
- ConstantInt::get(Type::getInt32Ty(Ctx), Idx++), "", InsPoint);
+ ConstantInt::get(Type::getInt32Ty(Ctx), Idx++), "", InsPoint->getIterator());
}
WorkList.splice(WorkList.begin(), ReplList);
return Repl;
@@ -235,10 +235,10 @@ void SPIRVRegularizer::visitCallScalToVec(CallInst *CI, StringRef MangledName,
auto ConstInt = ConstantInt::get(IntegerType::get(CI->getContext(), 32), 0);
PoisonValue *PVal = PoisonValue::get(Arg0Ty);
Instruction *Inst =
- InsertElementInst::Create(PVal, CI->getOperand(1), ConstInt, "", CI);
+ InsertElementInst::Create(PVal, CI->getOperand(1), ConstInt, "", CI->getIterator());
ElementCount VecElemCount = cast<VectorType>(Arg0Ty)->getElementCount();
Constant *ConstVec = ConstantVector::getSplat(VecElemCount, ConstInt);
- Value *NewVec = new ShuffleVectorInst(Inst, PVal, ConstVec, "", CI);
+ Value *NewVec = new ShuffleVectorInst(Inst, PVal, ConstVec, "", CI->getIterator());
CI->setOperand(1, NewVec);
CI->replaceUsesOfWith(OldF, NewF);
CI->mutateFunctionType(NewF->getFunctionType());
diff --git a/llvm/lib/Target/SPIRV/SPIRVStripConvergentIntrinsics.cpp b/llvm/lib/Target/SPIRV/SPIRVStripConvergentIntrinsics.cpp
index b632d784977678..c87048b93f80fc 100644
--- a/llvm/lib/Target/SPIRV/SPIRVStripConvergentIntrinsics.cpp
+++ b/llvm/lib/Target/SPIRV/SPIRVStripConvergentIntrinsics.cpp
@@ -62,7 +62,7 @@ class SPIRVStripConvergentIntrinsics : public FunctionPass {
return;
auto *NewCall = CallBase::removeOperandBundle(
- CI, LLVMContext::OB_convergencectrl, CI);
+ CI, LLVMContext::OB_convergencectrl, CI->getIterator());
NewCall->copyMetadata(*CI);
CI->replaceAllUsesWith(NewCall);
ToRemove.insert(CI);
More information about the llvm-commits
mailing list