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

Diana Picus via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 26 08:44:22 PDT 2017


rovka added a comment.

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.


https://reviews.llvm.org/D32529





More information about the llvm-commits mailing list