[PATCH] D50652: [libcxx] By default, do not use internal_linkage to hide symbols from the ABI

[Nico Weber via Phabricator via cfe-commits]

thakis added a comment.

I haven't read all the messages in these threads, forgive me if someone asked this already. It's a bit weird to me that we have to override this behavior in Chromium while the default is different. Why isn't the executable size blowup we see in chromium a problem for everyone else too? Is the plan to fix ld64's string pooling at the same time as rolling this change out, and this is just a workaround for people who have head libc++ but not head ld64?


Repository:
  rCXX libc++

https://reviews.llvm.org/D50652