[PATCH] D49934: AMDGPU: Allow fp32-denormals feaure for r600 targets

Jan Vesely via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 1 08:05:15 PDT 2018


This revision was automatically updated to reflect the committed changes.
Closed by commit rL338569: AMDGPU: Allow fp32-denormals feature for r600 targets (authored by jvesely, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D49934?vs=157756&id=158543#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D49934

Files:
  llvm/trunk/lib/Target/AMDGPU/AMDGPU.td
  llvm/trunk/lib/Target/AMDGPU/AMDGPUFeatures.td


Index: llvm/trunk/lib/Target/AMDGPU/AMDGPUFeatures.td
===================================================================
--- llvm/trunk/lib/Target/AMDGPU/AMDGPUFeatures.td
+++ llvm/trunk/lib/Target/AMDGPU/AMDGPUFeatures.td
@@ -19,6 +19,15 @@
   "Enable single precision FMA (not as fast as mul+add, but fused)"
 >;
 
+// Some instructions do not support denormals despite this flag. Using
+// fp32 denormals also causes instructions to run at the double
+// precision rate for the device.
+def FeatureFP32Denormals : SubtargetFeature<"fp32-denormals",
+  "FP32Denormals",
+  "true",
+  "Enable single precision denormal handling"
+>;
+
 class SubtargetFeatureLocalMemorySize <int Value> : SubtargetFeature<
   "localmemorysize"#Value,
   "LocalMemorySize",
Index: llvm/trunk/lib/Target/AMDGPU/AMDGPU.td
===================================================================
--- llvm/trunk/lib/Target/AMDGPU/AMDGPU.td
+++ llvm/trunk/lib/Target/AMDGPU/AMDGPU.td
@@ -267,15 +267,6 @@
 // Subtarget Features (options and debugging)
 //===------------------------------------------------------------===//
 
-// Some instructions do not support denormals despite this flag. Using
-// fp32 denormals also causes instructions to run at the double
-// precision rate for the device.
-def FeatureFP32Denormals : SubtargetFeature<"fp32-denormals",
-  "FP32Denormals",
-  "true",
-  "Enable single precision denormal handling"
->;
-
 // Denormal handling for fp64 and fp16 is controlled by the same
 // config register when fp16 supported.
 // TODO: Do we need a separate f16 setting when not legal?


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D49934.158543.patch
Type: text/x-patch
Size: 1586 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180801/2a6743eb/attachment.bin>


More information about the llvm-commits mailing list