Re: [PATCH, PR63307] Fix generation of new declarations in random order

2015-01-21 Thread Jakub Jelinek
On Wed, Oct 29, 2014 at 04:05:27PM +, Zamyatin, Igor wrote: The question remains, are the decls all you need from the traversal (i.e. what you need to act upon)? From my earlier skim of the original code that wasn't that obvious. You can have in decl_map at least also BLOCKs,

RE: [PATCH, PR63307] Fix generation of new declarations in random order

2014-10-29 Thread Zamyatin, Igor
The question remains, are the decls all you need from the traversal (i.e. what you need to act upon)? From my earlier skim of the original code that wasn't that obvious. You can have in decl_map at least also BLOCKs, perhaps types too, what else? Jakub, Seems the BLOCKs are the only

Re: [PATCH, PR63307] Fix generation of new declarations in random order

2014-10-21 Thread Jakub Jelinek
On Thu, Oct 16, 2014 at 11:06:34AM -0600, Jeff Law wrote: We really prefer fully specified sorts. For a qsort callback, this doesn't look fully specified. With that fixed, this should be OK. jeff Thanks for the review. Here is the updated version. Is it ok? Yes, this is good for

RE: [PATCH, PR63307] Fix generation of new declarations in random order

2014-10-21 Thread Zamyatin, Igor
For some reasons it passed bootstrap locally... -Original Message- From: Jakub Jelinek [mailto:ja...@redhat.com] Sent: Tuesday, October 21, 2014 12:15 PM To: Zamyatin, Igor; Jeff Law Cc: GCC Patches (gcc-patches@gcc.gnu.org) Subject: Re: [PATCH, PR63307] Fix generation of new

Re: [PATCH, PR63307] Fix generation of new declarations in random order

2014-10-21 Thread Jakub Jelinek
On Tue, Oct 21, 2014 at 10:14:56AM +0200, Jakub Jelinek wrote: I've bootstrapped/regtested on i686-linux the following quick fix, bootstrapped on x86_64-linux too, in the middle of regtesting there. If it succeeds, I'll commit as obvious, so that people can continue working on the trunk. Ah,

RE: [PATCH, PR63307] Fix generation of new declarations in random order

2014-10-16 Thread Zamyatin, Igor
c-family/Changelog: 2014-10-03 Igor Zamyatin igor.zamya...@intel.com PR c/63307 * cilk.c: Include vec.h. (struct cilk_decls): New structure. (wrapper_parm_cb): Split this function to... (fill_decls_vec): ...this... (create_parm_list): ...and this.

Re: [PATCH, PR63307] Fix generation of new declarations in random order

2014-10-16 Thread Jeff Law
On 10/16/14 09:16, Zamyatin, Igor wrote: c-family/Changelog: 2014-10-03 Igor Zamyatin igor.zamya...@intel.com PR c/63307 * cilk.c: Include vec.h. (struct cilk_decls): New structure. (wrapper_parm_cb): Split this function to... (fill_decls_vec):

Re: [PATCH, PR63307] Fix generation of new declarations in random order

2014-10-10 Thread Jeff Law
On 10/04/14 05:54, Zamyatin, Igor wrote: Hi! The following patch does fix random order for new decls generation during Cilk_spawn generation. As Jakub suggested in the PR first we deal with vectors, then sort them and only then perform necessary generation of new decls. Bootstrapped and