[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