diff options
author | Andrew Burgess <andrew.burgess@embecosm.com> | 2021-09-23 18:20:09 +0100 |
---|---|---|
committer | Andrew Burgess <andrew.burgess@embecosm.com> | 2021-10-04 10:52:35 +0100 |
commit | 86c1ed137a1e9cc5ec73cb61836cebf304996aa8 (patch) | |
tree | 1a43d88fc62b24de9373da930a6b9d25205eb547 /gdbsupport/config.in | |
parent | adc82fdb715184fa8e9acf47179e67acc82b8375 (diff) | |
download | gdb-86c1ed137a1e9cc5ec73cb61836cebf304996aa8.zip gdb-86c1ed137a1e9cc5ec73cb61836cebf304996aa8.tar.gz gdb-86c1ed137a1e9cc5ec73cb61836cebf304996aa8.tar.bz2 |
gdbsupport: remove attempt to define TARGET_WORD_SIZE
In the gdbsupport configure.ac file, there is an attempt to define
TARGET_WORD_SIZE. This is done by running grep on the file
../bfd/bfd-in3.h.
The problem with this is, the file bfd-in3.h is generated into the bfd
build directory when bfd is configured, and there is no dependency
between the gdbsupport module and the bfd module, so, for example, if
I do:
$ ../src/configure
$ make all-gdbsupport
Then bfd will neither be configured, or built. In this case
TARGET_WORD_SIZE ends up being defined, but with no value because the
grep on bfd-in3.h fails.
However, it turns out that this doesn't matter; we don't actually use
TARGET_WORD_SIZE anywhere.
My proposal in this commit is to just remove the definition of
TARGET_WORD_SIZE, the alternative would be to add a dependency between
configure-gdbsupport and configure-bfd into Makefile.def, but adding a
dependency for something we don't need seems pretty pointless.
Diffstat (limited to 'gdbsupport/config.in')
-rw-r--r-- | gdbsupport/config.in | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/gdbsupport/config.in b/gdbsupport/config.in index f46e261..e4e6fc0 100644 --- a/gdbsupport/config.in +++ b/gdbsupport/config.in @@ -354,9 +354,6 @@ /* Define to 1 if you have the ANSI C header files. */ #undef STDC_HEADERS -/* Define to the word size for the target. */ -#undef TARGET_WORD_SIZE - /* Enable extensions on AIX 3, Interix. */ #ifndef _ALL_SOURCE # undef _ALL_SOURCE |