From a20f46c319ebb59fa32958ff3529b7f1ade68789 Mon Sep 17 00:00:00 2001 From: Hans Wennborg Date: Fri, 13 Jun 2014 20:59:54 +0000 Subject: [PATCH] Add support for the /EP argument to clang-cl This maps the /EP argument to both -E and -P. Patch by Ehsan Akhgari! Differential Reviion: http://reviews.llvm.org/D4133 git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@210935 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/clang/Driver/CLCompatOptions.td | 3 ++- lib/Driver/Driver.cpp | 3 ++- lib/Driver/Tools.cpp | 6 ++++++ test/Driver/cl-options.c | 6 +++++- 4 files changed, 15 insertions(+), 3 deletions(-) diff --git a/include/clang/Driver/CLCompatOptions.td b/include/clang/Driver/CLCompatOptions.td index a4881e24ff..e80a44b8f2 100644 --- a/include/clang/Driver/CLCompatOptions.td +++ b/include/clang/Driver/CLCompatOptions.td @@ -140,6 +140,8 @@ def _SLASH_Zs : CLFlag<"Zs">, HelpText<"Syntax-check only">, def _SLASH_M_Group : OptionGroup<"">, Group; +def _SLASH_EP : CLFlag<"EP">, + HelpText<"Disable linemarker output and preprocess to stdout">; def _SLASH_FA : CLFlag<"FA">, HelpText<"Output assembly code file during compilation">; def _SLASH_Fa : CLJoined<"Fa">, @@ -205,7 +207,6 @@ def _SLASH_clr : CLJoined<"clr">; def _SLASH_d2Zi_PLUS : CLFlag<"d2Zi+">; def _SLASH_doc : CLJoined<"doc">; def _SLASH_EH : CLJoined<"EH">; -def _SLASH_EP : CLFlag<"EP">; def _SLASH_FA_joined : CLJoined<"FA">; def _SLASH_favor : CLJoined<"favor">; def _SLASH_FC : CLFlag<"FC">; diff --git a/lib/Driver/Driver.cpp b/lib/Driver/Driver.cpp index 67e0122435..1a9de3aebf 100644 --- a/lib/Driver/Driver.cpp +++ b/lib/Driver/Driver.cpp @@ -152,9 +152,10 @@ const { Arg *PhaseArg = nullptr; phases::ID FinalPhase; - // -{E,M,MM} and /P only run the preprocessor. + // -{E,EP,P,M,MM} only run the preprocessor. if (CCCIsCPP() || (PhaseArg = DAL.getLastArg(options::OPT_E)) || + (PhaseArg = DAL.getLastArg(options::OPT__SLASH_EP)) || (PhaseArg = DAL.getLastArg(options::OPT_M, options::OPT_MM)) || (PhaseArg = DAL.getLastArg(options::OPT__SLASH_P))) { FinalPhase = phases::Preprocess; diff --git a/lib/Driver/Tools.cpp b/lib/Driver/Tools.cpp index 0df398b338..e49267acd1 100644 --- a/lib/Driver/Tools.cpp +++ b/lib/Driver/Tools.cpp @@ -4363,6 +4363,12 @@ void Clang::AddClangCLArgs(const ArgList &Args, ArgStringList &CmdArgs) const { if (!Args.hasArg(options::OPT_frtti, options::OPT_fno_rtti)) CmdArgs.push_back("-fno-rtti"); + // /EP should expand to -E -P. + if (Args.hasArg(options::OPT__SLASH_EP)) { + CmdArgs.push_back("-E"); + CmdArgs.push_back("-P"); + } + const Driver &D = getToolChain().getDriver(); Arg *MostGeneralArg = Args.getLastArg(options::OPT__SLASH_vmg); Arg *BestCaseArg = Args.getLastArg(options::OPT__SLASH_vmb); diff --git a/test/Driver/cl-options.c b/test/Driver/cl-options.c index eaf7e172d6..38cbb347c6 100644 --- a/test/Driver/cl-options.c +++ b/test/Driver/cl-options.c @@ -25,6 +25,11 @@ // E: "-E" // E: "-o" "-" +// RUN: %clang_cl /EP -### -- %s 2>&1 | FileCheck -check-prefix=EP %s +// EP: "-E" +// EP: "-P" +// EP: "-o" "-" + // RTTI is on by default; just check that we don't error. // RUN: %clang_cl /Zs /GR -- %s 2>&1 @@ -196,7 +201,6 @@ // RUN: /docname \ // RUN: /d2Zi+ \ // RUN: /EHsc \ -// RUN: /EP \ // RUN: /F \ // RUN: /FA \ // RUN: /FAc \ -- 2.40.0