1804461 - Remove `SingleByteRegs` from non x86 architectures.

Status: RESOLVED FIXED

(Core :: JavaScript Engine: JIT, enhancement, P5)

Description

[Nicolas B. Pierron [:nbp]]

x86 has al, ah, bl, bh, cl, ch, dl, dh single byte registers. This set is used to encode whether among all x64/x86 extended register we properly allocated one which has the ability of being used as a byte-size register.

Most likely for copy and paste reasons, other architectures inherited this trait, which is simply meaning less on other architectures which do not have the same legacy as i386 assembly.

See https://searchfox.org/mozilla-central/search?q=SingleByteReg&path=&case=false&regexp=false

Comment 1

[Michael Holloway]

Hi, I'd like to pick this up as a first SpiderMonkey task.

Comment 2

[Michael Holloway]

Attachment: Bug 1804461: Remove `SingleByteRegs` from non x86 architectures r?nbp

Remove SingleByteRegs constants from non-x86 architectures for which they
don't make sense.

Comment 3

[Nicolas B. Pierron [:nbp]]

Thanks for contributing! The patch sounds good.
The repository is under soft-code-freeze, I will merge it next Tuesday.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/49c9f7b06869
Remove `SingleByteRegs` from non x86 architectures r=nbp

Comment 5

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/49c9f7b06869