JonasToth added a comment.

F5426271: llvm_lib_target_x86_paths <https://reviews.llvm.org/F5426271>
Example output for `llvm/lib/Target/X86`
Running it over the whole `llvm/lib` codebase generates a lot of warnings. 
Please note, that it seems to be common to write code like this:

  int Val;
  switch(Val) {
    case 1: // something
    case 2: // something else
    case 16: // magic
  }
  llvm_unreachable("reason");

In some cases it has been taken care that no fallthrough happens, but it's not 
so simple to spot. Using `default: llvm_unreachable("reason");` would comply 
with the check.


https://reviews.llvm.org/D37808



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to