[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