[clang] [Clang][Analysis] Don't treat the default switch path as unreachable when all enumerators are covered (PR #123166)

[Jeremy Rifkin via cfe-commits]

================
@@ -170,16 +170,14 @@ void test_nested_switch() {
   }
 }
 
-// Test that if all the values of an enum covered, that the 'default' branch
-// is unreachable.
+// Test that a warning is not emitted if the code is unreachable.
 enum Values { A, B, C, D };
 void test_all_enums_covered(enum Values v) {
----------------
jeremy-rifkin wrote:

It looks like there's another test case that does effectively make an unreachable branch, I'll update this case as you suggested

https://github.com/llvm/llvm-project/pull/123166