512409 - nanojit support for Windows x64

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Makoto Kato [:m_kato]]

Attachment: WIP v0.1

You know, x86_64 backend of nanojit was removed at past month.  By bug 511919, although x86_64 backend into redux's code will be copied to tracemonkey tree, this is still broken.

Comment 1

[David Anderson [:dvander] - inactive, e-mail if emergency]

Cool! Graydon landed the NativeX64.h/.cpp files a few days ago so you can probably cut the patch size down a little :)

Getting it to work is still blocked on a little more merge progress (it has come a long way but a few headers are still in flux), and the tracer needs changes to the demotion logic to be compatible with 64-bit systems. Once the dust has started to clear we'll probably need a new LIR_float opcode to differentiate from LIR_quad.

Comment 2

[Makoto Kato [:m_kato]]

Attachment: WIP v0.2

Comment 3

[Makoto Kato [:m_kato]]

Attachment: patch v1

Comment 4

[Makoto Kato [:m_kato]]

Comment on attachment 402547 [details] [diff] [review]
patch v1

We will support 2 compiler (VC++ or GCC) for x64 Windows.  So, we have to use correct defines.

Comment 5

[Makoto Kato [:m_kato]]

http://hg.mozilla.org/tracemonkey/rev/54bc1f6a31ef

Comment 6

[Nikolay Shopik]

As it checked in shouldn't this be RESOLVED FIXED?

Comment 7

[David Anderson [:dvander] - inactive, e-mail if emergency]

(In reply to comment #6)
> As it checked in shouldn't this be RESOLVED FIXED?

When it lands on mozilla-central. Right now we're using the status whiteboard to denote tracemonkey landings.

Comment 8

[Robert Sayre]

http://hg.mozilla.org/mozilla-central/rev/54bc1f6a31ef