Re: [PATCH RESEND v3 4/8] host-utils: Implemented unsigned 256-by-128 division

[Richard Henderson]

On 5/25/22 06:49, Lucas Mateus Castro(alqotel) wrote:
> From: "Lucas Mateus Castro (alqotel)"<lucas.araujo@eldorado.org.br>
>
> Based on already existing QEMU implementation, created an unsigned 256
> bit by 128 bit division needed to implement the vector divide extended
> unsigned instruction from PowerISA3.1
>
> Signed-off-by: Lucas Mateus Castro (alqotel)<lucas.araujo@eldorado.org.br>
> ---
> This patch had received Reviewed-by by Richard Henderson pending on the
> placemente of clz128 being moved to int128.h, but clz128 ended up being changed
> to accommodate to int128.h (i.e. the lack of clz64), so out of precaution I'd
> like to request a review of the clz128 implementation
> ---
>   include/qemu/host-utils.h |   2 +
>   include/qemu/int128.h     |  38 +++++++++++
>   util/host-utils.c         | 129 ++++++++++++++++++++++++++++++++++++++
>   3 files changed, 169 insertions(+)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~