[PATCH] PR14795 : -frewrite-includes sometimes results in incorrect line number
Eli Friedman
eli.friedman at gmail.com
Mon Jul 15 17:29:43 PDT 2013
On Sun, Jul 14, 2013 at 2:16 PM, Lubos Lunak <l.lunak at suse.cz> wrote:
> On Monday 01 of July 2013, Eli Friedman wrote:
>> On Sat, Jun 29, 2013 at 11:35 PM, Lubos Lunak <l.lunak at suse.cz> wrote:
>> > Hello,
>> >
>> > could somebody please review and commit the atached patch for pr14795?
>> > Thank
>> > you.
>>
>> --- a/lib/Lex/Lexer.cpp
>> +++ b/lib/Lex/Lexer.cpp
>> @@ -2372,8 +2372,9 @@ bool Lexer::LexEndOfFile(Token &Result, const char
>> *CurPtr) {
>> FormTokenWithChars(Result, CurPtr, tok::eod);
>>
>> // Restore comment saving mode, in case it was disabled for directive.
>> - resetExtendedTokenMode();
>> - return true; // Have a token.
>> + if (PP)
>> + resetExtendedTokenMode();
>> + return true; // Have a token.
>> }
>>
>> // If we are in raw mode, return this event as an EOF token. Let the
>> caller
>>
>> How is this related?
>
> I had a crash there without this when I wrote the patch. I don't remember the
> circumstances though and the random testcase I tried doesn't trigger it.
Okay.
Please commit without this change.
-Eli
More information about the cfe-commits
mailing list