> The patch fixes that by calculating the size of the first element by
> taking the size of the outer mode and subtracting the size of the second
> element.
> 
> I've added an assert to make sure that the second element is not also a
> const_int, as a vec_concat of const_ints doesn't make sense as far as I can
> see.

I'm not sure about the assert, can't we just punt in this case?

> Bootstrapped and tested on aarch64-none-linux-gnu,
> arm-none-linux-gnueabihf, x86_64-linux-gnu.
> This bug appears on trunk, 4.9 and 4.8, so it's not a regression on the
> release branches but it is a wrong-code bug.

I think that the fix would be acceptable for GCC 5 without the assert.

-- 
Eric Botcazou

Reply via email to