From cb5724c71e396729b7c0dd6a2a8aff20444dee09 Mon Sep 17 00:00:00 2001 From: Nikita Popov Date: Fri, 5 Jun 2020 23:18:26 +0200 Subject: [PATCH] [CGP] Remove unnecessary MaybeAlign use (NFC) Stores now always have an alignment. --- llvm/lib/CodeGen/CodeGenPrepare.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/llvm/lib/CodeGen/CodeGenPrepare.cpp b/llvm/lib/CodeGen/CodeGenPrepare.cpp index 345b3d77d53d..4ed49757733c 100644 --- a/llvm/lib/CodeGen/CodeGenPrepare.cpp +++ b/llvm/lib/CodeGen/CodeGenPrepare.cpp @@ -7083,13 +7083,13 @@ static bool splitMergedValStore(StoreInst &SI, const DataLayout &DL, Value *Addr = Builder.CreateBitCast( SI.getOperand(1), SplitStoreType->getPointerTo(SI.getPointerAddressSpace())); + Align Alignment = SI.getAlign(); const bool IsOffsetStore = (IsLE && Upper) || (!IsLE && !Upper); - if (IsOffsetStore) + if (IsOffsetStore) { Addr = Builder.CreateGEP( SplitStoreType, Addr, ConstantInt::get(Type::getInt32Ty(SI.getContext()), 1)); - MaybeAlign Alignment = SI.getAlign(); - if (IsOffsetStore && Alignment) { + // When splitting the store in half, naturally one half will retain the // alignment of the original wider store, regardless of whether it was // over-aligned or not, while the other will require adjustment. -- GitLab