ping patches (regressions)

public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed

* ping patches (regressions)
@ 2004-10-17 17:55 Andrew Pinski
  2004-10-17 20:04 ` Roger Sayle
  2004-10-21 15:34 ` Geoffrey Keating
  0 siblings, 2 replies; 9+ messages in thread
From: Andrew Pinski @ 2004-10-17 17:55 UTC (permalink / raw)
  To: gcc-patches@gcc.gnu.org Patches

http://gcc.gnu.org/ml/gcc-patches/2004-10/msg00884.html
Gotos in conditional expression don't get a line note

http://gcc.gnu.org/ml/gcc-patches/2004-10/msg00314.html
Fix const-str-[34].m on darwin, the objective-C part has been approved

http://gcc.gnu.org/ml/gcc-patches/2004-10/msg00860.html
labels which have their address taken are removed

http://gcc.gnu.org/ml/gcc-patches/2004-10/msg00843.html
File names are wrong with IMA

http://gcc.gnu.org/ml/gcc-patches/2004-10/msg00800.html
Wrong file name in C++ error

http://gcc.gnu.org/ml/gcc-patches/2004-09/msg02909.html
Objective-C problem, bootstrap problem on x86-solaris

Thanks,
Andrew Pinski

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

* Re: ping patches (regressions)
  2004-10-17 17:55 ping patches (regressions) Andrew Pinski
@ 2004-10-17 20:04 ` Roger Sayle
  2004-10-17 20:21   ` Zack Weinberg
  2004-10-21 15:34 ` Geoffrey Keating
  1 sibling, 1 reply; 9+ messages in thread
From: Roger Sayle @ 2004-10-17 20:04 UTC (permalink / raw)
  To: Andrew Pinski; +Cc: gcc-patches


On Sun, 17 Oct 2004, Andrew Pinski wrote:
> http://gcc.gnu.org/ml/gcc-patches/2004-10/msg00884.html
> Gotos in conditional expression don't get a line note

This is OK for mainline, provided that you avoid introducing two
consective blank lines in the third hunk (as pointed out by Paolo).

Thanks,

Roger
--

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

* Re: ping patches (regressions)
  2004-10-17 20:04 ` Roger Sayle
@ 2004-10-17 20:21   ` Zack Weinberg
  0 siblings, 0 replies; 9+ messages in thread
From: Zack Weinberg @ 2004-10-17 20:21 UTC (permalink / raw)
  To: Roger Sayle; +Cc: Andrew Pinski, gcc-patches

Roger Sayle <roger@eyesopen.com> writes:

> On Sun, 17 Oct 2004, Andrew Pinski wrote:
>> http://gcc.gnu.org/ml/gcc-patches/2004-10/msg00884.html
>> Gotos in conditional expression don't get a line note
>
> This is OK for mainline, provided that you avoid introducing two
> consective blank lines in the third hunk (as pointed out by Paolo).

Also, never put anything else on the same line as a ^L character.

zw

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

* Re: ping patches (regressions)
  2004-10-17 17:55 ping patches (regressions) Andrew Pinski
  2004-10-17 20:04 ` Roger Sayle
@ 2004-10-21 15:34 ` Geoffrey Keating
  2004-10-21 15:56   ` Andrew Pinski
  2004-10-21 17:09   ` Andrew Pinski
  1 sibling, 2 replies; 9+ messages in thread
From: Geoffrey Keating @ 2004-10-21 15:34 UTC (permalink / raw)
  To: Andrew Pinski; +Cc: gcc-patches, zlaski

Andrew Pinski <pinskia@physics.uc.edu> writes:

> http://gcc.gnu.org/ml/gcc-patches/2004-10/msg00314.html
> Fix const-str-[34].m on darwin, the objective-C part has been approved

This is OK.

> http://gcc.gnu.org/ml/gcc-patches/2004-10/msg00860.html
> labels which have their address taken are removed

Is this still necessary?

> http://gcc.gnu.org/ml/gcc-patches/2004-10/msg00843.html
> File names are wrong with IMA

This is OK.

> http://gcc.gnu.org/ml/gcc-patches/2004-10/msg00800.html
> Wrong file name in C++ error

Is this still necessary?  When I tried your testcase, I got:

test.h: In function `int main()':
test.h:3: error: `int t::g' is private
test.cc:8: error: within this context

> http://gcc.gnu.org/ml/gcc-patches/2004-09/msg02909.html
> Objective-C problem, bootstrap problem on x86-solaris

Zem should look at this one.

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

* Re: ping patches (regressions)
  2004-10-21 15:34 ` Geoffrey Keating
