[PATCH] D52344: [Clang][CodeGen][ObjC]: Fix non-bridged CoreFoundation builds on ELF targets that use `-fconstant-cfstrings`.

Kristina Brooks via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 24 07:20:47 PDT 2018


kristina added a comment.

In https://reviews.llvm.org/D52344#1243149, @rjmccall wrote:

> I can respect wanting to change the rules on ELF, but it sounds like we do need to stick with the current section names.  Absent an ABI break to stop looking for the existing section names, CF will misbehave if they exist even if we change the compiler output, so effectively those identifiers are claimed in a way that cannot be incrementally changed.
>
> Also, it's probably correct for CF on non-Darwin OSes to stick to non-system namespaces anyway.


Yes makes sense, I kept it as `cfstring`, should be in that revision.


Repository:
  rC Clang

https://reviews.llvm.org/D52344





More information about the cfe-commits mailing list