On 4/29/24 1:14 AM, Eric Botcazou wrote:
Hi,

this removes the last parameter of choose_multiplier, which is unused, adds
another assertion and more details to the description and various comments.
Likewise to the closely related invert_mod2n, except for the last parameter.

Tested on x86-64/Linux, OK for the mainline?


2024-04-29  Eric Botcazou  <ebotca...@adacore.com>

        * expmed.h (choose_multiplier): Tweak description and remove last
        parameter.
        * expmed.cc (choose_multiplier): Likewise.  Add assertion for the
        third parameter and adds details to various comments.
        (invert_mod2n): Tweak description and add assertion for the first
        parameter.
        (expand_divmod): Adjust calls to choose_multiplier.
        * tree-vect-generic.cc (expand_vector_divmod): Likewise.
        * tree-vect-patterns.cc (vect_recog_divmod_pattern): Likewise.
OK. Consider waiting to commit though as we want to make it easy to cherry pick patches over to the release branch if needed.

Jeff

Reply via email to