[PATCH] D37471: [BasicTTIImpl] Remove Trunc and ZExt cost handling

[Hal Finkel via Phabricator via llvm-commits]

hfinkel added a comment.

> Since most transformations will be using TTI, which should then hold all the required information, just call the base getOperationCost without handling trunc and zext differently.

I don't understand what you mean. You're changing the default TTI implementation that most of the targets will use. Do you only want to call back when isTruncateFree/isZExtFree return false? I'd think that we'd want to continue handling the cases when they return true?


https://reviews.llvm.org/D37471