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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
