This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH1][PR71252] Fix missing swap to stmt_to_insert
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Kugan Vivekanandarajah <kugan dot vivekanandarajah at linaro dot org>,"gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>,Jakub Jelinek <jakub at redhat dot com>
- Date: Fri, 27 May 2016 19:25:19 +0200
- Subject: Re: [PATCH1][PR71252] Fix missing swap to stmt_to_insert
- Authentication-results: sourceware.org; auth=none
- References: <CAELXzTMNyYPtWbmFTbw1kos6UtFTnG8KSHhsfFDZyB29punS2Q at mail dot gmail dot com>
On May 27, 2016 4:12:06 PM GMT+02:00, Kugan Vivekanandarajah <kugan.vivekanandarajah@linaro.org> wrote:
>Hi,
>
>This fix the missing swap for stmt-to_insert. I tested this with the
>attached test case which is not valid any more due to some other
>commits. This I believe an is obvious fix and maybe the test case is
>needed.
>
>I am running bootstrap and regression testing on x86-64-linux gnu. Is
>this OK for trunk if the testing is fine ?
OK.
Richard.
>Thanks,
>
>Kugan
>
>
>gcc/ChangeLog:
>
>2016-05-28 Kugan Vivekanandarajah <kugan.vivekanandarajah@linaro.org>
>
>* tree-ssa-reassoc.c (swap_ops_for_binary_stmt): Fix swap such that
>
>all fields including stmt_to_insert are swapped.
>
>gcc/testsuite/ChangeLog:
>
>2016-05-28 Kugan Vivekanandarajah <kugan.vivekanandarajah@linaro.org>
>
>* gcc.dg/tree-ssa/pr71252-2.c: New test.