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 _______________________________________________ linaro-toolchain mailing list linaro-toolchain@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-toolchain
