[PATCH] D67105: [TargetLowering] Fix another potential FPE in expandFP_TO_UINT

Ulrich Weigand via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 9 05:55:02 PDT 2019


uweigand marked 2 inline comments as done.
uweigand added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/TargetLowering.cpp:5732
+    // IntOfs = select Sel, 0, 0x8000000000000000
+    // Result = fp_to_sint(Val - FltOfs) ^ IntOfs
 
----------------
xbolva00 wrote:
> Val is Src? New comment is confusing
Right, forgot to update this.  Should be fixed now, thanks!


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67105/new/

https://reviews.llvm.org/D67105





More information about the llvm-commits mailing list