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