[PATCH] D62350: ELF: Create synthetic sections for loadable partitions.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 5 01:05:20 PDT 2019


grimar added a comment.

I'd suggest adding a short description to each test case added.



================
Comment at: lld/test/ELF/partition-synthetic-sections.s:17
+
+// FIXME: llvm-objcopy does not preserve padding so for now we check the combined output file.
+// RUN: od -Ax -x %t | FileCheck %s -check-prefix=FILL
----------------
Was issue about that reported? We have a good practice in LLVM tools to add an URL of the issue
for each TODO in the test. That helps to cleanup such things a lot.


================
Comment at: lld/test/ELF/partition-synthetic-sections.s:23
+// CHECK: Section Headers:
+// CHECK: [ 1] .dynsym           DYNSYM          {{0*}}[[DYNSYM_ADDR:[^ ]*]]
+// CHECK-NEXT: .gnu.version      VERSYM          {{0*}}[[VERSYM_ADDR:[^ ]*]]
----------------
You do not need `[ 1]` I think.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62350/new/

https://reviews.llvm.org/D62350





More information about the llvm-commits mailing list