[PATCH] D12619: [Static Analyzer] Minor cleanups for the nullability checker.
Daniel Marjamäki via cfe-commits
cfe-commits at lists.llvm.org
Fri Sep 4 06:20:07 PDT 2015
danielmarjamaki added a subscriber: danielmarjamaki.
danielmarjamaki added a comment.
Overall.. LGTM
================
Comment at: lib/StaticAnalyzer/Checkers/NullabilityChecker.cpp:78
@@ -78,2 +77,3 @@
+ llvm_unreachable("Unexpected enumeration.");
return "";
}
----------------
After llvm_unreachable there is often not a return.
http://reviews.llvm.org/D12619
More information about the cfe-commits
mailing list