diff options
| author | Simon Marchi <simon.marchi@polymtl.ca> | 2021-04-08 09:49:30 -0400 |
|---|---|---|
| committer | Simon Marchi <simon.marchi@polymtl.ca> | 2021-04-08 09:49:30 -0400 |
| commit | 2b8d134be46071f6869ed45ebbf521d91c93fb66 (patch) | |
| tree | b7e9539146a59fd0cfa2dd562f217bbccbbf8aac /sim/mips | |
| parent | 16e311ab6d4d379da18ad03bc5373f621f488f41 (diff) | |
| download | fsf-binutils-gdb-2b8d134be46071f6869ed45ebbf521d91c93fb66.zip fsf-binutils-gdb-2b8d134be46071f6869ed45ebbf521d91c93fb66.tar.gz fsf-binutils-gdb-2b8d134be46071f6869ed45ebbf521d91c93fb66.tar.bz2 | |
sim: set ASAN_OPTIONS=detect_leaks=0 when running igen and opc2c
The igen/dgen and opc2c tools leak their heap-allocated memory (on
purpose) at program exit, which makes AddressSanitizer fail the tool
execution. This breaks the build, as it makes the tool return a
non-zero exit code.
Fix that by disabling leak detection through the setting of that
environment variable.
I also changed the opc2c rules for m32c to go through a temporary file.
What happened is that the failing opc2c would produce an incomplete file
(probably because ASan exits the process before stdout is flushed).
This meant that further make attempts didn't try to re-create the file,
as it already existed. A "clean" was therefore necessary. This can
also happen in regular builds if the user interrupts the build (^C) in
the middle of the opc2c execution and tries to resume it. Going to a
temporary file avoids this issue.
sim/m32c/ChangeLog:
* Makefile.in: Set ASAN_OPTIONS when running opc2c.
sim/mips/ChangeLog:
* Makefile.in: Set ASAN_OPTIONS when running igen.
sim/mn10300/ChangeLog:
* Makefile.in: Set ASAN_OPTIONS when running igen.
sim/ppc/ChangeLog:
* Makefile.in: Set ASAN_OPTIONS when running igen.
sim/v850/ChangeLog:
* Makefile.in: Set ASAN_OPTIONS when running igen.
Change-Id: I00f21d4dc1aff0ef73471925d41ce7c23e83e082
Diffstat (limited to 'sim/mips')
| -rw-r--r-- | sim/mips/ChangeLog | 4 | ||||
| -rw-r--r-- | sim/mips/Makefile.in | 20 |
2 files changed, 14 insertions, 10 deletions
diff --git a/sim/mips/ChangeLog b/sim/mips/ChangeLog index 8c2bc70..6d515f8 100644 --- a/sim/mips/ChangeLog +++ b/sim/mips/ChangeLog @@ -1,3 +1,7 @@ +2021-04-08 Simon Marchi <simon.marchi@polymtl.ca> + + * Makefile.in: Set ASAN_OPTIONS when running igen. + 2021-04-04 Steve Ellcey <sellcey@mips.com> Faraz Shahbazker <fshahbazker@wavecomp.com> diff --git a/sim/mips/Makefile.in b/sim/mips/Makefile.in index c66c6e8..0090645 100644 --- a/sim/mips/Makefile.in +++ b/sim/mips/Makefile.in @@ -149,7 +149,7 @@ BUILT_SRC_FROM_IGEN = \ $(BUILT_SRC_FROM_IGEN): tmp-igen tmp-igen: $(IGEN_INSN) $(IGEN_DC) ../igen/igen $(IGEN_INCLUDE) - ../igen/igen \ + $(IGEN) \ $(IGEN_TRACE) \ -I $(srcdir) \ -Werror \ @@ -220,7 +220,7 @@ BUILT_SRC_FROM_M16 = \ $(BUILT_SRC_FROM_M16): tmp-m16 tmp-m16: $(IGEN_INSN) $(IGEN_DC) ../igen/igen $(IGEN_INCLUDE) - ../igen/igen \ + $(IGEN) \ $(IGEN_TRACE) \ -I $(srcdir) \ -Werror \ @@ -255,7 +255,7 @@ tmp-m16: $(IGEN_INSN) $(IGEN_DC) ../igen/igen $(IGEN_INCLUDE) $(SHELL) $(srcdir)/../../move-if-change tmp-model.c m16_model.c $(SHELL) $(srcdir)/../../move-if-change tmp-support.h m16_support.h $(SHELL) $(srcdir)/../../move-if-change tmp-support.c m16_support.c - ../igen/igen \ + $(IGEN) \ $(IGEN_TRACE) \ -I $(srcdir) \ -Werror \ @@ -292,7 +292,7 @@ tmp-m16: $(IGEN_INSN) $(IGEN_DC) ../igen/igen $(IGEN_INCLUDE) $(SHELL) $(srcdir)/../../move-if-change tmp-model.c m32_model.c $(SHELL) $(srcdir)/../../move-if-change tmp-support.h m32_support.h $(SHELL) $(srcdir)/../../move-if-change tmp-support.c m32_support.c - ../igen/igen \ + $(IGEN) \ $(IGEN_TRACE) \ -I $(srcdir) \ -Werror \ @@ -346,7 +346,7 @@ BUILT_SRC_FROM_MICROMIPS = \ $(BUILT_SRC_FROM_MICROMIPS): tmp-micromips tmp-micromips: $(IGEN_INSN) $(IGEN_DC) ../igen/igen $(IGEN_INCLUDE) - ../igen/igen \ + $(IGEN) \ $(IGEN_TRACE) \ -I $(srcdir) \ -Werror \ @@ -391,7 +391,7 @@ tmp-micromips: $(IGEN_INSN) $(IGEN_DC) ../igen/igen $(IGEN_INCLUDE) micromips16_support.h $(SHELL) $(srcdir)/../../move-if-change tmp-support.c \ micromips16_support.c - ../igen/igen \ + $(IGEN) \ $(IGEN_TRACE) \ -I $(srcdir) \ -Werror \ @@ -436,7 +436,7 @@ tmp-micromips: $(IGEN_INSN) $(IGEN_DC) ../igen/igen $(IGEN_INCLUDE) micromips32_support.h $(SHELL) $(srcdir)/../../move-if-change tmp-support.c \ micromips32_support.c - ../igen/igen \ + $(IGEN) \ $(IGEN_TRACE) \ -I $(srcdir) \ -Werror \ @@ -481,7 +481,7 @@ tmp-micromips: $(IGEN_INSN) $(IGEN_DC) ../igen/igen $(IGEN_INCLUDE) micromips_m32_support.h $(SHELL) $(srcdir)/../../move-if-change tmp-support.c \ micromips_m32_support.c - ../igen/igen \ + $(IGEN) \ $(IGEN_TRACE) \ -I $(srcdir) \ -Werror \ @@ -521,7 +521,7 @@ tmp-mach-multi: $(IGEN_INSN) $(IGEN_DC) ../igen/igen $(IGEN_INCLUDE) m16*) e="-B 16 -H 15 -o $(M16_DC) -F 16" ;; \ *) e="-B 32 -H 31 -o $(IGEN_DC) -F $${f}" ;; \ esac; \ - ../igen/igen \ + $(IGEN) \ $(IGEN_TRACE) \ $${e} \ -I $(srcdir) \ @@ -574,7 +574,7 @@ tmp-mach-multi: $(IGEN_INSN) $(IGEN_DC) ../igen/igen $(IGEN_INCLUDE) done touch tmp-mach-multi tmp-itable-multi: $(IGEN_INSN) $(IGEN_DC) ../igen/igen $(IGEN_INCLUDE) - ../igen/igen \ + $(IGEN) \ $(IGEN_TRACE) \ -I $(srcdir) \ -Werror \ |