Conflicting warnings when handling enums in a switch with -Weverything

[StreetwareGames]

foo() warns about a default case missing, while foo2() warns about a default case being present when every enum case is already handled.

https://godbolt.org/z/WxooaoeMK

[llvmbot]

@llvm/issue-subscribers-clang-frontend

Author: None (SewerDweller)

foo() warns about a default case missing, while foo2() warns about a default case being present when every enum case is already handled.

https://godbolt.org/z/WxooaoeMK

[cor3ntin]

Hum, this is actually consistent with GCC https://godbolt.org/z/re8Y5W1Wv and it is a separate flag from -Wswitch.
I now remember that it was the intent of this flag (and note that it is not enabled under -Wall, -Wextra

[cor3ntin]

This was added by #73077

[jyknight]

This is correct. Some users want one, and some want the other, so both are provided.

It is expected behavior for -Weverything to include contradicting diagnostics, see the docs. As the docs also mention, it's not recommended to use -Weverything.