aboutsummaryrefslogtreecommitdiffstats
path: root/lib/CodeGen
diff options
context:
space:
mode:
authorChandler Carruth <chandlerc@gmail.com>2012-01-30 23:47:44 +0000
committerChandler Carruth <chandlerc@gmail.com>2012-01-30 23:47:44 +0000
commitbeb05952ce27b4039c9d8bea929f154edeb19ca0 (patch)
treecdbd3f07ca67008566e6882bd78aaa7c3e8ab240 /lib/CodeGen
parent893a045cdbdc08fdaa5a62a24838be2df35cb628 (diff)
downloadexternal_llvm-beb05952ce27b4039c9d8bea929f154edeb19ca0.zip
external_llvm-beb05952ce27b4039c9d8bea929f154edeb19ca0.tar.gz
external_llvm-beb05952ce27b4039c9d8bea929f154edeb19ca0.tar.bz2
Chris's constant data sequence refactoring actually enabled printing
vectors of all one bits to be printed more cleverly in the AsmPrinter. Unfortunately, the byte value for all one bits is the same with -fsigned-char as the error return of '-1'. Force this to be the unsigned byte value when returning it to avoid this problem, and update the test case for the shiny new behavior. Yay for building LLVM and Clang with -funsigned-char. Chris, please review, and let me know if there is any reason to not desire this change. It seems good on the surface, and certainly intended based on the code written. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@149299 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen')
-rw-r--r--lib/CodeGen/AsmPrinter/AsmPrinter.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/CodeGen/AsmPrinter/AsmPrinter.cpp b/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
index 78f3f1d..1e55de4 100644
--- a/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
+++ b/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
@@ -1558,7 +1558,7 @@ static int isRepeatedByteSequence(const ConstantDataSequential *V) {
char C = Data[0];
for (unsigned i = 1, e = Data.size(); i != e; ++i)
if (Data[i] != C) return -1;
- return C;
+ return static_cast<uint8_t>(C); // Ensure 255 is not returned as -1.
}