Re: [PATCH Commitred] Fix PR target/48767

2011-04-27 Thread Joseph S. Myers
On Wed, 27 Apr 2011, Kaz Kojima wrote: Joseph S. Myers jos...@codesourcery.com wrote: I think you should add a testcase to gcc.c-torture/compile, unless there is already one that this patch fixes. Ah, indeed. How about the attached testcase? Yes, that testcase looks like what I had in

[PATCH Commitred] Fix PR target/48767

2011-04-26 Thread Kaz Kojima
Hi, The attached target specific patch is to fix PR target/48767. In the problematic case, sh.c:sh_gimplify_va_arg_expr calls targetm.calls.must_pass_in_stack with void type as its 2nd argument which is unexpected by the callee. The patch is tested on sh4-unknown-linux-gnu with no new failures.

Re: [PATCH Commitred] Fix PR target/48767

2011-04-26 Thread Joseph S. Myers
I think you should add a testcase to gcc.c-torture/compile, unless there is already one that this patch fixes. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH Commitred] Fix PR target/48767

2011-04-26 Thread Kaz Kojima
Joseph S. Myers jos...@codesourcery.com wrote: I think you should add a testcase to gcc.c-torture/compile, unless there is already one that this patch fixes. Ah, indeed. How about the attached testcase? BTW, is it valid C? Regards, kaz -- ---