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

Chris Lattner clattner at apple.com
Mon Oct 5 15:32:58 PDT 2009


On Oct 3, 2009, at 6:56 PM, Jeffrey Yasskin 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.

This looks ok to me.  However, we really prefer to minimize the number  
of executable tests that are in llvm/test.  I understand that this is  
hard to impossible for JIT tests though.  Please try to aggregate them  
together into fewer larger tests.

-Chris



More information about the llvm-commits mailing list