[PATCH] D53679: [AArch64][GlobalISel] Fix the LegalityPredicate for lowerIf for G_LOAD/G_STORE
Daniel Sanders via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 24 18:45:58 PDT 2018
dsanders accepted this revision.
dsanders added a comment.
This revision is now accepted and ready to land.
LGTM with the test nit
Would you be willing to rename that `Size` member to `SizeInBits` to make the value clearer?
================
Comment at: test/CodeGen/AArch64/GlobalISel/legalize-load-v4s32.mir:5
+# CHECK: Legalize Machine IR for: load_v4s32
+# CHECK-NEXT: G_LOAD
+# CHECK-NOT: Lower
----------------
Could you include the full instruction (with operands and types) in this check?
https://reviews.llvm.org/D53679
More information about the llvm-commits
mailing list