Re: [PATCH] S/390: Wide int support.

2015-12-18 Thread Richard Sandiford
"Ulrich Weigand" writes: > Richard Sandiford wrote: >> "Ulrich Weigand" writes: >> > The problem is not DImode LABEL_REFs, but rather VOIDmode LABEL_REFs when >> > matched against a match_operand:DI. >> >> It'd be good to fix this in a more direct way

Re: [PATCH] S/390: Wide int support.

2015-12-15 Thread Richard Sandiford
"Ulrich Weigand" writes: > Dominik Vogt wrote: > >> +; Note: Although CONST_INT and CONST_DOUBLE are not handled in this >> predicate, >> +; at least one of them needs to appear or otherwise safe_predicate_mode will >> +; assume that a DImode LABEL_REF is not accepted either

Re: [PATCH] S/390: Wide int support.

2015-12-15 Thread Ulrich Weigand
Richard Sandiford wrote: > "Ulrich Weigand" writes: > > The problem is not DImode LABEL_REFs, but rather VOIDmode LABEL_REFs when > > matched against a match_operand:DI. > > It'd be good to fix this in a more direct way though, rather than > hack around it. It's possible

[PATCH] S/390: Wide int support.

2015-12-11 Thread Dominik Vogt
ogt <v...@linux.vnet.ibm.com> Date: Thu, 10 Dec 2015 11:51:08 +0100 Subject: [PATCH] S/390: Wide int support. This fixes the assertion in simplify-rtx.c:simplify_const_binary_operation() triggered by gcc.dg/pr68129_1.c. --- gcc/config/s390/predicates.md | 9 ++--- gcc/config/s390/s

Re: [PATCH] S/390: Wide int support.

2015-12-11 Thread Ulrich Weigand
Dominik Vogt wrote: > +; Note: Although CONST_INT and CONST_DOUBLE are not handled in this > predicate, > +; at least one of them needs to appear or otherwise safe_predicate_mode will > +; assume that a DImode LABEL_REF is not accepted either (see genrecog.c). The problem is not DImode

Re: [PATCH] S/390: Wide int support.

2015-12-11 Thread Andreas Krebbel
On 12/11/2015 03:20 PM, Dominik Vogt wrote: > The attached patch introduces wide int support to S/390 in order > to resolve a test case failure in gcc.dg/pr68129_1.c that is > caused by an assertion in > simplify-rtx.c:simplify_const_binary_operation(). Applied with the change suggested by Uli.