[PATCH] D32085: AMDGPU: Do not lower fast unsafe div for safe, f32, with fp32 denormals

Konstantin Zhuravlyov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 14 10:52:28 PDT 2017


kzhuravl added inline comments.


================
Comment at: lib/Target/AMDGPU/SIISelLowering.cpp:3384-3385
 
+  if (!Unsafe && VT == MVT::f32 && Subtarget->hasFP32Denormals())
+    return SDValue();
+
----------------
arsenm wrote:
> How is this different from the check right below in the constant case? Is it just the multiply by reciprocal case for non-constant below that is the problem?
Multiply by the reciprocal for non-constant case is the problem.


https://reviews.llvm.org/D32085





More information about the llvm-commits mailing list