[PATCH] D31482: [AMDGPU][GFX9] Set +fp32-denormals for >=gfx900 unless -cl-denorms-are-zero is set

Konstantin Zhuravlyov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 13 22:49:48 PDT 2017


kzhuravl added inline comments.


================
Comment at: lib/Basic/Targets.cpp:2114
 
+  static bool hasFullSpeedFMA(StringRef GPUName) {
+    return parseAMDGCNName(GPUName) >= GK_GFX9;
----------------
arsenm wrote:
> arsenm wrote:
> > FMAF32?
> Actually this also needs to specify full speed FMA with denorms. Full rate FMA is already a subtarget feature when denorms are disabled
I will do it in a separate patch. Thanks.


Repository:
  rL LLVM

https://reviews.llvm.org/D31482





More information about the cfe-commits mailing list