[llvm] r205171 - Try to fix MSan bootstrap bot: make ARM64Disassembler::getInstruction() always initialize Size argument.

Alexey Samsonov samsonov at google.com
Mon Mar 31 00:59:33 PDT 2014


Author: samsonov
Date: Mon Mar 31 02:59:33 2014
New Revision: 205171

URL: http://llvm.org/viewvc/llvm-project?rev=205171&view=rev
Log:
Try to fix MSan bootstrap bot: make ARM64Disassembler::getInstruction() always initialize Size argument.

Modified:
    llvm/trunk/lib/Target/ARM64/Disassembler/ARM64Disassembler.cpp

Modified: llvm/trunk/lib/Target/ARM64/Disassembler/ARM64Disassembler.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM64/Disassembler/ARM64Disassembler.cpp?rev=205171&r1=205170&r2=205171&view=diff
==============================================================================
--- llvm/trunk/lib/Target/ARM64/Disassembler/ARM64Disassembler.cpp (original)
+++ llvm/trunk/lib/Target/ARM64/Disassembler/ARM64Disassembler.cpp Mon Mar 31 02:59:33 2014
@@ -195,9 +195,11 @@ DecodeStatus ARM64Disassembler::getInstr
 
   uint8_t bytes[4];
 
+  Size = 0;
   // We want to read exactly 4 bytes of data.
   if (Region.readBytes(Address, 4, (uint8_t *)bytes) == -1)
     return Fail;
+  Size = 4;
 
   // Encoded as a small-endian 32-bit word in the stream.
   uint32_t insn =
@@ -209,8 +211,6 @@ DecodeStatus ARM64Disassembler::getInstr
   if (!result)
     return Fail;
 
-  Size = 4;
-
   return Success;
 }
 





More information about the llvm-commits mailing list