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/configure | |
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/configure')
-rwxr-xr-x | gdbsupport/configure | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/gdbsupport/configure b/gdbsupport/configure index a9dd02c..0880022 100755 --- a/gdbsupport/configure +++ b/gdbsupport/configure @@ -10286,13 +10286,6 @@ ac_compiler_gnu=$ac_cv_c_compiler_gnu -TARGET_WORD_SIZE=`sed -n 's,#define BFD_ARCH_SIZE \(.*\)$,\1,p' ../bfd/bfd-in3.h` - -cat >>confdefs.h <<_ACEOF -#define TARGET_WORD_SIZE $TARGET_WORD_SIZE -_ACEOF - - case ${host} in *mingw32*) |