Please apply 6 week old patch

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Please apply 6 week old patch

R P Herrold
The mention of c++ coming, induced me to retest

I bumped into a problem known for some time [it turns out],
and for which a patch has been sitting for six weeks
  http://llvm.org/bugs/show_bug.cgi?id=5803
  reprised at:
  http://llvm.org/bugs/show_bug.cgi?id=6319

I provide a reproducer, but obviously not a minimal one

What is the next step to get this commited to trunk, so I may
resume testing, continue to file, and hopefully clear all my
c++ issues before the anouncement date?

-- Russ herrold
_______________________________________________
cfe-dev mailing list
[hidden email]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
Reply | Threaded
Open this post in threaded view
|

Re: Please apply 6 week old patch

Daniel Dunbar
Thanks for the ping, applied/fixed here:
  http://llvm.org/viewvc/llvm-project?view=rev&revision=96511

It has a warning, because I think this is pretty grotty behavior. Once
the driver starts honoring diagnostic switches I will consider moving
it to -Wall or -pedantic, if it irritates users enough.

 - Daniel

On Wed, Feb 17, 2010 at 8:57 AM, R P Herrold <[hidden email]> wrote:

> The mention of c++ coming, induced me to retest
>
> I bumped into a problem known for some time [it turns out],
> and for which a patch has been sitting for six weeks
>        http://llvm.org/bugs/show_bug.cgi?id=5803
>        reprised at:
>        http://llvm.org/bugs/show_bug.cgi?id=6319
>
> I provide a reproducer, but obviously not a minimal one
>
> What is the next step to get this commited to trunk, so I may
> resume testing, continue to file, and hopefully clear all my
> c++ issues before the anouncement date?
>
> -- Russ herrold
> _______________________________________________
> cfe-dev mailing list
> [hidden email]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
>

_______________________________________________
cfe-dev mailing list
[hidden email]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
Reply | Threaded
Open this post in threaded view
|

Re: Please apply 6 week old patch

R P Herrold
On Wed, 17 Feb 2010, Daniel Dunbar wrote:

> Thanks for the ping, applied/fixed here:
>  http://llvm.org/viewvc/llvm-project?view=rev&revision=96511

and thanks for the commit  ;)

testing results in a bit

-- Russ herrold
_______________________________________________
cfe-dev mailing list
[hidden email]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev