Re: [Mono-dev] [PATCH] A fastpath dead code elimination

2005-11-17 Thread Zoltan Varga
Hi, The general JIT changes in the patch look harmless to me. There is a problem tough: the patch makes tests/marhal2.exe crash when run with -O=all. Zoltan On 11/15/05, Massimiliano Mantione [EMAIL PROTECTED] wrote: The

Re: [Mono-dev] [PATCH] A fastpath dead code elimination

2005-11-17 Thread Massimiliano Mantione
On Thu, 2005-11-17 at 13:23 +0100, Zoltan Varga wrote: The general JIT changes in the patch look harmless to me. There is a problem tough: the patch makes tests/marhal2.exe crash when run with -O=all. That looks more SSAPRE than the patch (or, better, it happens if you specify -O=ssapre and

Re: [Mono-dev] [PATCH] A fastpath dead code elimination

2005-11-15 Thread Miguel de Icaza
Hello, The alias analysis pass has O(n) complexity (n = code size), it is just a linear sweep on the list of instructions. Then, deadce operates one BB at a time, scanning the code linearly and using the liveness bits as start/end conditions (so it is O(n) as well). Is there is a threshold

Re: [Mono-dev] [PATCH] A fastpath dead code elimination

2005-11-15 Thread Massimiliano Mantione
On Tue, 2005-11-15 at 15:14 -0500, Miguel de Icaza wrote: Hello, The alias analysis pass has O(n) complexity (n = code size), it is just a linear sweep on the list of instructions. Then, deadce operates one BB at a time, scanning the code linearly and using the liveness bits as

Re: [Mono-dev] [PATCH] A fastpath dead code elimination

2005-11-15 Thread Ben Maurer
On Tue, 2005-11-15 at 15:14 -0500, Miguel de Icaza wrote: Hello, The alias analysis pass has O(n) complexity (n = code size), it is just a linear sweep on the list of instructions. Then, deadce operates one BB at a time, scanning the code linearly and using the liveness bits as