LP:602190 & LP:602285

[Daniel Jacobowitz]

On Wed, Aug 04, 2010 at 10:53:46PM +0800, Yao Qi wrote:
> LP:602190(https://bugs.launchpad.net/gcc-linaro/+bug/602190) and
> LP:602285(https://bugs.launchpad.net/gcc-linaro/+bug/602285) are
> related to this patch below.  You can get more details from comments
> of these bugs, since I've added my understand of the cause in
> comments.
> 
> This patch is to improve the performance of generated code, however,
> these two bugs are related to this patch(, correct me if I am wrong).
> Now, we have two options, 1) revert this patch, and make these test
> case pass; 2) keep this patch and fix test cases, 3) fix bugs and keep
> this patch,
> 
> What do you think?

Julian, weren't you looking at these?  You can avoid the param
adjustments by an explicit -funroll-loops or -fno-unroll-loops,
whichever is appropriate, in the options of the test case.

I do not think we should drop the patch.

-- 
Daniel Jacobowitz
CodeSourcery