[PATCH] D62350: ELF: Create synthetic sections for loadable partitions.
Peter Collingbourne via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 5 15:57:37 PDT 2019
pcc added inline comments.
================
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
----------------
grimar wrote:
> 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.
I looked into this a bit more and filed pr42145 and linked it here. Unfortunately fixing this isn't as simple as I first thought.
================
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:[^ ]*]]
----------------
grimar wrote:
> You do not need `[ 1]` I think.
I was using it to make sure that there were no section headers before `.dynsym`. But I've switched to using `CHECK-NEXT` here like I was doing elsewhere.
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