[PATCH] Fix left shifts of negative integers in AArch64 InstPrinter/Disassembler

Alexey Samsonov vonosmas at gmail.com
Fri Aug 29 15:25:15 PDT 2014


Hi t.p.northover,

Left shift of negative integer is an undefined behavior, and
is reported by UBSan. It's ok for imm values to be negative, so we can
just replace left shifts with multiplications.

http://reviews.llvm.org/D5132

Files:
  lib/Target/AArch64/Disassembler/AArch64Disassembler.cpp
  lib/Target/AArch64/InstPrinter/AArch64InstPrinter.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5132.13117.patch
Type: text/x-patch
Size: 2082 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140829/71b97613/attachment.bin>


More information about the llvm-commits mailing list