[llvm] 59bd206 - [NFC][ScalarizeMaskedMemIntrin] Fix unused variable warning
Yang Fan via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 28 23:17:49 PST 2021
Author: Yang Fan
Date: 2021-01-29T15:15:58+08:00
New Revision: 59bd2068e954e5d6d8396c4f27b94ad7f3abf3b1
URL: https://github.com/llvm/llvm-project/commit/59bd2068e954e5d6d8396c4f27b94ad7f3abf3b1
DIFF: https://github.com/llvm/llvm-project/commit/59bd2068e954e5d6d8396c4f27b94ad7f3abf3b1.diff
LOG: [NFC][ScalarizeMaskedMemIntrin] Fix unused variable warning
GCC warning:
```
/llvm-project/llvm/lib/Transforms/Scalar/ScalarizeMaskedMemIntrin.cpp: In function ‘void scalarizeMaskedStore(llvm::CallInst*, llvm::DomTreeUpdater*, bool&)’:
/llvm-project/llvm/lib/Transforms/Scalar/ScalarizeMaskedMemIntrin.cpp:295:15: warning: variable ‘IfBlock’ set but not used [-Wunused-but-set-variable]
295 | BasicBlock *IfBlock = CI->getParent();
| ^~~~~~~
/llvm-project/llvm/lib/Transforms/Scalar/ScalarizeMaskedMemIntrin.cpp: In function ‘void scalarizeMaskedScatter(llvm::CallInst*, llvm::DomTreeUpdater*, bool&)’:
/llvm-project/llvm/lib/Transforms/Scalar/ScalarizeMaskedMemIntrin.cpp:555:15: warning: variable ‘IfBlock’ set but not used [-Wunused-but-set-variable]
555 | BasicBlock *IfBlock = CI->getParent();
| ^~~~~~~
```
Added:
Modified:
llvm/lib/Transforms/Scalar/ScalarizeMaskedMemIntrin.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Transforms/Scalar/ScalarizeMaskedMemIntrin.cpp b/llvm/lib/Transforms/Scalar/ScalarizeMaskedMemIntrin.cpp
index a520ab3c63a4..11bd92c4334b 100644
--- a/llvm/lib/Transforms/Scalar/ScalarizeMaskedMemIntrin.cpp
+++ b/llvm/lib/Transforms/Scalar/ScalarizeMaskedMemIntrin.cpp
@@ -292,7 +292,6 @@ static void scalarizeMaskedStore(CallInst *CI, DomTreeUpdater *DTU,
IRBuilder<> Builder(CI->getContext());
Instruction *InsertPt = CI;
- BasicBlock *IfBlock = CI->getParent();
Builder.SetInsertPoint(InsertPt);
Builder.SetCurrentDebugLocation(CI->getDebugLoc());
@@ -369,7 +368,6 @@ static void scalarizeMaskedStore(CallInst *CI, DomTreeUpdater *DTU,
// Create "else" block, fill it in the next iteration
BasicBlock *NewIfBlock = ThenTerm->getSuccessor(0);
NewIfBlock->setName("else");
- IfBlock = NewIfBlock;
Builder.SetInsertPoint(NewIfBlock, NewIfBlock->begin());
}
@@ -552,7 +550,6 @@ static void scalarizeMaskedScatter(CallInst *CI, DomTreeUpdater *DTU,
IRBuilder<> Builder(CI->getContext());
Instruction *InsertPt = CI;
- BasicBlock *IfBlock = CI->getParent();
Builder.SetInsertPoint(InsertPt);
Builder.SetCurrentDebugLocation(CI->getDebugLoc());
@@ -618,7 +615,6 @@ static void scalarizeMaskedScatter(CallInst *CI, DomTreeUpdater *DTU,
// Create "else" block, fill it in the next iteration
BasicBlock *NewIfBlock = ThenTerm->getSuccessor(0);
NewIfBlock->setName("else");
- IfBlock = NewIfBlock;
Builder.SetInsertPoint(NewIfBlock, NewIfBlock->begin());
}
More information about the llvm-commits
mailing list