This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: ivopts cost oddities


Bernd Schmidt wrote:
> Bootstrapped, regression tested on i686-linux; committed as 119014.

And here's the patch.


Bernd

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 119013)
+++ ChangeLog	(working copy)
@@ -28,6 +28,9 @@
 	* config/bfin/bfin.c (bfin_function_ok_for_sibcall): Handle some
 	edge cases with local functions and TARGET_ID_SHARED_LIBRARY.
 
+	* tree-ssa-loop-ivopts.c (get_address_cost): Do not artificially
+	inflate costs for addresses with an out-of-bounds address.
+
 2006-11-19  Andrew Pinski  <pinskia@gmail.com>
 
 	PR rtl-opt/29879
Index: tree-ssa-loop-ivopts.c
===================================================================
--- tree-ssa-loop-ivopts.c	(revision 119010)
+++ tree-ssa-loop-ivopts.c	(working copy)
@@ -3561,10 +3561,7 @@ get_address_cost (bool symbol_present, b
     cost += multiply_by_cost (ratio, Pmode);
 
   if (s_offset && !offset_p && !symbol_present)
-    {
-      cost += add_cost (Pmode);
-      var_present = true;
-    }
+    cost += add_cost (Pmode);
 
   acost = costs[mem_mode][symbol_present][var_present][offset_p][ratio_p];
   return cost + acost;

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]