RE: Bug in expand_builtin_setjmp_receiver ?

2010-10-29 Thread Jon Beniston
Hi Fred, If you have access to a lm32 toolchain, can you test if gcc.c- torture/execute/built-in-setjmp.c passes at different optimization levels? For a SVN snapshot from yesterday, patched so it fixes the problem Nathan mentioned: FAIL: gcc.c-torture/execute/built-in-setjmp.c execution,

Re: Bug in expand_builtin_setjmp_receiver ?

2010-10-27 Thread Jon Beniston
Hi Nathan, lm32 has a gdb simulator available, so it should be fairly easy to write a board file for it if one doesn't already exist. Unfortunately, building lm32-elf is broken in several different ways right now. What problems do you have building lm32-elf? If you let me know, I can try

Re: Bug in expand_builtin_setjmp_receiver ?

2010-10-27 Thread Frédéric RISS
Hi Jon, Le mardi 26 octobre 2010 à 13:07 +0100, Jon Beniston a écrit : What problems do you have building lm32-elf? If you let me know, I can try to look in to them. If you have access to a lm32 toolchain, can you test if gcc.c-torture/execute/built-in-setjmp.c passes at different optimization

Re: Bug in expand_builtin_setjmp_receiver ?

2010-10-27 Thread Nathan Froyd
On Tue, Oct 26, 2010 at 01:07:26PM +0100, Jon Beniston wrote: lm32 has a gdb simulator available, so it should be fairly easy to write a board file for it if one doesn't already exist. Unfortunately, building lm32-elf is broken in several different ways right now. What problems do

Re: Bug in expand_builtin_setjmp_receiver ?

2010-10-25 Thread Frederic Riss
On 22 October 2010 20:17, Ian Lance Taylor i...@google.com wrote: Frederic Riss frederic.r...@gmail.com writes: OK... what's the best way forward on this? Do we just leave it as it is and wait until an official port needs complains about it? Should it be filled in bugzilla? Did you just

Re: Bug in expand_builtin_setjmp_receiver ?

2010-10-25 Thread Ian Lance Taylor
Frederic Riss frederic.r...@gmail.com writes: On 22 October 2010 20:17, Ian Lance Taylor i...@google.com wrote: Frederic Riss frederic.r...@gmail.com writes: OK... what's the best way forward on this? Do we just leave it as it is and wait until an official port needs complains about it?

Re: Bug in expand_builtin_setjmp_receiver ?

2010-10-22 Thread Frederic Riss
On 21 October 2010 16:49, Nathan Froyd froy...@codesourcery.com wrote: Is it easy to test lm32 on some simulator? lm32 has a gdb simulator available, so it should be fairly easy to write a board file for it if one doesn't already exist. Unfortunately, building lm32-elf is broken in several

Re: Bug in expand_builtin_setjmp_receiver ?

2010-10-22 Thread Ian Lance Taylor
Frederic Riss frederic.r...@gmail.com writes: On 21 October 2010 16:49, Nathan Froyd froy...@codesourcery.com wrote: Is it easy to test lm32 on some simulator? lm32 has a gdb simulator available, so it should be fairly easy to write a board file for it if one doesn't already exist.

Re: Bug in expand_builtin_setjmp_receiver ?

2010-10-21 Thread Frederic Riss
Hi Ian, On 19 October 2010 15:31, Ian Lance Taylor i...@google.com wrote: It should not be necessary to use STARTING_FRAME_OFFSET when using virtual_stack_vars_rtx, as it should be added in by the vregs pass.  See instantiate_new_reg, and note that var_offset is set to STARTING_FRAME_OFFSET.

Re: Bug in expand_builtin_setjmp_receiver ?

2010-10-21 Thread Nathan Froyd
On Thu, Oct 21, 2010 at 02:14:15PM +0200, Frederic Riss wrote: On 19 October 2010 15:31, Ian Lance Taylor i...@google.com wrote: However, I agree that it does seem that it should be added to or subtracted from hard_frame_pointer_rtx before setting virtual_stack_vars_rtx, or something.  I

Bug in expand_builtin_setjmp_receiver ?

2010-10-19 Thread Frederic Riss
Hi, in builtins.c:expand_builtin_setjmp_receiver I see the following code: 827  /* Now put in the code to restore the frame pointer, and argument 828     pointer, if needed.  */ 829 #ifdef HAVE_nonlocal_goto 830  if (! HAVE_nonlocal_goto) 831 #endif 832    { 833      emit_move_insn

Re: Bug in expand_builtin_setjmp_receiver ?

2010-10-19 Thread Ian Lance Taylor
Frederic Riss frederic.r...@gmail.com writes: in builtins.c:expand_builtin_setjmp_receiver I see the following code: 827  /* Now put in the code to restore the frame pointer, and argument 828     pointer, if needed.  */ 829 #ifdef HAVE_nonlocal_goto 830  if (! HAVE_nonlocal_goto) 831