[Mono-bugs] [Bug 60673][Wis] Changed - [PATCH] -O=loop sometimes adds padding in wrong places

bugzilla-daemon@bugzilla.ximian.com bugzilla-daemon@bugzilla.ximian.com
Tue, 10 Aug 2004 13:40:52 -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 lupus@ximian.com.


--- shadow/60673	2004-07-08 15:41:48.000000000 -0400
+++ shadow/60673.tmp.25658	2004-08-10 13:40:52.000000000 -0400
@@ -1,13 +1,13 @@
 Bug#: 60673
 Product: Mono: Runtime
 Version: unspecified
 OS: unknown
 OS Details: 
-Status: NEW   
+Status: RESOLVED   
+Resolution: FIXED
 Severity: Unknown
 Priority: Wishlist
 Component: misc
 AssignedTo: mono-bugs@ximian.com                            
 ReportedBy: bmaurer@users.sf.net               
 QAContact: mono-bugs@ximian.com
@@ -92,6 +92,11 @@
 patch i have on my tree. However, you can see the improvement in first
 case test case with just my patch.
 ------- Additional Comments From vargaz@freemail.hu  2004-07-08 15:41 -------
 This patch looks ok to me. Especially since 'loop' is not in the
 default optimization set.
+------- Additional Comments From lupus@ximian.com  2004-08-10 13:40 -------
+The patch is incorrect, since it aligns the wrong basic block (and
+slowed down 50% simple loops).
+I committed a fix to deal with fall-through checks in the condition.