[Mono-bugs] [Bug 53012][Wis] Changed - [PATCH] Optimize loops

[bugzilla-daemon@bugzilla.ximian.com]

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=53012

--- shadow/53012	2004-03-24 15:07:11.000000000 -0500
+++ shadow/53012.tmp.31975	2004-03-24 15:08:34.000000000 -0500
@@ -1,23 +1,23 @@
 Bug#: 53012
 Product: Mono: Compilers
 Version: unspecified
-OS: 
+OS: unknown
 OS Details: 
 Status: NEW   
 Resolution: 
-Severity: 
+Severity: Unknown
 Priority: Wishlist
 Component: C#
-AssignedTo: mono-bugs@ximian.com                            
+AssignedTo: bmaurer@users.sf.net                            
 ReportedBy: bmaurer@users.sf.net               
 QAContact: mono-bugs@ximian.com
 TargetMilestone: ---
 URL: 
 Cc: 
-Summary: Optimize loops
+Summary: [PATCH] Optimize loops
 
 Ok, a few items for discuession (thanks miguel, for getting me thinking
 abou this).
 
 class Test {
 	static void Main ()
@@ -81,6 +81,27 @@
 Again, we can replace brfalse with brtrue, and remove the unconditional branch.
 
 ------- Additional Comments From bmaurer@users.sf.net  2004-03-24 15:07 -------
 Created an attachment (id=7084)
 patch
 
+
+------- Additional Comments From bmaurer@users.sf.net  2004-03-24 15:08 -------
+This patch transforms an expression in the form
+
+if (foo)
+     goto bar;
+
+into
+
+ldfoo
+brtrue bar
+
+rather than
+
+ldfoo
+brfalse after
+br bar
+after:
+
+I got no regressions on the test suite, and ran peverify on the
+bootstrapped mcs.