[PATCH] D152034: [clang][NFC] refactors value type traits so we can have more than bools

[Nick Desaulniers via Phabricator via cfe-commits]

nickdesaulniers added a subscriber: kazu.
nickdesaulniers added inline comments.


================
Comment at: clang/lib/Sema/SemaExprCXX.cpp:5591-5594
+  default:
+    llvm_unreachable("reached the end of BuildTypeTrait because the type "
+                     "trait's type is unaccounted for");
+  }
----------------
nickdesaulniers wrote:
> ```
> llvm-project/clang/lib/Sema/SemaExprCXX.cpp:5591:3: warning: default label in switch which covers all enumeration values [-Wcovered-switch-default]
>   default:
>   ^
> ```
commit caf00725923c ("[Sema] Fix a warning")
via @kazu 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D152034/new/

https://reviews.llvm.org/D152034