mirror/CMake
1
0
Fork 0
mirror of https://github.com/Kitware/CMake.git synced 2026-01-01 19:30:13 -06:00
Code Issues Packages Projects Releases Wiki Activity

FASTBuild: Fix default MSVC compiler PDB paths

Browse Source 
When we pass a PDB output directory to the compiler in a path that
requires quoting, the trailing backslash must be escaped to be parsed
correctly by the compiler, e.g., `cl /Fd"path\with space\\"`.  However,
`fbuild` does not parse this correctly when extracting `/Fd`.  Work
around that bug by using a trailing forward slash in quotes instead.
This commit is contained in:
Brad King
2025-12-05 15:09:08 -05:00
parent dff020e679
commit 6f1fe8853d
1 changed files with 30 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

47
Source/cmFastbuildNormalTargetGenerator.cxx
View File

@@ -68,6 +68,12 @@ std::string const UNITY_BUILD_BATCH_SIZE("UNITY_BUILD_BATCH_SIZE");
std::string const SKIP_UNITY_BUILD_INCLUSION("SKIP_UNITY_BUILD_INCLUSION");
std::string const UNITY_GROUP("UNITY_GROUP");
#ifdef _WIN32
char const kPATH_SLASH = '\\';
#else
char const kPATH_SLASH = '/';
#endif
} // anonymous namespace
cmFastbuildNormalTargetGenerator::cmFastbuildNormalTargetGenerator(
@@ -810,24 +816,31 @@ void cmFastbuildNormalTargetGenerator::ComputePaths(
this->ConvertToFastbuildPath(linkerPDB));
}
}
if (GeneratorTarget->GetType() <= cmStateEnums::OBJECT_LIBRARY) {
std::string const pdbDir = GeneratorTarget->GetCompilePDBDirectory(Config);
LogMessage("GetCompilePDBDirectory: " + pdbDir);
EnsureDirectoryExists(pdbDir);
std::string pdbName = this->GeneratorTarget->GetCompilePDBName(Config);
LogMessage("GetCompilePDBName: " + pdbDir);
// If we don't have Compiler's PDB, we must add a trailing slash to satisfy
// MSVC.
bool needTrailingSlash = false;
if (pdbName.empty()) {
needTrailingSlash = true;
}
std::string const compilerPDB = cmStrCat(pdbDir, '\\', pdbName);
if (!compilerPDB.empty()) {
target.Variables["CompilerPDB"] = cmSystemTools::ConvertToOutputPath(
this->ConvertToFastbuildPath(compilerPDB) +
(needTrailingSlash ? "\\ " : ""));
std::string const compilerPDB = this->ComputeTargetCompilePDB(this->Config);
if (!compilerPDB.empty()) {
LogMessage("ComputeTargetCompilePDB: " + compilerPDB);
std::string compilerPDBArg = cmSystemTools::ConvertToOutputPath(
this->ConvertToFastbuildPath(compilerPDB));
if (cmHasSuffix(compilerPDB, '/')) {
// The compiler will choose the .pdb file name.
this->EnsureDirectoryExists(compilerPDB);
// ConvertToFastbuildPath dropped the trailing slash. Add it back.
// We do this after ConvertToOutputPath so that we can use a forward
// slash in the case that the argument is quoted.
if (cmHasSuffix(compilerPDBArg, '"')) {
// A quoted trailing backslash requires escaping, e.g., `/Fd"dir\\"`,
// but fbuild does not parse such arguments correctly as of 1.15.
// Always use a forward slash.
compilerPDBArg.insert(compilerPDBArg.size() - 1, 1, '/');
} else {
// An unquoted trailing slash or backslash is fine.
compilerPDBArg.push_back(kPATH_SLASH);
}
} else {
// We have an explicit .pdb path with file name.
this->EnsureParentDirectoryExists(compilerPDB);
}
target.Variables["CompilerPDB"] = std::move(compilerPDBArg);
}
std::string const impLibFullPath =
GeneratorTarget->GetFullPath(Config, cmStateEnums::ImportLibraryArtifact);