This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [tree-ssa] [PATCH]: Add some PRE regression tests


On Tue, 2003-11-25 at 13:01, Daniel Berlin wrote:

> Basically, if PRE is broken again, these tests will fail to compile due 
> to an abort or another error message (IE there is no need to execute 
> them).
> 
So, we don't need to scan a dump file making sure something that should
be gone really is?  Otherwise, they look OK.  I would only ask you to
add a comment to each test case explaining what it is that it's testing.

> Also, do i need a changelog entry for new tests (what changelog do they 
> go in?)
> 
There are two schools of thought here.  Some people add them, some
don't.  I personally add them, but I'm not going to enforce it.  If you
do, modify testsuite/ChangeLog.tree-ssa.


Diego.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]