summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBob Wilson <bob.wilson@apple.com>2013-09-26 21:00:51 +0000
committerBob Wilson <bob.wilson@apple.com>2013-09-26 21:00:51 +0000
commita1b4206a7018ca8f71e2d8564fc6f464d2bed1b7 (patch)
tree6b323dfe7b418b7c9e8ffbe95906d2c66854a1d9
parentcc64cc1773f2cf08e1c321a59659a9ccd2206338 (diff)
downloadbcm5719-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.td2
-rw-r--r--clang/test/Driver/integrated-as.s4
-rw-r--r--clang/tools/driver/cc1as_main.cpp2
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);
OpenPOWER on IntegriCloud