diff options
author | Brad King <brad.king@kitware.com> | 2023-03-30 11:18:55 (GMT) |
---|---|---|
committer | Kitware Robot <kwrobot@kitware.com> | 2023-03-30 11:19:05 (GMT) |
commit | dfe32d797e82b48d38648f580a621fe99c2405d4 (patch) | |
tree | f4fa3a4eababacf245f0aa6918ec21e63452e380 /Source/cmGlobalNinjaGenerator.cxx | |
parent | 2f3ba93102d1a3e6f4674b27b95853ea613d4bf6 (diff) | |
parent | 426f3295f61eaf2feb58c7e4b9ebe838b21cf453 (diff) | |
download | CMake-dfe32d797e82b48d38648f580a621fe99c2405d4.zip CMake-dfe32d797e82b48d38648f580a621fe99c2405d4.tar.gz CMake-dfe32d797e82b48d38648f580a621fe99c2405d4.tar.bz2 |
Merge topic 'ninja-performance'
426f3295f6 Ninja: Use more efficient data structures to collect outputs
Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: buildbot <buildbot@kitware.com>
Merge-request: !8371
Diffstat (limited to 'Source/cmGlobalNinjaGenerator.cxx')
-rw-r--r-- | Source/cmGlobalNinjaGenerator.cxx | 15 |
1 files changed, 3 insertions, 12 deletions
diff --git a/Source/cmGlobalNinjaGenerator.cxx b/Source/cmGlobalNinjaGenerator.cxx index 052f2e9..0c28776 100644 --- a/Source/cmGlobalNinjaGenerator.cxx +++ b/Source/cmGlobalNinjaGenerator.cxx @@ -1369,17 +1369,7 @@ void cmGlobalNinjaGenerator::AppendTargetDepends( } void cmGlobalNinjaGenerator::AppendTargetDependsClosure( - cmGeneratorTarget const* target, cmNinjaDeps& outputs, - const std::string& config, const std::string& fileConfig, bool genexOutput) -{ - cmNinjaOuts outs; - this->AppendTargetDependsClosure(target, outs, config, fileConfig, - genexOutput, true); - cm::append(outputs, outs); -} - -void cmGlobalNinjaGenerator::AppendTargetDependsClosure( - cmGeneratorTarget const* target, cmNinjaOuts& outputs, + cmGeneratorTarget const* target, std::unordered_set<std::string>& outputs, const std::string& config, const std::string& fileConfig, bool genexOutput, bool omit_self) { @@ -1400,7 +1390,8 @@ void cmGlobalNinjaGenerator::AppendTargetDependsClosure( // relevant for filling the cache entries properly isolated and a global // result set that is relevant for the result of the top level call to // AppendTargetDependsClosure. - cmNinjaOuts this_outs; // this will be the new cache entry + std::unordered_set<std::string> + this_outs; // this will be the new cache entry for (auto const& dep_target : this->GetTargetDirectDepends(target)) { if (!dep_target->IsInBuildSystem()) { |