This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Fix SLP wrong-code with VECTOR_BOOLEAN_TYPE_P (PR tree-optimization/71259)


On 8 June 2016 at 16:50, Jakub Jelinek <jakub@redhat.com> wrote:
> On Wed, Jun 08, 2016 at 04:44:00PM +0200, Christophe Lyon wrote:
>> I've tried the attached patch (which does only dg-options ->
>> dg-additional-options).
>> For GCC, it's better, except that on arm-none-eabi qemu complains about
>> an illegal instruction when asked to use arm926 and GCC is configured with
>> the default cpu. Maybe that's because check_vect does not have the expected
>
> check_vect installs a SIGILL handler and if the insn is invalid, excepts
> a signal to be raised.  Is that not the case with qemu?  Or is qemu just
> being too noisy?
>
qemu complains when executing check_vect's prologue, which contains
movw    r1, #35712
which is not supported either on arm926.

I don't know if the signal is not raised or if qemu hides it (I'm using qemu
user mode)

>> <DATE>  Christophe Lyon  <christophe.lyon@linaro.org>
>>
>>    * gcc.dg/vect/YYY.c: Use dg-additional options instead of dg-options.
>
> Please list all the changed tests in the ChangeLog (with : Likewise. for
> all but the first one).
>
Yes, that's what I meant.


> Ok with that change.
>
>         Jakub


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]