[llvm-dev] Question about __NSConstantString and __NSConstantString_tag
Ben Langmuir via llvm-dev
llvm-dev at lists.llvm.org
Thu Feb 25 08:41:10 PST 2016
> On Feb 25, 2016, at 7:49 AM, Brad King <brad.king at kitware.com> wrote:
> On 02/19/2016 10:35 AM, Ben Langmuir wrote:
>>> On Feb 19, 2016, at 7:16 AM, Brad King wrote:
>>>> __NSConstantString and __NSConstantString_tag are now exposed.
>>> To clarify, this is not so much about the appearance of new builtins
>>> but that the structure has fields with no name. Other builtin
>>> structures like __va_list_tag have names on their fields. FieldDecl
>>> derives (indirectly) from NamedDecl.
>>> Our concern is that AST processing tools expect to find a name on
>>> fields, and the changes in question introduced fields without names.
>>> Can they be given names, if only for AST consistency?
>> Yes, sounds like a good idea. I’ll take a look at this later today.
> Here is a patch that adds the field names.
I’m really sorry I totally forgot about this! Thanks for the patch. Committed as r261887
More information about the llvm-dev