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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to