[Patch,AVR]: ad PR49868: assemble 3-byte symbols
Georg-Johann Lay
avr@gjlay.de
Fri May 11 17:04:00 GMT 2012
Currently avr_assemble_integer emits an assembler warning to
make gas complain about missing feature
http://sourceware.org/PR13503
if a 3-byte address must be assembled.
As PR13503 is implemented now, avr-gcc can use this feature.
It's only needed for the new __memx address space.
Ok to install?
The feature is new in gcc and binutils and only rarely
used, IMHO. The current solution will throw a comprehensible
warning but generate bad code, with the patch old binutils
will throw a not really helpful diagnose but work with
new binutils...
Thus okay to backport? Or leave 4.7 as is?
Johann
PR target/46898
* config/avr/avr.c (avr_const_address_lo16): Remove.
(avr_assemble_integer): Print ".byte lo8(x)",
".byte hi8(x)", ".byte hh8(x)" instead of emit an assembler
.warning if 3-byte address is assembled.
* doc/extend.texi (AVR Named Address Spaces): Document that
binutils 2.23 is needed to assemble 3-byte addresses.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pr49868-byte3.diff
Type: text/x-patch
Size: 4871 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20120511/8e0264a8/attachment.bin>
More information about the Gcc-patches
mailing list