[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