<div dir="auto">I added a Sed step.  I double up the semicolon, and only convert for  Clang aarch64-unkown-windows.  Not ideal, but it works.<div dir="auto"><br></div><div dir="auto">I'm onto lld-link problems now.</div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">Thanks for all the input.</div><div dir="auto"></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Nov 11, 2019, 9:41 AM Peter Smith <<a href="mailto:peter.smith@linaro.org">peter.smith@linaro.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I don't think that there is an alternative separator for the Windows<br>
target as both the CommentString and SeparatorString are ";". I don't<br>
have a strong opinion over changing the comment character for AArch64<br>
Windows target. Probably best addressed by the people that use/develop<br>
on it. It is true that armasm(64) uses ";" as its comment character,<br>
but as Martin says many of the directives and other bits of syntax are<br>
different so at least some form of scripted conversion is likely to be<br>
needed anyway.<br>
<br>
Peter<br>
<br>
On Fri, 8 Nov 2019 at 20:31, Joel Winarske <<a href="mailto:joel.winarske@gmail.com" target="_blank" rel="noreferrer">joel.winarske@gmail.com</a>> wrote:<br>
><br>
> Ah yes, a comment...<br>
><br>
> Is there a seperator character that works?<br>
><br>
> On Fri, Nov 8, 2019, 5:43 AM Martin Storsjö <<a href="mailto:martin@martin.st" target="_blank" rel="noreferrer">martin@martin.st</a>> wrote:<br>
>><br>
>> On Fri, 8 Nov 2019, Peter Smith via cfe-dev wrote:<br>
>><br>
>> > Hello Joel,<br>
>> ><br>
>> > The root cause is that the AArch64 target is using ';' as a comment<br>
>> > character, whereas in GNU as (and many other llvm MC backends),  the<br>
>> > ';' is used as a statement separator. In your example only the text up<br>
>> > to ';' is being parsed.<br>
>> ><br>
>> > Looking at AArch64MCAsmInfo.cpp<br>
>> >  SeparatorString = "%%";<br>
>> >  CommentString = ";";<br>
>> ><br>
>> > I think that this is likely to have been derived from Darwin as I<br>
>> > can't get the "%%" to work as a statement separator for<br>
>> > aarch64-linux-gnu targets.<br>
>><br>
>> I don't know about the separator string, but if you look in<br>
>> AArch64MCAsmInfoELF (and AArch64MCAsmInfoGNUCOFF), the comment string is<br>
>> set to "//" instead.<br>
>><br>
>> I suggested changing this for the GNUCOFF (aka MinGW) target, as mingw-w64<br>
>> contains a number of assembler files that uses this pattern of piling up a<br>
>> few directives on one line.<br>
>><br>
>> As there's very little predecent for GNU style .s assembler for<br>
>> aarch64-windows-msvc targets (as the official tool from microsoft,<br>
>> armasm64, uses a totally different syntax), I'm guess it would be fine to<br>
>> change the comment string to "//" there as well, as there apparently is a<br>
>> demand for it?<br>
>><br>
>> With a target that doesn't treat ";" as a comment char, your assembler<br>
>> example errors out due to issues with the section and type directives<br>
>> though, now that they actually are interpreted and not stripped out as<br>
>> comments.<br>
>><br>
>> // Martin<br>
>><br>
</blockquote></div>