[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:47:20 PDT 2017


grimar added a comment.

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

> I don't understand the problem you are trying to fix. Looks like the test is correct to me. It verifies that .{init,fini}_array sections are sorted.


I do not think it was intended that test will have 6 input section in total: 3 .init_array and 3 .fini_array.
I believe intention was to create 10 input sections, or am I missing something ?


https://reviews.llvm.org/D35552





More information about the llvm-commits mailing list