kolan
/
CMake
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

cmGeneratorTarget: Trace cmSourceFile objects instead of strings. 

This reverses the decision in commit d38423ec (cmTarget: Add a
method to obtain list of filenames for sources., 2014-03-17).  The
cmSourceFile based API is preferred because that avoids creation of
many cmSourceFileLocation objects for matching strings, and the
result is cached by cmTarget.
This commit is contained in:
Stephen Kelly 2014-04-05 11:41:58 +02:00
parent c5b26f3bec
commit 92e2fbe103
1 changed files with 15 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
Source/cmGeneratorTarget.cxx
View File

@ -606,12 +606,12 @@ private:
cmGlobalGenerator const* GlobalGenerator;
typedef cmGeneratorTarget::SourceEntry SourceEntry;
SourceEntry* CurrentEntry;
std::queue<std::string> SourceQueue;
std::set<std::string> SourcesQueued;
std::queue<cmSourceFile*> SourceQueue;
std::set<cmSourceFile*> SourcesQueued;
typedef std::map<std::string, cmSourceFile*> NameMapType;
NameMapType NameMap;
void QueueSource(std::string const& name);
void QueueSource(cmSourceFile* sf);
void FollowName(std::string const& name);
void FollowNames(std::vector<std::string> const& names);
bool IsUtility(std::string const& dep);
@ -642,19 +642,19 @@ cmTargetTraceDependencies
{
configs.push_back("");
}
std::set<std::string> emitted;
std::set<cmSourceFile*> emitted;
for(std::vector<std::string>::const_iterator ci = configs.begin();
ci != configs.end(); ++ci)
{
std::vector<std::string> sources;
std::vector<cmSourceFile*> sources;
this->Target->GetSourceFiles(sources, *ci);
for(std::vector<std::string>::const_iterator si = sources.begin();
for(std::vector<cmSourceFile*>::const_iterator si = sources.begin();
si != sources.end(); ++si)
{
if(emitted.insert(*si).second && this->SourcesQueued.insert(*si).second)
cmSourceFile* sf = *si;
if(emitted.insert(sf).second && this->SourcesQueued.insert(sf).second)
{
this->SourceQueue.push(*si);
this->Makefile->GetOrCreateSource(*si);
this->SourceQueue.push(sf);
}
}
}
@ -673,8 +673,7 @@ void cmTargetTraceDependencies::Trace()
while(!this->SourceQueue.empty())
{
// Get the next source from the queue.
std::string src = this->SourceQueue.front();
cmSourceFile* sf = this->Makefile->GetSource(src);
cmSourceFile* sf = this->SourceQueue.front();
this->SourceQueue.pop();
this->CurrentEntry = &this->GeneratorTarget->SourceEntries[sf];
@ -702,14 +701,14 @@ void cmTargetTraceDependencies::Trace()
}
//----------------------------------------------------------------------------
void cmTargetTraceDependencies::QueueSource(std::string const& name)
void cmTargetTraceDependencies::QueueSource(cmSourceFile* sf)
{
if(this->SourcesQueued.insert(name).second)
if(this->SourcesQueued.insert(sf).second)
{
this->SourceQueue.push(name);
this->SourceQueue.push(sf);
// Make sure this file is in the target.
this->Target->AddSource(name);
this->Target->AddSource(sf->GetFullPath());
}
}
@ -731,7 +730,7 @@ void cmTargetTraceDependencies::FollowName(std::string const& name)
{
this->CurrentEntry->Depends.push_back(sf);
}
this->QueueSource(sf->GetFullPath());
this->QueueSource(sf);
}
}