[PATCH] D89485: [MachineOutliner] Do not outline debug instructions

[Jessica Paquette via Phabricator via llvm-commits]

paquette added a comment.

This seems correct to me. Outlined functions should not contain debug instructions.

Only thing is that it'd be a lot nicer to have a MIR test here. That would make it much clearer what the testcase is actually doing.



================
Comment at: llvm/test/CodeGen/ARM/machine-outliner-remove-debug-instr.ll:1
+; RUN: llc --verify-machineinstrs --stop-after=machine-outliner %s -o - | FileCheck %s
+target datalayout = "e-m:e-p:32:32-Fi8-i64:64-v128:64:128-a:0:32-n32-S64"
----------------
Can this be a MIR test instead?


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

https://reviews.llvm.org/D89485