From f6dd66b7b80e2fb42628df2593b3948149a53a5f Mon Sep 17 00:00:00 2001 From: Daniel Dunbar Date: Wed, 25 Mar 2009 06:08:46 +0000 Subject: [PATCH] Driver: Replace Option::ForwardToGCC by Option::DriverOption (which matches the flag in Options.def). git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@67679 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/clang/Driver/Option.h | 10 ++++++---- lib/Driver/OptTable.cpp | 7 +------ lib/Driver/Option.cpp | 2 +- 3 files changed, 8 insertions(+), 11 deletions(-) diff --git a/include/clang/Driver/Option.h b/include/clang/Driver/Option.h index 79b512e607..c7b6d0e08c 100644 --- a/include/clang/Driver/Option.h +++ b/include/clang/Driver/Option.h @@ -82,8 +82,8 @@ namespace driver { /// Always render this option joined with its value. bool ForceJoinedRender : 1; - /// Forward to generic GCC tools. - bool ForwardToGCC : 1; + /// This option is only for consumed by the driver. + bool DriverOption : 1; protected: Option(OptionClass Kind, options::ID ID, const char *Name, @@ -112,8 +112,10 @@ namespace driver { bool hasForceJoinedRender() const { return ForceJoinedRender; } void setForceJoinedRender(bool Value) { ForceJoinedRender = Value; } - bool hasForwardToGCC() const { return ForwardToGCC; } - void setForwardToGCC(bool Value) { ForwardToGCC = Value; } + bool isDriverOption() const { return DriverOption; } + void setDriverOption(bool Value) { DriverOption = Value; } + + bool hasForwardToGCC() const { return !DriverOption && !LinkerInput; } /// getUnaliasedOption - Return the final option this option /// aliases (itself, if the option has no alias). diff --git a/lib/Driver/OptTable.cpp b/lib/Driver/OptTable.cpp index baaa886fec..2e44999580 100644 --- a/lib/Driver/OptTable.cpp +++ b/lib/Driver/OptTable.cpp @@ -185,18 +185,13 @@ Option *OptTable::constructOption(options::ID id) const { case 'S': assert(info.Kind == Option::JoinedClass && "Invalid option."); Opt->setForceSeparateRender(true); break; - case 'd': Opt->setForwardToGCC(false); break; + case 'd': Opt->setDriverOption(true); break; case 'i': Opt->setNoOptAsInput(true); break; case 'l': Opt->setLinkerInput(true); break; case 'u': Opt->setUnsupported(true); break; } } - // Linker inputs shouldn't be forwarded to GCC as arguments (they - // will, however, be forwarded as inputs). - if (Opt->isLinkerInput()) - Opt->setForwardToGCC(false); - return Opt; } diff --git a/lib/Driver/Option.cpp b/lib/Driver/Option.cpp index 6ea02aaf40..624854815d 100644 --- a/lib/Driver/Option.cpp +++ b/lib/Driver/Option.cpp @@ -21,7 +21,7 @@ Option::Option(OptionClass _Kind, options::ID _ID, const char *_Name, : Kind(_Kind), ID(_ID), Name(_Name), Group(_Group), Alias(_Alias), Unsupported(false), LinkerInput(false), NoOptAsInput(false), ForceSeparateRender(false), ForceJoinedRender(false), - ForwardToGCC(true) + DriverOption(false) { // Multi-level aliases are not supported, and alias options cannot -- 2.50.1