[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