[PATCH] D85199: GlobalISel: Add generic instructions for memory intrinsics

Dominik Montada via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 17 05:52:33 PDT 2020


gargaroff added a comment.

In D85199#2221113 <https://reviews.llvm.org/D85199#2221113>, @arsenm wrote:

> Cast to minimum in IRTranslator. I decided making this a verifier requirement is too  strong since some target may have an instruction that looks like memcpy with a different integer size

Thank you for this! I think this is a good approach for this problem and we should be able to get rid of the legalizer workaround we have in place now.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D85199/new/

https://reviews.llvm.org/D85199



More information about the llvm-commits mailing list