Index: llvm/trunk/lib/Transforms/Scalar/GVN.cpp =================================================================== --- llvm/trunk/lib/Transforms/Scalar/GVN.cpp +++ llvm/trunk/lib/Transforms/Scalar/GVN.cpp @@ -1362,6 +1362,11 @@ } markInstructionForDeletion(IntrinsicI); return false; + } else if (isa(V)) { + // If it's not false, and constant, it must evaluate to true. This means our + // assume is assume(true), and thus, pointless, and we don't want to do + // anything more here. + return false; } Constant *True = ConstantInt::getTrue(V->getContext()); Index: llvm/trunk/test/Transforms/GVN/pr34908.ll =================================================================== --- llvm/trunk/test/Transforms/GVN/pr34908.ll +++ llvm/trunk/test/Transforms/GVN/pr34908.ll @@ -0,0 +1,13 @@ +; NOTE: Assertions have been autogenerated by utils/update_test_checks.py +; RUN: opt < %s -gvn -S | FileCheck %s + +define i1 @foo() { +; CHECK-LABEL: @foo( +; CHECK-NEXT: call void @llvm.assume(i1 undef) +; CHECK-NEXT: ret i1 undef +; + call void @llvm.assume(i1 undef) + ret i1 undef +} + +declare void @llvm.assume(i1)