Index: lib/Transforms/InstCombine/InstCombineCalls.cpp =================================================================== --- lib/Transforms/InstCombine/InstCombineCalls.cpp +++ lib/Transforms/InstCombine/InstCombineCalls.cpp @@ -2243,7 +2243,10 @@ break; } case Intrinsic::lifetime_start: - if (removeTriviallyEmptyRange(*II, Intrinsic::lifetime_start, + // Asan needs to poison memory to detect invalid access which is possible + // even for empty lifetime range. + if (!II->getFunction()->hasFnAttribute(Attribute::SanitizeAddress) && + removeTriviallyEmptyRange(*II, Intrinsic::lifetime_start, Intrinsic::lifetime_end, *this)) return nullptr; break; Index: test/Transforms/InstCombine/lifetime-asan.ll =================================================================== --- /dev/null +++ test/Transforms/InstCombine/lifetime-asan.ll @@ -0,0 +1,35 @@ +; RUN: opt < %s -instcombine -S | FileCheck %s + +declare void @llvm.lifetime.start(i64, i8* nocapture) +declare void @llvm.lifetime.end(i64, i8* nocapture) +declare void @foo(i8* nocapture) + +define void @asan() sanitize_address { +entry: + ; CHECK-LABEL: @asan( + %text = alloca i8, align 1 + + call void @llvm.lifetime.start(i64 1, i8* %text) + call void @llvm.lifetime.end(i64 1, i8* %text) + ; CHECK: call void @llvm.lifetime.start + ; CHECK-NEXT: call void @llvm.lifetime.end + + call void @foo(i8* %text) ; Keep alloca alive + + ret void +} + + +define void @no_asan() { +entry: + ; CHECK-LABEL: @no_asan( + %text = alloca i8, align 1 + + call void @llvm.lifetime.start(i64 1, i8* %text) + call void @llvm.lifetime.end(i64 1, i8* %text) + ; CHECK-NO: call void @llvm.lifetime + + call void @foo(i8* %text) ; Keep alloca alive + + ret void +}