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