I'm working on gcc and binutils at the moment to add Thread Local Storage to both, if you all don't mind I'll also add the importing local symbol feature to my bucket list as well once I'm done with that
best regards, Julian On Mon, Sep 9, 2024 at 4:40 PM Pali Rohár <[email protected]> wrote: > On Monday 09 September 2024 09:40:41 Jacek Caban wrote: > > On 7.09.2024 00:00, Pali Rohár wrote: > > > So maybe the imp symbol could be included only for gcc/binutils builds? > > > > > > I think that we don't want to introduce differences like that. Ideally, > we > > should be able to mix tools with crt builds with another tools (so, for > > example, use crt built with GCC with clang+lld and the other way around). > > Martin in other email pointed that there are such differences due to > llvm's dlltool is different from binutils's dlltool (usage of short vs > long imports and non-usage of weak symbols). So as I understood, mixing > is currently not possible. > > But I agree with you that mixing could be nice feature to have. > > > > > As I mentioned, I think that it's better to prevent people from adding > bogus > > dllimport in the future code than silently accepting it. There is a > concern > > that such code may already exist and if it does, we shouldn't break it. > But > > the fact that UCRT was fine without the symbol for years suggests that > it's > > not a problem in practice. > > > > > > Jacek > > As I said, feel free to drop this imp symbol completely. It was just a > my idea that it could be useful. But I fully understand your concern > that it was not there before and seems that it is not needed in > practice. > > I agree with you that the best would be to teach binutils's ld to do > conversion with warning, like it is in msvc or clang/llvm. But this is > long-term... > > > _______________________________________________ > Mingw-w64-public mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/mingw-w64-public > _______________________________________________ Mingw-w64-public mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
