public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug inline-asm/94095] New: Modifier 'a' do not work as described
@ 2020-03-09 10:02 frederic.recoules@univ-grenoble-alpes.fr
  2020-03-09 16:26 ` [Bug target/94095] " pinskia at gcc dot gnu.org
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: frederic.recoules@univ-grenoble-alpes.fr @ 2020-03-09 10:02 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94095

            Bug ID: 94095
           Summary: Modifier 'a' do not work as described
           Product: gcc
           Version: unknown
            Status: UNCONFIRMED
          Keywords: documentation
          Severity: normal
          Priority: P3
         Component: inline-asm
          Assignee: unassigned at gcc dot gnu.org
          Reporter: frederic.recoules@univ-grenoble-alpes.fr
  Target Milestone: ---

In https://gcc.gnu.org/onlinedocs/gcc/Extended-Asm.html 6.47.2.8, it is said
that 'a' "Print an absolute memory reference" (aka *%eax for 'att') but
experiments shows that it actually print '(%eax)'.

But yet, what is the intended behavior?
Is it actually valid to use 'a' modifier with 'p' constraint in

void *x = NULL;
__asm__ ("leal %a1, %0" : "=r" (x) : "p" (&x));

?

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug target/94095] Modifier 'a' do not work as described
  2020-03-09 10:02 [Bug inline-asm/94095] New: Modifier 'a' do not work as described frederic.recoules@univ-grenoble-alpes.fr
@ 2020-03-09 16:26 ` pinskia at gcc dot gnu.org
  2020-03-09 17:00 ` pinskia at gcc dot gnu.org
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: pinskia at gcc dot gnu.org @ 2020-03-09 16:26 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94095

--- Comment #1 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
The problem is just 'a' in the first (Modifier) column is wrong, it should have
been 'A'.  I will commit the patch in a few minutes.  Operand column is correct
to use 'A'.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug target/94095] Modifier 'a' do not work as described
  2020-03-09 10:02 [Bug inline-asm/94095] New: Modifier 'a' do not work as described frederic.recoules@univ-grenoble-alpes.fr
  2020-03-09 16:26 ` [Bug target/94095] " pinskia at gcc dot gnu.org
@ 2020-03-09 17:00 ` pinskia at gcc dot gnu.org
  2020-03-11 12:06 ` marxin at gcc dot gnu.org
  2020-03-11 15:35 ` pinskia at gcc dot gnu.org
  3 siblings, 0 replies; 5+ messages in thread
From: pinskia at gcc dot gnu.org @ 2020-03-09 17:00 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94095

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
   Target Milestone|---                         |10.0
             Status|UNCONFIRMED                 |RESOLVED

--- Comment #2 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
Fixed for GCC 10.  If someone else wants to backport the patch (it is trival),
I am ok with it.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug target/94095] Modifier 'a' do not work as described
  2020-03-09 10:02 [Bug inline-asm/94095] New: Modifier 'a' do not work as described frederic.recoules@univ-grenoble-alpes.fr
  2020-03-09 16:26 ` [Bug target/94095] " pinskia at gcc dot gnu.org
  2020-03-09 17:00 ` pinskia at gcc dot gnu.org
@ 2020-03-11 12:06 ` marxin at gcc dot gnu.org
  2020-03-11 15:35 ` pinskia at gcc dot gnu.org
  3 siblings, 0 replies; 5+ messages in thread
From: marxin at gcc dot gnu.org @ 2020-03-11 12:06 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94095

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |marxin at gcc dot gnu.org

--- Comment #3 from Martin Liška <marxin at gcc dot gnu.org> ---
commit r10-7092-ga931bb50fe77446058166b50eea4e53223ad7ef7
Author: Andrew Pinski <apinski@marvell.com>
Date:   Mon Mar 9 09:45:23 2020 -0700

    Fix 'A' operand modifier: PR inline-asm/94095

    The problem here is there was a typo in the documentation
    for the 'A' modifier in the table, it was recorded as 'a'
    in the table on the modifier column.

    Committed as obvious.

    2020-03-09  Andrew Pinski  <apinski@marvell.com>

            PR inline-asm/94095
            * doc/extend.texi (x86 Operand Modifiers): Fix column
            for 'A' modifier.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug target/94095] Modifier 'a' do not work as described
  2020-03-09 10:02 [Bug inline-asm/94095] New: Modifier 'a' do not work as described frederic.recoules@univ-grenoble-alpes.fr
                   ` (2 preceding siblings ...)
  2020-03-11 12:06 ` marxin at gcc dot gnu.org
@ 2020-03-11 15:35 ` pinskia at gcc dot gnu.org
  3 siblings, 0 replies; 5+ messages in thread
From: pinskia at gcc dot gnu.org @ 2020-03-11 15:35 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94095

--- Comment #4 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
(In reply to Martin Liška from comment #3)
I did not notice the git to bugzilla comment connection was not working until
yesterday and then forgot to update this one.  THanks for doing it.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2020-03-11 15:35 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-09 10:02 [Bug inline-asm/94095] New: Modifier 'a' do not work as described frederic.recoules@univ-grenoble-alpes.fr
2020-03-09 16:26 ` [Bug target/94095] " pinskia at gcc dot gnu.org
2020-03-09 17:00 ` pinskia at gcc dot gnu.org
2020-03-11 12:06 ` marxin at gcc dot gnu.org
2020-03-11 15:35 ` pinskia at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).