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

[Douglas Gregor]

On Oct 1, 2010, at 12:59 PM, Sebastian Redl wrote:

> Author: cornedbee
> Date: Fri Oct  1 14:59:12 2010
> New Revision: 115334
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=115334&view=rev
> Log:
> Record module loaders and module source order.
> 
> Modified:
>    cfe/trunk/include/clang/Serialization/ASTReader.h
>    cfe/trunk/lib/Serialization/ASTReader.cpp
> 
> Modified: cfe/trunk/include/clang/Serialization/ASTReader.h
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Serialization/ASTReader.h?rev=115334&r1=115333&r2=115334&view=diff
> ==============================================================================
> --- cfe/trunk/include/clang/Serialization/ASTReader.h (original)
> +++ cfe/trunk/include/clang/Serialization/ASTReader.h Fri Oct  1 14:59:12 2010
> @@ -193,7 +193,7 @@
>   /// \brief The AST consumer.
>   ASTConsumer *Consumer;
> 
> -  /// \brief Information that is needed for every file in the chain.
> +  /// \brief Information that is needed for every module.
>   struct PerFileData {
>     PerFileData();
>     ~PerFileData();
> @@ -318,12 +318,25 @@
>     ///
>     /// The dynamic type of this stat cache is always ASTStatCache
>     void *StatCache;
> -      
> +
>     /// \brief The number of preallocated preprocessing entities in the
>     /// preprocessing record.
>     unsigned NumPreallocatedPreprocessingEntities;
> +
> +    /// \brief The next module in source order.
> +    PerFileData *NextInSource;
> +
> +    /// \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"?

	- Doug