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

[Igor Kudrin via llvm-commits]

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:
> ikudrin wrote:
> > 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.
> Then maybe we should remove from them too? I think the purpose of this test is to test is Machine field, so checking values of miscellaneous Size fields seem excessive.
OK, but I don't think that cleaning the other cases is relevant to this patch.


http://reviews.llvm.org/D14831