[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