[PATCH] D24168: Refactor LICM pass in preparation for LoopSink pass.
    Chandler Carruth via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Oct  3 11:22:25 PDT 2016
    
    
  
chandlerc added inline comments.
> hfinkel wrote in LICM.cpp:520
> Why is this not returning a conservative answer?
This is a good question, it may just mean that "SafetyInfo" needs a better name or this needs a comment to explain why this is correct.
Alternatively, we could require that SafetyInfo is always provided.
https://reviews.llvm.org/D24168
    
    
More information about the llvm-commits
mailing list