1627255 - Rename the --disable-ion configure option to --disable-jit

Status: RESOLVED FIXED

(Core :: JavaScript Engine: JIT, task, P1)

Description

[Jan de Mooij [:jandem]]

At least 6 years overdue but let's finally do this.

I have a patch for this that will also suggest the new option if you try to use the old one.

Comment 1

[Jan de Mooij [:jandem]]

Attachment: Bug 1627255 part 1 - Remove the ENABLE_ION config item. r?glandium!

These days we can get the same information by checking JS_CODEGEN_NONE, so let's
do that.

Comment 2

[Jan de Mooij [:jandem]]

Attachment: Bug 1627255 part 2 - Rename the --enable-ion configure option to --enable-jit. r?glandium!

For many years now this has covered all our JIT codegen, not just Ion, so this is
a lot less confusing. Using --enable-ion/--disable-ion now results in an error that
suggests the new name.

Depends on D69578

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/00e0880582b2
part 1 - Remove the ENABLE_ION config item. r=glandium
https://hg.mozilla.org/integration/autoland/rev/ddd50e956bc0
part 2 - Rename the --enable-ion configure option to --enable-jit. r=glandium

Comment 4

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/00e0880582b2
https://hg.mozilla.org/mozilla-central/rev/ddd50e956bc0