[Bug target/39146] Unnecessary stack alignment

2009-06-12 Thread hjl dot tools at gmail dot com
--- Comment #20 from hjl dot tools at gmail dot com 2009-06-13 00:10 --- Fixed. -- hjl dot tools at gmail dot com changed: What|Removed |Added Status|UNCONFI

[Bug target/39146] Unnecessary stack alignment

2009-06-12 Thread hjl at gcc dot gnu dot org
--- Comment #19 from hjl at gcc dot gnu dot org 2009-06-12 12:54 --- Subject: Bug 39146 Author: hjl Date: Fri Jun 12 12:54:30 2009 New Revision: 148428 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=148428 Log: gcc/ 2009-06-12 Joey Ye PR middle-end/39146 * c

[Bug target/39146] Unnecessary stack alignment

2009-04-16 Thread pinskia at gcc dot gnu dot org
--- Comment #18 from pinskia at gcc dot gnu dot org 2009-04-16 18:33 --- Stop setting the target milestone unless it is a regression. -- pinskia at gcc dot gnu dot org changed: What|Removed |Added --

[Bug target/39146] Unnecessary stack alignment

2009-03-03 Thread hjl dot tools at gmail dot com
--- Comment #17 from hjl dot tools at gmail dot com 2009-03-03 17:36 --- A patch is posted at http://gcc.gnu.org/ml/gcc-patches/2009-03/msg00159.html -- hjl dot tools at gmail dot com changed: What|Removed |Added -

[Bug target/39146] Unnecessary stack alignment

2009-03-03 Thread hjl at gcc dot gnu dot org
--- Comment #16 from hjl at gcc dot gnu dot org 2009-03-03 17:14 --- Subject: Bug 39146 Author: hjl Date: Tue Mar 3 17:14:04 2009 New Revision: 144577 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=144577 Log: 2009-03-03 Joey Ye H.J. Lu PR middle-end/3

[Bug target/39146] Unnecessary stack alignment

2009-02-28 Thread hjl dot tools at gmail dot com
--- Comment #15 from hjl dot tools at gmail dot com 2009-02-28 21:43 --- (In reply to comment #14) > (In reply to comment #12) > > Created an attachment (id=17305) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17305&action=view) [edit] > > New patch attached > > > > Test finished

[Bug target/39146] Unnecessary stack alignment

2009-02-28 Thread hjl dot tools at gmail dot com
--- Comment #14 from hjl dot tools at gmail dot com 2009-02-28 18:51 --- (In reply to comment #12) > Created an attachment (id=17305) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17305&action=view) [edit] > New patch attached > > Test finished. No regression with emx_avx_sim. Wa

[Bug target/39146] Unnecessary stack alignment

2009-02-20 Thread hjl at gcc dot gnu dot org
--- Comment #13 from hjl at gcc dot gnu dot org 2009-02-21 00:01 --- Subject: Bug 39146 Author: hjl Date: Sat Feb 21 00:00:49 2009 New Revision: 144338 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=144338 Log: 2009-02-20 Joey Ye PR middle-end/39146 * cfgexpa

[Bug target/39146] Unnecessary stack alignment

2009-02-16 Thread Joey dot ye at intel dot com
--- Comment #12 from Joey dot ye at intel dot com 2009-02-16 08:49 --- Created an attachment (id=17305) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17305&action=view) New patch attached Test finished. No regression with emx_avx_sim. Wait to checkin to 4.5 -- Joey dot ye at i

[Bug target/39146] Unnecessary stack alignment

2009-02-12 Thread hjl dot tools at gmail dot com
--- Comment #11 from hjl dot tools at gmail dot com 2009-02-12 15:42 --- I think we can consider it for 4.5. -- hjl dot tools at gmail dot com changed: What|Removed |Added ---

[Bug target/39146] Unnecessary stack alignment

2009-02-12 Thread Joey dot ye at intel dot com
--- Comment #10 from Joey dot ye at intel dot com 2009-02-12 15:20 --- (In reply to comment #8) > We still have push and mov. I guess it may be the best we can do. > But please run full 32 and 64bit testsuite with your patch as well > as under emx-avx-sim. full 32/64 bit test pass with n

[Bug target/39146] Unnecessary stack alignment

2009-02-11 Thread Joey dot ye at intel dot com
--- Comment #9 from Joey dot ye at intel dot com 2009-02-12 02:40 --- (In reply to comment #8) > We still have push and mov. I guess it may be the best we can do. I believe so too. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39146

[Bug target/39146] Unnecessary stack alignment

2009-02-11 Thread hjl dot tools at gmail dot com
--- Comment #8 from hjl dot tools at gmail dot com 2009-02-12 02:33 --- (In reply to comment #7) > Created an attachment (id=17283) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17283&action=view) [edit] > A patch to fix this problem > > Impact to other test unknown. Test undergoi

[Bug target/39146] Unnecessary stack alignment

2009-02-11 Thread Joey dot ye at intel dot com
--- Comment #7 from Joey dot ye at intel dot com 2009-02-12 02:26 --- Created an attachment (id=17283) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17283&action=view) A patch to fix this problem Impact to other test unknown. Test undergoing. HJ, can you also help to verify and t

[Bug target/39146] Unnecessary stack alignment

2009-02-11 Thread hjl dot tools at gmail dot com
--- Comment #6 from hjl dot tools at gmail dot com 2009-02-12 01:58 --- locate_and_pad_parm in function.c has /* Remember if the outgoing parameter requires extra alignment on the calling function side. */ if (crtl->stack_alignment_needed < boundary) crtl->stack_alignment_

[Bug target/39146] Unnecessary stack alignment

2009-02-11 Thread Joey dot ye at intel dot com
--- Comment #5 from Joey dot ye at intel dot com 2009-02-12 01:45 --- Stack realign is finalized by stack_realign = (incoming_stack_boundary < (current_function_is_leaf ? crtl->max_used_stack_slot_alignment

[Bug target/39146] Unnecessary stack alignment

2009-02-11 Thread hjl dot tools at gmail dot com
--- Comment #4 from hjl dot tools at gmail dot com 2009-02-11 23:59 --- The dynamic stack alignment may not be easy to optimize out. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39146

[Bug target/39146] Unnecessary stack alignment

2009-02-11 Thread hjl dot tools at gmail dot com
--- Comment #3 from hjl dot tools at gmail dot com 2009-02-11 16:53 --- Also expand_one_var in cfgexpand.c has if (crtl->stack_alignment_estimated < align) { /* stack_alignment_estimated shouldn't change after stack realign decision made */

[Bug target/39146] Unnecessary stack alignment

2009-02-11 Thread hjl dot tools at gmail dot com
--- Comment #2 from hjl dot tools at gmail dot com 2009-02-11 15:15 --- This is caused by assign_parms in function.c: /* Estimate stack alignment from parameter alignment. */ if (SUPPORTS_STACK_ALIGNMENT) { unsigned int align = FUNCTION_ARG_BOUNDARY (data.

[Bug target/39146] Unnecessary stack alignment

2009-02-09 Thread Joey dot ye at intel dot com
--- Comment #1 from Joey dot ye at intel dot com 2009-02-10 05:35 --- Argument need 32 bytes alignment, No way to guarantee the argument won't be spilled. That's why stack adjustment is there. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39146