[PATCH] D105072: [GISel] Support llvm.memcpy.inline

Jon Roelofs via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 2 08:42:59 PDT 2021


jroelofs added inline comments.


================
Comment at: llvm/test/CodeGen/AArch64/GlobalISel/legalizer-info-validation.mir:632
 # DEBUG-NEXT: .. imm index coverage check SKIPPED: user-defined predicate detected
+# DEBUG-NEXT: G_MEMCPY_INLINE (opcode 219): 3 type indices, 0 imm indices
+# DEBUG-NEXT: .. type index coverage check SKIPPED: no rules defined
----------------
dyung wrote:
> In the future, please use a regex to match any number instead of hard coding a number in here as it causes the test to fail if a downstream copy has private changes which affect which number is assigned to this opcode.
> 
> I've fixed this up in f737d9794a40c066c9ccffb9ac277c1b70442ede.
Sorry about that. Thank you!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105072



More information about the llvm-commits mailing list