[PATCH] ARM FastISel fix load register classes

Chandler Carruth chandlerc at google.com
Sat Jun 8 21:20:51 PDT 2013


On Sat, Jun 8, 2013 at 4:37 PM, Renato Golin <renato.golin at linaro.org>wrote:

> Though, it's ok to commit this patch without a test if future patches will
> test it.


Doh, saw this thread after seeing JF's commit and commenting on it.

Renato, while in extreme cases (IE, when we simply have zero way of
constructing test cases at this point) this is true, I think it should be a
last resort. If at all possible, a test case should still go in with the
patch.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130608/b462519b/attachment.html>


More information about the llvm-commits mailing list