On Mon, Feb 29, 2016 at 3:53 AM, Yuri Rumyantsev <ysrum...@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.to...@gmail.com>: >> On Wed, Feb 10, 2016 at 2:26 AM, Yuri Rumyantsev <ysrum...@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 <ysrum...@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.