Re: [REPOST][PATCH v4] H8/300: sim: Fix simulator hang caused by qsort on Windows/MinGW

[Jeff Law <jeffrey.law@oss.qualcomm.com>]

On 10/5/26 12:05 AM, Jan Dubiec wrote:
> On Windows, qsort() uses an unstable sorting algorithm, which results
> in a "shuffled" opcode table rather than a properly sorted one, causing
> the entire simulator to hang.
>
> The simulator happens to work on Linux, but this behavior is not
> guaranteed, because the glibc documentation clearly states that "If
> two elements compare equal, their order after sorting is unpredictable."
>
> This patch introduces two additional sort keys to the instruction
> comparator function, making the resulting opcode table as close as
> possible to the one that would be produced by a stable sorting algorithm.
>
> It also fixes an issue in the instruction decoder where, for example,
> mov.l @er7+,er1 is recognized as ldm.l @er7+,(er0-er1) if the opcode
> table is sorted "the wrong way", i.e. when ldm appears before mov
> in the table.
>
> No regressions were found.
This seems generally sensible to me.  I'd probably kill that ldm/stm 
special case, particularly if the new comparator avoids the problem.

I'd instead be more interested in tripping an assert if we ever have two 
things compare equal


jeff