public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/54854] New: [avr] Deprecate and finally remove the -mshort-calls command line option
@ 2012-10-08 6:51 gjl at gcc dot gnu.org
2012-10-08 6:52 ` [Bug target/54854] " gjl at gcc dot gnu.org
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: gjl at gcc dot gnu.org @ 2012-10-08 6:51 UTC (permalink / raw)
To: gcc-bugs
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.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug target/54854] [avr] Deprecate and finally remove the -mshort-calls command line option
2012-10-08 6:51 [Bug target/54854] New: [avr] Deprecate and finally remove the -mshort-calls command line option gjl at gcc dot gnu.org
@ 2012-10-08 6:52 ` gjl at gcc dot gnu.org
2012-10-08 10:14 ` gjl at gcc dot gnu.org
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: gjl at gcc dot gnu.org @ 2012-10-08 6:52 UTC (permalink / raw)
To: gcc-bugs
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug target/54854] [avr] Deprecate and finally remove the -mshort-calls command line option
2012-10-08 6:51 [Bug target/54854] New: [avr] Deprecate and finally remove the -mshort-calls command line option gjl at gcc dot gnu.org
2012-10-08 6:52 ` [Bug target/54854] " gjl at gcc dot gnu.org
@ 2012-10-08 10:14 ` gjl at gcc dot gnu.org
2012-10-08 10:27 ` gjl at gcc dot gnu.org
2012-10-10 21:47 ` gjl at gcc dot gnu.org
3 siblings, 0 replies; 5+ messages in thread
From: gjl at gcc dot gnu.org @ 2012-10-08 10:14 UTC (permalink / raw)
To: gcc-bugs
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug target/54854] [avr] Deprecate and finally remove the -mshort-calls command line option
2012-10-08 6:51 [Bug target/54854] New: [avr] Deprecate and finally remove the -mshort-calls command line option gjl at gcc dot gnu.org
2012-10-08 6:52 ` [Bug target/54854] " gjl at gcc dot gnu.org
2012-10-08 10:14 ` gjl at gcc dot gnu.org
@ 2012-10-08 10:27 ` gjl at gcc dot gnu.org
2012-10-10 21:47 ` gjl at gcc dot gnu.org
3 siblings, 0 replies; 5+ messages in thread
From: gjl at gcc dot gnu.org @ 2012-10-08 10:27 UTC (permalink / raw)
To: gcc-bugs
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug target/54854] [avr] Deprecate and finally remove the -mshort-calls command line option
2012-10-08 6:51 [Bug target/54854] New: [avr] Deprecate and finally remove the -mshort-calls command line option gjl at gcc dot gnu.org
` (2 preceding siblings ...)
2012-10-08 10:27 ` gjl at gcc dot gnu.org
@ 2012-10-10 21:47 ` gjl at gcc dot gnu.org
3 siblings, 0 replies; 5+ messages in thread
From: gjl at gcc dot gnu.org @ 2012-10-10 21:47 UTC (permalink / raw)
To: gcc-bugs
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.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-10-10 21:47 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-10-08 6:51 [Bug target/54854] New: [avr] Deprecate and finally remove the -mshort-calls command line option gjl at gcc dot gnu.org
2012-10-08 6:52 ` [Bug target/54854] " gjl at gcc dot gnu.org
2012-10-08 10:14 ` gjl at gcc dot gnu.org
2012-10-08 10:27 ` gjl at gcc dot gnu.org
2012-10-10 21:47 ` gjl 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).