Re: [Patch, AVR]: AVR backend cleanup: use elfos.h

2011-06-27 Thread Denis Chertykov
2011/6/27 Georg-Johann Lay : > or > > [ ] Remove the undefs so that we have .ident. >    There's nothing terrific about it, it just adds >    some text to .comment section.  I guess it's missing >    because noone ever added it. This one. Denis.

Re: [Patch, AVR]: AVR backend cleanup: use elfos.h

2011-06-27 Thread Georg-Johann Lay
Denis Chertykov schrieb: 2011/6/27 Georg-Johann Lay: http://gcc.gnu.org/ml/gcc-patches/2011-06/msg02011.html [...] Is there a specific reason not to emit .ident? If not, I would remove the #undefs of IDENT_ASM_OP and ASM_OUTPUT_IDENT from avr/elf.h. Sorry, I forgot. Denis. It's just a m

Re: [Patch, AVR]: AVR backend cleanup: use elfos.h

2011-06-27 Thread Denis Chertykov
2011/6/27 Georg-Johann Lay : > Denis Chertykov wrote: >> 2011/6/27 Georg-Johann Lay : >>> This patch removes ELF dependencies from avr.h and redundancies from >>> avr.h and avr.c >>> >>> The AVR BE defines many things that are readily available in elfos.h. >>> ELF is the only target supported by av

Re: [Patch, AVR]: AVR backend cleanup: use elfos.h

2011-06-27 Thread Bernd Schmidt
On 06/27/11 19:34, Georg-Johann Lay wrote: > Denis Chertykov wrote: >> Approved. >> > http://gcc.gnu.org/ml/gcc-patches/2011-06/msg02011.html > > Is that also for the config.gcc part? > Or does it need review from global reviewer? Denis can approve AVR changes in config.gcc. Bernd

Re: [Patch, AVR]: AVR backend cleanup: use elfos.h

2011-06-27 Thread Georg-Johann Lay
Denis Chertykov wrote: > 2011/6/27 Georg-Johann Lay : >> This patch removes ELF dependencies from avr.h and redundancies from >> avr.h and avr.c >> >> The AVR BE defines many things that are readily available in elfos.h. >> ELF is the only target supported by avr-gcc. >> >> The patch includes elfos

Re: [Patch, AVR]: AVR backend cleanup: use elfos.h

2011-06-27 Thread Denis Chertykov
2011/6/27 Georg-Johann Lay : > This patch removes ELF dependencies from avr.h and redundancies from > avr.h and avr.c > > The AVR BE defines many things that are readily available in elfos.h. > ELF is the only target supported by avr-gcc. > > The patch includes elfos.h in config.gcc for all avr* ta

[Patch, AVR]: AVR backend cleanup: use elfos.h

2011-06-27 Thread Georg-Johann Lay
This patch removes ELF dependencies from avr.h and redundancies from avr.h and avr.c The AVR BE defines many things that are readily available in elfos.h. ELF is the only target supported by avr-gcc. The patch includes elfos.h in config.gcc for all avr* targets together with a new file avr/elf.h.