diff --git a/clang/lib/CodeGen/CodeGenPGO.cpp b/clang/lib/CodeGen/CodeGenPGO.cpp index 71bd1ca351df..24b035d67598 100644 --- a/clang/lib/CodeGen/CodeGenPGO.cpp +++ b/clang/lib/CodeGen/CodeGenPGO.cpp @@ -788,6 +788,8 @@ CodeGenPGO::applyFunctionAttributes(llvm::IndexedInstrProfReader *PGOReader, void CodeGenPGO::emitCounterIncrement(CGBuilderTy &Builder, unsigned Counter) { if (!CGM.getCodeGenOpts().ProfileInstrGenerate || !RegionCounterMap) return; + if (!Builder.GetInsertPoint()) + return; auto *I8PtrTy = llvm::Type::getInt8PtrTy(CGM.getLLVMContext()); Builder.CreateCall4(CGM.getIntrinsic(llvm::Intrinsic::instrprof_increment), llvm::ConstantExpr::getBitCast(FuncNameVar, I8PtrTy), diff --git a/clang/test/Profile/c-unreachable-after-switch.c b/clang/test/Profile/c-unreachable-after-switch.c new file mode 100644 index 000000000000..077043751058 --- /dev/null +++ b/clang/test/Profile/c-unreachable-after-switch.c @@ -0,0 +1,15 @@ +// RUN: %clang_cc1 -O3 -triple x86_64-apple-macosx10.10 -main-file-name c-unreachable-after-switch.c %s -o - -emit-llvm -fprofile-instr-generate | FileCheck %s + +// CHECK: @[[C:__llvm_profile_counters_foo]] = hidden global [3 x i64] zeroinitializer + +// CHECK-LABEL: @foo() +// CHECK: store {{.*}} @[[C]], i64 0, i64 0 +void foo() { + // CHECK: store {{.*}} @[[C]], i64 0, i64 2 + switch (0) { + default: + return; + } + // We shouldn't emit the unreachable counter. This used to crash in GlobalDCE. + // CHECK-NOT: store {{.*}} @[[SWC]], i64 0, i64 1} +}