[Mono-bugs] [Bug 59006][Wis] Changed - [PATCH] [perf] stelem.ref perf

bugzilla-daemon@bugzilla.ximian.com bugzilla-daemon@bugzilla.ximian.com
Tue, 29 Jun 2004 20:02:42 -0400 (EDT)


Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by bmaurer@users.sf.net.

http://bugzilla.ximian.com/show_bug.cgi?id=59006

--- shadow/59006	2004-06-29 20:01:23.000000000 -0400
+++ shadow/59006.tmp.7765	2004-06-29 20:02:42.000000000 -0400
@@ -1,23 +1,23 @@
 Bug#: 59006
 Product: Mono: Runtime
 Version: unspecified
-OS: 
+OS: unknown
 OS Details: 
 Status: NEW   
 Resolution: 
-Severity: 
+Severity: Unknown
 Priority: Wishlist
 Component: misc
 AssignedTo: mono-bugs@ximian.com                            
 ReportedBy: bmaurer@users.sf.net               
 QAContact: mono-bugs@ximian.com
 TargetMilestone: ---
 URL: 
 Cc: 
-Summary: [perf] stelem.ref perf
+Summary: [PATCH] [perf] stelem.ref perf
 
 Our performance on the following test case is 7x worse than MS.NET
 
 using System;
 
 class T {
@@ -44,6 +44,15 @@
 the LMF stuff, I was only 2x slower.
 
 ------- Additional Comments From bmaurer@users.sf.net  2004-06-29 20:01 -------
 Created an attachment (id=8394)
 a faster version
 
+
+------- Additional Comments From bmaurer@users.sf.net  2004-06-29 20:02 -------
+This patch implements a fast case for the stelem.ref opcode in managed
+code. If the quick test fails, the code falls back on to managed code.
+
+This gave me a nice 3% boost in MCS.
+
+I was able to do a full bootstrap with this patch. I also added some
+regression tests (in the patch) which passed.