On Wed, May 09, 2012 at 09:02:14AM +0800, Dehao Chen wrote:
> Sorry for the error. Here is a new patch to fix them:
>
> gcc/testsuite/ChangeLog:
> 2012-05-08 Dehao Chen
>
> * gcc.dg/predict-1.c: Remove the replicated text in this text.
> * gcc.dg/predict-2.c: Likewise.
> * gc
Sorry for the error. Here is a new patch to fix them:
gcc/testsuite/ChangeLog:
2012-05-08 Dehao Chen
* gcc.dg/predict-1.c: Remove the replicated text in this text.
* gcc.dg/predict-2.c: Likewise.
* gcc.dg/predict-3.c: Likewise.
* gcc.dg/predict-4.c: Likewise.
On Fri, May 04, 2012 at 09:46:22PM +0800, Dehao Chen wrote:
> Thanks for the prompt response. Attached is the updated patch.
>
> Passed bootstrap and all regression tests.
All the new testcases fail for me, on both x86_64-linux and i686-linux,
apparently because of incorrectly committed patch (ea
Hi, Honza,
Thanks for the prompt response. Attached is the updated patch.
Passed bootstrap and all regression tests.
Thanks,
Dehao
Index: gcc/testsuite/gcc.dg/predict-3.c
===
--- gcc/testsuite/gcc.dg/predict-3.c(revision 0)
+++
> gcc/ChangeLog
> 2012-05-04 Dehao Chen
>
> * predict.c (find_qualified_ssa_name): New
> (find_ssa_name_in_expr): New
> (find_ssa_name_in_assign_stmt): New
> (is_comparison_with_loop_invariant_p): New
> (is_bound_expr_similar): New
> (predict_iv_comparison):
http://gcc.gnu.org/ml/gcc-patches/2012-01/msg00377.html
I've also refined the patch, adding support for comparison of IV to
its initial value.
The updated patch is attached, and you can also it in:
http://codereview.appspot.com/5504086/
Passed bootstrap and all gcc regression tests.
Thanks,
Deh
http://gcc.gnu.org/ml/gcc-patches/2012-01/msg00377.html
Thanks!
Dehao
