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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to