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 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
