Skip to content

Commit 009fe07

Browse files
committed
[Alignment] Remove multiply by MaybeAlign
1 parent 2b089e9 commit 009fe07

File tree

2 files changed

+2
-12
lines changed

2 files changed

+2
-12
lines changed

llvm/include/llvm/Support/Alignment.h

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -319,11 +319,6 @@ inline Align operator*(Align Lhs, uint64_t Rhs) {
319319
return Align(Lhs.value() * Rhs);
320320
}
321321

322-
inline MaybeAlign operator*(MaybeAlign Lhs, uint64_t Rhs) {
323-
assert(Rhs > 0 && "Rhs must be positive");
324-
return Lhs ? Lhs.getValue() * Rhs : MaybeAlign();
325-
}
326-
327322
inline Align operator/(Align Lhs, uint64_t Divisor) {
328323
assert(llvm::isPowerOf2_64(Divisor) &&
329324
"Divisor must be positive and a power of 2");

llvm/lib/Transforms/Instrumentation/DataFlowSanitizer.cpp

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2746,13 +2746,8 @@ void DFSanVisitor::visitMemTransferInst(MemTransferInst &I) {
27462746
auto *MTI = cast<MemTransferInst>(
27472747
IRB.CreateCall(I.getFunctionType(), I.getCalledOperand(),
27482748
{DestShadow, SrcShadow, LenShadow, I.getVolatileCst()}));
2749-
if (ClPreserveAlignment) {
2750-
MTI->setDestAlignment(I.getDestAlign() * DFSF.DFS.ShadowWidthBytes);
2751-
MTI->setSourceAlignment(I.getSourceAlign() * DFSF.DFS.ShadowWidthBytes);
2752-
} else {
2753-
MTI->setDestAlignment(Align(DFSF.DFS.ShadowWidthBytes));
2754-
MTI->setSourceAlignment(Align(DFSF.DFS.ShadowWidthBytes));
2755-
}
2749+
MTI->setDestAlignment(DFSF.getShadowAlign(I.getDestAlign().valueOrOne()));
2750+
MTI->setSourceAlignment(DFSF.getShadowAlign(I.getSourceAlign().valueOrOne()));
27562751
if (ClEventCallbacks) {
27572752
IRB.CreateCall(DFSF.DFS.DFSanMemTransferCallbackFn,
27582753
{RawDestShadow,

0 commit comments

Comments
 (0)