[PATCH] D50582: ValueTracking: Handle more instructions in isKnownNeverNaN

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 19 10:46:19 PDT 2018


spatel added inline comments.


================
Comment at: lib/Analysis/ValueTracking.cpp:2921-2923
   // TODO: Handle instructions and potentially recurse like other 'isKnown'
   // functions. For example, the result of sitofp is never NaN.
 
----------------
spatel wrote:
> Remove stale comment.
Comment is still here. Also, we should have tests for sitofp/uitofp.


https://reviews.llvm.org/D50582





More information about the llvm-commits mailing list