yggdrasil/binutils-gdb
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

sim: igen: drop config.h & header checking

Browse Source 
While the configure script was checking for a bunch of headers, only
one of them was conditionally included in the source (unistd.h).  The
rest were always included.  Based on those usage this whole time, we
can reasonably assume that the build also has unistd.h.

All the other files including config.h never actually used any defines
from the header.
This commit is contained in:
Mike Frysinger 2021-02-22 00:51:34 -05:00
parent 88d7273afd
commit 9f34b60a43
12 changed files with 58 additions and 1125 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
sim/igen/ChangeLog
View File

@ -1,3 +1,17 @@
2021-02-28 Mike Frysinger <vapier@gentoo.org>
* Makefile.in (filter_filename.o): Delete unused target.
(distclean): Remove config.h.
(config.h, stamp-h): Delete targets.
* configure.ac (AC_CONFIG_HEADER): Delete call.
(AC_CHECK_HEADERS, AC_HEADER_DIRENT): Likewise.
(AC_OUTPUT): Remove 2nd arg.
* filter.c: Remove config.h include.
* filter_host.c, igen.c, lf.c, misc.c, misc.h: Likewise.
* table.c: Likewise. Remove HAVE_UNISTD_H check.
* configure: Regenerate.
* config.in: Removed.
2021-02-28 Mike Frysinger <vapier@gentoo.org>
* Makefile.in (program_transform_name, AR, RANLIB): Delete.

9
sim/igen/Makefile.in
View File

@ -88,9 +88,6 @@ all: igen
.c.o:
$(CC_FOR_BUILD) -c $(BUILD_CFLAGS) $<
filter_filename.o: filter_filename.c filter_filename.h config.h ppc-config.h
IGEN_OBJS=\
table.o \
lf.o misc.o \
@ -159,7 +156,7 @@ clean mostlyclean:
rm -f tmp-* *.[oasi] core igen
distclean realclean: clean
rm -f TAGS Makefile config.cache config.status config.h defines.h stamp-h config.log
rm -f TAGS Makefile config.cache config.status defines.h config.log
maintainer-clean: distclean
rm -f *~ *.log ppc-config.h core *.core
@ -167,10 +164,6 @@ maintainer-clean: distclean
Makefile: Makefile.in config.status
CONFIG_FILES=Makefile CONFIG_HEADERS= $(SHELL) ./config.status
config.h: stamp-h ; @true
stamp-h: config.in config.status
CONFIG_FILES= CONFIG_HEADERS=config.h:config.in $(SHELL) ./config.status
config.status: configure
$(SHELL) ./config.status --recheck

64
sim/igen/config.in
View File

@ -1,64 +0,0 @@
/* config.in. Generated from configure.ac by autoheader. */
/* Define to 1 if you have the <dirent.h> header file, and it defines `DIR'.
*/
#undef HAVE_DIRENT_H
/* Define to 1 if you have the <inttypes.h> header file. */
#undef HAVE_INTTYPES_H
/* Define to 1 if you have the <memory.h> header file. */
#undef HAVE_MEMORY_H
/* Define to 1 if you have the <ndir.h> header file, and it defines `DIR'. */
#undef HAVE_NDIR_H
/* Define to 1 if you have the <stdint.h> header file. */
#undef HAVE_STDINT_H
/* Define to 1 if you have the <stdlib.h> header file. */
#undef HAVE_STDLIB_H
/* Define to 1 if you have the <strings.h> header file. */
#undef HAVE_STRINGS_H
/* Define to 1 if you have the <string.h> header file. */
#undef HAVE_STRING_H
/* Define to 1 if you have the <sys/dir.h> header file, and it defines `DIR'.
*/
#undef HAVE_SYS_DIR_H
/* Define to 1 if you have the <sys/ndir.h> header file, and it defines `DIR'.
*/
#undef HAVE_SYS_NDIR_H
/* Define to 1 if you have the <sys/stat.h> header file. */
#undef HAVE_SYS_STAT_H
/* Define to 1 if you have the <sys/types.h> header file. */
#undef HAVE_SYS_TYPES_H
/* Define to 1 if you have the <unistd.h> header file. */
#undef HAVE_UNISTD_H
/* Define to the address where bug reports for this package should be sent. */
#undef PACKAGE_BUGREPORT
/* Define to the full name of this package. */
#undef PACKAGE_NAME
/* Define to the full name and version of this package. */
#undef PACKAGE_STRING
/* Define to the one symbol short name of this package. */
#undef PACKAGE_TARNAME
/* Define to the home page for this package. */
#undef PACKAGE_URL
/* Define to the version of this package. */
#undef PACKAGE_VERSION
/* Define to 1 if you have the ANSI C header files. */
#undef STDC_HEADERS

1074
sim/igen/configure vendored
View File

File diff suppressed because it is too large Load Diff

9
sim/igen/configure.ac
View File

@ -18,14 +18,7 @@ fi
AC_CANONICAL_SYSTEM
AC_CONFIG_HEADER(config.h:config.in)
# NB: You can assume C11 headers exist.
AC_CHECK_HEADERS(sys/stat.h sys/types.h unistd.h)
AC_HEADER_DIRENT
AC_SUBST(CC_FOR_BUILD)
AC_SUBST(CFLAGS_FOR_BUILD)
AC_OUTPUT(Makefile,
[case x$CONFIG_HEADERS in xconfig.h:config.in) echo > stamp-h ;; esac])
AC_OUTPUT(Makefile)

3
sim/igen/filter.c
View File

@ -21,9 +21,6 @@
#include <stdio.h>
#include "config.h"
#include <string.h>
#include "misc.h"

1
sim/igen/filter_host.c
View File

@ -19,7 +19,6 @@
You should have received a copy of the GNU General Public License
along with this program. If not, see <http://www.gnu.org/licenses/>. */
#include "config.h"
#include "filter_host.h"
/* Shorten traces by eliminating the directory component to filenames. */

1
sim/igen/igen.c
View File

@ -25,7 +25,6 @@
#include "misc.h"
#include "lf.h"
#include "table.h"
#include "config.h"
#include "filter.h"
#include "igen.h"

1
sim/igen/lf.c
View File

@ -25,7 +25,6 @@
#include <stdarg.h>
#include <ctype.h>
#include "config.h"
#include "misc.h"
#include "lf.h"

1
sim/igen/misc.c
View File

@ -25,7 +25,6 @@
#include <stdarg.h>
#include <ctype.h>
#include "config.h"
#include "misc.h"
#include <stdlib.h>

3
sim/igen/misc.h
View File

@ -23,9 +23,6 @@
/* Frustrating header junk */
#include "config.h"
enum
{
default_insn_bit_size = 32,

3
sim/igen/table.c
View File

@ -27,14 +27,11 @@
#include <fcntl.h>
#include <ctype.h>
#include "config.h"
#include "misc.h"
#include "lf.h"
#include "table.h"
#ifdef HAVE_UNISTD_H
#include <unistd.h>
#endif
#include <stdlib.h>
typedef struct _open_table open_table;