Re: [Open64-devel] (no subject) (make_libdeps change version 2)

2012-04-05 Thread Sun Chan
I guess I read wrong. I am not the one to review build makefiles :-) Sun On Fri, Apr 6, 2012 at 1:41 PM, Gilmore, Doug wrote: >> -Original Message- >> From: Sun Chan [mailto:sun.c...@gmail.com] >> Sent: Thursday, April 05, 2012 7:19 PM >> To: Gilmore, Doug >> Cc: open64-devel >> Subject:

Re: [Open64-devel] (no subject) (make_libdeps change version 2)

2012-04-05 Thread Gilmore, Doug
> -Original Message- > From: Sun Chan [mailto:sun.c...@gmail.com] > Sent: Thursday, April 05, 2012 7:19 PM > To: Gilmore, Doug > Cc: open64-devel > Subject: Re: [Open64-devel] (no subject) > > looks like you folks are making the binary one monolithic executable, no? > Sun This was exposed

Re: [Open64-devel] Code review request for an HTABLE bug [WOPT]

2012-04-05 Thread Jian-Xin Lai
Looks good to me. 2012/4/5 Wu Yongchong > Hi, all > > Can someone help review this patch. > > 1. Bug descript > The hash function for VN_BINARY_EXPR will first canonicalize the > expression. as the following code in file opt_vn_expr_taxonomy.h:509 > size_t hash() > { > Is_True(!has_bo

Re: [Open64-devel] (no subject)

2012-04-05 Thread Sun Chan
looks like you folks are making the binary one monolithic executable, no? Sun On Fri, Apr 6, 2012 at 2:41 AM, Gilmore, Doug wrote: > Folks probably noticed that I backed out my "make_libdeps rule" > change. > > I attached a new version of the patch that works with old > versions of make. > > Coul

[Open64-devel] (no subject)

2012-04-05 Thread Gilmore, Doug
Folks probably noticed that I backed out my "make_libdeps rule" change. I attached a new version of the patch that works with old versions of make. Could someone review this change when they have the chance? Thanks, Doug make_libdeps_v2.patch Description: make_libdeps_v2.patch ---