[llvm-commits] [patch] Add a test for PR3043

Jeffrey Yasskin jyasskin at google.com
Mon Oct 5 12:59:22 PDT 2009


Ping?

(Sorry for being pushy, but the PR5116 fix that this makes safe is
blocking our Q3 release.)

On Sat, Oct 3, 2009 at 6:56 PM, Jeffrey Yasskin <jyasskin at google.com> wrote:
> In fixing bugs around stubs, it would be nice to have tests for the
> bugs that have already been fixed so I don't reopen them. Here's one
> for http://llvm.org/bugs/show_bug.cgi?id=3043. Since it's my first
> addition to the dejagnu test suite, I'd appreciate any stylistic
> pointers. I tested the test by disabling r59265, and indeed the test
> fails.
>
> Evan, could you write a test for the bug
> http://llvm.org/viewvc/llvm-project?view=rev&revision=60822 fixed? My
> current fix for PR5116 simply reverts that change, and I want to know
> if I'm breaking something or if the original bug has now been fixed
> some other way.
>
> Thanks,
> Jeffrey
>



More information about the llvm-commits mailing list