[llvm] r350378 - [memcpyopt] Remove a few unnecessary isVolatile() checks. NFC
Xin Tong via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 3 18:13:22 PST 2019
Author: trentxintong
Date: Thu Jan 3 18:13:22 2019
New Revision: 350378
URL: http://llvm.org/viewvc/llvm-project?rev=350378&view=rev
Log:
[memcpyopt] Remove a few unnecessary isVolatile() checks. NFC
We already checked for isSimple() on the store.
Modified:
llvm/trunk/lib/Transforms/Scalar/MemCpyOptimizer.cpp
Modified: llvm/trunk/lib/Transforms/Scalar/MemCpyOptimizer.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/MemCpyOptimizer.cpp?rev=350378&r1=350377&r2=350378&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/MemCpyOptimizer.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/MemCpyOptimizer.cpp Thu Jan 3 18:13:22 2019
@@ -675,13 +675,11 @@ bool MemCpyOptPass::processStore(StoreIn
if (UseMemMove)
M = Builder.CreateMemMove(
SI->getPointerOperand(), findStoreAlignment(DL, SI),
- LI->getPointerOperand(), findLoadAlignment(DL, LI), Size,
- SI->isVolatile());
+ LI->getPointerOperand(), findLoadAlignment(DL, LI), Size);
else
M = Builder.CreateMemCpy(
SI->getPointerOperand(), findStoreAlignment(DL, SI),
- LI->getPointerOperand(), findLoadAlignment(DL, LI), Size,
- SI->isVolatile());
+ LI->getPointerOperand(), findLoadAlignment(DL, LI), Size);
LLVM_DEBUG(dbgs() << "Promoting " << *LI << " to " << *SI << " => "
<< *M << "\n");
@@ -770,8 +768,8 @@ bool MemCpyOptPass::processStore(StoreIn
if (!Align)
Align = DL.getABITypeAlignment(T);
IRBuilder<> Builder(SI);
- auto *M = Builder.CreateMemSet(SI->getPointerOperand(), ByteVal,
- Size, Align, SI->isVolatile());
+ auto *M =
+ Builder.CreateMemSet(SI->getPointerOperand(), ByteVal, Size, Align);
LLVM_DEBUG(dbgs() << "Promoting " << *SI << " to " << *M << "\n");
More information about the llvm-commits
mailing list