[Patch AArch64] Fix for PR62040

Kyrill Tkachov kyrylo.tkachov@arm.com
Wed Aug 20 11:27:00 GMT 2014


Hi Carrot,

cc'ing the aarch64 maintainers...

On 20/08/14 00:43, Carrot Wei wrote:
> Hi
>
> Current AArch64 backend can generate rtl expressions like
> (vec_duplicate:DI (const_int 0 [0])), which causes ICE in
> simplify_const_unary_operation because vec_duplicate should generate
> vector mode only.
>
> As suggested by Andrew in the bug entry, I split the original insn
> patterns to avoid scalar mode vec_duplicate expression.

The documentation does say that vec_concat can work on scalars, so it 
seems ok to me at a glance (but I can't approve it myself).

Would be nice to have an addition to the testsuite though...

Kyrill

> Passed regression tests on qemu without failure.
> OK for trunk and 4.9 branch?
>
> thanks
> Guozhi Wei
>
> 2014-08-19  Guozhi Wei  <carrot@google.com>
>
>          PR target/62040
>          * config/aarch64/iterators.md (VQ_NO2E, VQ_2E): New iterators.
>          * config/aarch64/aarch64-simd.md (move_lo_quad_internal_<mode>): Split
>          it into two patterns.
>          (move_lo_quad_internal_be_<mode>): Likewise.




More information about the Gcc-patches mailing list