From d6c3c908df237d21646bf270e3650b1e32078a88 Mon Sep 17 00:00:00 2001 From: Aaron Ballman Date: Mon, 27 Nov 2023 11:17:06 -0500 Subject: [PATCH] Remove the embed path group; it's not necessary --- clang/include/clang/Driver/Options.td | 12 +++--------- clang/lib/Driver/ToolChains/Clang.cpp | 6 +----- 2 files changed, 4 insertions(+), 14 deletions(-) diff --git a/clang/include/clang/Driver/Options.td b/clang/include/clang/Driver/Options.td index 6773ce6258d0b9..9523e32a5f8af6 100644 --- a/clang/include/clang/Driver/Options.td +++ b/clang/include/clang/Driver/Options.td @@ -114,11 +114,6 @@ def IncludePath_Group : OptionGroup<"">, Group, DocBrief<[{ Flags controlling how ``#include``\s are resolved to files.}]>; -def EmbedPath_Group : OptionGroup<"">, Group, - DocName<"Embed path management">, - DocBrief<[{ -Flags controlling how ``#embed``\s and similar are resolved to files.}]>; - def I_Group : OptionGroup<"">, Group, DocFlatten; def i_Group : OptionGroup<"">, Group, DocFlatten; def clang_i_Group : OptionGroup<"">, Group, DocFlatten; @@ -830,10 +825,9 @@ will be ignored}]>; def L : JoinedOrSeparate<["-"], "L">, Flags<[RenderJoined]>, Group, Visibility<[ClangOption, FlangOption]>, MetaVarName<"">, HelpText<"Add directory to library search path">; -def embed_dir_EQ : Joined<["--"], "embed-dir=">, - Flags<[RenderJoined]>, Group, - Visibility<[ClangOption, CC1Option]>, - MetaVarName<"">, HelpText<"Add directory to embed search path">; +def embed_dir_EQ : Joined<["--"], "embed-dir=">, Group, + Visibility<[ClangOption, CC1Option]>, MetaVarName<"">, + HelpText<"Add directory to embed search path">; def MD : Flag<["-"], "MD">, Group, HelpText<"Write a depfile containing user and system headers">; def MMD : Flag<["-"], "MMD">, Group, diff --git a/clang/lib/Driver/ToolChains/Clang.cpp b/clang/lib/Driver/ToolChains/Clang.cpp index c820b5ddaff387..2d73f42772a29d 100644 --- a/clang/lib/Driver/ToolChains/Clang.cpp +++ b/clang/lib/Driver/ToolChains/Clang.cpp @@ -1351,8 +1351,7 @@ void Clang::AddPreprocessingOptions(Compilation &C, const JobAction &JA, Args.addAllArgs(CmdArgs, {options::OPT_D, options::OPT_U, options::OPT_I_Group, - options::OPT_F, options::OPT_index_header_map, - options::OPT_EmbedPath_Group}); + options::OPT_F, options::OPT_index_header_map}); // Add -Wp, and -Xpreprocessor if using the preprocessor. @@ -8280,9 +8279,6 @@ void ClangAs::ConstructJob(Compilation &C, const JobAction &JA, // Pass along any -I options so we get proper .include search paths. Args.AddAllArgs(CmdArgs, options::OPT_I_Group); - // Pass along any --embed-dir or similar options so we get proper embed paths. - Args.AddAllArgs(CmdArgs, options::OPT_EmbedPath_Group); - // Determine the original source input. auto FindSource = [](const Action *S) -> const Action * { while (S->getKind() != Action::InputClass) {