[cfe-commits] r115334 - in /cfe/trunk: include/clang/Serialization/ASTReader.h lib/Serialization/ASTReader.cpp

[Douglas Gregor]

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