Re: [PATCH] S/390: PR target/79240: Fix assertion in s390_extzv_shift_ok.

2017-01-26 Thread Jakub Jelinek
On Thu, Jan 26, 2017 at 05:43:13PM +0100, Dominik Vogt wrote: > > If the predicates are supposed to ensure it, then I think the assert is > > fine. > > Is it guaranteed that the predicate conditions are evaluated > before executing the conditions? Yes. You can see it in insn-recog.c...

Re: [PATCH] S/390: PR target/79240: Fix assertion in s390_extzv_shift_ok.

2017-01-26 Thread Dominik Vogt
On Thu, Jan 26, 2017 at 05:33:18PM +0100, Jakub Jelinek wrote: > On Thu, Jan 26, 2017 at 05:11:27PM +0100, Dominik Vogt wrote: > > The patch fixes the s390x crash reported in PR 79240: > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79240 > > Regression tested and bootstrapped on s390x biarch and

Re: [PATCH] S/390: PR target/79240: Fix assertion in s390_extzv_shift_ok.

2017-01-26 Thread Jakub Jelinek
On Thu, Jan 26, 2017 at 05:11:27PM +0100, Dominik Vogt wrote: > The patch fixes the s390x crash reported in PR 79240: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79240 > Regression tested and bootstrapped on s390x biarch and s390. > gcc/ChangeLog-pr79240 > > PR target/79240 > *

[PATCH] S/390: PR target/79240: Fix assertion in s390_extzv_shift_ok.

2017-01-26 Thread Dominik Vogt
240 * gcc.target/s390/pr79240.c: New test. >From 0c9bc9c84a98c90df54c58dbdc299e064b3a9c5f Mon Sep 17 00:00:00 2001 From: Dominik Vogt <v...@linux.vnet.ibm.com> Date: Thu, 26 Jan 2017 13:50:46 +0100 Subject: [PATCH] S/390: PR target/79240: Fix assertion in s390_extzv_shift_ok.