Skip to content

Commit

Permalink
[lldb][NFC] Fix warning about missing switch cases
Browse files Browse the repository at this point in the history
These types were recently added in D62960 but it seems the patch didn't
consider LLDB which causes a bunch of compiler warnings about
missing enum values. It seems this feature isn't fully implemented yet,
so I don't think we can write any test for this. For now lets just add
the missing types to our usual list of unsupported types.

llvm-svn: 368424
  • Loading branch information
Teemperor committed Aug 9, 2019
1 parent 01eb869 commit 339b5d1
Showing 1 changed file with 14 additions and 0 deletions.
14 changes: 14 additions & 0 deletions lldb/source/Symbol/ClangASTContext.cpp
Expand Up @@ -5258,6 +5258,20 @@ lldb::Encoding ClangASTContext::GetEncoding(lldb::opaque_compiler_type_t type,
case clang::BuiltinType::OCLIntelSubgroupAVCImeSingleRefStreamin:
case clang::BuiltinType::OCLIntelSubgroupAVCImeDualRefStreamin:
break;

case clang::BuiltinType::SveBool:
case clang::BuiltinType::SveInt8:
case clang::BuiltinType::SveInt16:
case clang::BuiltinType::SveInt32:
case clang::BuiltinType::SveInt64:
case clang::BuiltinType::SveUint8:
case clang::BuiltinType::SveUint16:
case clang::BuiltinType::SveUint32:
case clang::BuiltinType::SveUint64:
case clang::BuiltinType::SveFloat16:
case clang::BuiltinType::SveFloat32:
case clang::BuiltinType::SveFloat64:
break;
}
break;
// All pointer types are represented as unsigned integer encodings. We may
Expand Down

0 comments on commit 339b5d1

Please sign in to comment.