Skip to content

Commit

Permalink
Supply missing break in case statement.
Browse files Browse the repository at this point in the history
Summary:
All the tests pass without hitting the situation mentioned in the FIXME, so,
per Aaron Smith's suggestion, this case will now return unconditionally.

Subscribers: sanjoy, mgorny, JDevlieghere

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

llvm-svn: 325188
  • Loading branch information
amccarth-google committed Feb 14, 2018
1 parent b39bcc0 commit 1e368d1
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions lldb/source/Plugins/SymbolFile/PDB/PDBASTParser.cpp
Expand Up @@ -107,9 +107,7 @@ CompilerType GetBuiltinTypeForPDBEncodingAndBitSize(
case PDB_BuiltinType::None:
return CompilerType();
case PDB_BuiltinType::Void:
// FIXME: where is non-zero size of `void` from?
if (width == 0)
return clang_ast->GetBasicType(eBasicTypeVoid);
return clang_ast->GetBasicType(eBasicTypeVoid);
case PDB_BuiltinType::Bool:
return clang_ast->GetBasicType(eBasicTypeBool);
case PDB_BuiltinType::Long:
Expand Down

0 comments on commit 1e368d1

Please sign in to comment.