648865 - Remove WINCE code from js/src/assember/*

Status: VERIFIED FIXED

(Core :: JavaScript Engine, defect)

Description

[Ed Morley [:emorley]]

Bug 647389 removed WINCE/WINMO code from most of spidermonkey. However, it was requested that some parts of the patch be broken out into separate bugs, since they covered code from upstream sources.

From bug 647389 comment 37:

> >   - js/src/assembler is shared or stolen code too, but I don't know the
> >     policy there. Maybe we've forked it. Paul, ask dmandelin or dvander?
> 
> File a follow-on, submit the subset of the patch in js/src/assembler/ and ask
> cdleary for a review (This code originally landed in cdleary's Yarr patch, and
> it's unclear if this code came from upstream, in which case we won't delete
> it).

Comment 1

[Ed Morley [:emorley]]

Attachment: Patch v1 (based on TM rev 4ace629bb676)

Comment 2

[Chris Leary [:cdleary] (not checking bugmail)]

Comment on attachment 524953 [details] [diff] [review]
Patch v1 (based on TM rev 4ace629bb676)

I'm passing review on this over to dmandelin -- I think we want to leave it in to make upstream merge comparisons more trivial, but getting a second opinion sounds like a good idea. :-)

Comment 3

[Ed Morley [:emorley]]

(In reply to comment #2)
> ...I think we want to leave it in
> to make upstream merge comparisons more trivial, but getting a second opinion
> sounds like a good idea. :-)

Cool yeah WONTFIX would make more sense if it's still being synced with upstream; thought it worth asking either way, given that the patch had already been created for bug 647389 before I realised :-)

Comment 4

[David Mandelin [:dmandelin]]

Comment on attachment 524953 [details] [diff] [review]
Patch v1 (based on TM rev 4ace629bb676)

I hear WINCE is dead, so let's take it. It looks very unlikely to cause merge conflicts.

Comment 5

[Ed Morley [:emorley]]

Attachment: Patch v1 (updated to TM tip)

Updated to TM tip.

Comment 6

[Ed Morley [:emorley]]

(In reply to comment #4)
> I hear WINCE is dead, so let's take it. It looks very unlikely to cause merge
> conflicts.

Great, thanks for the review :-)

Comment 7

[Ed Morley [:emorley]]

http://hg.mozilla.org/tracemonkey/rev/3246e3124fe7

Thanks Boris :-)

Comment 8

[Chris Leary [:cdleary] (not checking bugmail)]

http://hg.mozilla.org/mozilla-central/rev/3246e3124fe7

Comment 9

[Ed Morley [:emorley]]

Thanks Chris :-)

Comment 10

[Ed Morley [:emorley]]

http://mxr.mozilla.org/mozilla-central/search?string=wince&find=/js/src/assember/
...returns 0 results.

-> Verified.