Changeset: 6b3969c84089 for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=6b3969c84089
Modified Files:
        monetdb5/mal/mal_instruction.c
Branch: xid
Log Message:

Enlarge the search space for instruction localisation
The heuristic to search for an embedded instruction when its argument list
increases, should not be a constant, but related to the number of arguments.
The heuristic allows for 2 instructions being added for each argument.


diffs (15 lines):

diff --git a/monetdb5/mal/mal_instruction.c b/monetdb5/mal/mal_instruction.c
--- a/monetdb5/mal/mal_instruction.c
+++ b/monetdb5/mal/mal_instruction.c
@@ -1513,8 +1513,9 @@ pushArgument(MalBlkPtr mb, InstrPtr p, i
                 * the stmtblock after pushing the arguments. In sql_gencode
                 * this alone would be >100 places and in the optimizers >
                 * 30. In almost all cases the instructions have few
-                * parameters. */
-               pclimit = mb->stop - 8;
+                * parameters. 
+                * Search at least beyond the size of the argument list */
+               pclimit = mb->stop - 2 * p->maxarg;
                pclimit = pclimit < 0 ? 0 : pclimit;
                for (pc = mb->stop - 1; pc >= pclimit; pc--)
                        if (mb->stmt[pc] == p) {
_______________________________________________
Checkin-list mailing list
[email protected]
http://mail.monetdb.org/mailman/listinfo/checkin-list

Reply via email to