BaselineCompiler: Rename frameReg and spReg to BaselineFrameReg and BaselineStackReg

RESOLVED FIXED

Status

()

Core
JavaScript Engine
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: djvj, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Not strictly x64-related, but just a quick cleanup in preparation for that.
(Reporter)

Comment 1

5 years ago
Created attachment 677801 [details] [diff] [review]
patch.
Attachment #677801 - Flags: review?(jdemooij)
Comment on attachment 677801 [details] [diff] [review]
patch.

Review of attachment 677801 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/ion/x86/BaselineRegisters-x86.h
@@ +13,5 @@
>  namespace js {
>  namespace ion {
>  
> +static const Register BaselineFrameReg = ebp;
> +static const Register BaselineStackReg = esp;

We have to update x64 too.
Attachment #677801 - Flags: review?(jdemooij) → review+
(Reporter)

Updated

5 years ago
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.