[PATCH] D44464: allow custom OptBisect classes set to LLVMContext
    Fedor Sergeev via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Mar 30 04:00:19 PDT 2018
    
    
  
fedor.sergeev added inline comments.
================
Comment at: lib/IR/LLVMContextImpl.h:1362
   /// analysis.
   OptPassGate &getOptPassGate();
+
----------------
yrouban wrote:
> andrew.w.kaylor wrote:
> > I missed this in the last review. This function should be const.
> D44821 did not change this. LLVMContextImpl::getOptBisect() used to be non-const.
> 
I agree with Andrew, it is natural to have it const.
https://reviews.llvm.org/D44464
    
    
More information about the llvm-commits
mailing list