[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