If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

bustage Win64 build due to UnsetVMFrameRegs

RESOLVED FIXED

Status

()

Core
JavaScript Engine
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: m_kato, Assigned: m_kato)

Tracking

(Blocks: 1 bug)

Trunk
x86_64
Windows Vista
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
link -NOLOGO -OUT:js.exe -PDB:js.pdb  -NXCOMPAT -DYNAMICBASE  -DEBUG -DEBUGTYPE:
CV   js.obj jsworkers.obj  c:/Workspace/hg.mozilla.org/objdir-tm-win64-dbg/dist/
lib/nspr4.lib c:/Workspace/hg.mozilla.org/objdir-tm-win64-dbg/dist/lib/plc4.lib
c:/Workspace/hg.mozilla.org/objdir-tm-win64-dbg/dist/lib/plds4.lib @../js_static
.lib.fake kernel32.lib user32.lib gdi32.lib winmm.lib wsock32.lib advapi32.lib
LINK : js.exe not found or not built by the last incremental link; performing fu
ll link
   Creating library js.lib and object js.exp
TrampolineMasmX64.obj : error LNK2019: unresolved external symbol UnsetVMFrameRe
gs referenced in function JaegerTrampoline
(Assignee)

Comment 1

7 years ago
Created attachment 474625 [details] [diff] [review]
fix
(Assignee)

Updated

7 years ago
Attachment #474625 - Flags: review?(dvander)
Comment on attachment 474625 [details] [diff] [review]
fix

>+        /* we need pad the stack */

Nit: Reads better as "X64 needs to pad the stack."
Attachment #474625 - Flags: review?(dvander) → review+
(Assignee)

Comment 3

7 years ago
http://hg.mozilla.org/tracemonkey/rev/aa65a38ae90b
Whiteboard: fixed-in-tracemonkey

Comment 4

7 years ago
(In reply to comment #3)
> http://hg.mozilla.org/tracemonkey/rev/aa65a38ae90b

Perfect.
Do we know when the next merge will be?
Could I please ask for landing this on trunk since it blocks my testing on the 64-bit build machines?
Blocks: 471090

Comment 5

7 years ago
http://hg.mozilla.org/mozilla-central/rev/aa65a38ae90b
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.