From 5ab002ea11921c4766912cfa7ca98756271f87dc Mon Sep 17 00:00:00 2001 From: Brad King Date: Sat, 18 Mar 2023 09:55:18 -0400 Subject: cmCxxModuleMapper: Remove redundant path conversion callbacks Two calls to `PathForGenerator` were applied to values returned by `BmiGeneratorPathForModule`, that already calls `PathForGenerator`. --- Source/cmCxxModuleMapper.cxx | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/Source/cmCxxModuleMapper.cxx b/Source/cmCxxModuleMapper.cxx index 7952dfc..2c57c05 100644 --- a/Source/cmCxxModuleMapper.cxx +++ b/Source/cmCxxModuleMapper.cxx @@ -239,8 +239,7 @@ std::set CxxModuleUsageSeed( for (auto const& p : object.Provides) { if (auto bmi_loc = loc.BmiGeneratorPathForModule(p.LogicalName)) { // XXX(cxx-modules): How to support header units? - usages.AddReference(p.LogicalName, loc.PathForGenerator(*bmi_loc), - LookupMethod::ByName); + usages.AddReference(p.LogicalName, *bmi_loc, LookupMethod::ByName); } } @@ -268,8 +267,7 @@ std::set CxxModuleUsageSeed( } if (bmi_loc) { - usages.AddReference(r.LogicalName, loc.PathForGenerator(*bmi_loc), - r.Method); + usages.AddReference(r.LogicalName, *bmi_loc, r.Method); } } } -- cgit v0.12