[PATCH] D24512: AMDGPU: Fix target options fp32/64-denormals

Tom Stellard via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 13 09:58:06 PDT 2016


tstellarAMD accepted this revision.
tstellarAMD added a comment.
This revision is now accepted and ready to land.

Once small comment otherwise. LGTM.


================
Comment at: lib/Basic/Targets.cpp:1962
@@ -1961,3 +1961,3 @@
   bool hasLDEXPF:1;
-  bool hasDenormSupport:1;
+  bool hasFP32DenormSupport:1;
 
----------------
I think this name is a little confusing, because the hardware does support  fp32 denorms.  I would change this to something like hasFullSpeedFP32Denorms.


https://reviews.llvm.org/D24512





More information about the cfe-commits mailing list