[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