Re: [PATCH] Fix failing assertion in calls.c:store_unaligned_arguments_into_pseudos

2013-11-14 Thread Ulrich Weigand
Jeff Law wrote: > On 11/11/13 07:32, Ulrich Weigand wrote: > > 2013-11-11 Ulrich Weigand > > > > * calls.c (store_unaligned_arguments_into_pseudos): Skip PARALLEL > > arguments. > OK, so after a lot of worrying, I think this is OK. I kept thinking > this had to tie into the BLKmode ret

Re: [PATCH] Fix failing assertion in calls.c:store_unaligned_arguments_into_pseudos

2013-11-13 Thread Jeff Law
On 11/11/13 07:32, Ulrich Weigand wrote: Hello, when implementing the new ABI for powerpc64le-linux, I ran into an assertion failure in store_unaligned_arguments_into_pseudos: gcc_assert (args[i].partial % UNITS_PER_WORD == 0); This can happen in the new ABI since we pass "homogene

Re: [PATCH] Fix failing assertion in calls.c:store_unaligned_arguments_into_pseudos

2013-11-13 Thread Jeff Law
On 11/11/13 12:10, Ulrich Weigand wrote: Jeff Law wrote: On 11/11/13 07:32, Ulrich Weigand wrote: However, looking more closely, it seems store_unaligned_arguments_into_pseudos is not really useful for PARALLEL arguments in the first place. What this routine does is load arguments into args[i]

Re: [PATCH] Fix failing assertion in calls.c:store_unaligned_arguments_into_pseudos

2013-11-11 Thread Ulrich Weigand
Jeff Law wrote: > On 11/11/13 07:32, Ulrich Weigand wrote: > > However, looking more closely, it seems > > store_unaligned_arguments_into_pseudos > > is not really useful for PARALLEL arguments in the first place. What this > > routine does is load arguments into args[i].aligned_regs. But if we

Re: [PATCH] Fix failing assertion in calls.c:store_unaligned_arguments_into_pseudos

2013-11-11 Thread Jeff Law
On 11/11/13 07:32, Ulrich Weigand wrote: Hello, when implementing the new ABI for powerpc64le-linux, I ran into an assertion failure in store_unaligned_arguments_into_pseudos: gcc_assert (args[i].partial % UNITS_PER_WORD == 0); This can happen in the new ABI since we pass "homogene

[PATCH] Fix failing assertion in calls.c:store_unaligned_arguments_into_pseudos

2013-11-11 Thread Ulrich Weigand
Hello, when implementing the new ABI for powerpc64le-linux, I ran into an assertion failure in store_unaligned_arguments_into_pseudos: gcc_assert (args[i].partial % UNITS_PER_WORD == 0); This can happen in the new ABI since we pass "homogeneous structures" consisting of soleley floati