[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