Re: [hatari-devel] Debugger FPU bug ? |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/hatari-devel Archives
]
- To: hatari-devel@xxxxxxxxxxxxxxxxxxx
- Subject: Re: [hatari-devel] Debugger FPU bug ?
- From: Thomas Huth <th.huth@xxxxxxxxx>
- Date: Wed, 12 Feb 2020 19:03:58 +0100
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/simple; d=posteo.de; s=2017; t=1581530640; bh=ZJuIsMFh86MDzr/SPfbNtVB09/avXz2lVY1lLDyqzII=; h=Date:From:To:Subject:From; b=bIFd0egXBSwPVaD9m1GLHDZQoJcE3no8FXVHQPKzQgJ25O7VjLzfPy4wBXcMU6tco IldMxKLqDaia0+UDjzBBTCggrZVW2hYvGMQY6PaN9b5SEtc/9Rf+2MD+/btinZLzOr +6Ids8MptgZYxK0jeKoEbEe7MALzfCC9tCgGW7OJpS3dJhls8CpSt5OpJtxeSCmWQB 77ykziJBjhrXCwIf1Ds2nrFZ+lrj3v4NhiVR7eg1GJmzEf5erGjuGH8p2H9Y1mne/x lrQ5iKldVl9K7LtOQAFo2JyWhotY9onx3JHISCCC7Va0hYAGzUtp1fGseJr3DJSIDf uHK+vLG9TxfAQ==
Am Mon, 10 Feb 2020 22:46:02 +0100
schrieb Nicolas Pomarède <npomarede@xxxxxxxxxxxx>:
> Le 10/02/2020 à 00:02, Eero Tamminen a écrit :
> > Hi,
> >
> > Ok, so it's the external disassembler which is
> > buggy.
Again... Sigh, it's really annoying to maintain this piece of code :-(
> > Nicolas, where we are with removing the external
> > one, once the UAE core one gets features that
> > were missing from it?
The deprecation note has been added in commit c79254943 - that was
after the release of version 2.2.1. So we should still keep it for at
least one more public release that the users have the possibility to
read the deprecation note before we remove it.
By the way, Nicolas, it's been more than one year now since the last
release, what do you think about doing a new release soon?
> I don't remember if it was decided to drop it completely, but at
> least we can make internal WinUAE cpu core disassembler the default
> now, as it was much improved lately (MMU and FPU opcodes) and should
> now be more accurate than external disassembler in all cases.
>
> Note that external disassembler can still be useful in the sense that
> its output can be more or less recompiled with devpac for example. On
> the opposite WinUAE's internal debugger add many useful infos (memory
> content, effective address display), but this can make each disasm
> line not directly assemblable without editing it first.
>
> For now, maybe it's just enough to change this line in configuration.c
>
> ConfigureParams.Debugger.bDisasmUAE = false
>
> and to use 'true' instead of 'false'
Good idea, I've just commited a patch for this now.
Thomas