@ 2004-10-21 15:56   ` Andrew Pinski
  2004-10-21 16:08     ` Geoff Keating
  2004-10-21 17:09   ` Andrew Pinski
  1 sibling, 1 reply; 9+ messages in thread
From: Andrew Pinski @ 2004-10-21 15:56 UTC (permalink / raw)
  To: Geoffrey Keating; +Cc: gcc-patches@gcc.gnu.org Patches


On Oct 21, 2004, at 11:23 AM, Geoffrey Keating wrote:

>> http://gcc.gnu.org/ml/gcc-patches/2004-10/msg00800.html
>> Wrong file name in C++ error
>
> Is this still necessary?  When I tried your testcase, I got:
>
> test.h: In function `int main()':
> test.h:3: error: `int t::g' is private
> test.cc:8: error: within this context

Note:
> test.h: In function `int main()':

The file name is wrong there there which gives a false
sense of where the function is.

Thanks,
Andrew Pinski

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

* Re: ping patches (regressions)
  2004-10-21 15:56   ` Andrew Pinski
@ 2004-10-21 16:08     ` Geoff Keating
  0 siblings, 0 replies; 9+ messages in thread
From: Geoff Keating @ 2004-10-21 16:08 UTC (permalink / raw)
  To: Andrew Pinski; +Cc: gcc-patches@gcc.gnu.org Patches

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


On 21/10/2004, at 8:30 AM, Andrew Pinski wrote:

>
> On Oct 21, 2004, at 11:23 AM, Geoffrey Keating wrote:
>
>>> http://gcc.gnu.org/ml/gcc-patches/2004-10/msg00800.html
>>> Wrong file name in C++ error
>>
>> Is this still necessary?  When I tried your testcase, I got:
>>
>> test.h: In function `int main()':
>> test.h:3: error: `int t::g' is private
>> test.cc:8: error: within this context
>
> Note:
>> test.h: In function `int main()':
>
> The file name is wrong there there which gives a false
> sense of where the function is.

Doh!  I read those error messages twice and still didn't see that.

The patch is OK then.


[-- Attachment #2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 2410 bytes --]

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

* Re: ping patches (regressions)
  2004-10-21 15:34 ` Geoffrey Keating
  2004-10-21 15:56   ` Andrew Pinski
@ 2004-10-21 17:09   ` Andrew Pinski
  1 sibling, 0 replies; 9+ messages in thread
From: Andrew Pinski @ 2004-10-21 17:09 UTC (permalink / raw)
  To: Geoffrey Keating; +Cc: gcc-patches@gcc.gnu.org Patches


On Oct 21, 2004, at 11:23 AM, Geoffrey Keating wrote:

>> http://gcc.gnu.org/ml/gcc-patches/2004-10/msg00860.html
>> labels which have their address taken are removed
>
> Is this still necessary?

This was already approved and applied.

Thanks,
Andrew Pinski

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

* ping patches (regressions)
@ 2005-02-21 18:21 Andrew Pinski
  2005-02-21 18:48 ` Roger Sayle
  0 siblings, 1 reply; 9+ messages in thread
From: Andrew Pinski @ 2005-02-21 18:21 UTC (permalink / raw)
  To: patch Patches

I am going to ping some important patches which fixes regressions.

http://gcc.gnu.org/ml/gcc-patches/2005-02/msg00657.html
Fix PR 19937, Fixes binutils miscompiling, forgot to fold_convert a
value in IV-OPTS.

Well that was the only one which I could find right now.

-- Pinski

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

* Re: ping patches (regressions)
  2005-02-21 18:21 Andrew Pinski
@ 2005-02-21 18:48 ` Roger Sayle
  0 siblings, 0 replies; 9+ messages in thread
From: Roger Sayle @ 2005-02-21 18:48 UTC (permalink / raw)
  To: Zdenek Dvorak; +Cc: Andrew Pinski, gcc-patches


On Mon, 21 Feb 2005, Andrew Pinski wrote:
> http://gcc.gnu.org/ml/gcc-patches/2005-02/msg00657.html
> Fix PR 19937, Fixes binutils miscompiling, forgot to fold_convert a
> value in IV-OPTS.

This is OK for mainline.


> Well that was the only one which I could find right now.
:-)

Roger
--

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

end of thread, other threads:[~2005-02-21 15:06 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-10-17 17:55 ping patches (regressions) Andrew Pinski
2004-10-17 20:04 ` Roger Sayle
2004-10-17 20:21   ` Zack Weinberg
2004-10-21 15:34 ` Geoffrey Keating
2004-10-21 15:56   ` Andrew Pinski
2004-10-21 16:08     ` Geoff Keating
2004-10-21 17:09   ` Andrew Pinski
2005-02-21 18:21 Andrew Pinski
2005-02-21 18:48 ` Roger Sayle

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).