Bug 315463 (MONO61445) - [PATCH] use test eax,eax for cmp eax, 0
Summary: [PATCH] use test eax,eax for cmp eax, 0
Status: RESOLVED FIXED
Alias: MONO61445
Product: Mono: Runtime
Classification: Mono
Component: misc (show other bugs)
Version: unspecified
Hardware: Other Other
: P3 - Medium : Normal
Target Milestone: ---
Assignee: Mono Bugs
QA Contact: Mono Bugs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-07-12 21:12 UTC by Ben Maurer
Modified: 2007-09-15 21:24 UTC (History)
1 user (show)

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---

Attachments
mini-test-eax-eax.patch (1.50 KB, patch)
2004-07-12 21:13 UTC, Thomas Wiest 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Wiest 2007-09-15 18:46:37 UTC 


---- Reported by bmaurer@users.sf.net 2004-07-12 14:12:53 MST ----

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:10 MST ----

Created an attachment (id=166445)
mini-test-eax-eax.patch




---- Additional Comments From miguel@ximian.com 2004-07-17 01:12:39 MST ----

Looks ok to me to go in.

CCing Zoltan to get permission to commit.



---- Additional Comments From bmaurer@users.sf.net 2004-08-14 18:46:26 MST ----

Done

Imported an attachment (id=166445)

Unknown operating system unknown. Setting to default OS "Other".