[PATCH] [ConstantFolding] Fix wrong folding of intrinsic 'convert.from.fp16'.
Ahmed Bougacha
ahmed.bougacha at gmail.com
Thu May 14 10:47:41 PDT 2015
Better indeed, go ahead!
-Ahmed
================
Comment at: lib/Analysis/ConstantFolding.cpp:1549
@@ -1546,3 +1548,3 @@
APFloat::opStatus status =
- Val.convert(APFloat::IEEEsingle, APFloat::rmNearestTiesToEven, &lost);
+ Val.convert(FPS, APFloat::rmNearestTiesToEven, &lost);
----------------
Nit: I'd put the Ty->getFltSemantics() here instead of in the variable, but I'm fine with both ways.
http://reviews.llvm.org/D9771
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list