[PATCH] D21739: [TTI] Add functions determining if int parameters/returns should be zeroext/signext.

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 28 18:53:31 PDT 2016


hfinkel added a comment.

In http://reviews.llvm.org/D21739#469090, @davidxl wrote:

> Looks reasonable to me. Added Hal for a second look.


If a given target is not compiled in, then the defaults TTI provides might be wrong for the target. I think you need to add some what for the transformation to detect whether or not it is getting an accurate answer so it can abort the transformation in that case. We shouldn't miscompile IR just because we don't have a particular target available.


Repository:
  rL LLVM

http://reviews.llvm.org/D21739





More information about the llvm-commits mailing list