[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