Re: [PATCH] Fix up cmove expansion (PR target/58864, take 2)

2014-11-07 Thread Andrew Pinski
On Mon, Dec 2, 2013 at 2:51 PM, Jakub Jelinek ja...@redhat.com wrote: Hi! On Sat, Nov 30, 2013 at 12:38:30PM +0100, Eric Botcazou wrote: Rather than adding do_pending_stack_adjust () in all the places, especially when it isn't clear whether emit_conditional_move will be called at all and

Re: [PATCH] Fix up cmove expansion (PR target/58864, take 2)

2013-12-03 Thread Richard Biener
On Mon, 2 Dec 2013, Jeff Law wrote: On 12/02/13 15:51, Jakub Jelinek wrote: Hi! On Sat, Nov 30, 2013 at 12:38:30PM +0100, Eric Botcazou wrote: Rather than adding do_pending_stack_adjust () in all the places, especially when it isn't clear whether emit_conditional_move will be

[PATCH] Fix up cmove expansion (PR target/58864, take 2)

2013-12-02 Thread Jakub Jelinek
Hi! On Sat, Nov 30, 2013 at 12:38:30PM +0100, Eric Botcazou wrote: Rather than adding do_pending_stack_adjust () in all the places, especially when it isn't clear whether emit_conditional_move will be called at all and whether it will actually do do_pending_stack_adjust (), I chose to add

Re: [PATCH] Fix up cmove expansion (PR target/58864, take 2)

2013-12-02 Thread Jeff Law
On 12/02/13 15:51, Jakub Jelinek wrote: Hi! On Sat, Nov 30, 2013 at 12:38:30PM +0100, Eric Botcazou wrote: Rather than adding do_pending_stack_adjust () in all the places, especially when it isn't clear whether emit_conditional_move will be called at all and whether it will actually do

Re: [PATCH] Fix up cmove expansion (PR target/58864)

2013-11-30 Thread Richard Biener
Jakub Jelinek ja...@redhat.com wrote: Hi! The following testcase ICEs because expand_cond_expr_using_cmove calls emit_conditional_move (which calls do_pending_stack_adjust under some circumstances), but when that fails, just removes all the insns generated by emit_conditional_move (and perhaps

Re: [PATCH] Fix up cmove expansion (PR target/58864)

2013-11-30 Thread Eric Botcazou
Rather than adding do_pending_stack_adjust () in all the places, especially when it isn't clear whether emit_conditional_move will be called at all and whether it will actually do do_pending_stack_adjust (), I chose to add two new functions to save/restore the pending stack adjustment state,

[PATCH] Fix up cmove expansion (PR target/58864)

2013-11-29 Thread Jakub Jelinek
Hi! The following testcase ICEs because expand_cond_expr_using_cmove calls emit_conditional_move (which calls do_pending_stack_adjust under some circumstances), but when that fails, just removes all the insns generated by emit_conditional_move (and perhaps some earlier ones too), thus it removes