Fwd: NDS32 V3 backend

Joerg Sonnenberger via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 14 12:54:55 PST 2017


On Fri, Jan 13, 2017 at 05:01:22PM +0100, Tobias Grosser via llvm-commits wrote:
> I had a quick look at your patches. It seems you do not have any test
> cases (except in the last patch). If this is true, I am afraid this will
> be one of the first things reviewers will miss.

To expand on this slightly: ideally, each new phase of the backend
should have patches that test that the newly added part works. This
makes it useful to hook up MC ASAP so that you can test instruction
patterns etc.

Joerg


More information about the llvm-commits mailing list