RE: Remove splay_tree from gimplify.c

2015-05-27 Thread Aditya K
> Date: Wed, 27 May 2015 18:41:55 +0200 > From: ja...@redhat.com > To: l...@redhat.com > CC: hiradi...@msn.com; gcc-patches@gcc.gnu.org > Subject: Re: Remove splay_tree from gimplify.c > > On Wed, May 27, 2015 at 10:34:46AM -0600,

Re: Remove splay_tree from gimplify.c

2015-05-27 Thread Jakub Jelinek
On Wed, May 27, 2015 at 10:34:46AM -0600, Jeff Law wrote: > So the question here is whether or not the other uses are commonly looking > up elements we've already searched for -- that's the whole purpose of a > splay tree, to improve lookup performance for commonly hit items. First of all, this is

Re: Remove splay_tree from gimplify.c

2015-05-27 Thread Jeff Law
don't need to update the tree in these cases at least. I have pasted the patch which removes splay_tree with hash_map. Another patch (attached) removes splay_tree with std::map. Please review the patches. Thanks, -Aditya gcc/ChangeLog: 2015-05-17 hiraditya Remove

Remove splay_tree from gimplify.c

2015-05-17 Thread Aditya K
se cases at least. I have pasted the patch which removes splay_tree with hash_map. Another patch (attached) removes splay_tree with std::map. Please review the patches. Thanks, -Aditya gcc/ChangeLog: 2015-05-17  hiraditya       Remove splay_tree from gimplify.c         * gimplify.c (struct g