[Mono-devel-list] [PATCH] isinst/castclass optimizations

[Paolo Molaro]

On 03/12/04 Ben Maurer wrote:
> Ok, apparently 1 is causing regressions, so i have extracted 2 out and
> attached it below. I will fix part 1 and send it later.

Good, but keep in mind that a patch done that way won't be accepted.
You duplicate lots of code and it's not needed: bloat and complexity
are better kept out of the jit.

> This patch simply special cases sealed types, such as blah is string.

What are the results of running the corlib test suite, the test suite in
mono/tests and the regression tests in mini/ before and after the patch?
Also: when a change can be cleanly done without moving code around, it's
better done that way: we are not in a contest to produce the larger
diffs:-)
There is another issue, too: the check can be done in a faster way
so why optimize it only a bit? While we are at it we should go the full
way. I'll let you think about the better way to optimize it (it's
simple, really) to not spoil you the excitement of finding it
(but, of course, you can ask and I'll tell you).

lupus

-- 
-----------------------------------------------------------------
lupus at debian.org                                     debian/rules
lupus at ximian.com                             Monkeys do it better