[m-rev.] for review: faster merging of varsets

Zoltan Somogyi zoltan.somogyi at runbox.com
Sun Jan 13 13:32:25 AEDT 2019


For review by anyone.

The diff implements the proposed faster algorithm just once,
even though it is applicable to four predicates. I plan to address
any review comments on that one predicate, and *then* copy
the approach to the other three.

If someone would undertake to run tuning benchmarks for me,
I would write the required benchmarking code. (The compiler
at the moment goes out of its way to avoid calling these predicates
with argument sizes where the old algorithm is too slow,
so I don't think it would be a good benchmark for this.
The main objective of this diff is the avoidance of this need.)

Zoltan.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Log.varset
Type: application/octet-stream
Size: 1817 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20190113/bd2bc693/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: DIFF.varset
Type: application/octet-stream
Size: 17924 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20190113/bd2bc693/attachment-0001.obj>


More information about the reviews mailing list