[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cfe-dev
Subject:    Re: [cfe-dev] Confusing comments in PCHReader
From:       Douglas Gregor <dgregor () apple ! com>
Date:       2009-07-18 7:32:05
Message-ID: 48759AEB-916D-4A60-8993-639C57C12720 () apple ! com
[Download RAW message or body]


On Jul 7, 2009, at 5:03 PM, Zhongxing Xu wrote:

> In file PCHReader.cpp:1472: "Initialization of builtins and library
> builtins occurs before the ...". But actually we do not initialize any
> builtins if pch file is included, see clang-cc.cpp:2292.

Ah, right. This comment was written before the builtins were  
effectively moved into the PCH file (rather than being re-created each  
time the compiler is executed).

> Only keywords
> and pragmas are initialized at that point. Shall we change this
> comment to: "Initialization of keywords and pragmas occurs before
> ..."?

Yes, please.

> And since keywords can not be redefined, only pragmas can be
> repopulated. Is it right?


I believe that's right. One could look at the contents of the  
identifier table at this point in PCH loading to see if any other  
identifiers are sneaking in.

	- Doug
_______________________________________________
cfe-dev mailing list
cfe-dev@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
[prev in list] [next in list] [prev in thread] [next in thread]