[PATCH] D19178: Broaden FoldItoFPtoI to try and establish whether the integer value fits into the float type

Carlos Liam via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 15 14:01:01 PDT 2016


aarzee added a comment.

In http://reviews.llvm.org/D19178#402921, @deadalnix wrote:

> Wouldn't it make sense to add a test also for case where it is NOT safe to fold ?


That should be covered by the existing tests, namely 13, 17, and 19.


http://reviews.llvm.org/D19178





More information about the llvm-commits mailing list