Re: [SH] Check FPSCR.PR for fldi0, fldi1 insns in simulator

[Simon Marchi <simark@simark.ca>]

On 2026-03-16 21:18, Oleg Endo wrote:
> On Wed, 2026-03-11 at 10:39 -0400, Simon Marchi wrote:
>>
>> Ok, well since we're in "undefined behavior" territory we could argue
>> that the current sim behavior is fine.  But I think the patch is still
>> good, traping is a more useful behavior to catch something you don't
>> want to happen.
> 
> Yes, the current sim behavior is not wrong per se.  But like you said,
> having it trap is more useful for sim's main purpose -- pre-verifying code. 
> 
> 
>>>> The patch LGTM, but I am not a maintainer of the sim, so:
>>>>
>>>> Reviewed-By: Simon Marchi <simon.marchi@efficios.com>
>>>>
>>>> One note, please put the relevant information (the body of your email)
>>>> into the commit log itself).
>>>
>>> Yeah, sure I can expand the comment of the commit.  Updated patch/commit
>>> attached.
>>
>> I would give it a week for Andrew or Mike to review and approve, but
>> otherwise I would feel comfortable merging it.
>>
> 
> Alright.  Last time I've checked I had write access.  I can commit & push it
> myself.  Just let me know.

Yes, please go ahead.

Simon