[Bug target/54854] New: [avr] Deprecate and finally remove the -mshort-calls command line option

[Bug target/54854] New: [avr] Deprecate and finally remove the -mshort-calls command line option

[gjl at gcc dot gnu.org]

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

             Bug #: 54854
           Summary: [avr] Deprecate and finally remove the -mshort-calls
                    command line option
    Classification: Unclassified
           Product: gcc
           Version: 4.8.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
        AssignedTo: gjl@gcc.gnu.org
        ReportedBy: gjl@gcc.gnu.org
            Target: avr


-mshort-calls uses RJMP/RCALL regardless of the flash size of the target
device.  This will lead to a linker error if a JMP/CALL target is out of reach
of RJMP/RCALL:

file.c:(.text+0x1234): relocation truncated to fit: R_AVR_13_PCREL against
symbol `function' defined in in module.o

The -mshort-calls option dates back to the times before the linker could relax
JMP/CALL to RJMP/RCALL if the jump offset allows this.

The above error message is a blocker and hard to understand for users.

Instead of -mstort-calls, the compiler can be called with -mrelax or
-Wl,--relax to direct the linker to use the shortest possible instruction.

[Bug target/54854] [avr] Deprecate and finally remove the -mshort-calls command line option

[gjl at gcc dot gnu.org]

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

Georg-Johann Lay <gjl at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P4
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2012-10-08
            Version|4.8.0                       |4.7.2
   Target Milestone|---                         |4.8.0
     Ever Confirmed|0                           |1

[Bug target/54854] [avr] Deprecate and finally remove the -mshort-calls command line option

[gjl at gcc dot gnu.org]

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

--- Comment #1 from Georg-Johann Lay <gjl at gcc dot gnu.org> 2012-10-08 10:14:01 UTC ---
Author: gjl
Date: Mon Oct  8 10:13:56 2012
New Revision: 192201

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=192201
Log:
    PR target/54854
    * doc/invoke.texi (AVR Options): Remove -mshort-calls.
    * config/avr/avr.opt (-mshort-calls): Remove option.
    * config/avr/avr.h (AVR_HAVE_JMP_CALL): Don't depend on
    TARGET_SHORT_CALLS.


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/avr/avr.h
    trunk/gcc/config/avr/avr.opt
    trunk/gcc/doc/invoke.texi

[Bug target/54854] [avr] Deprecate and finally remove the -mshort-calls command line option

[gjl at gcc dot gnu.org]

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

--- Comment #2 from Georg-Johann Lay <gjl at gcc dot gnu.org> 2012-10-08 10:27:26 UTC ---
Author: gjl
Date: Mon Oct  8 10:27:15 2012
New Revision: 192202

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=192202
Log:
    PR target/54854
    * doc/invoke.texi (AVR Options): Deprecate -mshort-calls.


Modified:
    branches/gcc-4_7-branch/gcc/ChangeLog
    branches/gcc-4_7-branch/gcc/doc/invoke.texi

[Bug target/54854] [avr] Deprecate and finally remove the -mshort-calls command line option

[gjl at gcc dot gnu.org]

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

Georg-Johann Lay <gjl at gcc dot gnu.org> changed:

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

--- Comment #3 from Georg-Johann Lay <gjl at gcc dot gnu.org> 2012-10-10 21:47:31 UTC ---
Done.