On Mon, 8 Mar 2021 19:32:10 +0800 Bin Meng <bmeng...@gmail.com> wrote:
> On Mon, Mar 8, 2021 at 7:18 PM Marek Behun <marek.be...@nic.cz> wrote: > > > > On Mon, 8 Mar 2021 18:44:58 +0800 > > Bin Meng <bmeng...@gmail.com> wrote: > > > > > Could you investigate why? > > > > I could, but I don't understand why exactly I should > > - Linux is also using --whole-archive > > - it is working > > - I have other things I would like to work on > > > > Maybe you could look into this? :) > > Yes, I can look into this. I wonder if you already knew this which > could save some time as this is a normal review process, asking for > clarifications if something isn't clear. If I knew what? Whether it is used in Linux also? To be honest I wasn't sure, I looked into Linux' sources before replying. The thing is, when I first worked on these patches, I just started with some old LTO patches for Linux (which were not even merged then), and played with the options until LTO worked for sandbox. Then the things started to get complicated and I rebased the series many times, so I did not remember which came from where. Marek
