diff options
| author | Richard Smith <richard-llvm@metafoo.co.uk> | 2012-08-24 23:29:28 +0000 |
|---|---|---|
| committer | Richard Smith <richard-llvm@metafoo.co.uk> | 2012-08-24 23:29:28 +0000 |
| commit | 1144af3c9b4da48cd581156e05b24261c8de366a (patch) | |
| tree | 49da576a97bfdab702528643450798baa54c790f /lib/Target/CellSPU/SPUAsmPrinter.cpp | |
| parent | cac59d8ae815596f4f6b77d1a5414c0591168ea5 (diff) | |
| download | external_llvm-1144af3c9b4da48cd581156e05b24261c8de366a.zip external_llvm-1144af3c9b4da48cd581156e05b24261c8de366a.tar.gz external_llvm-1144af3c9b4da48cd581156e05b24261c8de366a.tar.bz2 | |
Fix integer undefined behavior due to signed left shift overflow in LLVM.
Reviewed offline by chandlerc.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@162623 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/CellSPU/SPUAsmPrinter.cpp')
| -rw-r--r-- | lib/Target/CellSPU/SPUAsmPrinter.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/lib/Target/CellSPU/SPUAsmPrinter.cpp b/lib/Target/CellSPU/SPUAsmPrinter.cpp index 03d5a9a..3396e8b 100644 --- a/lib/Target/CellSPU/SPUAsmPrinter.cpp +++ b/lib/Target/CellSPU/SPUAsmPrinter.cpp @@ -130,8 +130,7 @@ namespace { void printS10ImmOperand(const MachineInstr *MI, unsigned OpNo, raw_ostream &O) { - short value = (short) (((int) MI->getOperand(OpNo).getImm() << 16) - >> 16); + short value = MI->getOperand(OpNo).getImm(); assert((value >= -(1 << 9) && value <= (1 << 9) - 1) && "Invalid s10 argument"); O << value; @@ -140,8 +139,7 @@ namespace { void printU10ImmOperand(const MachineInstr *MI, unsigned OpNo, raw_ostream &O) { - short value = (short) (((int) MI->getOperand(OpNo).getImm() << 16) - >> 16); + short value = MI->getOperand(OpNo).getImm(); assert((value <= (1 << 10) - 1) && "Invalid u10 argument"); O << value; } |