[Bug ada/55243] STAMP variable is not defined in t-avr

[gjl at gcc dot gnu.org]

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

--- Comment #25 from Georg-Johann Lay <gjl at gcc dot gnu.org> 2012-12-14 18:26:07 UTC ---
Created attachment 28960
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=28960
Don't use STAMP to please Ada

(In reply to comment #24)
> > What I don't understand is what is bad with Rolf's proposal of defining STAMP?
> 
> We simply don't need to stamp anything for the gnattools.

Notice $(srcdir)/gcc/ada/gcc-interface/Makefile.in reads:

# target overrides
ifneq ($(tmake_file),)
include $(tmake_file)
endif

# host overrides
ifneq ($(xmake_file),)
include $(xmake_file)
endif

There are other backends like x86 and mips that use STAMP in their t-snip, so I
wonder how you can conclude that ada does not include code that might require
STAMP?

Here is a patch that bypasses the need of STAMP in t-avr.  Rolf can test it.


    PR55243
    * config/avr/t-avr: Don't automatically rebuild
    $(srcdir)/config/avr/t-multilib
    $(srcdir)/config/avr/avr-tables.opt
    $(srcdir)/doc/avr-mmcu.texi
    (avr-mcus): New phony target to build them on request.
    (s-avr-mlib, s-avr-mmcu-texi): Remove.
    * avr/avr-mcus.def: Adjust comments.