------- Comment #22 from hjl dot tools at gmail dot com 2009-10-10 14:46 ------- (In reply to comment #20) > (In reply to comment #19) > > Without a testcase, people may not review the patch. > > > > May be just include your tests from Bug 40838? (even without testing) > From http://gcc.gnu.org/bugzilla/attachment.cgi?id=18656 > Your suggestions? > > I understand so there are same results with your gcc-4.4-pr40838-*.patch, but > realigning whole code and exclude "misaligned" AMD CPUs. And warrantied to > realign in any place. > > Your gcc-4.4-pr40838-3.patch lost realigning in unknown place - runtime > segfaults in Seamonkey. *-4.patch I don't test becouse alredy use simple
We can't check Seamonkey into gcc/testsuite. We need something much smaller. > "-mstackrealign" solution - first there are dummy testing - surfing with > Seamonkey until sometimes segfault, second - I got new desktop with Athlon > (but > was alredy satisfyed by "-mstackrealign" on old Celeron). Now I prefer to use > safe "-mstackrealign" solution for Intel CPUs on 32bit servers and to be sure > in safe SSE code. And see no visual defferences in perfomance whith global > "-mstackrealign". > If you think adding -mstackrealign to your build is an acceptable solution, we can close this bug. Otherwise, please test my latest gcc 4.4 patch for PR 40838. Thanks. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41156
