[PATCH] D142484: [LinkerWrapper] Only import static libraries with needed symbols

Wed Mar 8 10:37:55 PST 2023

DiggerLin added a comment.

In D142484#4080202 <https://reviews.llvm.org/D142484#4080202>, @Jake-Egan wrote:

> In D142484#4079909 <https://reviews.llvm.org/D142484#4079909>, @jhuber6 wrote:
>
>> In D142484#4079869 <https://reviews.llvm.org/D142484#4079869>, @Jake-Egan wrote:
>>
>>> Hi, this new test fails on AIX. Could you take a look?
>>> https://lab.llvm.org/buildbot/#/builders/214/builds/5477/steps/6/logs/FAIL__Clang__linker-wrapper-libs_c
>>
>> I might just put that this test requires Linux. Because I don't think I can debug it without access to the system. We don't really support offloading on anything but Linux anyway.
>
> I think that makes sense. We'd need time to investigate it on AIX anyways.

I put a patch to fix the problem. https://reviews.llvm.org/D145600

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142484