Have regdat.sh always rewrite the output register file
authorPatrick Palka <patrick@parcs.ath.cx>
Fri, 10 Jan 2014 03:55:50 +0000 (22:55 -0500)
committerTom Tromey <tromey@redhat.com>
Mon, 13 Jan 2014 19:53:53 +0000 (12:53 -0700)
commitbdf61915fc102bc07c12aa450376d7402bd427b4
tree702d1094eea7869f65e241e1ef0488b7e74068d7
parentf71e1a8ddb5a41dddb0dea92aefe694e26d4c174
Have regdat.sh always rewrite the output register file

The rules for generating the output register files look like:

amd64.c : $(srcdir)/../regformats/i386/amd64.dat $(regdat_sh)
$(SHELL) $(regdat_sh) $(srcdir)/../regformats/i386/amd64.dat amd64.c

According to this rule, if regdat.sh is newer than amd64.c, then
regdat.sh shall be invoked on amd64.dat.  According to regdat.sh, if the
script determines that the output file amd64.c has not changed, then it
will not overwrite the existing output file.  This means that a
subsequent invocation of make will trigger the above rule again as
regdat.sh will be perpetually newer than amd64.c.

This then shows up in the make output like so:

/bin/bash ./../regformats/regdat.sh ./../regformats/i386/amd64-linux.dat amd64-linux.c
amd64-linux.c unchanged.
/bin/bash ./../regformats/regdat.sh ./../regformats/i386/amd64-avx-linux.dat amd64-avx-linux.c
amd64-avx-linux.c unchanged.
...

To fix this pathological behavior, it suffices to have regdat.sh
unconditionally rewrite the output register file.

On my machine, which has a regdat.sh file that is newer than some of the
input register files, this change speeds up every invocation of make
under gdb/ by about 5 seconds.
gdb/ChangeLog
gdb/regformats/regdat.sh