From: "Martin Liška" <mliska@suse.cz>
To: gcc-patches@gcc.gnu.org
Subject: Re: [PATCH 3/5] Fix ICE in tree-chkp-opt.c (PR tree-optimization/79631).
Date: Thu, 09 Mar 2017 09:36:00 -0000 [thread overview]
Message-ID: <8d14756c-333d-7851-b674-01803395b7a3@suse.cz> (raw)
In-Reply-To: <84e861ef8f06f875dce32ad3d665b45626d7497c.1488881229.git.mliska@suse.cz>
PING^1:
On 03/02/2017 06:15 PM, marxin wrote:
> gcc/ChangeLog:
>
> 2017-03-06 Martin Liska <mliska@suse.cz>
>
> PR tree-optimization/79631
> * tree-chkp-opt.c (chkp_is_constant_addr): Call
> tree_int_cst_sign_bit just for INTEGER constants.
>
> gcc/testsuite/ChangeLog:
>
> 2017-03-06 Martin Liska <mliska@suse.cz>
>
> PR tree-optimization/79631
> * gcc.target/i386/mpx/pr79631.c: New test.
> ---
> gcc/testsuite/gcc.target/i386/mpx/pr79631.c | 15 +++++++++++++++
> gcc/tree-chkp-opt.c | 3 ++-
> 2 files changed, 17 insertions(+), 1 deletion(-)
> create mode 100644 gcc/testsuite/gcc.target/i386/mpx/pr79631.c
>
> diff --git a/gcc/testsuite/gcc.target/i386/mpx/pr79631.c b/gcc/testsuite/gcc.target/i386/mpx/pr79631.c
> new file mode 100644
> index 00000000000..075d46b835f
> --- /dev/null
> +++ b/gcc/testsuite/gcc.target/i386/mpx/pr79631.c
> @@ -0,0 +1,15 @@
> +/* { dg-do compile { target { ! x32 } } } */
> +/* { dg-options "-fcheck-pointer-bounds -mmpx -O2" } */
> +
> +typedef struct { int _mp_size; } mpz_t[1];
> +int a, b;
> +void fn1()
> +{
> + mpz_t c[1][b];
> + for (;;) {
> + int d = 0 >= 0 ? 0 == 0 ? c[0][0]->_mp_size ? -1 : 0 : 0 : 0,
> + e = 0 >= 0 ? 0 == 0 ? c[1][1]->_mp_size ? -1 : 0 : 0 : 0;
> + if (d != e)
> + a++;
> + }
> +}
> diff --git a/gcc/tree-chkp-opt.c b/gcc/tree-chkp-opt.c
> index ebe05459773..286f7853921 100644
> --- a/gcc/tree-chkp-opt.c
> +++ b/gcc/tree-chkp-opt.c
> @@ -241,7 +241,8 @@ chkp_is_constant_addr (const address_t &addr, int *sign)
> return false;
> else if (integer_zerop (addr.pol[0].cst))
> *sign = 0;
> - else if (tree_int_cst_sign_bit (addr.pol[0].cst))
> + else if (TREE_CODE (addr.pol[0].cst) == INTEGER_CST
> + && tree_int_cst_sign_bit (addr.pol[0].cst))
> *sign = -1;
> else
> *sign = 1;
>
next prev parent reply other threads:[~2017-03-09 9:36 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-07 10:09 [PATCH 0/5] Fix various MPX issues marxin
2017-03-07 10:09 ` [PATCH 1/5] Fix *_CST ICEs connected to MPX marxin
2017-03-07 10:17 ` Rainer Orth
2017-03-07 14:48 ` Martin Liška
2017-03-07 14:53 ` Richard Biener
2017-03-07 16:01 ` Martin Liška
2017-03-08 11:00 ` Richard Biener
2017-03-09 11:40 ` Martin Liška
2017-03-13 12:28 ` Richard Biener
2017-03-13 13:02 ` Martin Liška
2017-03-13 13:08 ` Richard Biener
2017-03-13 13:33 ` Martin Liška
2017-03-14 23:58 ` Ilya Enkovich
2017-03-15 10:12 ` Martin Liška
2017-03-15 17:09 ` Jeff Law
2017-03-07 10:09 ` [PATCH 5/5] Support BIT_FIELD_REF in MPX (PR ipa/79764) marxin
2017-03-07 10:18 ` Rainer Orth
2017-03-07 14:49 ` Martin Liška
2017-03-07 15:00 ` Richard Biener
2017-03-07 10:09 ` [PATCH 3/5] Fix ICE in tree-chkp-opt.c (PR tree-optimization/79631) marxin
2017-03-09 9:36 ` Martin Liška [this message]
2017-03-09 10:05 ` Richard Biener
2017-03-09 10:12 ` Martin Liška
2017-03-07 10:09 ` [PATCH 4/5] Disable -fcheck-pointer-bounds with sanitizers marxin
2017-03-07 16:12 ` Jeff Law
2017-03-07 16:21 ` Jakub Jelinek
2017-03-09 10:02 ` Martin Liška
2017-03-10 13:09 ` [PATCH] MPX: Fix option handling Martin Liška
2017-03-10 13:14 ` Jakub Jelinek
2017-03-17 12:17 ` Rainer Orth
2017-03-21 11:21 ` Martin Liška
2017-03-07 10:09 ` [PATCH 2/5] Get bounds for a PARM_DECL (PR ipa/79761) marxin
2017-03-07 10:16 ` Rainer Orth
2017-03-07 14:48 ` Martin Liška
2017-03-07 14:57 ` Richard Biener
2017-03-07 16:07 ` Martin Liška
2017-03-07 16:17 ` Jeff Law
2017-03-08 11:01 ` Richard Biener
2017-03-09 14:19 ` [PATCH 6/N] Do not warn -Wsuggest-attribute=noreturn for main.chkp (PR, middle-end/78339) Martin Liška
2017-03-13 12:29 ` Richard Biener
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8d14756c-333d-7851-b674-01803395b7a3@suse.cz \
--to=mliska@suse.cz \
--cc=gcc-patches@gcc.gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).