902908 - Rename js/src/ion to js/src/jit

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jan de Mooij [:jandem]]

Attachment: Patch

As discussed on the mailing list last week, this patch renames js/src/ion to js/src/jit.

It also renames the ifdefs in the header files, for instance ion_TypePolicy_h -> jit_TypePolicy_h

What may be a problem for some people is that the "jit-test" directory and the "jit" directory both start with "jit", so the shell does not complete "jit<tab>" to "jit/" If anybody thinks that's a problem please speak up..

I can build a shell with this patch, but still need to send this to Try.

Comment 1

[Luke Wagner [:luke]]

Comment on attachment 787502 [details] [diff] [review]
Patch

Great!  As for 'jit-test', hopefully, in the long term, we'll roll that into src/tests so that *gasp* we only have a single shell testing harness.

Comment 2

[Jan de Mooij [:jandem]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/fe7a314efca0

Sorry for breaking everybody's patches.. Though "hg rebase" should take care of that just fine. If you don't use hg rebase, replacing ion/ with jit/ in the patch file should work..

Comment 3

[Jan de Mooij [:jandem]]

Follow-up to touch CLOBBER:

https://hg.mozilla.org/integration/mozilla-inbound/rev/105f90f2e28b

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/fe7a314efca0
https://hg.mozilla.org/mozilla-central/rev/105f90f2e28b

Comment 5

[Nicholas Nethercote [inactive]]

Any plans to rename the |ion| namespace?

Comment 6

[Jan de Mooij [:jandem]]

(In reply to Nicholas Nethercote [:njn] from comment #5)
> Any plans to rename the |ion| namespace?

Yup, that's next. I also want to rename

IonRuntime -> JitRuntime
IonCompartment -> JitCompartment
IonCode -> JitCode
IonFrames -> JitFrames

(These are used by both JITs.)

Comment 7

[Hannes Verschore [:h4writer]]

While you are busy, what about renaming "jit-tests.py --ion" to "jit-tests.py --jit". IIRC --ion also tests --eager-baseline.

Comment 8

[Nicolas B. Pierron [:nbp]]

(In reply to Jan de Mooij [:jandem] from comment #6)
> Yup, that's next. I also want to rename
> 
> IonFrames -> JitFrames

Do you mind if I do the renaming of the last one?  As I currently have a *Huge* pile of patches waiting for review on these files. (Bug 878503)

Otherwise, somebody can just review these patches (22+ … and more coming) such as I land them before you do the modifications, but I guess this will require that nobody enforce that these patches should not be reviewed.

Comment 9

[Jan de Mooij [:jandem]]

Attachment: Patch for aurora

Renaming js/src/ion to js/src/jit makes backporting JIT patches to aurora, beta and ESR24 a lot more annoying. As suggested in bug 909499 comment 2, we'd like to backport these changes to aurora and beta (including ESR24). To quote Waldo:

> Failing that, I think we should seriously consider redoing this rename for
> esr24 as well.  As long as we're careful about it just being renaming, I
> think it'd be worth it to avoid ten months of backport conflict-fixing.  To
> be honest, that scares me more than the possibility of screwing something up
> in a single, concerted renaming backport-session.

Comment 10

[Jan de Mooij [:jandem]]

Attachment: Patch for beta

See comment 9.

Comment 11

[Luke Wagner [:luke]]

Comment on attachment 796606 [details] [diff] [review]
Patch for aurora

Thanks for doing this Jan.

Comment 12

[Jan de Mooij [:jandem]]

Pushed to try:

Beta: https://tbpl.mozilla.org/?tree=Try&rev=70cf061b3b2c
Aurora: https://tbpl.mozilla.org/?tree=Try&rev=2947743914ba

Comment 13

[Ryan VanderMeulen [:RyanVM]]

Try pushes are green.

Comment 14

[bhavana bajaj [:bajaj]]

(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #13)
> Try pushes are green.

That was quick :), Looks good to land , approving !

Comment 15

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/e8d49bc63f2c
https://hg.mozilla.org/releases/mozilla-beta/rev/c253a1011418

Comment 16

[u279076]

Assuming no QA needed here. Please remove [qa-] from the whiteboard and add the verifyme keyword if this needs QA.