Bug 315556 (MONO61797) - [PATCH] avoid two comparisons for bool b = x != y
Summary: [PATCH] avoid two comparisons for bool b = x != y
Status: RESOLVED FIXED
Alias: MONO61797
Product: Mono: Runtime
Classification: Mono
Component: misc (show other bugs)
Version: unspecified
Hardware: Other Other
: P3 - Medium : Enhancement
Target Milestone: ---
Assignee: Mono Bugs
QA Contact: Mono Bugs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-07-21 23:10 UTC by Ben Maurer
Modified: 2007-09-15 21:24 UTC (History)
0 users

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

Attachments
mini-cne.patch (3.90 KB, patch)
2004-07-21 23:10 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:47:39 UTC 


---- Reported by bmaurer@users.sf.net 2004-07-21 16:10:21 MST ----

there is no cli opcode to do `compare not equal'. As a result, mcs emits:

..
ceq
ldc.i4.0
ceq

This patch uses only one comparison for this operation.

A similar thing could be done with the other ops, however they seem to be
so infrequent to make this patch not worth it.



---- Additional Comments From bmaurer@users.sf.net 2004-07-21 16:10:37 MST ----

Created an attachment (id=166504)
mini-cne.patch




---- Additional Comments From bmaurer@users.sf.net 2004-07-24 01:54:20 MST ----

*** https://bugzilla.novell.com/show_bug.cgi?id=MONO60697 has been marked as a duplicate of this bug. ***



---- Additional Comments From lupus@ximian.com 2004-07-28 12:15:23 MST ----

Please commit, but first add the proper tests to mini/basic.cs.



---- Additional Comments From bmaurer@users.sf.net 2004-07-28 16:30:51 MST ----

its in

Imported an attachment (id=166504)

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