[PATCH] D18576: Initial implementation of optimization bisect
    Mehdi AMINI via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Mar 29 18:08:45 PDT 2016
    
    
  
joker.eph added inline comments.
================
Comment at: lib/IR/OptBisect.cpp:118
@@ +117,3 @@
+    "Lower 'expect' Intrinsics"
+};
+
----------------
Ew, that's pretty terrible to have this inlined here. But I agree it is not easy to decouple...
Maybe having a new (optional?) API on the Passes themselves may be cleaner.
Something like `bool isRequiredForCorrectness()`?
Repository:
  rL LLVM
http://reviews.llvm.org/D18576
    
    
More information about the llvm-commits
mailing list