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

[Sebastian Redl]

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.

Sebastian