[PATCH] D42157: [clang-cl] Let /FA output use intel assembly.

2020-06-26 Thread naoyuki nabana via Phabricator via cfe-commits
Cartman0 added a comment. hi! I try to assemble the .asm outputted from `clang-cl` by ML/ML64.exe on windows10. The ".asm" files outputted from `clang-cl /FA` / are supports to assemble by ML/ML64.exe on windows? when assemble by ML64.exe, occured " A2008:syntax error" and A2044:invalid chara

Re: [PATCH] D42157: [clang-cl] Let /FA output use intel assembly.

2018-01-17 Thread Yvan Roux via cfe-commits
On 17 January 2018 at 17:05, Nico Weber wrote: > r322674 attempts to fix. Fixed on ARM bots. Thanks > On Wed, Jan 17, 2018 at 10:54 AM, Yvan Roux via cfe-commits > wrote: >> >> Hi Nico, >> >> On 17 January 2018 at 14:35, Nico Weber via Phabricator via >> cfe-commits wrote: >> > thakis closed t

[PATCH] D42157: [clang-cl] Let /FA output use intel assembly.

2018-01-17 Thread Bill Seurer via Phabricator via cfe-commits
seurer added a comment. This patch fails on powerpc64 http://lab.llvm.org:8011/builders/clang-ppc64be-linux/builds/14380 http://lab.llvm.org:8011/builders/clang-ppc64le-linux/builds/13463 - TEST 'Clang :: Driver/masm.c' FAILED Script: --- /home/buildbots/ppc64le-clang-

Re: [PATCH] D42157: [clang-cl] Let /FA output use intel assembly.

2018-01-17 Thread Nico Weber via cfe-commits
r322674 attempts to fix. On Wed, Jan 17, 2018 at 10:54 AM, Yvan Roux via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Hi Nico, > > On 17 January 2018 at 14:35, Nico Weber via Phabricator via > cfe-commits wrote: > > thakis closed this revision. > > thakis added a comment. > > > > r322652,

Re: [PATCH] D42157: [clang-cl] Let /FA output use intel assembly.

2018-01-17 Thread Yvan Roux via cfe-commits
Hi Nico, On 17 January 2018 at 14:35, Nico Weber via Phabricator via cfe-commits wrote: > thakis closed this revision. > thakis added a comment. > > r322652, thanks! This commit broke ARM bots: http://lab.llvm.org:8011/builders/clang-cmake-armv7-a15/builds/14839 Cheers, Yvan > > https://review

[PATCH] D42157: [clang-cl] Let /FA output use intel assembly.

2018-01-17 Thread Nico Weber via Phabricator via cfe-commits
thakis closed this revision. thakis added a comment. r322652, thanks! https://reviews.llvm.org/D42157 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D42157: [clang-cl] Let /FA output use intel assembly.

2018-01-17 Thread Hans Wennborg via Phabricator via cfe-commits
hans accepted this revision. hans added a comment. This revision is now accepted and ready to land. lgtm, thanks for fixing! https://reviews.llvm.org/D42157 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailma

[PATCH] D42157: [clang-cl] Let /FA output use intel assembly.

2018-01-16 Thread Nico Weber via Phabricator via cfe-commits
thakis created this revision. thakis added a reviewer: hans. cl.exe's assembly output is in intel syntax, so clang-cl's should be too, PR35031. https://reviews.llvm.org/D42157 Files: lib/Driver/ToolChains/Clang.cpp test/Driver/masm.c Index: test/Driver/masm.c