[PATCH] D78335: [GIsel][LegalizerHelper] Account for debug insts when creating mem libcalls

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 17 15:48:01 PDT 2020


vsk marked an inline comment as done.
vsk added inline comments.


================
Comment at: llvm/test/CodeGen/AArch64/GlobalISel/legalize-memcpy-et-al.mir:3
 # RUN: llc -march=aarch64 -run-pass=legalizer -verify-machineinstrs %s -o - | FileCheck %s
+# RUN: llc -debugify-and-strip-all-safe -march=aarch64 -run-pass=legalizer -verify-machineinstrs %s -o - | FileCheck %s
 ---
----------------
arsenm wrote:
> Does this end up stressing the case with multiple dbg instructions? I think tests with explicit dbg instructions would be more helpful here
This should be covered by the added test. PTAL, thanks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78335





More information about the llvm-commits mailing list