[PATCH] D98445: [ELF] Special case --shuffle-sections=-1 to reverse input sections
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 15 19:38:00 PDT 2021
MaskRay marked 2 inline comments as done.
MaskRay added inline comments.
================
Comment at: lld/test/ELF/shuffle-sections.s:30
+## As a special case, -1 reverses sections as a stable transform.
+# RUN: ld.lld --symbol-ordering-file %t_order.txt --shuffle-sections=-1 %t.o -o %t-1.out
+# RUN: llvm-readelf -x .text %t-1.out | FileCheck %s --check-prefix=SHUFFLE3
----------------
jhenderson wrote:
> Do you need the `--symbol-ordering-file` option here? It seems like it's unrelated to testing whether -1 reverses the section order. If there's an interaction you want to test between the two, perhaps we should have a case with and a case without.
The -1 test does not need it. I'll delete it.
`--symbol-ordering-file` has been tested by the lines above so the interaction does not need re-test.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D98445/new/
https://reviews.llvm.org/D98445
More information about the llvm-commits
mailing list