[PATCH] microblaze: microblaze.md: Use VOID instead of SI to fix "((void (*)(void)) 0)()" issue

2014-09-25 Thread Chen Gang
Need use VOID instead of SI, or when real VOIDmode comes, it does not match SImode, so cause issue. This patch can fix this issue and pass testsuite. The related test code ('void' will cause CALL instead of SET): typedef void (*T)(void); f1 () { ((T) 0)(); } The related error: [ro

Re: [PATCH] microblaze: microblaze.md: Use VOID instead of SI to fix "((void (*)(void)) 0)()" issue

2014-09-25 Thread Michael Eager
On 09/25/14 07:03, Chen Gang wrote: Need use VOID instead of SI, or when real VOIDmode comes, it does not match SImode, so cause issue. This patch can fix this issue and pass testsuite. Did you forget to attach the patch? -- Michael Eagerea...@eagercon.com 1960 Park Blvd., Palo Alto, CA 94

Re: [PATCH] microblaze: microblaze.md: Use VOID instead of SI to fix "((void (*)(void)) 0)()" issue

2014-09-25 Thread Michael Eager
On 09/25/14 10:38, Michael Eager wrote: On 09/25/14 07:03, Chen Gang wrote: Need use VOID instead of SI, or when real VOIDmode comes, it does not match SImode, so cause issue. This patch can fix this issue and pass testsuite. Did you forget to attach the patch? Never mind. My eyes were play

Re: [PATCH] microblaze: microblaze.md: Use VOID instead of SI to fix "((void (*)(void)) 0)()" issue

2014-09-25 Thread Chen Gang
Yeah, the comments are big, but the body is the smallest. Send from Lenovo A788t. Michael Eager wrote: >On 09/25/14 10:38, Michael Eager wrote: >> On 09/25/14 07:03, Chen Gang wrote: >>> Need use VOID instead of SI, or when real VOIDmode comes, it does not >>> match SImode, so cause issue. Th

Re: [PATCH] microblaze: microblaze.md: Use VOID instead of SI to fix "((void (*)(void)) 0)()" issue

2014-09-27 Thread Michael Eager
On 09/25/14 07:03, Chen Gang wrote: Need use VOID instead of SI, or when real VOIDmode comes, it does not match SImode, so cause issue. This patch can fix this issue and pass testsuite. The related test code ('void' will cause CALL instead of SET): typedef void (*T)(void); f1 () {

Re: [PATCH] microblaze: microblaze.md: Use VOID instead of SI to fix "((void (*)(void)) 0)()" issue

2014-09-27 Thread Chen Gang
I guess it is not in our test case, or after this patch, the new should get a little better result than the old (at present, they are same). I shall try to add related case into testsuite within this month. Thanks Send from Lenovo A788t. Michael Eager wrote: >On 09/25/14 07:03, Chen Gang wrot

Re: [PATCH] microblaze: microblaze.md: Use VOID instead of SI to fix "((void (*)(void)) 0)()" issue

2014-09-27 Thread Chen Gang
And excuse me, I am not quite familiar with adding testsuite, so during I am trying, welcome any related ideas, suggestions or completions. Thanks. On 9/28/14 3:08, Chen Gang wrote: > I guess it is not in our test case, or after this patch, the new > should get a little better result than the ol

Re: [PATCH] microblaze: microblaze.md: Use VOID instead of SI to fix "((void (*)(void)) 0)()" issue

2014-09-29 Thread Michael Eager
On 09/25/14 07:03, Chen Gang wrote: 2014-09-25 Chen Gang gcc: * config/microblaze/microblaze.md (call_internal1): Use VOID instead of SI to fix "((void (*)(void)) 0)()" issue gcc/testsuite/: 2014-09-28 Chen Gang * gcc.c-torture/compile/calls-void.c: New test. C