Teach -Wunreachable-code about dead code caused by macro expansions. This should suppress false positives resulting from 'assert' and friends.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@138576 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/Analysis/ReachableCode.cpp b/lib/Analysis/ReachableCode.cpp
index e3194cb..4931771 100644
--- a/lib/Analysis/ReachableCode.cpp
+++ b/lib/Analysis/ReachableCode.cpp
@@ -86,12 +86,10 @@
}
static bool isValidDeadStmt(const Stmt *S) {
- SourceLocation Loc = S->getLocStart();
- if (!(Loc.isValid() && !Loc.isMacroID()))
+ if (S->getLocStart().isInvalid())
return false;
- if (const BinaryOperator *BO = dyn_cast<BinaryOperator>(S)) {
+ if (const BinaryOperator *BO = dyn_cast<BinaryOperator>(S))
return BO->getOpcode() != BO_Comma;
- }
return true;
}
@@ -144,6 +142,12 @@
}
continue;
}
+
+ // Specially handle macro-expanded code.
+ if (S->getLocStart().isMacroID()) {
+ count += clang::reachable_code::ScanReachableFromBlock(Block, Reachable);
+ continue;
+ }
if (isDeadCodeRoot(Block)) {
reportDeadCode(S, CB);
diff --git a/test/Sema/warn-unreachable.c b/test/Sema/warn-unreachable.c
index 80f32cd..8db36b7 100644
--- a/test/Sema/warn-unreachable.c
+++ b/test/Sema/warn-unreachable.c
@@ -114,3 +114,15 @@
}
}
+// Handle unreachable code triggered by macro expansions.
+void __myassert_rtn(const char *, const char *, int, const char *) __attribute__((__noreturn__));
+
+#define myassert(e) \
+ (__builtin_expect(!(e), 0) ? __myassert_rtn(__func__, __FILE__, __LINE__, #e) : (void)0)
+
+void test_assert() {
+ myassert(0 && "unreachable");
+ return; // no-warning
+}
+
+