[PATCH] D78937: [CostModel] Use isExtLoad in BasicTTI

[Sam Parker via Phabricator via llvm-commits]

samparker added a comment.

> It is dangerous with how we have things set up at the moment for anything to look at the _type_ of the context instruction

But now we're checking that the queried type matches the type of context instruction too, so I'm not really what other conclusion to draw other than that the user asking: what is the cost of //this// instruction? Based on the currently limited API, what scenario do you think this would not generally be true?


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

https://reviews.llvm.org/D78937