diff options
author | Bob Wilson <bob.wilson@apple.com> | 2013-09-26 21:00:51 +0000 |
---|---|---|
committer | Bob Wilson <bob.wilson@apple.com> | 2013-09-26 21:00:51 +0000 |
commit | a1b4206a7018ca8f71e2d8564fc6f464d2bed1b7 (patch) | |
tree | 6b323dfe7b418b7c9e8ffbe95906d2c66854a1d9 | |
parent | cc64cc1773f2cf08e1c321a59659a9ccd2206338 (diff) | |
download | bcm5719-llvm-a1b4206a7018ca8f71e2d8564fc6f464d2bed1b7.tar.gz bcm5719-llvm-a1b4206a7018ca8f71e2d8564fc6f464d2bed1b7.zip |
Fix up fallout from r187156.
The previous change caused the driver to translate -Wa,-L to the
-msave-temp-labels option for cc1as, but cc1as did not accept that option.
This patch follows the same approach used for similar options (-relax-all,
-noexecstack) in the previous patch.
llvm-svn: 191458
-rw-r--r-- | clang/include/clang/Driver/CC1AsOptions.td | 2 | ||||
-rw-r--r-- | clang/test/Driver/integrated-as.s | 4 | ||||
-rw-r--r-- | clang/tools/driver/cc1as_main.cpp | 2 |
3 files changed, 5 insertions, 3 deletions
diff --git a/clang/include/clang/Driver/CC1AsOptions.td b/clang/include/clang/Driver/CC1AsOptions.td index baa769156dc..b536724f242 100644 --- a/clang/include/clang/Driver/CC1AsOptions.td +++ b/clang/include/clang/Driver/CC1AsOptions.td @@ -33,7 +33,7 @@ def I : JoinedOrSeparate<["-"], "I">, MetaVarName<"<directory>">, HelpText<"Add directory to include search path">; def n : Flag<["-"], "n">, HelpText<"Don't automatically start assembly file with a text section">; -def L : Flag<["-"], "L">, +def msave_temp_labels : Flag<["-"], "msave-temp-labels">, HelpText<"Save temporary labels in the symbol table. " "Note this may change .s semantics, it should almost never be used " "on compiler generated code!">; diff --git a/clang/test/Driver/integrated-as.s b/clang/test/Driver/integrated-as.s index c168f67b46c..51ce31cc51f 100644 --- a/clang/test/Driver/integrated-as.s +++ b/clang/test/Driver/integrated-as.s @@ -1,8 +1,10 @@ // RUN: %clang -### -c -integrated-as %s 2>&1 | FileCheck %s - // CHECK: cc1as // CHECK-NOT: -relax-all +// RUN: %clang -### -c -integrated-as -Wa,-L %s 2>&1 | FileCheck --check-prefix=OPT_L %s +// OPT_L: msave-temp-labels + // RUN: not %clang -c -integrated-as -Wa,--compress-debug-sections %s 2>&1 | FileCheck --check-prefix=INVALID %s // INVALID: error: unsupported argument '--compress-debug-sections' to option 'Wa,' diff --git a/clang/tools/driver/cc1as_main.cpp b/clang/tools/driver/cc1as_main.cpp index 80ab1ab22e9..31cd236b84c 100644 --- a/clang/tools/driver/cc1as_main.cpp +++ b/clang/tools/driver/cc1as_main.cpp @@ -184,7 +184,7 @@ bool AssemblerInvocation::CreateFromArgs(AssemblerInvocation &Opts, // Language Options Opts.IncludePaths = Args->getAllArgValues(OPT_I); Opts.NoInitialTextSection = Args->hasArg(OPT_n); - Opts.SaveTemporaryLabels = Args->hasArg(OPT_L); + Opts.SaveTemporaryLabels = Args->hasArg(OPT_msave_temp_labels); Opts.GenDwarfForAssembly = Args->hasArg(OPT_g); Opts.DwarfDebugFlags = Args->getLastArgValue(OPT_dwarf_debug_flags); Opts.DwarfDebugProducer = Args->getLastArgValue(OPT_dwarf_debug_producer); |