Re: [PATCH] allocate combine.c:LOG_LINKS in an obstack

2011-04-05 Thread Nathan Froyd
On Tue, Apr 05, 2011 at 12:29:45PM -0700, H.J. Lu wrote: > On Tue, Apr 5, 2011 at 11:28 AM, Nathan Froyd > wrote: > >> This broke compilation on AUTO_INC_DEC targets.  Currently putting > >> together a fix and testing via cross to powerpc-eabispe. > > > > ...and here's the patch I'm going to inst

Re: [PATCH] allocate combine.c:LOG_LINKS in an obstack

2011-04-05 Thread H.J. Lu
On Tue, Apr 5, 2011 at 11:28 AM, Nathan Froyd wrote: > On Tue, Apr 05, 2011 at 11:22:56AM -0700, Nathan Froyd wrote: >> On Tue, Apr 05, 2011 at 09:59:43AM -0400, Nathan Froyd wrote: >> > On Mon, Apr 04, 2011 at 02:49:54PM -0400, Nathan Froyd wrote: >> > > This patch does just what $SUBJECT suggest

Re: [PATCH] allocate combine.c:LOG_LINKS in an obstack

2011-04-05 Thread H.J. Lu
On Tue, Apr 5, 2011 at 11:28 AM, Nathan Froyd wrote: > On Tue, Apr 05, 2011 at 11:22:56AM -0700, Nathan Froyd wrote: >> On Tue, Apr 05, 2011 at 09:59:43AM -0400, Nathan Froyd wrote: >> > On Mon, Apr 04, 2011 at 02:49:54PM -0400, Nathan Froyd wrote: >> > > This patch does just what $SUBJECT suggest

Re: [PATCH] allocate combine.c:LOG_LINKS in an obstack

2011-04-05 Thread Nathan Froyd
On Tue, Apr 05, 2011 at 11:22:56AM -0700, Nathan Froyd wrote: > On Tue, Apr 05, 2011 at 09:59:43AM -0400, Nathan Froyd wrote: > > On Mon, Apr 04, 2011 at 02:49:54PM -0400, Nathan Froyd wrote: > > > This patch does just what $SUBJECT suggests. > > > > v2, now with obstacks! > > This broke compilat

Re: [PATCH] allocate combine.c:LOG_LINKS in an obstack

2011-04-05 Thread Nathan Froyd
On Tue, Apr 05, 2011 at 09:59:43AM -0400, Nathan Froyd wrote: > On Mon, Apr 04, 2011 at 02:49:54PM -0400, Nathan Froyd wrote: > > This patch does just what $SUBJECT suggests. > > v2, now with obstacks! This broke compilation on AUTO_INC_DEC targets. Currently putting together a fix and testing v

Re: [PATCH] allocate combine.c:LOG_LINKS in an obstack

2011-04-05 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/05/11 08:51, Nathan Froyd wrote: > On Tue, Apr 05, 2011 at 04:42:27PM +0200, Steven Bosscher wrote: >> On Tue, Apr 5, 2011 at 3:59 PM, Nathan Froyd >> wrote: >>> v2, now with obstacks! >> >> @findex LOG_LINKS >> @item LOG_LINKS (@var{i}) >> A l

Re: [PATCH] allocate combine.c:LOG_LINKS in an obstack

2011-04-05 Thread Steven Bosscher
On Tue, Apr 5, 2011 at 4:51 PM, Nathan Froyd wrote: > On Tue, Apr 05, 2011 at 04:42:27PM +0200, Steven Bosscher wrote: >> On Tue, Apr 5, 2011 at 3:59 PM, Nathan Froyd >> wrote: >> > v2, now with obstacks! >> >> @findex LOG_LINKS >> @item LOG_LINKS (@var{i}) >> A list (chain of @code{insn_list} e

Re: [PATCH] allocate combine.c:LOG_LINKS in an obstack

2011-04-05 Thread Nathan Froyd
On Tue, Apr 05, 2011 at 04:42:27PM +0200, Steven Bosscher wrote: > On Tue, Apr 5, 2011 at 3:59 PM, Nathan Froyd wrote: > > v2, now with obstacks! > > @findex LOG_LINKS > @item LOG_LINKS (@var{i}) > A list (chain of @code{insn_list} expressions) giving information about > dependencies between inst

Re: [PATCH] allocate combine.c:LOG_LINKS in an obstack

2011-04-05 Thread Steven Bosscher
On Tue, Apr 5, 2011 at 3:59 PM, Nathan Froyd wrote: > On Mon, Apr 04, 2011 at 02:49:54PM -0400, Nathan Froyd wrote: >> This patch does just what $SUBJECT suggests. > > v2, now with obstacks! Still one TODO in doc/rtl.texi: @findex LOG_LINKS @item LOG_LINKS (@var{i}) A list (chain of @code{insn_l

Re: [PATCH] allocate combine.c:LOG_LINKS in an obstack

2011-04-05 Thread Richard Guenther
On Tue, Apr 5, 2011 at 3:59 PM, Nathan Froyd wrote: > On Mon, Apr 04, 2011 at 02:49:54PM -0400, Nathan Froyd wrote: >> This patch does just what $SUBJECT suggests. > > v2, now with obstacks! > > Tested on x86_64-unknown-linux-gnu.  OK to commit? Ok. Thanks, Richard. > -Nathan > >        * combi

Re: [PATCH] allocate combine.c:LOG_LINKS in an obstack

2011-04-05 Thread Nathan Froyd
On Mon, Apr 04, 2011 at 02:49:54PM -0400, Nathan Froyd wrote: > This patch does just what $SUBJECT suggests. v2, now with obstacks! Tested on x86_64-unknown-linux-gnu. OK to commit? -Nathan * combine.c: Include obstack.h (struct insn_link): Define. (uid_log_links): Adju