Re: Fix GCC bug causing bootstrap failure with vectorizer turned on

2013-07-19 Thread Xinliang David Li
We all missed it when just staring at the code :) David On Fri, Jul 19, 2013 at 5:45 AM, Jeff Law wrote: > On 07/18/2013 10:49 AM, Xinliang David Li wrote: >> >> The difference is that the relative order of DECL_UIDs do not change >> whether debug info is on or not, but there is no such guarante

Re: Fix GCC bug causing bootstrap failure with vectorizer turned on

2013-07-19 Thread Jeff Law
On 07/18/2013 10:49 AM, Xinliang David Li wrote: The difference is that the relative order of DECL_UIDs do not change whether debug info is on or not, but there is no such guarantee when hashing is involved. Yea. I feel like an idiot for missing the hash vs comparison. Add a ChangeLog and this

Re: Fix GCC bug causing bootstrap failure with vectorizer turned on

2013-07-18 Thread Jakub Jelinek
On Thu, Jul 18, 2013 at 10:45:19AM -0600, Jeff Law wrote: > If I understand you correctly, you're claiming that the DECL_UID can > vary based on whether or not we're emitting debug info. This can be > seen in iterative_hash_expr: Yes, for e.g. SSA_NAME_VERSION, we require that it is the same in b

Re: Fix GCC bug causing bootstrap failure with vectorizer turned on

2013-07-18 Thread Xinliang David Li
The difference is that the relative order of DECL_UIDs do not change whether debug info is on or not, but there is no such guarantee when hashing is involved. David On Thu, Jul 18, 2013 at 9:45 AM, Jeff Law wrote: > On 07/12/2013 04:13 PM, Cong Hou wrote: >> >> GCC bootstrap failed with loop vec

Re: Fix GCC bug causing bootstrap failure with vectorizer turned on

2013-07-18 Thread Jeff Law
On 07/12/2013 04:13 PM, Cong Hou wrote: GCC bootstrap failed with loop vectorizer turned on by default at O2. The symptom is that the comparison between stage2&3 compilers fails. The root cause is a bug in the file "tree-vect-data-refs.c", where a qsort() function call is used to sort a group of

Re: Fix GCC bug causing bootstrap failure with vectorizer turned on

2013-07-15 Thread Richard Biener
Cong Hou wrote: >Hi Richard > >Thank you for you reply. The code generation is affected when >generating >conditions for alias checks using the function > >static void vect_create_cond_for_alias_checks (loop_vec_info >loop_vinfo, >tree * cond_expr) > >from the file > >*tree-vect-loop-manip.c* > >

Re: Fix GCC bug causing bootstrap failure with vectorizer turned on

2013-07-13 Thread Richard Biener
Cong Hou wrote: >GCC bootstrap failed with loop vectorizer turned on by default at O2. >The symptom is that the comparison between stage2&3 compilers fails. >The root cause is a bug in the file "tree-vect-data-refs.c", where a >qsort() function call is used to sort a group of data references usin

Fix GCC bug causing bootstrap failure with vectorizer turned on

2013-07-12 Thread Cong Hou
GCC bootstrap failed with loop vectorizer turned on by default at O2. The symptom is that the comparison between stage2&3 compilers fails. The root cause is a bug in the file "tree-vect-data-refs.c", where a qsort() function call is used to sort a group of data references using a comparison functio