[PATCH] D40692: AMDGPU/EG: Add a new FeatureFMA and use it to selectively enable FMA instruction

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 1 08:39:59 PST 2017


arsenm accepted this revision.
arsenm added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: lib/Target/AMDGPU/R600ISelLowering.cpp:214-217
+  if (!Subtarget->hasFMA()) {
+    setOperationAction(ISD::FMA, MVT::f32, Expand);
+    setOperationAction(ISD::FMA, MVT::f64, Expand);
+  }
----------------
There's not much point in doing this since it will just turn into an unsupported libcall


Repository:
  rL LLVM

https://reviews.llvm.org/D40692





More information about the llvm-commits mailing list