george.karpenkov added a comment.
I'm not sure why you could get away with removing those llvm_unreachable cases?
================
Comment at: lib/StaticAnalyzer/Core/AnalyzerOptions.cpp:373
- default:
- llvm_unreachable("Invalid mode.");
case UMK_Shallow:
----------------
Why?
================
Comment at: lib/StaticAnalyzer/Core/AnalyzerOptions.cpp:418
- default:
- llvm_unreachable("Invalid mode.");
case UMK_Shallow:
----------------
Why?
================
Comment at: lib/StaticAnalyzer/Core/CoreEngine.cpp:72
- default:
- llvm_unreachable("Unexpected case");
}
----------------
Why?
https://reviews.llvm.org/D53274
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
