MicroMIPS specific little endian fixup data byte ordering.
authorZoran Jovanovic <zoran.jovanovic@imgtec.com>
Thu, 3 Apr 2014 12:01:01 +0000 (12:01 +0000)
committerZoran Jovanovic <zoran.jovanovic@imgtec.com>
Thu, 3 Apr 2014 12:01:01 +0000 (12:01 +0000)
Differential Revision: http://llvm-reviews.chandlerc.com/D3245

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@205528 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/Mips/MCTargetDesc/MipsAsmBackend.cpp
test/MC/Mips/micromips-el-fixup-data.s [new file with mode: 0644]

index 541f8b6f91863d32102c790aa93f928136700f28..0f99ecc5484f74ea54446bb66d6f07e15f26c739 100644 (file)
@@ -112,6 +112,22 @@ MCObjectWriter *MipsAsmBackend::createObjectWriter(raw_ostream &OS) const {
     MCELFObjectTargetWriter::getOSABI(OSType), IsLittle, Is64Bit);
 }
 
+// Little-endian fixup data byte ordering:
+//   mips32r2:   a | b | x | x
+//   microMIPS:  x | x | a | b
+
+static bool needsMMLEByteOrder(unsigned Kind) {
+  return Kind >= Mips::fixup_MICROMIPS_26_S1 &&
+         Kind < Mips::LastTargetFixupKind;
+}
+
+// Calculate index for microMIPS specific little endian byte order
+static unsigned calculateMMLEIndex(unsigned i) {
+  assert(i <= 3 && "Index out of range!");
+
+  return (1 - i / 2) * 2 + i % 2;
+}
+
 /// ApplyFixup - Apply the \p Value for given \p Fixup into the provided
 /// data fragment, at the offset specified by the fixup and following the
 /// fixup kind as appropriate.
@@ -149,8 +165,12 @@ void MipsAsmBackend::applyFixup(const MCFixup &Fixup, char *Data,
   // Grab current value, if any, from bits.
   uint64_t CurVal = 0;
 
+  bool microMipsLEByteOrder = needsMMLEByteOrder((unsigned) Kind);
+
   for (unsigned i = 0; i != NumBytes; ++i) {
-    unsigned Idx = IsLittle ? i : (FullSize - 1 - i);
+    unsigned Idx = IsLittle ? (microMipsLEByteOrder ? calculateMMLEIndex(i)
+                                                    : i)
+                            : (FullSize - 1 - i);
     CurVal |= (uint64_t)((uint8_t)Data[Offset + Idx]) << (i*8);
   }
 
@@ -160,7 +180,9 @@ void MipsAsmBackend::applyFixup(const MCFixup &Fixup, char *Data,
 
   // Write out the fixed up bytes back to the code/data bits.
   for (unsigned i = 0; i != NumBytes; ++i) {
-    unsigned Idx = IsLittle ? i : (FullSize - 1 - i);
+    unsigned Idx = IsLittle ? (microMipsLEByteOrder ? calculateMMLEIndex(i)
+                                                    : i)
+                            : (FullSize - 1 - i);
     Data[Offset + Idx] = (uint8_t)((CurVal >> (i*8)) & 0xff);
   }
 }
diff --git a/test/MC/Mips/micromips-el-fixup-data.s b/test/MC/Mips/micromips-el-fixup-data.s
new file mode 100644 (file)
index 0000000..2293f63
--- /dev/null
@@ -0,0 +1,25 @@
+# RUN: llvm-mc %s -triple=mipsel-unknown-linux -mcpu=mips32r2 \
+# RUN:   -mattr=+micromips 2>&1 -filetype=obj > %t.o
+# RUN: llvm-objdump %t.o -triple mipsel -mattr=+micromips -d | FileCheck %s
+
+# Check that fixup data is writen in the microMIPS specific little endian
+# byte order.
+
+    .text
+    .globl  main
+    .align  2
+    .type   main,@function
+    .set    micromips
+    .set    nomips16
+    .ent    main
+main:
+    addiu   $sp, $sp, -16
+    bnez    $9, lab1
+
+# CHECK:    09 b4 04 00    bne $9, $zero, 8
+
+    addu    $zero, $zero, $zero
+lab1:
+    jr  $ra
+    addiu   $sp, $sp, 16
+    .end    main