[Mono-bugs] [Bug 61445][Nor] Changed - [PATCH] use test eax,eax for cmp eax, 0
bugzilla-daemon@bugzilla.ximian.com
bugzilla-daemon@bugzilla.ximian.com
Sat, 17 Jul 2004 01:12:39 -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 miguel@ximian.com.
http://bugzilla.ximian.com/show_bug.cgi?id=61445
--- shadow/61445 2004-07-12 14:13:10.000000000 -0400
+++ shadow/61445.tmp.6530 2004-07-17 01:12:39.000000000 -0400
@@ -1,26 +1,30 @@
Bug#: 61445
Product: Mono: Runtime
Version: unspecified
-OS:
+OS: unknown
OS Details:
Status: NEW
Resolution:
-Severity:
-Priority: Wishlist
+Severity: Unknown
+Priority: Normal
Component: misc
AssignedTo: mono-bugs@ximian.com
ReportedBy: bmaurer@users.sf.net
QAContact: mono-bugs@ximian.com
TargetMilestone: ---
URL:
-Cc:
Summary: [PATCH] use test eax,eax for cmp eax, 0
Right now, we only use test eax,eax if it is an equality comparison.
However, test eax, eax is *EXACTLY* the same as cmp eax, 0, for all cases.
This patch uses it in all cases (as it is shorter).
------- Additional Comments From bmaurer@users.sf.net 2004-07-12 14:13 -------
Created an attachment (id=8533)
mini-test-eax-eax.patch
+
+------- Additional Comments From miguel@ximian.com 2004-07-17 01:12 -------
+Looks ok to me to go in.
+
+CCing Zoltan to get permission to commit.