From: Nico Weber Date: Wed, 17 Jan 2018 13:34:20 +0000 (+0000) Subject: [clang-cl] Let /FA output use intel assembly. X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=614f5c2b1bdf24b24845d8be274b3bdb448cc37f;p=clang [clang-cl] Let /FA output use intel assembly. cl's assembly output is in intel syntax, so clang-cl's should be too, PR35031. https://reviews.llvm.org/D42157 git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@322652 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Driver/ToolChains/Clang.cpp b/lib/Driver/ToolChains/Clang.cpp index 3d374a4c6f..f30782d8bf 100644 --- a/lib/Driver/ToolChains/Clang.cpp +++ b/lib/Driver/ToolChains/Clang.cpp @@ -1761,6 +1761,9 @@ void Clang::AddX86TargetArgs(const ArgList &Args, getToolChain().getDriver().Diag(diag::err_drv_unsupported_option_argument) << A->getOption().getName() << Value; } + } else if (getToolChain().getDriver().IsCLMode()) { + CmdArgs.push_back("-mllvm"); + CmdArgs.push_back("-x86-asm-syntax=intel"); } // Set flags to support MCU ABI. diff --git a/test/Driver/masm.c b/test/Driver/masm.c index 17c639358b..41b50effc5 100644 --- a/test/Driver/masm.c +++ b/test/Driver/masm.c @@ -2,11 +2,13 @@ // RUN: %clang -target i386-unknown-linux -masm=att -S %s -### 2>&1 | FileCheck --check-prefix=CHECK-ATT %s // RUN: %clang -target i386-unknown-linux -S -masm=somerequired %s -### 2>&1 | FileCheck --check-prefix=CHECK-SOMEREQUIRED %s // RUN: %clang -target arm-unknown-eabi -S -masm=intel %s -### 2>&1 | FileCheck --check-prefix=CHECK-ARM %s +// RUN: %clang_cl /FA -### -- %s 2>&1 | FileCheck --check-prefix=CHECK-CL %s int f() { // CHECK-INTEL: -x86-asm-syntax=intel // CHECK-ATT: -x86-asm-syntax=att // CHECK-SOMEREQUIRED: error: unsupported argument 'somerequired' to option 'masm=' // CHECK-ARM: warning: argument unused during compilation: '-masm=intel' +// CHECK-CL: -x86-asm-syntax=intel return 0; }