This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH PR69652, Regression]
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Yuri Rumyantsev <ysrumyan at gmail dot com>
- Cc: Richard Biener <richard dot guenther at gmail dot com>, Jakub Jelinek <jakub at redhat dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>, Igor Zamyatin <izamyatin at gmail dot com>
- Date: Mon, 29 Feb 2016 05:01:52 -0800
- Subject: Re: [PATCH PR69652, Regression]
- Authentication-results: sourceware.org; auth=none
- References: <CAEoMCqTxQBsKSOunf7797JWfziaf5_aP6xSyYkFjLjRf9z0Z+w at mail dot gmail dot com> <20160204164017 dot GO3017 at tucnak dot redhat dot com> <CAEoMCqRSt8PLGoycPehzMJ3aFo16GfXyRBJy-PjUeUyYpZ_9qQ at mail dot gmail dot com> <CAFiYyc2KoVygGQ4g2Y3-3QGrRv_L2R1cBoXac8SDLxzgf3NHHQ at mail dot gmail dot com> <CAEoMCqSXWyr1LGUXjryDZrjKC2Hu2gq_cOEv40cv1GoDjG7RqA at mail dot gmail dot com> <CAMe9rOrMAzUhwFafd8i+cjJV7vs-yGPnXWMLxim03ONPpETJHg at mail dot gmail dot com> <CAEoMCqSDoyrPhtd_v3XQfS1YLGj-6GUUwOVxjwwuRNN-3+VcmA at mail dot gmail dot com>
On Mon, Feb 29, 2016 at 3:53 AM, Yuri Rumyantsev <ysrumyan@gmail.com> wrote:
> This test simply checks that ICE is not occurred but not any
> vectorization issues.
Can we remove
/* { dg-options "-O2 -ffast-math -ftree-vectorize " } */
then?
H.J.
> Best regards.
> Yuri.
>
> 2016-02-28 20:29 GMT+03:00 H.J. Lu <hjl.tools@gmail.com>:
>> On Wed, Feb 10, 2016 at 2:26 AM, Yuri Rumyantsev <ysrumyan@gmail.com> wrote:
>>> Thanks Richard for your comments.
>>> I changes algorithm to remove dead scalar statements as you proposed.
>>>
>>> Bootstrap and regression testing did not show any new failures on x86-64.
>>> Is it OK for trunk?
>>>
>>> Changelog:
>>> 2016-02-10 Yuri Rumyantsev <ysrumyan@gmail.com>
>>>
>>> PR tree-optimization/69652
>>> * tree-vect-loop.c (optimize_mask_stores): Move declaration of STMT1
>>> to nested loop, did source re-formatting, skip debug statements,
>>> add check on statement with volatile operand, remove dead scalar
>>> statements.
>>>
>>> gcc/testsuite/ChangeLog:
>>> * gcc.dg/torture/pr69652.c: New test.
>>>
>>>
>>
>> /* { dg-do compile } */
>> /* { dg-options "-O2 -ffast-math -ftree-vectorize " } */
>> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>>
>> Any particular reason why it should be in gcc.dg/torture, not in
>> gcc.dg/vect? -O2 here is overridden by -Ox.
>>
>> /* { dg-additional-options "-mavx" { target { i?86-*-* x86_64-*-* } } } */
>>
>>
>>
>> --
>> H.J.
--
H.J.