arsenm accepted this revision.
arsenm added a comment.
This revision is now accepted and ready to land.
LGTM with f32 clarification
================
Comment at: lib/Basic/Targets.cpp:2114
+ static bool hasFullSpeedFMA(StringRef GPUName) {
+ return parseAMDGCNName(GPUName) >= GK_GFX9;
----------------
FMAF32?
================
Comment at: lib/Basic/Targets.cpp:2115
+ static bool hasFullSpeedFMA(StringRef GPUName) {
+ return parseAMDGCNName(GPUName) >= GK_GFX9;
+ }
----------------
We should probably add a new subtarget feature for this, but that's a separate
patch
https://reviews.llvm.org/D31482
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
