[PATCH] D14831: [ELF/AArch64] Accept -m aarch64linux.

Igor Kudrin via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 19 17:44:27 PST 2015


ikudrin added inline comments.

================
Comment at: test/ELF/emulation.s:167-173
@@ +166,9 @@
+# AARCH64-NEXT:   ]
+# AARCH64-NEXT:   HeaderSize: 64
+# AARCH64-NEXT:   ProgramHeaderEntrySize: 56
+# AARCH64-NEXT:   ProgramHeaderCount:
+# AARCH64-NEXT:   SectionHeaderEntrySize: 64
+# AARCH64-NEXT:   SectionHeaderCount:
+# AARCH64-NEXT:   StringTableSectionIndex:
+# AARCH64-NEXT: }
+
----------------
ruiu wrote:
> Remove these lines as the other test cases doesn't test these fields.
As far as I can see, both X86-64 and PPC64 have the same lines, X86 has similar lines and only MIPS and MIPSEL miss them.


http://reviews.llvm.org/D14831





More information about the llvm-commits mailing list