[cfe-dev] Patch to allow comment translators implementation

Chris Lattner clattner at apple.com
Mon Jan 18 14:36:26 PST 2010


>> Il 12/01/2010 22:40, Chris Lattner ha scritto:
>>>> The only proposal I see that would not change a lot of things is to
>>>> remove Token argument from CommentHandler and to specify that they
>>>> always shall use EnterTokenStream also when they need to insert  
>>>> only one
>>>> token.
>>>> Then at end of Preprocessor::HandleComment if any of the comment
>>>> handlers has informed the caller to have inserted some tokens the  
>>>> first
>>>> one available is fetched and returned by  
>>>> Preprocessor::HandleComment.
>>>
>>> Yes, I think this is a good plan.
>>
>> You find attached a revised patch (based on r93512) that work as  
>> designed.
>>
>> The patch passes all clang tests.
>
> Looks fine to me. Chris?

Looks good to me, committed in r93795.  I didn't commit the lib/Lex/ 
PPDirectives.cpp hunk, because it didn't look necessary.  Please  
correct me if I'm wrong.

-Chris



More information about the cfe-dev mailing list