[cfe-dev] ThreadSpecified visibility in VarDecl
Kevin Tew
tewk at tewk.com
Mon Sep 8 11:01:18 PDT 2008
Attempt #2
Kevin
Daniel Dunbar wrote:
> Two comments:
>
> (1) This should be a bitfield with width 1:
> + bool ThreadSpecified;
>
> (2) These should be formatted to fit in 80-columns:
> + : ValueDecl(DK, DC, L, Id, T, PrevDecl), Init(0),
> ThreadSpecified(false) { SClass = SC; }
> and
> + NewVD = VarDecl::Create(Context, CurContext,
> D.getIdentifierLoc(), II, R, SC, LastDeclarator);
> + NewVD->setThreadSpecified(ThreadSpecified);
>
> - Daniel
>
> On Mon, Sep 8, 2008 at 9:50 AM, Kevin Tew <tewk at tewk.com> wrote:
>
>> Makes ThreadSpecified visible during AST traversals.
>>
>> Kevin Tew
>>
>>
>> _______________________________________________
>> cfe-dev mailing list
>> cfe-dev at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
>>
>>
>>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vardecl_threadspecified_2.patch
Type: text/x-diff
Size: 2716 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20080908/2ec966a4/attachment.patch>
More information about the cfe-dev
mailing list