[PATCH] D142443: [ORC] Drop Comdat when discarding IR symbol

Jonas Hahnfeld via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 24 09:28:34 PST 2023


Hahnfeld added a comment.

In D142443#4077610 <https://reviews.llvm.org/D142443#4077610>, @sunho wrote:

> What happens when we actually have duplicate comdat any definitions? IIRC I've seen that in msvcrt.lib.

I'm not sure I understand your question. I think this is partially what https://reviews.llvm.org/D138264 is about? De-duplicating Comdat symbols on Linux doesn't seem to work as of now, hence the `weak` definition in `weak-comdat.ll` (but I'm not really into this Comdat business, not sure if a non-weak definition should work there).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142443



More information about the llvm-commits mailing list