[PATCH] D15816: [TTI] Add hook for contextual cast estimates

Matthew Simpson via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 19 10:37:11 PST 2016


mssimpso added a comment.

Hal,

Thanks very much for the suggestions, and I apologize for the delayed response. I agree the initial approach is unsatisfying. There is definitely a balance to be struck here between providing context to the cost model and having to unnecessarily re-encode information. The question of how much context to provide is also problematic. Although not a high priority for me at the moment, I do plan to evaluate your idea soon. So thanks again for the feedback!


http://reviews.llvm.org/D15816





More information about the llvm-commits mailing list