diff options
author | David Majnemer <david.majnemer@gmail.com> | 2015-03-06 18:04:22 +0000 |
---|---|---|
committer | David Majnemer <david.majnemer@gmail.com> | 2015-03-06 18:04:22 +0000 |
commit | be09e8e5cf8495fc81cee39de028d6f62451fc03 (patch) | |
tree | 13d882289ed08a07418cc81a3fcf9a8946c3befa /clang/lib/AST | |
parent | ef648462d26c62898e338806c405a937d9f9bb1d (diff) | |
download | bcm5719-llvm-be09e8e5cf8495fc81cee39de028d6f62451fc03.tar.gz bcm5719-llvm-be09e8e5cf8495fc81cee39de028d6f62451fc03.zip |
Sema: The i8 suffix should yield a literal of type char
We would make i8 literals turn into signed char instead of char. This
is incompatible with MSVC.
This fixes PR22824.
llvm-svn: 231494
Diffstat (limited to 'clang/lib/AST')
-rw-r--r-- | clang/lib/AST/StmtPrinter.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/clang/lib/AST/StmtPrinter.cpp b/clang/lib/AST/StmtPrinter.cpp index d6e0debee14..8f0edd07568 100644 --- a/clang/lib/AST/StmtPrinter.cpp +++ b/clang/lib/AST/StmtPrinter.cpp @@ -1069,7 +1069,8 @@ void StmtPrinter::VisitIntegerLiteral(IntegerLiteral *Node) { // Emit suffixes. Integer literals are always a builtin integer type. switch (Node->getType()->getAs<BuiltinType>()->getKind()) { default: llvm_unreachable("Unexpected type for integer literal!"); - case BuiltinType::SChar: OS << "i8"; break; + case BuiltinType::Char_S: + case BuiltinType::Char_U: OS << "i8"; break; case BuiltinType::UChar: OS << "Ui8"; break; case BuiltinType::Short: OS << "i16"; break; case BuiltinType::UShort: OS << "Ui16"; break; |