[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