[Bug c/59946] -mpcrel -O2 produces illegal asm code

[Bug c/59946] -mpcrel -O2 produces illegal asm code

[vincent.riviere at freesbee dot fr]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59946

Vincent Riviere <vincent.riviere at freesbee dot fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |vincent.riviere at freesbee dot fr

--- Comment #1 from Vincent Riviere <vincent.riviere at freesbee dot fr> ---
Confirmed with the following compilers:

m68k-atari-mint-gcc 4.7.1
m68k-atari-mint-gcc 4.6.4
m68k-elf-gcc 4.5.2

[Bug c/59946] -mpcrel -O2 produces illegal asm code

[mikpelinux at gmail dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59946

--- Comment #2 from Mikael Pettersson <mikpelinux at gmail dot com> ---
Reproduced with m68k-elf and m68k-linux toolchains built from binutils-2.23.2
and gcc-4.9-20140119, 4.8.2, and 4.7.3.  Removing "-m68000" causes the ".l"
suffix to disappear and gas to accept the generated code.

[Bug target/59946] -mpcrel -O2 produces illegal asm code

[law at redhat dot com]

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

Jeffrey A. Law <law at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2015-01-19
     Ever confirmed|0                           |1

[Bug target/59946] -mpcrel -O2 produces illegal asm code

[law at redhat dot com]

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

--- Comment #3 from Jeffrey A. Law <law at redhat dot com> ---
The m68000 does not support compare imm,pc-relative.  Sadly the various
comparison patterns play things fast and loose in terms of what order they emit
their operands.  That makes it nontrivial to write operand predicates and
constraints which would allow the PC-relative addresses.

I think the best solution is going to be to disallow pc-relative memory
addresses in the comparison patterns/expanders.

[Bug target/59946] -mpcrel -O2 produces illegal asm code

[law at gcc dot gnu.org]

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

--- Comment #4 from Jeffrey A. Law <law at gcc dot gnu.org> ---
Author: law
Date: Wed Jan 21 06:17:50 2015
New Revision: 219927

URL: https://gcc.gnu.org/viewcvs?rev=219927&root=gcc&view=rev
Log:
2015-01-20  Jeff Law  <law@redhat.com>

    PR target/59946
    * config/m68k/m68k.md (Comparison expanders and patterns): Do not
    allow pc-relative addresses in operand predicates or constraints.

    PR target/59946
    * gcc.target/m68k/pr59946.c: New test.

Added:
    trunk/gcc/testsuite/gcc.target/m68k/pr59946.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/m68k/m68k.md
    trunk/gcc/testsuite/ChangeLog

[Bug target/59946] -mpcrel -O2 produces illegal asm code

[law at redhat dot com]

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

Jeffrey A. Law <law at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #5 from Jeffrey A. Law <law at redhat dot com> ---
Fixed on trunk.  No plans to backport to any release branches.