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

[Erich Keane via Phabricator via cfe-commits]

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.


I think I know how to fix the LLDB code, can you hold off on reverting and let me take a look?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73967