[PATCH] D49650: Targets/AMDGPU: Don't set fp32-denormals feature for r600

[Jan Vesely via Phabricator via cfe-commits]

jvesely created this revision.
jvesely added reviewers: arsenm, tstellar.
Herald added subscribers: t-tye, tpr, dstuttard, yaxunl, nhaehnle, wdng, 
kzhuravl.

This feature was removed in r335942


Repository:
  rC Clang

https://reviews.llvm.org/D49650

Files:
  lib/Basic/Targets/AMDGPU.cpp


Index: lib/Basic/Targets/AMDGPU.cpp
===================================================================
--- lib/Basic/Targets/AMDGPU.cpp
+++ lib/Basic/Targets/AMDGPU.cpp
@@ -205,7 +205,7 @@
     if (I == "+fp64-fp16-denormals" || I == "-fp64-fp16-denormals")
       hasFP64Denormals = true;
   }
-  if (!hasFP32Denormals)
+  if (!hasFP32Denormals && isAMDGCN(getTriple()))
     TargetOpts.Features.push_back(
         (Twine(CGOptsGPU.HasFastFMAF && !CGOpts.FlushDenorm
                    ? '+'



Index: lib/Basic/Targets/AMDGPU.cpp
===================================================================
--- lib/Basic/Targets/AMDGPU.cpp
+++ lib/Basic/Targets/AMDGPU.cpp
@@ -205,7 +205,7 @@
     if (I == "+fp64-fp16-denormals" || I == "-fp64-fp16-denormals")
       hasFP64Denormals = true;
   }
-  if (!hasFP32Denormals)
+  if (!hasFP32Denormals && isAMDGCN(getTriple()))
     TargetOpts.Features.push_back(
         (Twine(CGOptsGPU.HasFastFMAF && !CGOpts.FlushDenorm
                    ? '+'

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