[PATCH] D25722: Improved cost model for FDIV and FSQRT

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 29 04:37:00 PDT 2016


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

LGTM with one minor



================
Comment at: lib/Target/X86/X86TargetTransformInfo.cpp:1227
+    { ISD::FSQRT, MVT::f32,   18 }, // Nehalem from http://www.agner.org/
+    { ISD::FSQRT, MVT::v4f32, 18 }, // Nehalem from http://www.agner.org/
   };
----------------
Better to use the Pentium III costs F32 = 28, VF432 = 56


https://reviews.llvm.org/D25722





More information about the llvm-commits mailing list