[PATCH] D48402: [mingw] Fix GCC ABI compatibility for comdat things
Pirama Arumuga Nainar via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 22 11:48:17 PDT 2018
pirama added a comment.
In https://reviews.llvm.org/D48402#1140046, @rnk wrote:
> In https://reviews.llvm.org/D48402#1140019, @pirama wrote:
>
> > Sections:
> > ....
> >
> > 3 .text$___Z3fooi 00000013 00000000 00000000 0000012c 2**4
> > CONTENTS, ALLOC, LOAD, RELOC, READONLY, CODE, LINK_ONCE_DISCARD
> >
> > Could that explain the difference?
>
>
> Woops, one too many calls to `getNameWithPrefix`.
I replaced `getMangler().getNameWithPrefix(SecName, Symbol, DL);` in `appendComdatSymbolForMinGW` with `SecName.append(Symbol.begin(), Symbol.end());`. This removes the extra underscore but still doesn't associate a comdat with the section. Is there anything else that I can try?
Repository:
rL LLVM
https://reviews.llvm.org/D48402
More information about the llvm-commits
mailing list