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.