Closed Bug 412614 Opened 17 years ago Closed 16 years ago

port tamarin vm to 64 bit windows and mac osx

Categories

(Tamarin Graveyard :: Virtual Machine, enhancement)

x86
All
enhancement
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: cpeyer, Assigned: rwinchel)

Details

Attachments

(1 file)

- port tamarin vm to 64 bit windows and mac
- verify vm compliance tests pass on 64 os
- when work is complete we can setup automated build and test
 
 
Transferred Comments:

Dan Schaffer - Thu Nov 15 09:33:11 CST 2007
open to Rob for flash player 10.
Assigned to me.
Status: NEW → ASSIGNED
Assignee: nobody → rwinchel
Attachment #298716 - Flags: review?
Attachment #298716 - Flags: review?(rreitmai)
Attachment #298716 - Flags: review?(edwsmith)
Comment on attachment 298716 [details] [diff] [review]
Proposed changes for Windows x64 support

Lots of (int) casting going on!

Besides here are a few more specific comments;

AmdAssembler64 Lines 664-674   - avmasert(0) still in there.  
 Lines 871-916    - verbose turned off all the time?
CodegenMIR.cpp
 Lines 7394-7400   - what happened to IA_32 case?
 Lines 8467-8481  - MOV() macro not surrounded by {}  might be dangerous.
Interpreter.cpp
 Lines 244-249  - motivation for moving decimalParam wasn't clear
Attachment #298716 - Flags: review?(rreitmai) → review+
Attachment #298716 - Flags: review?(wsharp)
Attachment #298716 - Flags: review?(wsharp) → review+
Attachment #298716 - Flags: review?(edwsmith) → review+
Attachment #298716 - Flags: review?
is this still relevant or should it be closed?
close it, imo
resolved, current redux is 64b clean.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
actually I don't think it's 100% clean on Mac, is it? (probably could be with little effort tho)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: