public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [Ping][Patch, testsuite]: Don't xfail sibcalls on AVR
@ 2011-03-29  9:38 Georg-Johann Lay
  2011-04-05  9:06 ` [Ping #2][Patch, " Georg-Johann Lay
  2011-04-08 23:18 ` [Ping][Patch, " Weddington, Eric
  0 siblings, 2 replies; 11+ messages in thread
From: Georg-Johann Lay @ 2011-03-29  9:38 UTC (permalink / raw)
  To: gcc-patches; +Cc: Anatoly Sokolov, Denis Chertykov, Eric Weddington

[-- Attachment #1: Type: text/plain, Size: 206 bytes --]

Target avr now supports tail calls, so don't xfail on that.

testsuite/

2011-03-29  Georg-Johann Lay  <avr@gjlay.de>

	* gcc.dg/sibcall-3.c: Don't xfail on AVR.
	* gcc.dg/sibcall-4.c: Don't xfail on AVR.


[-- Attachment #2: testsuite.diff --]
[-- Type: text/x-patch, Size: 1853 bytes --]

Index: testsuite/gcc.dg/sibcall-3.c
===================================================================
--- testsuite/gcc.dg/sibcall-3.c	(Revision 171039)
+++ testsuite/gcc.dg/sibcall-3.c	(Arbeitskopie)
@@ -5,7 +5,7 @@
    Copyright (C) 2002 Free Software Foundation Inc.
    Contributed by Hans-Peter Nilsson  <hp@bitrange.com>  */
 
-/* { dg-do run { xfail { { arc-*-* avr-*-* cris-*-* crisv32-*-* h8300-*-* hppa*64*-*-* m32r-*-* m68hc1?-*-* mcore-*-* mn10300-*-* xstormy16-*-* v850*-*-* vax-*-* xtensa*-*-* } || { arm*-*-* && { ! arm32 } } } } } */
+/* { dg-do run { xfail { { arc-*-* cris-*-* crisv32-*-* h8300-*-* hppa*64*-*-* m32r-*-* m68hc1?-*-* mcore-*-* mn10300-*-* xstormy16-*-* v850*-*-* vax-*-* xtensa*-*-* } || { arm*-*-* && { ! arm32 } } } } } */
 /* -mlongcall disables sibcall patterns.  */
 /* { dg-skip-if "" { powerpc*-*-* } { "-mlongcall" } { "" } } */
 /* { dg-options "-O2 -foptimize-sibling-calls" } */
Index: testsuite/gcc.dg/sibcall-4.c
===================================================================
--- testsuite/gcc.dg/sibcall-4.c	(Revision 171039)
+++ testsuite/gcc.dg/sibcall-4.c	(Arbeitskopie)
@@ -5,7 +5,7 @@
    Copyright (C) 2002 Free Software Foundation Inc.
    Contributed by Hans-Peter Nilsson  <hp@bitrange.com>  */
 
-/* { dg-do run { xfail { { arc-*-* avr-*-* cris-*-* crisv32-*-* h8300-*-* hppa*64*-*-* m32r-*-* m68hc1?-*-* mcore-*-* mn10300-*-* xstormy16-*-* v850*-*-* vax-*-* xtensa*-*-* } || { arm*-*-* && { ! arm32 } } } } } */
+/* { dg-do run { xfail { { arc-*-* cris-*-* crisv32-*-* h8300-*-* hppa*64*-*-* m32r-*-* m68hc1?-*-* mcore-*-* mn10300-*-* xstormy16-*-* v850*-*-* vax-*-* xtensa*-*-* } || { arm*-*-* && { ! arm32 } } } } } */
 /* -mlongcall disables sibcall patterns.  */
 /* { dg-skip-if "" { powerpc*-*-* } { "-mlongcall" } { "" } } */
 /* { dg-options "-O2 -foptimize-sibling-calls" } */


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [Ping #2][Patch, testsuite]: Don't xfail sibcalls on AVR
  2011-03-29  9:38 [Ping][Patch, testsuite]: Don't xfail sibcalls on AVR Georg-Johann Lay
@ 2011-04-05  9:06 ` Georg-Johann Lay
  2011-04-05 23:01   ` Weddington, Eric
  2011-04-08 23:18 ` [Ping][Patch, " Weddington, Eric
  1 sibling, 1 reply; 11+ messages in thread
From: Georg-Johann Lay @ 2011-04-05  9:06 UTC (permalink / raw)
  To: gcc-patches; +Cc: Anatoly Sokolov, Denis Chertykov, Eric Weddington

Georg-Johann Lay schrieb:
> Target avr now supports tail calls, so don't xfail on that.
> 
> testsuite/
> 
> 2011-03-29  Georg-Johann Lay  <avr@gjlay.de>
> 
> 	* gcc.dg/sibcall-3.c: Don't xfail on AVR.
> 	* gcc.dg/sibcall-4.c: Don't xfail on AVR.
> 
> 

^ permalink raw reply	[flat|nested] 11+ messages in thread

* RE: [Ping #2][Patch, testsuite]: Don't xfail sibcalls on AVR
  2011-04-05  9:06 ` [Ping #2][Patch, " Georg-Johann Lay
@ 2011-04-05 23:01   ` Weddington, Eric
  2011-04-06 10:54     ` Georg-Johann Lay
  0 siblings, 1 reply; 11+ messages in thread
From: Weddington, Eric @ 2011-04-05 23:01 UTC (permalink / raw)
  To: Georg-Johann Lay, gcc-patches; +Cc: Anatoly Sokolov, Denis Chertykov



> -----Original Message-----
> From: Georg-Johann Lay [mailto:avr@gjlay.de]
> Sent: Tuesday, April 05, 2011 3:06 AM
> To: gcc-patches@gcc.gnu.org
> Cc: Anatoly Sokolov; Denis Chertykov; Weddington, Eric
> Subject: Re: [Ping #2][Patch, testsuite]: Don't xfail sibcalls on AVR
> 
> Georg-Johann Lay schrieb:
> > Target avr now supports tail calls, so don't xfail on that.
> >
> > testsuite/
> >
> > 2011-03-29  Georg-Johann Lay  <avr@gjlay.de>
> >
> > 	* gcc.dg/sibcall-3.c: Don't xfail on AVR.
> > 	* gcc.dg/sibcall-4.c: Don't xfail on AVR.
> >
> >

Please commit, Johann. Thanks.

Eric

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [Ping #2][Patch, testsuite]: Don't xfail sibcalls on AVR
  2011-04-05 23:01   ` Weddington, Eric
@ 2011-04-06 10:54     ` Georg-Johann Lay
  2011-04-06 12:19       ` Weddington, Eric
  0 siblings, 1 reply; 11+ messages in thread
From: Georg-Johann Lay @ 2011-04-06 10:54 UTC (permalink / raw)
  To: Weddington, Eric; +Cc: gcc-patches, Anatoly Sokolov, Denis Chertykov

Weddington, Eric schrieb:
> 
>> -----Original Message-----
>> From: Georg-Johann Lay [mailto:avr@gjlay.de]
>> Sent: Tuesday, April 05, 2011 3:06 AM
>> To: gcc-patches@gcc.gnu.org
>> Cc: Anatoly Sokolov; Denis Chertykov; Weddington, Eric
>> Subject: Re: [Ping #2][Patch, testsuite]: Don't xfail sibcalls on AVR
>>
>> Georg-Johann Lay schrieb:
>>> Target avr now supports tail calls, so don't xfail on that.
>>>
>>> testsuite/
>>>
>>> 2011-03-29  Georg-Johann Lay  <avr@gjlay.de>
>>>
>>> 	* gcc.dg/sibcall-3.c: Don't xfail on AVR.
>>> 	* gcc.dg/sibcall-4.c: Don't xfail on AVR.
>>>
> 
> Please commit, Johann. Thanks.
> 
> Eric

I don't have write privilege, and I don't think I want write access at
the moment.

Johann



^ permalink raw reply	[flat|nested] 11+ messages in thread

* RE: [Ping #2][Patch, testsuite]: Don't xfail sibcalls on AVR
  2011-04-06 10:54     ` Georg-Johann Lay
@ 2011-04-06 12:19       ` Weddington, Eric
  2011-04-06 12:55         ` Georg-Johann Lay
  0 siblings, 1 reply; 11+ messages in thread
From: Weddington, Eric @ 2011-04-06 12:19 UTC (permalink / raw)
  To: Georg-Johann Lay; +Cc: gcc-patches, Anatoly Sokolov, Denis Chertykov



> -----Original Message-----
> From: Georg-Johann Lay [mailto:avr@gjlay.de]
> Sent: Wednesday, April 06, 2011 4:55 AM
> To: Weddington, Eric
> Cc: gcc-patches@gcc.gnu.org; Anatoly Sokolov; Denis Chertykov
> Subject: Re: [Ping #2][Patch, testsuite]: Don't xfail sibcalls on AVR
> 
> >>> Target avr now supports tail calls, so don't xfail on that.
> >>>
> >>> testsuite/
> >>>
> >>> 2011-03-29  Georg-Johann Lay  <avr@gjlay.de>
> >>>
> >>> 	* gcc.dg/sibcall-3.c: Don't xfail on AVR.
> >>> 	* gcc.dg/sibcall-4.c: Don't xfail on AVR.
> >>>
> >
> > Please commit, Johann. Thanks.
> >
> > Eric
> 
> I don't have write privilege, and I don't think I want write access at
> the moment.

Oh! I didn't know. I thought you did.

Ok, let's see what I can do, then.

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [Ping #2][Patch, testsuite]: Don't xfail sibcalls on AVR
  2011-04-06 12:19       ` Weddington, Eric
@ 2011-04-06 12:55         ` Georg-Johann Lay
  2011-04-06 23:26           ` Gerald Pfeifer
  0 siblings, 1 reply; 11+ messages in thread
From: Georg-Johann Lay @ 2011-04-06 12:55 UTC (permalink / raw)
  To: Weddington, Eric; +Cc: gcc-patches, Anatoly Sokolov, Denis Chertykov

Weddington, Eric schrieb:
> 
>> -----Original Message-----
>> From: Georg-Johann Lay [mailto:avr@gjlay.de]
>> To: Weddington, Eric

>>> Please commit, Johann. Thanks.
>>>
>> I don't have write privilege, and I don't think I want write access at
>> the moment.
> 
> Oh! I didn't know. I thought you did.

I signed a fsf copyright assignment. AFAIK there is no automatism that
grants write access when copyright assignment is approved. I would
have to be maintainer or something like that.

Johann

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [Ping #2][Patch, testsuite]: Don't xfail sibcalls on AVR
  2011-04-06 12:55         ` Georg-Johann Lay
@ 2011-04-06 23:26           ` Gerald Pfeifer
  2011-04-08  9:30             ` Georg-Johann Lay
  0 siblings, 1 reply; 11+ messages in thread
From: Gerald Pfeifer @ 2011-04-06 23:26 UTC (permalink / raw)
  To: Georg-Johann Lay
  Cc: Weddington, Eric, gcc-patches, Anatoly Sokolov, Denis Chertykov

On Wed, 6 Apr 2011, Georg-Johann Lay wrote:
> I signed a fsf copyright assignment. AFAIK there is no automatism
> that grants write access when copyright assignment is approved.

Yes, this is correct.

> I would have to be maintainer or something like that.

Nope.  All you need is a maintainer (Eric, Denis and Anatoly all
qualifying) to sponsor your request.  Obviously that mostly makes
sense if you expect to contribute more than a handfull of patches
over the course of the next year, which we do hope will be the
case. :-)

Gerald

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [Ping #2][Patch, testsuite]: Don't xfail sibcalls on AVR
  2011-04-06 23:26           ` Gerald Pfeifer
@ 2011-04-08  9:30             ` Georg-Johann Lay
  2011-04-09  7:30               ` Ralf Wildenhues
  0 siblings, 1 reply; 11+ messages in thread
From: Georg-Johann Lay @ 2011-04-08  9:30 UTC (permalink / raw)
  To: Gerald Pfeifer
  Cc: Weddington, Eric, gcc-patches, Anatoly Sokolov, Denis Chertykov

Gerald Pfeifer schrieb:
> On Wed, 6 Apr 2011, Georg-Johann Lay wrote:
>> I signed a fsf copyright assignment. AFAIK there is no automatism
>> that grants write access when copyright assignment is approved.
> 
> Yes, this is correct.
> 
>> I would have to be maintainer or something like that.
> 
> Nope.  All you need is a maintainer (Eric, Denis and Anatoly all
> qualifying) to sponsor your request.  Obviously that mostly makes
> sense if you expect to contribute more than a handfull of patches
> over the course of the next year, which we do hope will be the
> case. :-)
> 
> Gerald

Thanks for that hint. Is that the "Write after Approval" section in
MAINTAINERS?

Having write permissions would make thinks easier, both for the
maintainers and for me, I think.

However, I have bunch of questions for that I don't find answers in
the wiki. As it is very easy *not* to find something in the gcc wiki,
I probably just missed it...

1) What is the exact procedure for this? Will it be like fsf copyright
   assignment and take more than a year (at least in my case) to pass?

2) Changelogs and svn logs are always in sync. Is there some setup
   in the svn server to do this magic? Are there custom scripts for
   this or is it all hand-made?

3) Ditto for PRs that get a notification as soon as a patch goes
   upstream.

4) Will there be someone like a mentor who I could ask such
   questions or get useful hints?

Johann

^ permalink raw reply	[flat|nested] 11+ messages in thread

* RE: [Ping][Patch, testsuite]: Don't xfail sibcalls on AVR
  2011-03-29  9:38 [Ping][Patch, testsuite]: Don't xfail sibcalls on AVR Georg-Johann Lay
  2011-04-05  9:06 ` [Ping #2][Patch, " Georg-Johann Lay
@ 2011-04-08 23:18 ` Weddington, Eric
  1 sibling, 0 replies; 11+ messages in thread
From: Weddington, Eric @ 2011-04-08 23:18 UTC (permalink / raw)
  To: Georg-Johann Lay, gcc-patches; +Cc: Anatoly Sokolov, Denis Chertykov



> -----Original Message-----
> From: Georg-Johann Lay [mailto:avr@gjlay.de]
> Sent: Tuesday, March 29, 2011 3:00 AM
> To: gcc-patches@gcc.gnu.org
> Cc: Anatoly Sokolov; Denis Chertykov; Weddington, Eric
> Subject: [Ping][Patch, testsuite]: Don't xfail sibcalls on AVR
> 
> Target avr now supports tail calls, so don't xfail on that.
> 
> testsuite/
> 
> 2011-03-29  Georg-Johann Lay  <avr@gjlay.de>
> 
> 	* gcc.dg/sibcall-3.c: Don't xfail on AVR.
> 	* gcc.dg/sibcall-4.c: Don't xfail on AVR.

Committed. Thanks for the patch, Johann.

Eric

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [Ping #2][Patch, testsuite]: Don't xfail sibcalls on AVR
  2011-04-08  9:30             ` Georg-Johann Lay
@ 2011-04-09  7:30               ` Ralf Wildenhues
  2011-04-09 15:40                 ` Gerald Pfeifer
  0 siblings, 1 reply; 11+ messages in thread
From: Ralf Wildenhues @ 2011-04-09  7:30 UTC (permalink / raw)
  To: Georg-Johann Lay
  Cc: Gerald Pfeifer, Weddington, Eric, gcc-patches, Anatoly Sokolov,
	Denis Chertykov

* Georg-Johann Lay wrote on Fri, Apr 08, 2011 at 11:30:29AM CEST:
> Gerald Pfeifer schrieb:
> > Nope.  All you need is a maintainer (Eric, Denis and Anatoly all
> > qualifying) to sponsor your request.  Obviously that mostly makes
> > sense if you expect to contribute more than a handfull of patches
> > over the course of the next year, which we do hope will be the
> > case. :-)

> Thanks for that hint. Is that the "Write after Approval" section in
> MAINTAINERS?

Yes.

> However, I have bunch of questions for that I don't find answers in
> the wiki. As it is very easy *not* to find something in the gcc wiki,
> I probably just missed it...
> 
> 1) What is the exact procedure for this? Will it be like fsf copyright
>    assignment and take more than a year (at least in my case) to pass?

Somebody says "you can mention me as sponsor", you put him or her in the
approval in the form you fill out after reading
http://gcc.gnu.org/svnwrite.html and wait for the login to be approved.
In my case, it took 3 hours, but I wouldn't worry if it took a weekend
plus a couple of workdays until one of the sourceware people has time.

> 2) Changelogs and svn logs are always in sync. Is there some setup
>    in the svn server to do this magic? Are there custom scripts for
>    this or is it all hand-made?

FWIW, some people use a script named mklog[1], a while ago I wrote
vc-chlog[2] for generating stub ChangeLog entries.  I generate the svn
log entries from the ChangeLog entries with a couple of editor macros,
dunno if there's public scripting for that.

> 3) Ditto for PRs that get a notification as soon as a patch goes
>    upstream.

The mechanism is triggered by the
        PR category/12345

entries in the log.  The category is the 'Component' field in bugzilla.

> 4) Will there be someone like a mentor who I could ask such
>    questions or get useful hints?

I'm not aware of a special mentor program for new contributors, but
people here assume that beginners make beginner errors, read and search
mailing lists and/or the wiki and web pages, and learn.  I wouldn't
worry too much, you'll be corrected.  ;-)

Cheers,
Ralf

[1] http://sourceware.org/gdb/wiki/DeveloperTips
[2] http://www.gnu.org/software/vc-dwim/

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [Ping #2][Patch, testsuite]: Don't xfail sibcalls on AVR
  2011-04-09  7:30               ` Ralf Wildenhues
@ 2011-04-09 15:40                 ` Gerald Pfeifer
  0 siblings, 0 replies; 11+ messages in thread
From: Gerald Pfeifer @ 2011-04-09 15:40 UTC (permalink / raw)
  To: Ralf Wildenhues
  Cc: Georg-Johann Lay, Weddington, Eric, gcc-patches, Anatoly Sokolov,
	Denis Chertykov

On Sat, 9 Apr 2011, Ralf Wildenhues wrote:
>> 4) Will there be someone like a mentor who I could ask such
>>    questions or get useful hints?
> I'm not aware of a special mentor program for new contributors

I would say, however, that the one approving the request for a
gcc.gnu.org account should be ready (and offering) to help.

Gerald

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2011-04-09 15:40 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-03-29  9:38 [Ping][Patch, testsuite]: Don't xfail sibcalls on AVR Georg-Johann Lay
2011-04-05  9:06 ` [Ping #2][Patch, " Georg-Johann Lay
2011-04-05 23:01   ` Weddington, Eric
2011-04-06 10:54     ` Georg-Johann Lay
2011-04-06 12:19       ` Weddington, Eric
2011-04-06 12:55         ` Georg-Johann Lay
2011-04-06 23:26           ` Gerald Pfeifer
2011-04-08  9:30             ` Georg-Johann Lay
2011-04-09  7:30               ` Ralf Wildenhues
2011-04-09 15:40                 ` Gerald Pfeifer
2011-04-08 23:18 ` [Ping][Patch, " Weddington, Eric

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).