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

Yaxun Liu via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 13 10:02:26 PDT 2016


yaxunl added inline comments.

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


https://reviews.llvm.org/D24512





More information about the cfe-commits mailing list