[LLVMdev] Tree Stability + Testing
Chris Lattner
sabre at nondot.org
Tue Mar 4 11:17:11 PST 2008
Hi Everyone,
This is just a reminder to test patches before they go in. As the pace of
development on LLVM is increasing, making sure your patch is solid is
increasingly important. This is because every change has some inherent
risk of breakage, and the more patches that go in, the higher the chance
that the tree will be broken when the nightly testers run.
Some guidelines:
0. Always build the tree before committing.
1. Always run make check at a minimum before checking in nontrivial
patches (e.g. comment changes).
2. Almost all patches should run a subset of llvm-test to verify that they
didn't break anything obvious. My personal favorite is
MultiSource/Benchmarks, but any other reasonable subset is ok.
3. For major changes, please run a large subset of the test suite, such as
all of MultiSource.
4. For particularly invasive or risky patches, please run the full
llvm-test suite and verify no regressions before checking stuff in.
There are exceptions to these rules of course: if your code is not run by
default, or is only enabled in 'llc-beta' mode, you basically just need to
make sure that the code builds and does not cause regressions in 'make
check', regardless of how risky it is.
The goal of the nightly tester is to find subtle/rare problems (the sort
that cause 3-4 programs to fail in a nightly tester), or problems that you
can't be reasonably expected to test (e.g. a problem that only manifests
on darwin/ppc, but you're testing on itanium/linux).
Think green, the tree appreciates it! :)
-Chris
--
http://nondot.org/sabre/
http://llvm.org/
More information about the llvm-dev
mailing list