diff --git a/llvm/lib/Transforms/Scalar/SROA.cpp b/llvm/lib/Transforms/Scalar/SROA.cpp --- a/llvm/lib/Transforms/Scalar/SROA.cpp +++ b/llvm/lib/Transforms/Scalar/SROA.cpp @@ -1688,15 +1688,17 @@ bool IsThen = SuccBB == HeadBI->getSuccessor(0); int SuccIdx = IsThen ? 0 : 1; auto *NewMemOpBB = SuccBB == Tail ? Head : SuccBB; + auto &CondMemOp = cast(*I.clone()); if (NewMemOpBB != Head) { NewMemOpBB->setName(Head->getName() + (IsThen ? ".then" : ".else")); if (isa(I)) ++NumLoadsPredicated; else ++NumStoresPredicated; - } else + } else { + CondMemOp.dropUBImplyingAttrsAndUnknownMetadata(); ++NumLoadsSpeculated; - auto &CondMemOp = cast(*I.clone()); + } CondMemOp.insertBefore(NewMemOpBB->getTerminator()); Value *Ptr = SI.getOperand(1 + SuccIdx); if (auto *PtrTy = Ptr->getType(); diff --git a/llvm/test/Transforms/SROA/select-load-with-nonnull.ll b/llvm/test/Transforms/SROA/select-load-with-nonnull.ll new file mode 100644 --- /dev/null +++ b/llvm/test/Transforms/SROA/select-load-with-nonnull.ll @@ -0,0 +1,21 @@ +; NOTE: Assertions have been autogenerated by utils/update_test_checks.py +; RUN: opt < %s -passes='sroa' -S | FileCheck %s + +; When promoting speculative instruction, metadata that may trigger immediate UB should be dropped. +define void @load_of_select_with_noundef_nonnull(ptr %buffer, i1 %b) { +; CHECK-LABEL: @load_of_select_with_noundef_nonnull( +; CHECK-NEXT: br i1 [[B:%.*]], label [[DOTTHEN:%.*]], label [[DOTCONT:%.*]] +; CHECK: .then: +; CHECK-NEXT: [[LOAD_PTR_THEN_VAL:%.*]] = load ptr, ptr [[BUFFER:%.*]], align 8, !nonnull !0, !noundef !0 +; CHECK-NEXT: br label [[DOTCONT]] +; CHECK: .cont: +; CHECK-NEXT: [[LOAD_PTR:%.*]] = phi ptr [ [[LOAD_PTR_THEN_VAL]], [[DOTTHEN]] ], [ undef, [[TMP0:%.*]] ] +; CHECK-NEXT: ret void +; + %ub_ptr = alloca ptr + %select_ptr = select i1 %b, ptr %buffer, ptr %ub_ptr + %load_ptr = load ptr, ptr %select_ptr, !nonnull !1, !noundef !1 + ret void +} + +!1 = !{}