[llvm-commits] CVS: llvm/lib/CWriter/Writer.cpp
Chris Lattner
lattner at cs.uiuc.edu
Mon Jun 16 07:10:01 PDT 2003
Changes in directory llvm/lib/CWriter:
Writer.cpp updated: 1.92 -> 1.93
---
Log message:
Fix bug: CBackend/2003-06-11-HexConstant.ll
Fix bug: CBackend/2003-06-11-LiteralStringProblem.ll
---
Diffs of the changes:
Index: llvm/lib/CWriter/Writer.cpp
diff -u llvm/lib/CWriter/Writer.cpp:1.92 llvm/lib/CWriter/Writer.cpp:1.93
--- llvm/lib/CWriter/Writer.cpp:1.92 Fri Jun 6 02:10:24 2003
+++ llvm/lib/CWriter/Writer.cpp Mon Jun 16 07:09:09 2003
@@ -285,24 +285,35 @@
bool isString = (ETy == Type::SByteTy || ETy == Type::UByteTy);
// Make sure the last character is a null char, as automatically added by C
- if (CPA->getNumOperands() == 0 ||
- !cast<Constant>(*(CPA->op_end()-1))->isNullValue())
+ if (isString && (CPA->getNumOperands() == 0 ||
+ !cast<Constant>(*(CPA->op_end()-1))->isNullValue()))
isString = false;
if (isString) {
Out << "\"";
+ // Keep track of whether the last number was a hexadecimal escape
+ bool LastWasHex = false;
+
// Do not include the last character, which we know is null
for (unsigned i = 0, e = CPA->getNumOperands()-1; i != e; ++i) {
unsigned char C = (ETy == Type::SByteTy) ?
(unsigned char)cast<ConstantSInt>(CPA->getOperand(i))->getValue() :
(unsigned char)cast<ConstantUInt>(CPA->getOperand(i))->getValue();
- if (isprint(C)) {
+ // Print it out literally if it is a printable character. The only thing
+ // to be careful about is when the last letter output was a hex escape
+ // code, in which case we have to be careful not to print out hex digits
+ // explicitly (the C compiler things it is a continuation of the previous
+ // character, arg...)
+ //
+ if (isprint(C) && (!LastWasHex || !isxdigit(C))) {
+ LastWasHex = false;
if (C == '"' || C == '\\')
Out << "\\" << C;
else
Out << C;
} else {
+ LastWasHex = false;
switch (C) {
case '\n': Out << "\\n"; break;
case '\t': Out << "\\t"; break;
@@ -313,8 +324,9 @@
case '\'': Out << "\\\'"; break;
default:
Out << "\\x";
- Out << ( C/16 < 10) ? ( C/16 +'0') : ( C/16 -10+'A');
- Out << ((C&15) < 10) ? ((C&15)+'0') : ((C&15)-10+'A');
+ Out << (char)(( C/16 < 10) ? ( C/16 +'0') : ( C/16 -10+'A'));
+ Out << (char)(((C&15) < 10) ? ((C&15)+'0') : ((C&15)-10+'A'));
+ LastWasHex = true;
break;
}
}
More information about the llvm-commits
mailing list