[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