[PATCH] D32529: [GlobalISel] Make multi-step legalization work.

Kristof Beyls via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 26 09:53:41 PDT 2017


kristof.beyls added a comment.

In https://reviews.llvm.org/D32529#738192, @rovka wrote:

> Thanks for splitting this out! The ARM part looks excellent :) 
>  I think you could add some target-independent tests for this in unittests/CodeGen/GlobalISel/LegalizerInfoTest.cpp, to make sure it works to WidenScalar/MoreElements etc to a type that has a Libcall or Custom action. That way we won't have to depend on ARM for testing this functionality.


A yes, that makes sense. Now added.


https://reviews.llvm.org/D32529





More information about the llvm-commits mailing list