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
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
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
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