[PATCH] Value soft float calls as more expensive in the inliner.

Eric Christopher echristo at gmail.com
Tue Feb 3 14:49:31 PST 2015


Can probably expand the testcase a bit more with CHECK/CHECK-NOT for the various calls? One inline comment.

Otherwise LGTM.

-eric


================
Comment at: lib/Analysis/IPA/InlineCost.cpp:911
@@ +910,3 @@
+    // If the instruction is floating point, and the target says this operation is
+    // expensive or the function has the "use-soft-float" attribute, this will
+    // eventually become a library call.  Treat the cost as such.
----------------
s/will/may.

http://reviews.llvm.org/D6936

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list