[PATCH] D134337: [clang] [Driver] More flexible rules for loading default configs (WIP)

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 21 12:44:37 PDT 2022


MaskRay added inline comments.


================
Comment at: clang/lib/Driver/Driver.cpp:6195
+
+  assert(false && "Unhandled Mode");
+  return "clang";
----------------
If the switch covers all enum members, we typically use llvm_unreachable to work around a GCC -Wswitch warning.
Clang correctly knows all branches are covered and do not need `llvm_unreachable`


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

https://reviews.llvm.org/D134337



More information about the cfe-commits mailing list