diff options
| author | Thomas Huth <thuth@redhat.com> | 2018-10-09 09:40:33 +0200 |
|---|---|---|
| committer | Thomas Huth <thuth@redhat.com> | 2018-10-16 17:57:23 +0200 |
| commit | ba3fa397a1b7b1c43d8cff33f7fdfce52298d287 (patch) | |
| tree | f2e8ff183b8db6e05ccf4dabe5c4dfb69cb6ba45 | |
| parent | c3776f4de5919f6e5f0e7d38a368634b194a4c4d (diff) | |
| download | focaccia-qemu-ba3fa397a1b7b1c43d8cff33f7fdfce52298d287.tar.gz focaccia-qemu-ba3fa397a1b7b1c43d8cff33f7fdfce52298d287.zip | |
target/cris/translate: Get rid of qemu_log_separate()
The gen_BUG() function calls already cpu_abort(), which prints the information to stderr and the log already. So instead of additionally printing the dc->pc via fprintf() and qemu_log here, too, we can simply pass this information to cpu_abort() instead. Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> Signed-off-by: Thomas Huth <thuth@redhat.com>
| -rw-r--r-- | target/cris/translate.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/target/cris/translate.c b/target/cris/translate.c index 4ae1c04daf..11b2c11174 100644 --- a/target/cris/translate.c +++ b/target/cris/translate.c @@ -137,11 +137,7 @@ typedef struct DisasContext { static void gen_BUG(DisasContext *dc, const char *file, int line) { - fprintf(stderr, "BUG: pc=%x %s %d\n", dc->pc, file, line); - if (qemu_log_separate()) { - qemu_log("BUG: pc=%x %s %d\n", dc->pc, file, line); - } - cpu_abort(CPU(dc->cpu), "%s:%d\n", file, line); + cpu_abort(CPU(dc->cpu), "%s:%d pc=%x\n", file, line, dc->pc); } static const char *regnames_v32[] = |