[PATCH] D100536: [clang][deps] NFC: Remove unused FullDependencies member
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG95588c0da4ef: [clang][deps] NFC: Remove unused FullDependencies member (authored by jansvoboda11). Changed prior to commit: https://reviews.llvm.org/D100536?vs=337660=338500#toc Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100536/new/ https://reviews.llvm.org/D100536 Files: clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h clang/lib/Tooling/DependencyScanning/DependencyScanningTool.cpp Index: clang/lib/Tooling/DependencyScanning/DependencyScanningTool.cpp === --- clang/lib/Tooling/DependencyScanning/DependencyScanningTool.cpp +++ clang/lib/Tooling/DependencyScanning/DependencyScanningTool.cpp @@ -16,10 +16,10 @@ std::vector FullDependencies::getAdditionalCommandLine( std::function LookupPCMPath, std::function LookupModuleDeps) const { - std::vector Ret = AdditionalNonPathCommandLine; - - Ret.push_back("-fno-implicit-modules"); - Ret.push_back("-fno-implicit-module-maps"); + std::vector Ret{ + "-fno-implicit-modules", + "-fno-implicit-module-maps", + }; std::vector PCMPaths; std::vector ModMapPaths; Index: clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h === --- clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h +++ clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h @@ -38,13 +38,6 @@ /// determined that the differences are benign for this compilation. std::vector ClangModuleDeps; - /// A partial addtional set of command line arguments that can be used to - /// build this translation unit. - /// - /// Call \c getFullAdditionalCommandLine() to get a command line suitable for - /// appending to the original command line to pass to clang. - std::vector AdditionalNonPathCommandLine; - /// Get additional arguments suitable for appending to the original Clang /// command line. /// Index: clang/lib/Tooling/DependencyScanning/DependencyScanningTool.cpp === --- clang/lib/Tooling/DependencyScanning/DependencyScanningTool.cpp +++ clang/lib/Tooling/DependencyScanning/DependencyScanningTool.cpp @@ -16,10 +16,10 @@ std::vector FullDependencies::getAdditionalCommandLine( std::function LookupPCMPath, std::function LookupModuleDeps) const { - std::vector Ret = AdditionalNonPathCommandLine; - - Ret.push_back("-fno-implicit-modules"); - Ret.push_back("-fno-implicit-module-maps"); + std::vector Ret{ + "-fno-implicit-modules", + "-fno-implicit-module-maps", + }; std::vector PCMPaths; std::vector ModMapPaths; Index: clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h === --- clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h +++ clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h @@ -38,13 +38,6 @@ /// determined that the differences are benign for this compilation. std::vector ClangModuleDeps; - /// A partial addtional set of command line arguments that can be used to - /// build this translation unit. - /// - /// Call \c getFullAdditionalCommandLine() to get a command line suitable for - /// appending to the original command line to pass to clang. - std::vector AdditionalNonPathCommandLine; - /// Get additional arguments suitable for appending to the original Clang /// command line. /// ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D100536: [clang][deps] NFC: Remove unused FullDependencies member
dexonsmith accepted this revision. dexonsmith added a comment. This revision is now accepted and ready to land. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100536/new/ https://reviews.llvm.org/D100536 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D100536: [clang][deps] NFC: Remove unused FullDependencies member
jansvoboda11 created this revision. jansvoboda11 added reviewers: Bigcheese, dexonsmith. jansvoboda11 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This patch removes the `FullDependencies::AdditionalNonPathCommandLine` member, as it's value-initialized and never mutated. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D100536 Files: clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h clang/lib/Tooling/DependencyScanning/DependencyScanningTool.cpp Index: clang/lib/Tooling/DependencyScanning/DependencyScanningTool.cpp === --- clang/lib/Tooling/DependencyScanning/DependencyScanningTool.cpp +++ clang/lib/Tooling/DependencyScanning/DependencyScanningTool.cpp @@ -16,10 +16,10 @@ std::vector FullDependencies::getAdditionalCommandLine( std::function LookupPCMPath, std::function LookupModuleDeps) const { - std::vector Ret = AdditionalNonPathCommandLine; - - Ret.push_back("-fno-implicit-modules"); - Ret.push_back("-fno-implicit-module-maps"); + std::vector Ret{ + "-fno-implicit-modules", + "-fno-implicit-module-maps", + }; std::vector PCMPaths; std::vector ModMapPaths; Index: clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h === --- clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h +++ clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h @@ -38,13 +38,6 @@ /// determined that the differences are benign for this compilation. std::vector ClangModuleDeps; - /// A partial addtional set of command line arguments that can be used to - /// build this translation unit. - /// - /// Call \c getFullAdditionalCommandLine() to get a command line suitable for - /// appending to the original command line to pass to clang. - std::vector AdditionalNonPathCommandLine; - /// Gets the full addtional command line suitable for appending to the /// original command line to pass to clang. /// Index: clang/lib/Tooling/DependencyScanning/DependencyScanningTool.cpp === --- clang/lib/Tooling/DependencyScanning/DependencyScanningTool.cpp +++ clang/lib/Tooling/DependencyScanning/DependencyScanningTool.cpp @@ -16,10 +16,10 @@ std::vector FullDependencies::getAdditionalCommandLine( std::function LookupPCMPath, std::function LookupModuleDeps) const { - std::vector Ret = AdditionalNonPathCommandLine; - - Ret.push_back("-fno-implicit-modules"); - Ret.push_back("-fno-implicit-module-maps"); + std::vector Ret{ + "-fno-implicit-modules", + "-fno-implicit-module-maps", + }; std::vector PCMPaths; std::vector ModMapPaths; Index: clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h === --- clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h +++ clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h @@ -38,13 +38,6 @@ /// determined that the differences are benign for this compilation. std::vector ClangModuleDeps; - /// A partial addtional set of command line arguments that can be used to - /// build this translation unit. - /// - /// Call \c getFullAdditionalCommandLine() to get a command line suitable for - /// appending to the original command line to pass to clang. - std::vector AdditionalNonPathCommandLine; - /// Gets the full addtional command line suitable for appending to the /// original command line to pass to clang. /// ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits