[PATCH] D79162: [Analysis] TTI: Add CastContextHint for getCastInstrCost

[Sam Parker via Phabricator via llvm-commits]

samparker added a comment.

> What do you mean by "doing it at a higher level"?

I mean that trying to attach extra bits of information to //individual// instruction costs hooks doesn't scale. Also, if we want to add in-depth information, such as the legalization object, then we'd probably have to create a whole new API for the vectorizer to use too. Instead, we could enable TTI to calculate the cost of the loop, not just each instruction. This would give us the freedom to evaluate all the memory operations, evaluating their extends/truncs together, and enable us to make a good decision.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79162/new/

https://reviews.llvm.org/D79162