Index: cfe/trunk/lib/CodeGen/CoverageMappingGen.cpp =================================================================== --- cfe/trunk/lib/CodeGen/CoverageMappingGen.cpp +++ cfe/trunk/lib/CodeGen/CoverageMappingGen.cpp @@ -716,6 +716,16 @@ terminateRegion(S); } + void VisitCallExpr(const CallExpr *E) { + VisitStmt(E); + + // Terminate the region when we hit a noreturn function. + // (This is helpful dealing with switch statements.) + QualType CalleeType = E->getCallee()->getType(); + if (getFunctionExtInfo(*CalleeType).getNoReturn()) + terminateRegion(E); + } + void VisitWhileStmt(const WhileStmt *S) { extendRegion(S); Index: cfe/trunk/test/CoverageMapping/md.cpp =================================================================== --- cfe/trunk/test/CoverageMapping/md.cpp +++ cfe/trunk/test/CoverageMapping/md.cpp @@ -27,6 +27,17 @@ #include "Inputs/md.def" } +// CHECK: bar +// CHECK-NEXT: File 0, [[@LINE+3]]:12 -> [[@LINE+8]]:2 = #0 +bool isVal1(); +bool isVal2(); +bool bar() { + #define HANDLE_MD(X) is##X() || + return +#include "Inputs/md.def" + 0; +} + int main(int argc, const char *argv[]) { foo(MD::Val1); return 0; Index: cfe/trunk/test/CoverageMapping/switch.cpp =================================================================== --- cfe/trunk/test/CoverageMapping/switch.cpp +++ cfe/trunk/test/CoverageMapping/switch.cpp @@ -97,3 +97,16 @@ break; } } + +void abort(void) __attribute((noreturn)); + // CHECK: noret +int noret(int x) { // CHECK-NEXT: File 0, [[@LINE]]:18 -> [[@LINE+9]]:2 + switch (x) { + default: // CHECK-NEXT: File 0, [[@LINE]]:3 -> [[@LINE+1]]:12 + abort(); + case 1: // CHECK-NEXT: File 0, [[@LINE]]:3 -> [[@LINE+1]]:13 + return 5; + case 2: // CHECK-NEXT: File 0, [[@LINE]]:3 -> [[@LINE+1]]:14 + return 10; + } +}