[PATCH] D54251: [MSP430] Fix encodeInstruction() for big endian hosts

[Michael Skvortsov via Phabricator via llvm-commits]

mskvortsov created this revision.
mskvortsov added a reviewer: asl.
Herald added a subscriber: llvm-commits.

Repository:
  rL LLVM

https://reviews.llvm.org/D54251

Files:
  lib/Target/MSP430/MCTargetDesc/MSP430MCCodeEmitter.cpp


Index: lib/Target/MSP430/MCTargetDesc/MSP430MCCodeEmitter.cpp
===================================================================
--- lib/Target/MSP430/MCTargetDesc/MSP430MCCodeEmitter.cpp
+++ lib/Target/MSP430/MCTargetDesc/MSP430MCCodeEmitter.cpp
@@ -91,12 +91,11 @@
   Offset = 2;
 
   uint64_t BinaryOpCode = getBinaryCodeForInstr(MI, Fixups, STI);
-  const uint16_t *Words = reinterpret_cast<uint16_t const *>(&BinaryOpCode);
   size_t WordCount = Size / 2;
 
-  for (size_t i = 0; i < WordCount; ++i) {
-    uint16_t Word = Words[i];
-    support::endian::write(OS, Word, support::little);
+  while (WordCount--) {
+    support::endian::write(OS, (uint16_t)BinaryOpCode, support::little);
+    BinaryOpCode >>= 16;
   }
 }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54251.173131.patch
Type: text/x-patch
Size: 730 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181108/732b118d/attachment.bin>