Author: krasimir
Date: Tue May  7 06:59:30 2019
New Revision: 360146

URL: http://llvm.org/viewvc/llvm-project?rev=360146&view=rev
Log:
[lldb] Add MacroQualified switch cases for r360109

Summary:
r360109 added a new enum case, causing lldb build to fail with several errors 
like:
lldb/source/Symbol/ClangASTContext.cpp:4342:11: error: enumeration value 
'MacroQualified' not handled in switch [-Werror,-Wswitch]
  switch (qual_type->getTypeClass()) {
          ^
This adds the missing switch cases.
I'm not an lldb maintainer and just used my best judgement that it's probably 
expected that we break in these cases. Feel free to ping / revert / fix this 
change if this behavior is not appropriate.

Reviewers: gribozavr

Reviewed By: gribozavr

Differential Revision: https://reviews.llvm.org/D61640

Modified:
    lldb/trunk/source/Symbol/ClangASTContext.cpp

Modified: lldb/trunk/source/Symbol/ClangASTContext.cpp
URL: 
http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Symbol/ClangASTContext.cpp?rev=360146&r1=360145&r2=360146&view=diff
==============================================================================
--- lldb/trunk/source/Symbol/ClangASTContext.cpp (original)
+++ lldb/trunk/source/Symbol/ClangASTContext.cpp Tue May  7 06:59:30 2019
@@ -4469,6 +4469,8 @@ ClangASTContext::GetTypeClass(lldb::opaq
 
   case clang::Type::DependentAddressSpace:
     break;
+  case clang::Type::MacroQualified:
+    break;
   }
   // We don't know hot to display this type...
   return lldb::eTypeClassOther;
@@ -5337,6 +5339,8 @@ lldb::Encoding ClangASTContext::GetEncod
 
   case clang::Type::DependentAddressSpace:
     break;
+  case clang::Type::MacroQualified:
+    break;
   }
   count = 0;
   return lldb::eEncodingInvalid;
@@ -5504,6 +5508,8 @@ lldb::Format ClangASTContext::GetFormat(
 
   case clang::Type::DependentAddressSpace:
     break;
+  case clang::Type::MacroQualified:
+    break;
   }
   // We don't know hot to display this type...
   return lldb::eFormatBytes;


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

Reply via email to