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