[PATCH] D20337: [MC] Support symbolic expressions in assembly directives

Nirav Davé via llvm-commits llvm-commits at lists.llvm.org
Wed May 25 08:01:47 PDT 2016


Yes, gas does use it.

Petr, can you add a short comment to this patch just so it's noted.

I believe this will be the only place where we may leave an error save
adding inline assembly with the fno-integrated-as flag.

-Nirav



On Wed, May 25, 2016 at 10:51 AM, Rafael EspĂ­ndola <
rafael.espindola at gmail.com> wrote:

> On 25 May 2016 at 10:45, James Y Knight <jyknight at google.com> wrote:
> > The .s does have a way to carry the location. GCC emits location lines
> > around inline asm, like this:
> > #APP
> > # 3 "/tmp/test-asm.c" 1
> >         cmp %ax, %ax
> > # 0 "" 2
> > #NO_APP
> >
> > Clang doesn't do that. Clang emits for the same code:
> >         #APP
> >         cmpw    %ax, %ax
> >         #NO_APP
> >
> > Emitting location directives that seems like it's really a separate
> patch,
> > IMO.
>
> Agreed. Does gas use that to print warning/errors these days?
>
> Cheers,
> Rafael
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160525/a80c4d81/attachment.html>


More information about the llvm-commits mailing list