Yes, I think so. The bug is that the arguments of tree_int_cst_compare() may not be constant integers. This patch should take care of it.
thanks, Cong On Fri, Nov 8, 2013 at 12:06 PM, H.J. Lu <hjl.to...@gmail.com> wrote: > On Fri, Nov 8, 2013 at 10:34 AM, Cong Hou <co...@google.com> wrote: >> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59050 >> >> This is my bad. I forget to check the test result for gfortran. With >> this patch the bug should be fixed (tested on x86-64). >> >> >> thanks, >> Cong >> >> >> diff --git a/gcc/ChangeLog b/gcc/ChangeLog >> index 90b01f2..e62c672 100644 >> --- a/gcc/ChangeLog >> +++ b/gcc/ChangeLog >> @@ -1,3 +1,8 @@ >> +2013-11-08 Cong Hou <co...@google.com> >> + >> + PR tree-optimization/59050 >> + * tree-vect-data-refs.c (comp_dr_addr_with_seg_len_pair): Bug fix. >> + > > Many SPEC CPU 2000 tests failed with > > costab.c: In function 'HandleCoinc2': > costab.c:1565:17: internal compiler error: tree check: expected > integer_cst, have plus_expr in tree_int_cst_lt, at tree.c:7083 > void HandleCoinc2 ( cos1, cos2, hdfactor ) > ^ > 0xb6e084 tree_check_failed(tree_node const*, char const*, int, char const*, > ...) > ../../src-trunk/gcc/tree.c:9477 > 0xb6ffe4 tree_check > ../../src-trunk/gcc/tree.h:2914 > 0xb6ffe4 tree_int_cst_lt(tree_node const*, tree_node const*) > ../../src-trunk/gcc/tree.c:7083 > 0xb70020 tree_int_cst_compare(tree_node const*, tree_node const*) > ../../src-trunk/gcc/tree.c:7093 > 0xe53f1c comp_dr_addr_with_seg_len_pair > ../../src-trunk/gcc/tree-vect-data-refs.c:2672 > 0xe5cbb5 vec<dr_addr_with_seg_len_pair_t, va_heap, > vl_embed>::qsort(int (*)(void const*, void const*)) > ../../src-trunk/gcc/vec.h:941 > 0xe5cbb5 vec<dr_addr_with_seg_len_pair_t, va_heap, vl_ptr>::qsort(int > (*)(void const*, void const*)) > ../../src-trunk/gcc/vec.h:1620 > 0xe5cbb5 vect_prune_runtime_alias_test_list(_loop_vec_info*) > ../../src-trunk/gcc/tree-vect-data-refs.c:2845 > 0xb39382 vect_analyze_loop_2 > ../../src-trunk/gcc/tree-vect-loop.c:1716 > 0xb39382 vect_analyze_loop(loop*) > ../../src-trunk/gcc/tree-vect-loop.c:1807 > 0xb4f78f vectorize_loops() > ../../src-trunk/gcc/tree-vectorizer.c:360 > Please submit a full bug report, > with preprocessed source if appropriate. > Please include the complete backtrace with any bug report. > See <http://gcc.gnu.org/bugs.html> for instructions. > specmake[3]: *** [costab.o] Error 1 > specmake[3]: *** Waiting for unfinished jobs.... > > Will this patch fix them? > > > -- > H.J.
