[PATCH] D105210: [lld-macho] Ignore debug symbols for now.
    Jez Ng via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Jul  1 15:22:16 PDT 2021
    
    
  
int3 added a comment.
Ah, `ld -r`... that bit us before in D104502: [lld-macho] Handle non-extern symbols marked as private extern <https://reviews.llvm.org/D104502> :)
> with a checked-in .o file, unless someone can tell me a way to produce it within the test
Let's use yaml2obj instead. Other LLVM tool tests use raw .o files sometimes, but I get the sense that it's a legacy decision and people are pushing back on further additions.
As a bonus, you can inline the YAML within the test file via `split-file --no-leading-lines`, see e.g. `dylink-ordinal.s`.
================
Comment at: lld/test/MachO/bug_50812.s:5
+
+# %lld -lSystem -platform_version macos 11.3 11.0 -arch x86_64  %S/Inputs/bug_50812.o
----------------
you're missing a RUN: here (which the test infra has rightly noted)
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105210/new/
https://reviews.llvm.org/D105210
    
    
More information about the llvm-commits
mailing list