694641 - [nanojit] PPC broken in tamarin-redux (and likely nanojit-central)

Status: RESOLVED WONTFIX

(Tamarin Graveyard :: Virtual Machine, defect, P1)

Description

[Rick Reitmaier]

As a result of landing the patches associated with bug 624164, I have most likely broken PPC builds.

I'll attach the patches sent by Virgil to this bug in case anyone want to pick up where he left off, to ensure that PPC is functioning.

Comment 1

[Rick Reitmaier]

Attachment: Virgil's ppc compilation fix

Comment 2

[Cameron Kaiser [:spectre]]

Strange. Why would those be needed?

Also, I don't see the logic where asm_label() turns off swapping still:

     void Assembler::asm_label() {
-        // do nothing right now
+        // disable swapping for the next instruction; it's a branch point.
+        _lastOpcode.reg1 = NoSwap;
     }

Comment 3

[Rick Reitmaier]

Im(In reply to Cameron Kaiser from comment #2)
> Strange. Why would those be needed?

I'm not sure...he mentioned using xcode compiling 64b ppc.


> Also, I don't see the logic where asm_label() turns off swapping still:
> 
>      void Assembler::asm_label() {
> -        // do nothing right now
> +        // disable swapping for the next instruction; it's a branch point.
> +        _lastOpcode.reg1 = NoSwap;
>      }

I think this is the bit that was removed when I landed the patches en masse.  If you're up to it and can tweak virgils patch, I can review and land it.

Comment 4

[Lars T Hansen]

This blocks normal Mac development and testing (Xcode builds normally build PPC).  Please fix this with the utmost urgency or back out the broken assembler changes.  (Backing out changesets 6641 and 6642 results in code that compiles and passes acceptance testing.)

Comment 5

[Tamarin Bot]

changeset: 6655:50b5a11cbd14
user:      Lars T Hansen <[e-mail]>
summary:   For 694641: disabled PPC builds in Xcode project (r=lhansen)

http://hg.mozilla.org/tamarin-redux/rev/50b5a11cbd14

Comment 6

[Cameron Kaiser [:spectre]]

The 64 bit code is not tested much so that could be it (comment 3). I'm looking at that segment, but I have no objection to backing it out until then.

Comment 7

[Rick Reitmaier]

Attachment: fixup ppc

virgils' patch + add back missing asm_label() code.  Compiles on xcode both 32/64.

Comment 8

[Rick Reitmaier]

(In reply to Lars T Hansen from comment #4)
> This blocks normal Mac development and testing

The tamarin build system no longer performs ppc builds;  disabling ppc in the xcode project is the most straightforward option, as per your comment 5.

Comment 9

[Cameron Kaiser [:spectre]]

Comment on attachment 567550 [details] [diff] [review]
fixup ppc

Beat me to it :) This looks okay. If I find any fallout in TenFourFox I'll submit a separate bug but we don't use the 64-bit stuff, so I don't think there will be any problem.

Comment 10

[Rick Reitmaier]

rreitmai http://hg.mozilla.org/projects/nanojit-central/rev/19b6e1d68ea6

Comment 11

[Tamarin Bot]

changeset: 6734:60a467992d8b
user:      Rick Reitmaier <rreitmai>
summary:   Bug 694641 - [nanojit]  PPC broken in tamarin-redux (and likely nanojit-central) (r=spectre)

http://hg.mozilla.org/tamarin-redux/rev/60a467992d8b

Comment 12

[Sylvestre Ledru [:Sylvestre]]

Tamarin is a dead project now. Mass WONTFIX.

Comment 13

[Sylvestre Ledru [:Sylvestre]]

Tamarin isn't maintained anymore. WONTFIX remaining bugs.