diff options
author | Stafford Horne <shorne@gmail.com> | 2019-03-28 06:40:30 +0900 |
---|---|---|
committer | Stafford Horne <shorne@gmail.com> | 2019-03-28 06:40:30 +0900 |
commit | b6061d4d383b08966d16a0b0f72c69f35fc4beb9 (patch) | |
tree | bbd1767c1d65b7702ae7f2ac5cf1a28437890749 /sim | |
parent | ef9866970ce6683d40465fb7c3168f87a1dcd1b7 (diff) | |
download | fsf-binutils-gdb-b6061d4d383b08966d16a0b0f72c69f35fc4beb9.zip fsf-binutils-gdb-b6061d4d383b08966d16a0b0f72c69f35fc4beb9.tar.gz fsf-binutils-gdb-b6061d4d383b08966d16a0b0f72c69f35fc4beb9.tar.bz2 |
sim/common: Fix warnings: "warning: implicit declaration of function..."
During building of several cgen simulator's I notices the below
warnings. Adding includes fixes these.
Including config.h allows stdio.h to properly configure itself to expose
asprintf().
The other warnings for abort, free, memset, strlen are trivial.
Warnings:
../../../binutils-gdb/sim/or1k/../common/sim-watch.c: In function ‘sim_watchpoint_install’:
../../../binutils-gdb/sim/or1k/../common/sim-watch.c:415:10: warning: implicit declaration of function ‘asprintf’; did you mean ‘vasprintf’? [-Wimplicit-function-declaration]
if (asprintf (&name, "watch-%s-%s",
^~~~~~~~
vasprintf
../../../binutils-gdb/sim/lm32/../common/hw-device.c: In function ‘hw_strdup’:
../../../binutils-gdb/sim/lm32/../common/hw-device.c:59:34: warning: implicit declaration of function ‘strlen’ [-Wimplicit-function-declaration]
char *dup = hw_zalloc (me, strlen (str) + 1);
^~~~~~
../../../binutils-gdb/sim/lm32/../common/hw-events.c: In function ‘hw_event_queue_schedule’:
../../../binutils-gdb/sim/lm32/../common/hw-events.c:92:3: warning: implicit declaration of function ‘memset’ [-Wimplicit-function-declaration]
memset (&dummy, 0, sizeof dummy);
^~~~~~
../../../binutils-gdb/sim/lm32/../common/hw-handles.c: In function ‘hw_handle_remove_ihandle’:
../../../binutils-gdb/sim/lm32/../common/hw-handles.c:211:4: warning: implicit declaration of function ‘free’ [-Wimplicit-function-declaration]
free (delete);
^~~~
../../../binutils-gdb/sim/lm32/../common/sim-fpu.c: In function ‘pack_fpu’:
../../../binutils-gdb/sim/lm32/../common/sim-fpu.c:292:7: warning: implicit declaration of function ‘abort’ [-Wimplicit-function-declaration]
abort ();
^~~~~
sim/common/ChangeLog:
* sim-options.c: Include "config.h".
Include <stdio.h>.
* sim-watch.c: Include "config.h".
Include <stdio.h>.
* hw-device.c: Include <string.h>.
* hw-events.c: Include <string.h>.
* hw-handles.c: Include <stdlib.h>.
* sim-fpu.c: Include <stdlib.h>.
Diffstat (limited to 'sim')
-rw-r--r-- | sim/common/ChangeLog | 11 | ||||
-rw-r--r-- | sim/common/hw-device.c | 4 | ||||
-rw-r--r-- | sim/common/hw-events.c | 3 | ||||
-rw-r--r-- | sim/common/hw-handles.c | 3 | ||||
-rw-r--r-- | sim/common/sim-fpu.c | 3 | ||||
-rw-r--r-- | sim/common/sim-options.c | 2 | ||||
-rw-r--r-- | sim/common/sim-watch.c | 2 |
7 files changed, 28 insertions, 0 deletions
diff --git a/sim/common/ChangeLog b/sim/common/ChangeLog index 653a2fb..53dde49 100644 --- a/sim/common/ChangeLog +++ b/sim/common/ChangeLog @@ -1,5 +1,16 @@ 2019-03-28 Stafford Horne <shorne@gmail.com> + * sim-options.c: Include "config.h". + Include <stdio.h>. + * sim-watch.c: Include "config.h". + Include <stdio.h>. + * hw-device.c: Include <string.h>. + * hw-events.c: Include <string.h>. + * hw-handles.c: Include <stdlib.h>. + * sim-fpu.c: Include <stdlib.h>. + +2019-03-28 Stafford Horne <shorne@gmail.com> + * Make-common.in (sim-arange_h): Remove sim-arange.c * sim-arange.c: Remove SIM_ARANGE_C. Add ifdef for _SIM_ARANGE_C_. diff --git a/sim/common/hw-device.c b/sim/common/hw-device.c index ee1bfad..458ee22 100644 --- a/sim/common/hw-device.c +++ b/sim/common/hw-device.c @@ -27,6 +27,10 @@ #include <stdlib.h> #endif +#if HAVE_STRING_H +#include <string.h> +#endif + /* Address methods */ const hw_unit * diff --git a/sim/common/hw-events.c b/sim/common/hw-events.c index e652336..f78be2a 100644 --- a/sim/common/hw-events.c +++ b/sim/common/hw-events.c @@ -23,6 +23,9 @@ along with this program. If not, see <http://www.gnu.org/licenses/>. */ #include "sim-events.h" +#if HAVE_STRING_H +#include <string.h> +#endif /* The hw-events object is implemented using sim-events */ diff --git a/sim/common/hw-handles.c b/sim/common/hw-handles.c index 2848b9b..d056562 100644 --- a/sim/common/hw-handles.c +++ b/sim/common/hw-handles.c @@ -23,6 +23,9 @@ #include "hw-main.h" #include "hw-base.h" +#if HAVE_STDLIB_H +#include <stdlib.h> +#endif struct hw_handle_mapping { diff --git a/sim/common/sim-fpu.c b/sim/common/sim-fpu.c index 81cdbf5..74f5fd4 100644 --- a/sim/common/sim-fpu.c +++ b/sim/common/sim-fpu.c @@ -41,6 +41,9 @@ along with this program. If not, see <http://www.gnu.org/licenses/>. */ #include "sim-io.h" #include "sim-assert.h" +#ifdef HAVE_STDLIB_H +#include <stdlib.h> +#endif /* Debugging support. If digits is -1, then print all digits. */ diff --git a/sim/common/sim-options.c b/sim/common/sim-options.c index 69aebfe..dc4a712 100644 --- a/sim/common/sim-options.c +++ b/sim/common/sim-options.c @@ -17,6 +17,7 @@ GNU General Public License for more details. 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 "sim-main.h" #ifdef HAVE_STRING_H #include <string.h> @@ -29,6 +30,7 @@ along with this program. If not, see <http://www.gnu.org/licenses/>. */ #include <stdlib.h> #endif #include <ctype.h> +#include <stdio.h> #include "libiberty.h" #include "sim-options.h" #include "sim-io.h" diff --git a/sim/common/sim-watch.c b/sim/common/sim-watch.c index 6c357f8..174336b 100644 --- a/sim/common/sim-watch.c +++ b/sim/common/sim-watch.c @@ -17,12 +17,14 @@ GNU General Public License for more details. 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 "sim-main.h" #include "sim-options.h" #include "sim-assert.h" #include <ctype.h> +#include <stdio.h> #ifdef HAVE_STRING_H #include <string.h> |