[PATCH] D73967: Implement _ExtInt as an extended int type specifier.

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 17 10:47:33 PDT 2020


erichkeane added a comment.

In D73967#1989145 <https://reviews.llvm.org/D73967#1989145>, @saugustine wrote:

> This change breaks the lldb build with
>
> lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp:3958:11: error: enumeration values 'DependentExtInt' and 'ExtInt' not handled in switch [-Werror,-Wswitch]
>
>   switch (qual_type->getTypeClass()) {
>           ^
>
> lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp:4633:11: error: enumeration values 'DependentExtInt' and 'ExtInt' not handled in switch [-Werror,-Wswitch]
>
>   switch (qual_type->getTypeClass()) {
>           ^
>
> lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp:4889:11: error: enumeration values 'DependentExtInt' and 'ExtInt' not handled in switch [-Werror,-Wswitch]
>
>   switch (qual_type->getTypeClass()) {
>   
>
> I would fix it, but I don't know the lldb code well enough.
>
> I am reverting this change shortly.


Also, can you share the buildbot that failed here?  I'd like to keep an eye on it after my fix.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73967





More information about the cfe-commits mailing list