Hi!

On Thu, Jan 09, 2020 at 07:34:31PM -0500, Michael Meissner wrote:
> +static rtx
> +adjust_vec_address_pcrel (rtx addr, rtx element_offset, rtx base_tmp)
> +{
> +  rtx new_addr = NULL;

You don't need this variable, you can just return the value directly in
every case, and then you don't need all these "else"s either, or all
these nested blocks.  The NULL here is never used, either (you call
gcc_unreachable instead of returning it).

But, okay for trunk.  Thanks!


Segher

Reply via email to