Re: [Open64-devel] Gatekeeper code review request for AMD 4.2.5.1 merge

2011-07-14 Thread Jian-Xin Lai
This patch looks fine to me. Please go ahead. 2011/7/15 Berg, Michael : > All:  The attached patch file contains(I have left it in one files, it is > not large): > > > > 1.)    Updates for alignment usage on BD > > 2.)    EBO register pressure heuristic tunning > > 3.)    A set of horizontal add p

[Open64-devel] Gatekeeper code review request for AMD 4.2.5.1 merge

2011-07-14 Thread Berg, Michael
All: The attached patch file contains(I have left it in one files, it is not large): 1.)Updates for alignment usage on BD 2.)EBO register pressure heuristic tunning 3.)A set of horizontal add peephole opts for BD 4.)More forms of replicate updates for BD 5.)Support for n

Re: [Open64-devel] Gatekeeper code review request for AMD 4.2.5.1 merge

2011-07-10 Thread Sun Chan
1.p has no assertion. Can you add? E.g. I would assert num_bbs no less than 1. The func LOOP_Block_Merge is not targ_x86, is it? As long as the call to it is TARG_X8664, it should be fine. 2.p, you are using bool, please use BOOL like that of 1.p if the segment: bool uses_destructive_d

Re: [Open64-devel] Gatekeeper code review request for AMD 4.2.5.1 merge

2011-07-08 Thread Berg, Michael
It's been a while, can a gatekeeper pls have a look at this patch. Thx, m From: Berg, Michael Sent: Friday, July 01, 2011 12:14 PM To: 'open64-devel@lists.sourceforge.net' Subject: Gatekeeper code review request for AMD 4.2.5.1 merge All: The attached patch files are: 1.)Remainder loop c