This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[PATCH] Handle BUILT_IN_EH_* in estimate_num_insns
- From: Richard Guenther <rguenther at suse dot de>
- To: gcc-patches at gcc dot gnu dot org
- Cc: Jan Hubicka <jh at suse dot de>, rth at redhat dot com
- Date: Tue, 19 Jan 2010 16:46:13 +0100 (CET)
- Subject: [PATCH] Handle BUILT_IN_EH_* in estimate_num_insns
Currently they are accounted by the move costs of their argument
and result types plus the fixed call-cost. Certainly a bit too much
as they should expand to register operations. Thus, simply account
them as such - and register operations are free according to
estimate_num_insns.
Any better suggestions? Both BUILT_IN_EH_FILTER and
BUILT_IN_EH_COPY_VALUES are quite common in tramp3d IL.
Thanks,
Richard.
2010-01-19 Richard Guenther <rguenther@suse.de>
* tree-inline.c (estimate_num_insns): Handle EH builtins.
Index: gcc/tree-inline.c
===================================================================
*** gcc/tree-inline.c (revision 156035)
--- gcc/tree-inline.c (working copy)
*************** estimate_num_insns (gimple stmt, eni_wei
*** 3310,3315 ****
--- 3310,3321 ----
cost = weights->target_builtin_call_cost;
break;
+ /* Exception state returns or moves registers around. */
+ case BUILT_IN_EH_FILTER:
+ case BUILT_IN_EH_POINTER:
+ case BUILT_IN_EH_COPY_VALUES:
+ return 0;
+
default:
break;
}