[PATCH] D104721: [lld-macho] add tests for ICF, plus cleanups

Jez Ng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 22 16:05:23 PDT 2021


int3 added a comment.

Nice! Just one request: could we fix and test this too? https://reviews.llvm.org/D103292#inline-992625



================
Comment at: lld/test/MachO/icf.s:6
+# RUN: llvm-mc -filetype=obj -triple=x86_64-apple-darwin19.0.0 %t/abs.s -o %t/abs.o
+# RUN: %lld -pie -lSystem --icf=all -o %t/main %t/main.o %t/abs.o
 # RUN: llvm-objdump -d --syms %t/main | FileCheck %s
----------------
`-pie` shouldn't be necessary, `%lld` expands out to include a modern platform_version that implies PIE


================
Comment at: lld/test/MachO/icf.s:107
+
+### No fold: the absolute symbol reference differs
+
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104721



More information about the llvm-commits mailing list