On Wed, 4 Aug 2010 22:53:46 +0800 Yao Qi yao@codesourcery.com 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?
I don't particularly like the "CSL LOCAL" markers in a non-CSL-local branch, but I don't think that's what you're asking :-).
We're talking about the Linaro 4.4 branch here, right? I'm planning to add appropriate "-fno-unroll-loops" options to affected tests on the 4.5 branch (that is not done yet), but I wasn't planning to do the same for 4.4. I don't see why we can't though.
Julian