[PATCH] D86360: Add new hidden option -print-changed which only reports changes to IR

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 17 22:01:14 PDT 2020


MaskRay added inline comments.


================
Comment at: llvm/test/Other/change-printer.ll:6
+; Simple functionality check.
+; RUN: opt -S -print-changed -passes=instsimplify 2>&1 -o /dev/null < %s | FileCheck %s --check-prefix=CHECK_SIMPLE
+;
----------------
ychen wrote:
> For check prefixes, using dash is much more common than underscore.
I am fine with underscores because several directives use dashes and underscores are more distinguishable.


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

https://reviews.llvm.org/D86360



More information about the llvm-commits mailing list