On 06/08/2016 11:31 AM, Jan Hubicka wrote:
I think 20 insns to copy for loop header is way too much. The constant came
from jump.c that was operating with quite different IL and compiler.
This patch adds --param for it so we can fine tune it for new millenia.

+@item max-loop-headers-insns
+The maximum number of insns in loop header duplicated by copy loop headers 
pass.
+

"the copy loop headers pass", here and in params.def.

-      int limit = 20;
+      int ninsns = PARAM_VALUE (PARAM_MAX_LOOP_HEADER_INSNS);
+      int limit = ninsns;

The naming is somewhat unfortunate, I think limit should be the initial limit, and something like remaining_limit should be the name of the one that counts down.

Otherwise ok.


Bernd

Reply via email to