[PATCH] D35608: Migrate SimplifyLibCalls to new OptimizationRemarkEmitter

Adam Nemet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 25 22:31:10 PDT 2017


anemet accepted this revision.
anemet added a comment.

Really minor nit below.  No need to upload the patch after this change, just go ahead and land it.



================
Comment at: test/Transforms/Util/libcalls-opt-remarks.ll:4-5
+; RUN: cat %t | FileCheck -check-prefix=YAML %s
+; RUN: opt < %s -passes='require<opt-remark-emit>,instcombine' -o /dev/null -S -pass-remarks=instcombine \
+; RUN:     2>&1 | FileCheck %s
+
----------------
Assuming the old PM stuff will be removed at some point, it would be good to test the YAML output in this case too.


https://reviews.llvm.org/D35608





More information about the llvm-commits mailing list