Re: [qemu-s390x] [PATCH v3 12/15] s390x/tcg: Implement XxC and checks for most FP instructions

[David Hildenbrand]

On 18.02.19 13:27, David Hildenbrand wrote:
> With the floating-point extension facility
> - CONVERT FROM LOGICAL
> - CONVERT TO LOGICAL
> - CONVERT TO FIXED
> - CONVERT FROM FIXED
> - LOAD FP INTEGER
> have both, a rounding mode specification and the inexact-exception control
> (XxC). Other instructions will be handled separatly.
> 
> Check for valid rounding modes and forward also the XxC (via m4). To avoid
> a lot of boilerplate code and changes to the helpers, combine both, the
> m3 and m4 field in a combined 32 bit TCG variable. Perform checks at
> a central place, taking in account if the m3 or m4 field was ignore
> before the floating-point extension facility was introduced.
> 
> Signed-off-by: David Hildenbrand <address@hidden>
> ---


Here as well (from v2 review)

Reviewed-by: Richard Henderson <address@hidden>

-- 

Thanks,

David / dhildenb