[PATCH] D130434: [MachineFunctionPass] Support -print-changed and -print-changed=quiet

Tue Jul 26 10:13:08 PDT 2022

MaskRay added inline comments.

================
Comment at: llvm/test/Other/print-changed-machine.ll:2
+; REQUIRES: aarch64-registered-target
+; RUN: llc -filetype=null -mtriple=aarch64 -O0 -print-changed %s 2>&1 | FileCheck %s --check-prefixes=VERBOSE,VERBOSE-BAR
+; RUN: llc -filetype=null -mtriple=aarch64 -O0 -print-changed -filter-print-funcs=foo %s 2>&1 | FileCheck %s --check-prefixes=VERBOSE,NO-BAR
----------------
aeubanks wrote:
> I'm worried about how brittle this test is
> 
> making the input MIR would probably help a little
> 
> I'm also not sure which target is the simplest in terms backend passes
This should be quite stable. aarch64 -O0 defaults to global isel and -O* will switch too. The IRTranslator/Legalizer lines will likely stay unchanged.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130434