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

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/TargetLowering.cpp:5715
+  else {
+    // Likewise if we prove at compile time that the argument is in range.
+    Cst = DAG.getConstantFP(APF, dl, SrcVT);
----------------
I'm not sure what you're trying to do here.  If the input value is a constant, we should just constant-fold the operation.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D67105