[PATCH] D49700: [LLD] [COFF] Treat .xdata/.pdata$<sym> as implicitly associative to <sym> for MinGW

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 24 15:01:36 PDT 2018


mstorsjo added a comment.

In https://reviews.llvm.org/D49700#1174154, @ruiu wrote:

> Sorry, I didn't follow the discussion as to how much important it is to be compatible with gcc, but I guess that's important for you? I don't think that the new code you are adding is not complicated, so as long as we can keep the overall complexity of these functions low, adding new code should be fine.


No, compatibility with gcc produced object files isn't very important to me here.

There's currently three (more or less) commonly used mingw configurations:

- gcc + ld.bfd (the normal default one)
- clang + ld.bfd (@rnk fixed an issue for this case in llvm, which triggered all of this)
- clang + lld (the one I mostly work on)

The fourth plausible combo would be gcc + lld, but I don't know of anybody using that, and I'm pretty sure there are other issues that one would run into in that case.


https://reviews.llvm.org/D49700





More information about the llvm-commits mailing list