[PATCH] D24830: [ELF] - Linkerscript: implemented BYTE/SHORT/LONG/QUAD commands.

[Rafael Ávila de Espíndola via llvm-commits]

rafael accepted this revision.
rafael added a comment.
This revision is now accepted and ready to land.

LGTM with nits, but wait so see if Rui is OK with it too.


================
Comment at: test/ELF/linkerscript/data-commands.s:2
@@ +1,3 @@
+# REQUIRES: x86
+# RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux %s -o %t.o
+
----------------
Add a big endian test.

================
Comment at: test/ELF/linkerscript/data-commands.s:21
@@ +20,3 @@
+# CHECK:      Contents of section .foo:
+# CHECK-NEXT:  00e8 ff11ff22 11ff4433 2211ff88 77665544
+# CHECK-NEXT:  00f8 332211
----------------
Don't check the address.


https://reviews.llvm.org/D24830