This revision was automatically updated to reflect the committed changes.
Closed by commit rL281357: AMDGPU: Fix target options fp32/64-denormals
(authored by yaxunl).
Changed prior to commit:
https://reviews.llvm.org/D24512?vs=71183&id=71201#toc
Repository:
rL LLVM
https://reviews.llvm.org
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 su
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 hasFP32Deno