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 though, rather than >> hack around it. It's

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 that the trick will stop

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 (see genrecog.c). > >

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. T

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 LABEL_REF