On Oct 1, 2010, at 3:19 PM, Sebastian Redl wrote:

> 
> On Oct 1, 2010, at 1:02 PM, Douglas Gregor wrote:
> 
>>> +    /// \brief All the modules that loaded this one. Can contain NULL for
>>> +    /// directly loaded modules.
>>> +    llvm::SmallVector<PerFileData *, 1> Loaders;
>>> };
>> 
>> Do you really mean "can contain NULL" or do you mean "might be empty"?
> 
> I really mean can contain NULL. A NULL loader means the currently compiled 
> stuff, because there's no PerFileData for that. If this wasn't possible, 
> there would be no way to indicate that a module has been loaded both as a 
> dependency and directly.

Okay, thanks!

        - Doug
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to