[Lldb-commits] [lldb] [lldb] Add unreachable after fully covered switches, avoid GCC warnings. NFC. (PR #159327)

[David Spickett via lldb-commits]

DavidSpickett wrote:

Taking one as an example:
```
  enum class FunctionNameRepresentation {
    eName,
    eNameWithArgs,
    eNameWithNoArgs
  };
```
And we do have a return for each of the cases.

So it feels like a limitation in GCC's analysis, is that right or did they make a decision to warn in these cases? I can see some value in making people mark the "unused" exit path like you might a fallthrough case.

If you have some background on it please add it to the PR description. I'll probably want to cite it again at some point :)

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