[PATCH] D123860: GlobalISel: Relax handling of G_ASSERT_* with source register classes

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 21 17:43:13 PDT 2022


arsenm added inline comments.


================
Comment at: llvm/test/MachineVerifier/test_g_assert_zext_register_bank_class.mir:32
    %implicit_physreg:gpr(s32) = G_ASSERT_ZEXT %class, 16, implicit-def $w0
+
+   %nothing:_(s32) = G_IMPLICIT_DEF
----------------
paquette wrote:
> maybe we should also test like
> 
> ```
> %nothing64:_(s64) = G_IMPLICIT_DEF
> %dst_32:_(s32) = G_ASSERT_ZEXT %nothing64, 4
> ```
That's already illegal and tested in test_g_assert_zext


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

https://reviews.llvm.org/D123860



More information about the llvm-commits mailing list