[PATCH] D35552: [ELF] - Fix init_fini_priority.s test.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 20 10:59:08 PDT 2017


grimar added a comment.

In https://reviews.llvm.org/D35552#816422, @ruiu wrote:

> What I don't really understand is that you changed the test input but didn't change the expected output, so this change doesn't demonstrate what was wrong and what it should have been.


I can add a check of llvm-mc output to validate that we have 5+5 input sections now and show initial order.
I did not do that because usually we do not check llvm-mc output, but only linker result which is the same with or without patch.


https://reviews.llvm.org/D35552





More information about the llvm-commits mailing list