WebKit Bugzilla
Attachment 371956 Details for
Bug 198604
: Refactoring of architectural Register Information
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Requests
|
Help
|
New Account
|
Log In
Remember
[x]
|
Forgot Password
Login:
[x]
[patch]
Post-1 Review Patch
refactor-reginfo.patch (text/plain), 74.42 KB, created by
Paulo Matos
on 2019-06-12 07:54:03 PDT
(
hide
)
Description:
Post-1 Review Patch
Filename:
MIME Type:
Creator:
Paulo Matos
Created:
2019-06-12 07:54:03 PDT
Size:
74.42 KB
patch
obsolete
>commit bad01eeb6358691c80486101e80edb94ac80025d >Author: Paulo Matos <pmatos@linki.tools> >Date: Wed May 8 18:36:44 2019 +0200 > > Prototype for register information aggregation. > > This prototype works for ARM64, ARMv7, MIPS, X86 and X86_64, > which are all the supported architectures. > > <arch>Registers.h are specification files that describe available > registers and their properties on each architecture. > >diff --git a/Source/JavaScriptCore/CMakeLists.txt b/Source/JavaScriptCore/CMakeLists.txt >index d8f36400fd8..3bc725e7d7d 100644 >--- a/Source/JavaScriptCore/CMakeLists.txt >+++ b/Source/JavaScriptCore/CMakeLists.txt >@@ -435,7 +435,9 @@ set(JavaScriptCore_PRIVATE_FRAMEWORK_HEADERS > API/OpaqueJSString.h > > assembler/ARM64Assembler.h >+ assembler/ARM64Registers.h > assembler/ARMv7Assembler.h >+ assembler/ARMv7Registers.h > assembler/AbortReason.h > assembler/AbstractMacroAssembler.h > assembler/AssemblerBuffer.h >@@ -445,6 +447,7 @@ set(JavaScriptCore_PRIVATE_FRAMEWORK_HEADERS > assembler/CodeLocation.h > assembler/LinkBuffer.h > assembler/MIPSAssembler.h >+ assembler/MIPSRegisters.h > assembler/MacroAssembler.h > assembler/MacroAssemblerARM64.h > assembler/MacroAssemblerARMv7.h >@@ -455,7 +458,10 @@ set(JavaScriptCore_PRIVATE_FRAMEWORK_HEADERS > assembler/MacroAssemblerX86Common.h > assembler/MacroAssemblerX86_64.h > assembler/Printer.h >+ assembler/RegisterInfo.h > assembler/X86Assembler.h >+ assembler/X86Registers.h >+ assembler/X86_64Registers.h > > bindings/ScriptFunctionCall.h > bindings/ScriptObject.h >diff --git a/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj b/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj >index 6750a65b6e3..05109515efc 100644 >--- a/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj >+++ b/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj >@@ -1231,10 +1231,12 @@ > 86976E5F1FA3E8BC00E7C4E1 /* BigIntConstructor.h in Headers */ = {isa = PBXBuildFile; fileRef = 86976E571FA3754000E7C4E1 /* BigIntConstructor.h */; settings = {ATTRIBUTES = (Private, ); }; }; > 869EBCB70E8C6D4A008722CC /* ResultType.h in Headers */ = {isa = PBXBuildFile; fileRef = 869EBCB60E8C6D4A008722CC /* ResultType.h */; settings = {ATTRIBUTES = (Private, ); }; }; > 86ADD1450FDDEA980006EEC2 /* ARMv7Assembler.h in Headers */ = {isa = PBXBuildFile; fileRef = 86ADD1430FDDEA980006EEC2 /* ARMv7Assembler.h */; settings = {ATTRIBUTES = (Private, ); }; }; >+ 86ADD1450FDDEA980006FFCC /* ARMv7Registers.h in Headers */ = {isa = PBXBuildFile; fileRef = 86ADD1430FDDEA980006FFCC /* ARMv7Registers.h */; settings = {ATTRIBUTES = (Private, ); }; }; > 86ADD1460FDDEA980006EEC2 /* MacroAssemblerARMv7.h in Headers */ = {isa = PBXBuildFile; fileRef = 86ADD1440FDDEA980006EEC2 /* MacroAssemblerARMv7.h */; settings = {ATTRIBUTES = (Private, ); }; }; > 86C36EEA0EE1289D00B3DF59 /* MacroAssembler.h in Headers */ = {isa = PBXBuildFile; fileRef = 86C36EE90EE1289D00B3DF59 /* MacroAssembler.h */; settings = {ATTRIBUTES = (Private, ); }; }; > 86C568E111A213EE0007F7F0 /* MacroAssemblerMIPS.h in Headers */ = {isa = PBXBuildFile; fileRef = 86C568DE11A213EE0007F7F0 /* MacroAssemblerMIPS.h */; settings = {ATTRIBUTES = (Private, ); }; }; > 86C568E211A213EE0007F7F0 /* MIPSAssembler.h in Headers */ = {isa = PBXBuildFile; fileRef = 86C568DF11A213EE0007F7F0 /* MIPSAssembler.h */; settings = {ATTRIBUTES = (Private, ); }; }; >+ 86C568E211A213EE0007F7FF /* MIPSRegisters.h in Headers */ = {isa = PBXBuildFile; fileRef = 86C568DF11A213EE0007F7FF /* MIPSRegisters.h */; settings = {ATTRIBUTES = (Private, ); }; }; > 86CC85A10EE79A4700288682 /* JITInlines.h in Headers */ = {isa = PBXBuildFile; fileRef = 86CC85A00EE79A4700288682 /* JITInlines.h */; }; > 86CCEFDE0F413F8900FD7F9E /* JITCode.h in Headers */ = {isa = PBXBuildFile; fileRef = 86CCEFDD0F413F8900FD7F9E /* JITCode.h */; settings = {ATTRIBUTES = (Private, ); }; }; > 86D2221A167EF9440024C804 /* testapi.mm in Sources */ = {isa = PBXBuildFile; fileRef = 86D22219167EF9440024C804 /* testapi.mm */; }; >@@ -1283,6 +1285,9 @@ > 960097A60EBABB58007A7297 /* LabelScope.h in Headers */ = {isa = PBXBuildFile; fileRef = 960097A50EBABB58007A7297 /* LabelScope.h */; }; > 9688CB150ED12B4E001D649F /* AssemblerBuffer.h in Headers */ = {isa = PBXBuildFile; fileRef = 9688CB130ED12B4E001D649F /* AssemblerBuffer.h */; settings = {ATTRIBUTES = (Private, ); }; }; > 9688CB160ED12B4E001D649F /* X86Assembler.h in Headers */ = {isa = PBXBuildFile; fileRef = 9688CB140ED12B4E001D649F /* X86Assembler.h */; settings = {ATTRIBUTES = (Private, ); }; }; >+ 9688CB160ED12B4E001D6491 /* X86Registers.h in Headers */ = {isa = PBXBuildFile; fileRef = 9688CB140ED12B4E001D6491 /* X86Registers.h */; settings = {ATTRIBUTES = (Private, ); }; }; >+ 9688CB160ED12B4E001D6492 /* X86_64Registers.h in Headers */ = {isa = PBXBuildFile; fileRef = 9688CB140ED12B4E001D6492 /* X86_64Registers.h */; settings = {ATTRIBUTES = (Private, ); }; }; >+ 9688CB160ED12B4E001D6499 /* RegisterInfo.h in Headers */ = {isa = PBXBuildFile; fileRef = 9688CB140ED12B4E001D6499 /* RegisterInfo.h */; settings = {ATTRIBUTES = (Private, ); }; }; > 969A07230ED1CE3300F1F681 /* BytecodeGenerator.h in Headers */ = {isa = PBXBuildFile; fileRef = 969A07210ED1CE3300F1F681 /* BytecodeGenerator.h */; }; > 969A072A0ED1CE6900F1F681 /* Label.h in Headers */ = {isa = PBXBuildFile; fileRef = 969A07270ED1CE6900F1F681 /* Label.h */; }; > 969A072B0ED1CE6900F1F681 /* RegisterID.h in Headers */ = {isa = PBXBuildFile; fileRef = 969A07280ED1CE6900F1F681 /* RegisterID.h */; }; >@@ -1344,6 +1349,7 @@ > A18193E41B4E0CDB00FC1029 /* IntlCollatorPrototype.lut.h in Headers */ = {isa = PBXBuildFile; fileRef = A18193E21B4E0CDB00FC1029 /* IntlCollatorPrototype.lut.h */; }; > A1A009C01831A22D00CF8711 /* MacroAssemblerARM64.h in Headers */ = {isa = PBXBuildFile; fileRef = 8640923C156EED3B00566CB2 /* MacroAssemblerARM64.h */; settings = {ATTRIBUTES = (Private, ); }; }; > A1A009C11831A26E00CF8711 /* ARM64Assembler.h in Headers */ = {isa = PBXBuildFile; fileRef = 8640923B156EED3B00566CB2 /* ARM64Assembler.h */; settings = {ATTRIBUTES = (Private, ); }; }; >+ A1A009C11831A26E00CF8722 /* ARM64Registers.h in Headers */ = {isa = PBXBuildFile; fileRef = 8640923B156EED3B00566CC2 /* ARM64Registers.h */; settings = {ATTRIBUTES = (Private, ); }; }; > A1B9E23A1B4E0D6700BC7FED /* IntlCollator.h in Headers */ = {isa = PBXBuildFile; fileRef = A1B9E2341B4E0D6700BC7FED /* IntlCollator.h */; }; > A1B9E23C1B4E0D6700BC7FED /* IntlCollatorConstructor.h in Headers */ = {isa = PBXBuildFile; fileRef = A1B9E2361B4E0D6700BC7FED /* IntlCollatorConstructor.h */; }; > A1B9E23E1B4E0D6700BC7FED /* IntlCollatorPrototype.h in Headers */ = {isa = PBXBuildFile; fileRef = A1B9E2381B4E0D6700BC7FED /* IntlCollatorPrototype.h */; }; >@@ -3935,6 +3941,7 @@ > 863C6D991521111200585E4E /* YarrCanonicalize.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; name = YarrCanonicalize.h; path = yarr/YarrCanonicalize.h; sourceTree = "<group>"; }; > 863C6D9A1521111200585E4E /* YarrCanonicalizeUCS2.js */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.javascript; name = YarrCanonicalizeUCS2.js; path = yarr/YarrCanonicalizeUCS2.js; sourceTree = "<group>"; }; > 8640923B156EED3B00566CB2 /* ARM64Assembler.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = ARM64Assembler.h; sourceTree = "<group>"; }; >+ 8640923B156EED3B00566CC2 /* ARM64Registers.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = ARM64Registers.h; sourceTree = "<group>"; }; > 8640923C156EED3B00566CB2 /* MacroAssemblerARM64.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = MacroAssemblerARM64.h; sourceTree = "<group>"; }; > 865A30F0135007E100CDB49E /* JSCJSValueInlines.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = JSCJSValueInlines.h; sourceTree = "<group>"; }; > 866739D013BFDE710023D87C /* BigInteger.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = BigInteger.h; sourceTree = "<group>"; }; >@@ -3963,6 +3970,7 @@ > 86A054481556451B00445157 /* LowLevelInterpreter64.asm */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.asm.asm; lineEnding = 0; name = LowLevelInterpreter64.asm; path = llint/LowLevelInterpreter64.asm; sourceTree = "<group>"; }; > 86A90ECF0EE7D51F00AB350D /* JITArithmetic.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JITArithmetic.cpp; sourceTree = "<group>"; }; > 86ADD1430FDDEA980006EEC2 /* ARMv7Assembler.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = ARMv7Assembler.h; sourceTree = "<group>"; }; >+ 86ADD1430FDDEA980006FFCC /* ARMv7Registers.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = ARMv7Registers.h; sourceTree = "<group>"; }; > 86ADD1440FDDEA980006EEC2 /* MacroAssemblerARMv7.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = MacroAssemblerARMv7.h; sourceTree = "<group>"; }; > 86B5822C14D22F5F00A9C306 /* ProfileTreeNode.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = ProfileTreeNode.h; sourceTree = "<group>"; }; > 86B5822E14D2373B00A9C306 /* CodeProfile.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = CodeProfile.cpp; sourceTree = "<group>"; }; >@@ -3972,6 +3980,7 @@ > 86C36EE90EE1289D00B3DF59 /* MacroAssembler.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = MacroAssembler.h; sourceTree = "<group>"; }; > 86C568DE11A213EE0007F7F0 /* MacroAssemblerMIPS.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = MacroAssemblerMIPS.h; sourceTree = "<group>"; }; > 86C568DF11A213EE0007F7F0 /* MIPSAssembler.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = MIPSAssembler.h; sourceTree = "<group>"; }; >+ 86C568DF11A213EE0007F7FF /* MIPSRegisters.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = MIPSRegisters.h; sourceTree = "<group>"; }; > 86CC85A00EE79A4700288682 /* JITInlines.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = JITInlines.h; sourceTree = "<group>"; }; > 86CC85A20EE79B7400288682 /* JITCall.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JITCall.cpp; sourceTree = "<group>"; }; > 86CC85C30EE7A89400288682 /* JITPropertyAccess.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JITPropertyAccess.cpp; sourceTree = "<group>"; }; >@@ -4062,6 +4071,9 @@ > 960097A50EBABB58007A7297 /* LabelScope.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = LabelScope.h; sourceTree = "<group>"; }; > 9688CB130ED12B4E001D649F /* AssemblerBuffer.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = AssemblerBuffer.h; sourceTree = "<group>"; }; > 9688CB140ED12B4E001D649F /* X86Assembler.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = X86Assembler.h; sourceTree = "<group>"; }; >+ 9688CB140ED12B4E001D6491 /* X86Registers.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = X86Registers.h; sourceTree = "<group>"; }; >+ 9688CB140ED12B4E001D6492 /* X86_64Registers.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = X86_64Registers.h; sourceTree = "<group>"; }; >+ 9688CB140ED12B4E001D6499 /* RegisterInfo.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = RegisterInfo.h; sourceTree = "<group>"; }; > 969A07200ED1CE3300F1F681 /* BytecodeGenerator.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = BytecodeGenerator.cpp; sourceTree = "<group>"; }; > 969A07210ED1CE3300F1F681 /* BytecodeGenerator.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = BytecodeGenerator.h; sourceTree = "<group>"; }; > 969A07270ED1CE6900F1F681 /* Label.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = Label.h; sourceTree = "<group>"; }; >@@ -7837,7 +7849,9 @@ > AD412B351E7B57C0008AF157 /* AllowMacroScratchRegisterUsageIf.h */, > 8640923B156EED3B00566CB2 /* ARM64Assembler.h */, > FE1E2C3D2240D2F600F6B729 /* ARM64EAssembler.h */, >+ 8640923B156EED3B00566CC2 /* ARM64Registers.h */, > 86ADD1430FDDEA980006EEC2 /* ARMv7Assembler.h */, >+ 86ADD1430FDDEA980006FFCC /* ARMv7Registers.h */, > 9688CB130ED12B4E001D649F /* AssemblerBuffer.h */, > 86D3B2C110156BDE002865E7 /* AssemblerBufferWithConstantPool.h */, > 43C392AA1C3BEB0000241F53 /* AssemblerCommon.h */, >@@ -7866,6 +7880,7 @@ > 860161E20F3A83C100F84710 /* MacroAssemblerX86Common.h */, > 65860177185A8F5E00030EEE /* MaxFrameExtentForSlowPathCall.h */, > 86C568DF11A213EE0007F7F0 /* MIPSAssembler.h */, >+ 86C568DF11A213EE0007F7FF /* MIPSRegisters.h */, > FE63DD551EA9BC5D00103A69 /* Printer.cpp */, > FE63DD531EA9B60E00103A69 /* Printer.h */, > FE10AAF31F46826D009DEDC5 /* ProbeContext.cpp */, >@@ -7873,8 +7888,11 @@ > FEB41CCB1F73284200C5481E /* ProbeFrame.h */, > FE10AAE91F44D510009DEDC5 /* ProbeStack.cpp */, > FE10AAEA1F44D512009DEDC5 /* ProbeStack.h */, >+ 9688CB140ED12B4E001D6499 /* RegisterInfo.h */, > FE533CA01F217C310016A1FE /* testmasm.cpp */, > 9688CB140ED12B4E001D649F /* X86Assembler.h */, >+ 9688CB140ED12B4E001D6491 /* X86Registers.h */, >+ 9688CB140ED12B4E001D6492 /* X86_64Registers.h */, > ); > path = assembler; > sourceTree = "<group>"; >@@ -8617,7 +8635,9 @@ > 0F6B1CB91861244C00845D97 /* ArityCheckMode.h in Headers */, > A1A009C11831A26E00CF8711 /* ARM64Assembler.h in Headers */, > FE1E2C402240DD6200F6B729 /* ARM64EAssembler.h in Headers */, >+ A1A009C11831A26E00CF8722 /* ARM64Registers.h */, > 86ADD1450FDDEA980006EEC2 /* ARMv7Assembler.h in Headers */, >+ 86ADD1450FDDEA980006FFCC /* ARMv7Registers.h in Headers */, > 0F8335B81639C1EA001443B5 /* ArrayAllocationProfile.h in Headers */, > A7A8AF3517ADB5F3005AB174 /* ArrayBuffer.h in Headers */, > 0FFC99D5184EE318009C10AB /* ArrayBufferNeuteringWatchpointSet.h in Headers */, >@@ -9649,6 +9669,7 @@ > 7C008CE7187631B600955C24 /* Microtask.h in Headers */, > FE2A87601F02381600EB31B2 /* MinimumReservedZoneSize.h in Headers */, > 86C568E211A213EE0007F7F0 /* MIPSAssembler.h in Headers */, >+ 86C568E211A213EE0007F7FF /* MIPSRegisters.h in Headers */, > C4703CD7192844CC0013FBEA /* models.py in Headers */, > E3794E761B77EB97005543AE /* ModuleAnalyzer.h in Headers */, > 9F63434577274FAFB9336C38 /* ModuleNamespaceAccessCase.h in Headers */, >@@ -9775,6 +9796,7 @@ > E328C6C91DA432F900D255FD /* RegisterAtOffset.h in Headers */, > E328C6C81DA4306100D255FD /* RegisterAtOffsetList.h in Headers */, > 969A072B0ED1CE6900F1F681 /* RegisterID.h in Headers */, >+ 9688CB160ED12B4E001D6499 /* RegisterInfo.h in Headers */, > 623A37EC1B87A7C000754209 /* RegisterMap.h in Headers */, > 0FC314121814559100033232 /* RegisterSet.h in Headers */, > 0FD0E5F01E46BF250006AB08 /* RegisterState.h in Headers */, >@@ -10053,6 +10075,8 @@ > C2B6D75318A33793004A9301 /* WriteBarrierInlines.h in Headers */, > 0FC8150A14043BF500CFA603 /* WriteBarrierSupport.h in Headers */, > 9688CB160ED12B4E001D649F /* X86Assembler.h in Headers */, >+ 9688CB160ED12B4E001D6491 /* X86Registers.h in Headers */, >+ 9688CB160ED12B4E001D6492 /* X86_64Registers.h in Headers */, > 9959E92E1BD17FA4001AA413 /* xxd.pl in Headers */, > 451539B912DC994500EF7AC4 /* Yarr.h in Headers */, > E3282BBB1FE930AF00EDAF71 /* YarrErrorCode.h in Headers */, >diff --git a/Source/JavaScriptCore/assembler/ARM64Assembler.h b/Source/JavaScriptCore/assembler/ARM64Assembler.h >index 544b612c345..ae92882e51c 100644 >--- a/Source/JavaScriptCore/assembler/ARM64Assembler.h >+++ b/Source/JavaScriptCore/assembler/ARM64Assembler.h >@@ -27,6 +27,7 @@ > > #if ENABLE(ASSEMBLER) && CPU(ARM64) > >+#include "ARM64Registers.h" > #include "AssemblerBuffer.h" > #include "AssemblerCommon.h" > #include "CPU.h" >@@ -164,102 +165,35 @@ inline uint16_t getHalfword(uint64_t value, int which) > return value >> (which << 4); > } > >+// If modifying the namespace name, please be sure to modify it as well >+// in ARM64Registers.h on the macro definition of REGISTER_NAMESPACE > namespace ARM64Registers { > > typedef enum : int8_t { >- // Parameter/result registers. >- x0, >- x1, >- x2, >- x3, >- x4, >- x5, >- x6, >- x7, >- // Indirect result location register. >- x8, >- // Temporary registers. >- x9, >- x10, >- x11, >- x12, >- x13, >- x14, >- x15, >- // Intra-procedure-call scratch registers (temporary). >- x16, >- x17, >- // Platform Register (temporary). >- x18, >- // Callee-saved. >- x19, >- x20, >- x21, >- x22, >- x23, >- x24, >- x25, >- x26, >- x27, >- x28, >- // Special. >- fp, >- lr, >- sp, >- >- ip0 = x16, >- ip1 = x17, >- x29 = fp, >- x30 = lr, >- zr = 0x3f, >+#define REGISTER_ID(id, nsid, name, r, cs) id, >+ FOR_EACH_GP_REGISTER(REGISTER_ID) >+#undef REGISTER_ID >+ >+#define REGISTER_ALIAS(id, nsid, name, alias) id = alias, >+ FOR_EACH_REGISTER_ALIAS(REGISTER_ALIAS) >+#undef REGISTER_ALIAS >+ > InvalidGPRReg = -1, > } RegisterID; > > typedef enum : int8_t { >- pc, >- nzcv, >- fpsr >+#define REGISTER_ID(id, nsid, name) id, >+ FOR_EACH_SP_REGISTER(REGISTER_ID) >+#undef REGISTER_ID > } SPRegisterID; > > // ARM64 always has 32 FPU registers 128-bits each. See http://llvm.org/devmtg/2012-11/Northover-AArch64.pdf > // and Section 5.1.2 in http://infocenter.arm.com/help/topic/com.arm.doc.ihi0055b/IHI0055B_aapcs64.pdf. > // However, we only use them for 64-bit doubles. > typedef enum : int8_t { >- // Parameter/result registers. >- q0, >- q1, >- q2, >- q3, >- q4, >- q5, >- q6, >- q7, >- // Callee-saved (up to 64-bits only!). >- q8, >- q9, >- q10, >- q11, >- q12, >- q13, >- q14, >- q15, >- // Temporary registers. >- q16, >- q17, >- q18, >- q19, >- q20, >- q21, >- q22, >- q23, >- q24, >- q25, >- q26, >- q27, >- q28, >- q29, >- q30, >- q31, >+#define REGISTER_ID(id, nsid, name, r, cs) id, >+ FOR_EACH_FP_REGISTER(REGISTER_ID) >+#undef REGISTER_ID > InvalidFPRReg = -1, > } FPRegisterID; > >@@ -292,10 +226,9 @@ public: > { > ASSERT(id >= firstRegister() && id <= lastRegister()); > static const char* const nameForRegister[numberOfRegisters()] = { >- "r0", "r1", "r2", "r3", "r4", "r5", "r6", "r7", >- "r8", "r9", "r10", "r11", "r12", "r13", "r14", "r15", >- "r16", "r17", "r18", "r19", "r20", "r21", "r22", "r23", >- "r24", "r25", "r26", "r27", "r28", "fp", "lr", "sp" >+#define REGISTER_NAME(id, nsid, name, r, cs) name, >+ FOR_EACH_GP_REGISTER(REGISTER_NAME) >+#undef REGISTER_NAME > }; > return nameForRegister[id]; > } >@@ -304,7 +237,9 @@ public: > { > ASSERT(id >= firstSPRegister() && id <= lastSPRegister()); > static const char* const nameForRegister[numberOfSPRegisters()] = { >- "pc", "nzcv", "fpsr" >+#define REGISTER_NAME(id, nsid, name) name, >+ FOR_EACH_SP_REGISTER(REGISTER_NAME) >+#undef REGISTER_NAME > }; > return nameForRegister[id]; > } >@@ -313,10 +248,9 @@ public: > { > ASSERT(id >= firstFPRegister() && id <= lastFPRegister()); > static const char* const nameForRegister[numberOfFPRegisters()] = { >- "q0", "q1", "q2", "q3", "q4", "q5", "q6", "q7", >- "q8", "q9", "q10", "q11", "q12", "q13", "q14", "q15", >- "q16", "q17", "q18", "q19", "q20", "q21", "q22", "q23", >- "q24", "q25", "q26", "q27", "q28", "q29", "q30", "q31" >+#define REGISTER_NAME(id, nsid, name, r, cs) name, >+ FOR_EACH_FP_REGISTER(REGISTER_NAME) >+#undef REGISTER_NAME > }; > return nameForRegister[id]; > } >diff --git a/Source/JavaScriptCore/assembler/ARM64Registers.h b/Source/JavaScriptCore/assembler/ARM64Registers.h >new file mode 100644 >index 00000000000..83261e6e63a >--- /dev/null >+++ b/Source/JavaScriptCore/assembler/ARM64Registers.h >@@ -0,0 +1,177 @@ >+/* >+ * Copyright (C) 2019 Metrological Group B.V. >+ * Copyright (C) 2019 Igalia S.L. >+ * >+ * Redistribution and use in source and binary forms, with or without >+ * modification, are permitted provided that the following conditions >+ * are met: >+ * 1. Redistributions of source code must retain the above copyright >+ * notice, this list of conditions and the following disclaimer. >+ * 2. Redistributions in binary form must reproduce the above copyright >+ * notice, this list of conditions and the following disclaimer in the >+ * documentation and/or other materials provided with the distribution. >+ * >+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. ``AS IS'' AND ANY >+ * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE >+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR >+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR >+ * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, >+ * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, >+ * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR >+ * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY >+ * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT >+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE >+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. >+ */ >+ >+#pragma once >+ >+#include <wtf/Platform.h> >+ >+#define REGISTER_NAMESPACE ARM64Registers >+#define n(id) id, REGISTER_NAMESPACE::id, #id >+ >+#define FOR_EACH_REGISTER(macro) \ >+ FOR_EACH_GP_REGISTER(macro) \ >+ FOR_EACH_FP_REGISTER(macro) >+ >+#define macro_forward(macro, ...) macro(__VA_ARGS__) >+ >+// We don't include LR in the set of callee-save registers even though it technically belongs >+// there. This is because we use this set to describe the set of registers that need to be saved >+// beyond what you would save by the platform-agnostic "preserve return address" and "restore >+// return address" operations in CCallHelpers. >+ >+#if !PLATFORM(IOS_FAMILY) >+#define FOR_EACH_GP_REGISTER(macro) \ >+ /* Parameter/result registers. */ \ >+ macro_forward(macro, n(x0), 0, 0) \ >+ macro_forward(macro, n(x1), 0, 0) \ >+ macro_forward(macro, n(x2), 0, 0) \ >+ macro_forward(macro, n(x3), 0, 0) \ >+ macro_forward(macro, n(x4), 0, 0) \ >+ macro_forward(macro, n(x5), 0, 0) \ >+ macro_forward(macro, n(x6), 0, 0) \ >+ macro_forward(macro, n(x7), 0, 0) \ >+ /* Indirect result location register. */ \ >+ macro_forward(macro, n(x8), 0, 0) \ >+ /* Temporary registers. */ \ >+ macro_forward(macro, n(x9), 0, 0) \ >+ macro_forward(macro, n(x10), 0, 0) \ >+ macro_forward(macro, n(x11), 0, 0) \ >+ macro_forward(macro, n(x12), 0, 0) \ >+ macro_forward(macro, n(x13), 0, 0) \ >+ macro_forward(macro, n(x14), 0, 0) \ >+ macro_forward(macro, n(x15), 0, 0) \ >+ /* Intra-procedure-call scratch registers (temporary). */ \ >+ macro_forward(macro, n(x16), 0, 0) \ >+ macro_forward(macro, n(x17), 0, 0) \ >+ /* Platform Register (temporary). */ \ >+ macro_forward(macro, n(x18), 0, 0) \ >+ /* Callee-saved. */ \ >+ macro_forward(macro, n(x19), 0, 1) \ >+ macro_forward(macro, n(x20), 0, 1) \ >+ macro_forward(macro, n(x21), 0, 1) \ >+ macro_forward(macro, n(x22), 0, 1) \ >+ macro_forward(macro, n(x23), 0, 1) \ >+ macro_forward(macro, n(x24), 0, 1) \ >+ macro_forward(macro, n(x25), 0, 1) \ >+ macro_forward(macro, n(x26), 0, 1) \ >+ macro_forward(macro, n(x27), 0, 1) \ >+ macro_forward(macro, n(x28), 0, 1) \ >+ /* Special. */ \ >+ macro_forward(macro, n(fp), 0, 1) \ >+ macro_forward(macro, n(lr), 1, 0) \ >+ macro_forward(macro, n(sp), 0, 0) >+#else >+#define FOR_EACH_GP_REGISTER(macro) \ >+ /* Parameter/result registers. */ \ >+ macro_forward(macro, n(x0), 0, 0) \ >+ macro_forward(macro, n(x1), 0, 0) \ >+ macro_forward(macro, n(x2), 0, 0) \ >+ macro_forward(macro, n(x3), 0, 0) \ >+ macro_forward(macro, n(x4), 0, 0) \ >+ macro_forward(macro, n(x5), 0, 0) \ >+ macro_forward(macro, n(x6), 0, 0) \ >+ macro_forward(macro, n(x7), 0, 0) \ >+ /* Indirect result location register. */ \ >+ macro_forward(macro, n(x8), 0, 0) \ >+ /* Temporary registers. */ \ >+ macro_forward(macro, n(x9), 0, 0) \ >+ macro_forward(macro, n(x10), 0, 0) \ >+ macro_forward(macro, n(x11), 0, 0) \ >+ macro_forward(macro, n(x12), 0, 0) \ >+ macro_forward(macro, n(x13), 0, 0) \ >+ macro_forward(macro, n(x14), 0, 0) \ >+ macro_forward(macro, n(x15), 0, 0) \ >+ /* Intra-procedure-call scratch registers (temporary). */ \ >+ macro_forward(macro, n(x16), 0, 0) \ >+ macro_forward(macro, n(x17), 0, 0) \ >+ /* Platform Register (temporary). */ \ >+ macro_forward(macro, n(x18), 1, 0) \ >+ /* Callee-saved. */ \ >+ macro_forward(macro, n(x19), 0, 1) \ >+ macro_forward(macro, n(x20), 0, 1) \ >+ macro_forward(macro, n(x21), 0, 1) \ >+ macro_forward(macro, n(x22), 0, 1) \ >+ macro_forward(macro, n(x23), 0, 1) \ >+ macro_forward(macro, n(x24), 0, 1) \ >+ macro_forward(macro, n(x25), 0, 1) \ >+ macro_forward(macro, n(x26), 0, 1) \ >+ macro_forward(macro, n(x27), 0, 1) \ >+ macro_forward(macro, n(x28), 0, 1) \ >+ /* Special. */ \ >+ macro_forward(macro, n(fp), 0, 1) \ >+ macro_forward(macro, n(lr), 1, 0) \ >+ macro_forward(macro, n(sp), 0, 0) >+#endif >+ >+#define FOR_EACH_REGISTER_ALIAS(macro) \ >+ macro_forward(macro, n(ip0), x16) \ >+ macro_forward(macro, n(ip1), x17) \ >+ macro_forward(macro, n(x29), fp) \ >+ macro_forward(macro, n(x30), lr) \ >+ macro_forward(macro, n(zr), 0x3f) >+ >+#define FOR_EACH_SP_REGISTER(macro) \ >+ macro_forward(macro, n(pc)) \ >+ macro_forward(macro, n(nzcv)) \ >+ macro_forward(macro, n(fpsr)) >+ >+#define FOR_EACH_FP_REGISTER(macro) \ >+ /* Parameter/result registers. */ \ >+ macro_forward(macro, n(q0), 0, 0) \ >+ macro_forward(macro, n(q1), 0, 0) \ >+ macro_forward(macro, n(q2), 0, 0) \ >+ macro_forward(macro, n(q3), 0, 0) \ >+ macro_forward(macro, n(q4), 0, 0) \ >+ macro_forward(macro, n(q5), 0, 0) \ >+ macro_forward(macro, n(q6), 0, 0) \ >+ macro_forward(macro, n(q7), 0, 0) \ >+ /* Callee-saved (up to 64-bits only!). */ \ >+ macro_forward(macro, n(q8), 0, 1) \ >+ macro_forward(macro, n(q9), 0, 1) \ >+ macro_forward(macro, n(q10), 0, 1) \ >+ macro_forward(macro, n(q11), 0, 1) \ >+ macro_forward(macro, n(q12), 0, 1) \ >+ macro_forward(macro, n(q13), 0, 1) \ >+ macro_forward(macro, n(q14), 0, 1) \ >+ macro_forward(macro, n(q15), 0, 1) \ >+ /* Temporary registers. */ \ >+ macro_forward(macro, n(q16), 0, 0) \ >+ macro_forward(macro, n(q17), 0, 0) \ >+ macro_forward(macro, n(q18), 0, 0) \ >+ macro_forward(macro, n(q19), 0, 0) \ >+ macro_forward(macro, n(q20), 0, 0) \ >+ macro_forward(macro, n(q21), 0, 0) \ >+ macro_forward(macro, n(q22), 0, 0) \ >+ macro_forward(macro, n(q23), 0, 0) \ >+ macro_forward(macro, n(q24), 0, 0) \ >+ macro_forward(macro, n(q25), 0, 0) \ >+ macro_forward(macro, n(q26), 0, 0) \ >+ macro_forward(macro, n(q27), 0, 0) \ >+ macro_forward(macro, n(q28), 0, 0) \ >+ macro_forward(macro, n(q29), 0, 0) \ >+ macro_forward(macro, n(q30), 0, 0) \ >+ macro_forward(macro, n(q31), 0, 0) >+ >diff --git a/Source/JavaScriptCore/assembler/ARMv7Assembler.h b/Source/JavaScriptCore/assembler/ARMv7Assembler.h >index 6fe04e44623..0c624242687 100644 >--- a/Source/JavaScriptCore/assembler/ARMv7Assembler.h >+++ b/Source/JavaScriptCore/assembler/ARMv7Assembler.h >@@ -30,6 +30,7 @@ > > #include "AssemblerBuffer.h" > #include "AssemblerCommon.h" >+#include "RegisterInfo.h" > #include <limits.h> > #include <wtf/Assertions.h> > #include <wtf/Vector.h> >@@ -37,132 +38,45 @@ > > namespace JSC { > >+// If modifying the namespace name, please be sure to modify it as well >+// in ARMv7Registers.h on the macro definition of REGISTER_NAMESPACE > namespace ARMRegisters { > > typedef enum : int8_t { >- r0, >- r1, >- r2, >- r3, >- r4, >- r5, >- r6, >- r7, >- r8, >- r9, >- r10, >- r11, >- r12, >- r13, >- r14, >- r15, >- >- fp = r7, // frame pointer >- sb = r9, // static base >- sl = r10, // stack limit >- ip = r12, >- sp = r13, >- lr = r14, >- pc = r15, >+#define REGISTER_ID(id, nsid, name, r, cs) id, >+ FOR_EACH_GP_REGISTER(REGISTER_ID) >+#undef REGISTER_ID >+ >+#define REGISTER_ALIAS(id, nsid, name, alias) id = alias, >+ FOR_EACH_REGISTER_ALIAS(REGISTER_ALIAS) >+#undef REGISTER_ALIAS > InvalidGPRReg = -1, > } RegisterID; > > typedef enum : int8_t { >- apsr, >- fpscr >+#define REGISTER_ID(id, nsid, name) id, >+ FOR_EACH_SP_REGISTER(REGISTER_ID) >+#undef REGISTER_ID > } SPRegisterID; > > typedef enum : int8_t { >- s0, >- s1, >- s2, >- s3, >- s4, >- s5, >- s6, >- s7, >- s8, >- s9, >- s10, >- s11, >- s12, >- s13, >- s14, >- s15, >- s16, >- s17, >- s18, >- s19, >- s20, >- s21, >- s22, >- s23, >- s24, >- s25, >- s26, >- s27, >- s28, >- s29, >- s30, >- s31, >+#define REGISTER_ID(id, nsid, name, r, cs) id, >+ FOR_EACH_FP_SINGLE_REGISTER(REGISTER_ID) >+#undef REGISTER_ID > } FPSingleRegisterID; > > typedef enum : int8_t { >- d0, >- d1, >- d2, >- d3, >- d4, >- d5, >- d6, >- d7, >- d8, >- d9, >- d10, >- d11, >- d12, >- d13, >- d14, >- d15, >-#if CPU(ARM_NEON) || CPU(ARM_VFP_V3_D32) >- d16, >- d17, >- d18, >- d19, >- d20, >- d21, >- d22, >- d23, >- d24, >- d25, >- d26, >- d27, >- d28, >- d29, >- d30, >- d31, >-#endif // CPU(ARM_NEON) || CPU(ARM_VFP_V3_D32) >+#define REGISTER_ID(id, nsid, name, r, cs) id, >+ FOR_EACH_FP_DOUBLE_REGISTER(REGISTER_ID) >+#undef REGISTER_ID > InvalidFPRReg = -1, > } FPDoubleRegisterID; > > #if CPU(ARM_NEON) > typedef enum : int8_t { >- q0, >- q1, >- q2, >- q3, >- q4, >- q5, >- q6, >- q7, >- q8, >- q9, >- q10, >- q11, >- q12, >- q13, >- q14, >- q15, >+#define REGISTER_ID(id, nsid, name, r, cs) id, >+ FOR_EACH_FP_QUAD_REGISTER(REGISTER_ID) >+#undef REGISTER_ID > } FPQuadRegisterID; > #endif // CPU(ARM_NEON) > >@@ -455,10 +369,9 @@ public: > { > ASSERT(id >= firstRegister() && id <= lastRegister()); > static const char* const nameForRegister[numberOfRegisters()] = { >- "r0", "r1", "r2", "r3", >- "r4", "r5", "r6", "fp", >- "r8", "r9", "r10", "r11", >- "ip", "sp", "lr", "pc" >+#define REGISTER_NAME(id, nsid, name, r, cs) name, >+ FOR_EACH_GP_REGISTER(REGISTER_NAME) >+#undef REGISTER_NAME > }; > return nameForRegister[id]; > } >@@ -467,7 +380,9 @@ public: > { > ASSERT(id >= firstSPRegister() && id <= lastSPRegister()); > static const char* const nameForRegister[numberOfSPRegisters()] = { >- "apsr", "fpscr" >+#define REGISTER_NAME(id, nsid, name) name, >+ FOR_EACH_SP_REGISTER(REGISTER_NAME) >+#undef REGISTER_NAME > }; > return nameForRegister[id]; > } >@@ -476,16 +391,9 @@ public: > { > ASSERT(id >= firstFPRegister() && id <= lastFPRegister()); > static const char* const nameForRegister[numberOfFPRegisters()] = { >- "d0", "d1", "d2", "d3", >- "d4", "d5", "d6", "d7", >- "d8", "d9", "d10", "d11", >- "d12", "d13", "d14", "d15", >-#if CPU(ARM_NEON) || CPU(ARM_VFP_V3_D32) >- "d16", "d17", "d18", "d19", >- "d20", "d21", "d22", "d23", >- "d24", "d25", "d26", "d27", >- "d28", "d29", "d30", "d31" >-#endif // CPU(ARM_NEON) || CPU(ARM_VFP_V3_D32) >+#define REGISTER_NAME(id, nsid, name, r, cs) name, >+ FOR_EACH_FP_DOUBLE_REGISTER(REGISTER_NAME) >+#undef REGISTER_NAME > }; > return nameForRegister[id]; > } >diff --git a/Source/JavaScriptCore/assembler/ARMv7Registers.h b/Source/JavaScriptCore/assembler/ARMv7Registers.h >new file mode 100644 >index 00000000000..b1f04c4195f >--- /dev/null >+++ b/Source/JavaScriptCore/assembler/ARMv7Registers.h >@@ -0,0 +1,200 @@ >+/* >+ * Copyright (C) 2019 Metrological Group B.V. >+ * Copyright (C) 2019 Igalia S.L. >+ * >+ * Redistribution and use in source and binary forms, with or without >+ * modification, are permitted provided that the following conditions >+ * are met: >+ * 1. Redistributions of source code must retain the above copyright >+ * notice, this list of conditions and the following disclaimer. >+ * 2. Redistributions in binary form must reproduce the above copyright >+ * notice, this list of conditions and the following disclaimer in the >+ * documentation and/or other materials provided with the distribution. >+ * >+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. ``AS IS'' AND ANY >+ * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE >+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR >+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR >+ * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, >+ * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, >+ * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR >+ * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY >+ * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT >+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE >+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. >+ */ >+#pragma once >+ >+#include <wtf/Platform.h> >+ >+#define REGISTER_NAMESPACE ARMRegisters >+#define n(id) id, REGISTER_NAMESPACE::id >+#define nn(id) id, REGISTER_NAMESPACE::id, #id >+ >+#define macro_forward(macro, ...) macro(__VA_ARGS__) >+ >+#define FOR_EACH_REGISTER(macro) \ >+ FOR_EACH_GP_REGISTER(macro) \ >+ FOR_EACH_FP_REGISTER(macro) >+ >+#define FOR_EACH_FP_REGISTER(macro) \ >+ FOR_EACH_FP_DOUBLE_REGISTER(macro) >+ >+#if !PLATFORM(IOS_FAMILY) >+#define FOR_EACH_GP_REGISTER(macro) \ >+ macro_forward(macro, nn(r0), 0, 0) \ >+ macro_forward(macro, nn(r1), 0, 0) \ >+ macro_forward(macro, nn(r2), 0, 0) \ >+ macro_forward(macro, nn(r3), 0, 0) \ >+ macro_forward(macro, nn(r4), 0, 1) \ >+ macro_forward(macro, nn(r5), 0, 1) \ >+ macro_forward(macro, nn(r6), 0, 1) \ >+ macro_forward(macro, nn(r7), 0, 0) \ >+ macro_forward(macro, nn(r8), 0, 1) \ >+ macro_forward(macro, nn(r9), 0, 1) \ >+ macro_forward(macro, nn(r10), 0, 1) \ >+ macro_forward(macro, nn(r11), 0, 1) \ >+ macro_forward(macro, n(r12), "ip", 0, 0) \ >+ macro_forward(macro, n(r13), "sp", 0, 0) \ >+ macro_forward(macro, n(r14), "lr", 1, 0) \ >+ macro_forward(macro, n(r15), "pc", 1, 0) >+#else >+#define FOR_EACH_GP_REGISTER(macro) \ >+ macro_forward(macro, nn(r0), 0, 0) \ >+ macro_forward(macro, nn(r1), 0, 0) \ >+ macro_forward(macro, nn(r2), 0, 0) \ >+ macro_forward(macro, nn(r3), 0, 0) \ >+ macro_forward(macro, nn(r4), 0, 1) \ >+ macro_forward(macro, nn(r5), 0, 1) \ >+ macro_forward(macro, nn(r6), 0, 1) \ >+ macro_forward(macro, nn(r7), 0, 0) \ >+ macro_forward(macro, nn(r8), 0, 1) \ >+ macro_forward(macro, nn(r9), 0, 0) \ >+ macro_forward(macro, nn(r10), 0, 1) \ >+ macro_forward(macro, nn(r11), 0, 1) \ >+ macro_forward(macro, n(r12), "ip", 0, 0) \ >+ macro_forward(macro, n(r13), "sp", 0, 0) \ >+ macro_forward(macro, n(r14), "lr", 1, 0) \ >+ macro_forward(macro, n(r15), "pc", 1, 0) >+#endif >+ >+#define FOR_EACH_REGISTER_ALIAS(macro) \ >+ macro_forward(macro, nn(fp), r7) \ >+ macro_forward(macro, nn(sb), r9) \ >+ macro_forward(macro, nn(sl), r10) \ >+ macro_forward(macro, nn(ip), r12) \ >+ macro_forward(macro, nn(sp), r13) \ >+ macro_forward(macro, nn(lr), r14) \ >+ macro_forward(macro, nn(pc), r15) >+ >+#define FOR_EACH_SP_REGISTER(macro) \ >+ macro_forward(macro, nn(apsr)) \ >+ macro_forward(macro, nn(fpscr)) >+ >+#define FOR_EACH_FP_SINGLE_REGISTER(macro) \ >+ macro_forward(macro, nn(s0), 0, 0) \ >+ macro_forward(macro, nn(s1), 0, 0) \ >+ macro_forward(macro, nn(s2), 0, 0) \ >+ macro_forward(macro, nn(s3), 0, 0) \ >+ macro_forward(macro, nn(s4), 0, 0) \ >+ macro_forward(macro, nn(s5), 0, 0) \ >+ macro_forward(macro, nn(s6), 0, 0) \ >+ macro_forward(macro, nn(s7), 0, 0) \ >+ macro_forward(macro, nn(s8), 0, 0) \ >+ macro_forward(macro, nn(s9), 0, 0) \ >+ macro_forward(macro, nn(s10), 0, 0) \ >+ macro_forward(macro, nn(s11), 0, 0) \ >+ macro_forward(macro, nn(s12), 0, 0) \ >+ macro_forward(macro, nn(s13), 0, 0) \ >+ macro_forward(macro, nn(s14), 0, 0) \ >+ macro_forward(macro, nn(s15), 0, 0) \ >+ macro_forward(macro, nn(s16), 0, 0) \ >+ macro_forward(macro, nn(s17), 0, 0) \ >+ macro_forward(macro, nn(s18), 0, 0) \ >+ macro_forward(macro, nn(s19), 0, 0) \ >+ macro_forward(macro, nn(s20), 0, 0) \ >+ macro_forward(macro, nn(s21), 0, 0) \ >+ macro_forward(macro, nn(s22), 0, 0) \ >+ macro_forward(macro, nn(s23), 0, 0) \ >+ macro_forward(macro, nn(s24), 0, 0) \ >+ macro_forward(macro, nn(s25), 0, 0) \ >+ macro_forward(macro, nn(s26), 0, 0) \ >+ macro_forward(macro, nn(s27), 0, 0) \ >+ macro_forward(macro, nn(s28), 0, 0) \ >+ macro_forward(macro, nn(s29), 0, 0) \ >+ macro_forward(macro, nn(s30), 0, 0) \ >+ macro_forward(macro, nn(s31), 0, 0) >+ >+#if CPU(ARM_NEON) || CPU(ARM_VFP_V3_D32) >+#define FOR_EACH_FP_DOUBLE_REGISTER(macro) \ >+ macro_forward(macro, nn(d0), 0, 0) \ >+ macro_forward(macro, nn(d1), 0, 0) \ >+ macro_forward(macro, nn(d2), 0, 0) \ >+ macro_forward(macro, nn(d3), 0, 0) \ >+ macro_forward(macro, nn(d4), 0, 0) \ >+ macro_forward(macro, nn(d5), 0, 0) \ >+ macro_forward(macro, nn(d6), 0, 0) \ >+ macro_forward(macro, nn(d7), 0, 0) \ >+ macro_forward(macro, nn(d8), 0, 0) \ >+ macro_forward(macro, nn(d9), 0, 0) \ >+ macro_forward(macro, nn(d10), 0, 0) \ >+ macro_forward(macro, nn(d11), 0, 0) \ >+ macro_forward(macro, nn(d12), 0, 0) \ >+ macro_forward(macro, nn(d13), 0, 0) \ >+ macro_forward(macro, nn(d14), 0, 0) \ >+ macro_forward(macro, nn(d15), 0, 0) \ >+ macro_forward(macro, nn(d16), 0, 0) \ >+ macro_forward(macro, nn(d17), 0, 0) \ >+ macro_forward(macro, nn(d18), 0, 0) \ >+ macro_forward(macro, nn(d19), 0, 0) \ >+ macro_forward(macro, nn(d20), 0, 0) \ >+ macro_forward(macro, nn(d21), 0, 0) \ >+ macro_forward(macro, nn(d22), 0, 0) \ >+ macro_forward(macro, nn(d23), 0, 0) \ >+ macro_forward(macro, nn(d24), 0, 0) \ >+ macro_forward(macro, nn(d25), 0, 0) \ >+ macro_forward(macro, nn(d26), 0, 0) \ >+ macro_forward(macro, nn(d27), 0, 0) \ >+ macro_forward(macro, nn(d28), 0, 0) \ >+ macro_forward(macro, nn(d29), 0, 0) \ >+ macro_forward(macro, nn(d30), 0, 0) \ >+ macro_forward(macro, nn(d31), 0, 0) >+#else >+#define FOR_EACH_FP_DOUBLE_REGISTER(macro) \ >+ macro_forward(macro, nn(d0), 0, 0) \ >+ macro_forward(macro, nn(d1), 0, 0) \ >+ macro_forward(macro, nn(d2), 0, 0) \ >+ macro_forward(macro, nn(d3), 0, 0) \ >+ macro_forward(macro, nn(d4), 0, 0) \ >+ macro_forward(macro, nn(d5), 0, 0) \ >+ macro_forward(macro, nn(d6), 0, 0) \ >+ macro_forward(macro, nn(d7), 0, 0) \ >+ macro_forward(macro, nn(d8), 0, 0) \ >+ macro_forward(macro, nn(d9), 0, 0) \ >+ macro_forward(macro, nn(d10), 0, 0) \ >+ macro_forward(macro, nn(d11), 0, 0) \ >+ macro_forward(macro, nn(d12), 0, 0) \ >+ macro_forward(macro, nn(d13), 0, 0) \ >+ macro_forward(macro, nn(d14), 0, 0) \ >+ macro_forward(macro, nn(d15), 0, 0) >+#endif >+ >+#if CPU(ARM_NEON) >+#define FOR_EACH_FP_QUAD_REGISTER(macro) \ >+ macro_forward(macro, nn(q0), 0, 0) \ >+ macro_forward(macro, nn(q1), 0, 0) \ >+ macro_forward(macro, nn(q2), 0, 0) \ >+ macro_forward(macro, nn(q3), 0, 0) \ >+ macro_forward(macro, nn(q4), 0, 0) \ >+ macro_forward(macro, nn(q5), 0, 0) \ >+ macro_forward(macro, nn(q6), 0, 0) \ >+ macro_forward(macro, nn(q7), 0, 0) \ >+ macro_forward(macro, nn(q8), 0, 0) \ >+ macro_forward(macro, nn(q9), 0, 0) \ >+ macro_forward(macro, nn(q10), 0, 0) \ >+ macro_forward(macro, nn(q11), 0, 0) \ >+ macro_forward(macro, nn(q12), 0, 0) \ >+ macro_forward(macro, nn(q13), 0, 0) \ >+ macro_forward(macro, nn(q14), 0, 0) \ >+ macro_forward(macro, nn(q15), 0, 0) >+#endif >diff --git a/Source/JavaScriptCore/assembler/MIPSAssembler.h b/Source/JavaScriptCore/assembler/MIPSAssembler.h >index 7e8e213dfce..ec43f289656 100644 >--- a/Source/JavaScriptCore/assembler/MIPSAssembler.h >+++ b/Source/JavaScriptCore/assembler/MIPSAssembler.h >@@ -32,6 +32,7 @@ > > #include "AssemblerBuffer.h" > #include "JITCompilationEffort.h" >+#include "MIPSRegisters.h" > #include <limits.h> > #include <wtf/Assertions.h> > #include <wtf/SegmentedVector.h> >@@ -40,117 +41,29 @@ namespace JSC { > > typedef uint32_t MIPSWord; > >+// If modifying the namespace name, please be sure to modify it as well >+// in MIPSRegisters.h on the macro definition of REGISTER_NAMESPACE > namespace MIPSRegisters { > typedef enum : int8_t { >- r0 = 0, >- r1, >- r2, >- r3, >- r4, >- r5, >- r6, >- r7, >- r8, >- r9, >- r10, >- r11, >- r12, >- r13, >- r14, >- r15, >- r16, >- r17, >- r18, >- r19, >- r20, >- r21, >- r22, >- r23, >- r24, >- r25, >- r26, >- r27, >- r28, >- r29, >- r30, >- r31, >- zero = r0, >- at = r1, >- v0 = r2, >- v1 = r3, >- a0 = r4, >- a1 = r5, >- a2 = r6, >- a3 = r7, >- t0 = r8, >- t1 = r9, >- t2 = r10, >- t3 = r11, >- t4 = r12, >- t5 = r13, >- t6 = r14, >- t7 = r15, >- s0 = r16, >- s1 = r17, >- s2 = r18, >- s3 = r19, >- s4 = r20, >- s5 = r21, >- s6 = r22, >- s7 = r23, >- t8 = r24, >- t9 = r25, >- k0 = r26, >- k1 = r27, >- gp = r28, >- sp = r29, >- fp = r30, >- ra = r31, >+#define REGISTER_ID(id, nsid, name, r, cs) id, >+ FOR_EACH_GP_REGISTER(REGISTER_ID) >+#undef REGISTER_ID >+#define REGISTER_ALIAS(id, alias) id = alias, >+ FOR_EACH_REGISTER_ALIAS(REGISTER_ALIAS) >+#undef REGISTER_ALIAS > InvalidGPRReg = -1, > } RegisterID; > > typedef enum : int8_t { >- fir = 0, >- fccr = 25, >- fexr = 26, >- fenr = 28, >- fcsr = 31, >- pc >+#define REGISTER_ID(id, nsid, name, idx) id = idx, >+ FOR_EACH_SP_REGISTER(REGISTER_ID) >+#undef REGISTER_ID > } SPRegisterID; > > typedef enum : int8_t { >- f0, >- f1, >- f2, >- f3, >- f4, >- f5, >- f6, >- f7, >- f8, >- f9, >- f10, >- f11, >- f12, >- f13, >- f14, >- f15, >- f16, >- f17, >- f18, >- f19, >- f20, >- f21, >- f22, >- f23, >- f24, >- f25, >- f26, >- f27, >- f28, >- f29, >- f30, >- f31, >+#define REGISTER_ID(id, ndsid, name, r, cs) id, >+ FOR_EACH_FP_REGISTER(REGISTER_ID) >+#undef REGISTER_ID > InvalidFPRReg = -1, > } FPRegisterID; > >@@ -179,46 +92,31 @@ public: > { > ASSERT(id >= firstRegister() && id <= lastRegister()); > static const char* const nameForRegister[numberOfRegisters()] = { >- "zero", "at", "v0", "v1", >- "a0", "a1", "a2", "a3", >- "t0", "t1", "t2", "t3", >- "t4", "t5", "t6", "t7" >+#define REGISTER_NAME(id, nsid, name, r, c) name, >+ FOR_EACH_GP_REGISTER(REGISTER_NAME) >+#undef REGISTER_NAME > }; > return nameForRegister[id]; > } > > static const char* sprName(SPRegisterID id) > { >- switch (id) { >- case MIPSRegisters::fir: >- return "fir"; >- case MIPSRegisters::fccr: >- return "fccr"; >- case MIPSRegisters::fexr: >- return "fexr"; >- case MIPSRegisters::fenr: >- return "fenr"; >- case MIPSRegisters::fcsr: >- return "fcsr"; >- case MIPSRegisters::pc: >- return "pc"; >- default: >- RELEASE_ASSERT_NOT_REACHED(); >- } >+ ASSERT(id >= firstSPRegister() && id <= lastSPRegister()); >+ static const char* const nameForRegister[numberOfSPRegisters()] = { >+#define REGISTER_NAME(id, nsid, name, idx) name, >+ FOR_EACH_SP_REGISTER(REGISTER_NAME) >+#undef REGISTER_NAME >+ }; >+ return nameForRegister[id]; > } > > static const char* fprName(FPRegisterID id) > { > ASSERT(id >= firstFPRegister() && id <= lastFPRegister()); > static const char* const nameForRegister[numberOfFPRegisters()] = { >- "f0", "f1", "f2", "f3", >- "f4", "f5", "f6", "f7", >- "f8", "f9", "f10", "f11", >- "f12", "f13", "f14", "f15" >- "f16", "f17", "f18", "f19" >- "f20", "f21", "f22", "f23" >- "f24", "f25", "f26", "f27" >- "f28", "f29", "f30", "f31" >+#define REGISTER_NAME(id, nsid, name, r, cs) name, >+ FOR_EACH_FP_REGISTER(REGISTER_NAME) >+#undef REGISTER_NAME > }; > return nameForRegister[id]; > } >diff --git a/Source/JavaScriptCore/assembler/MIPSRegisters.h b/Source/JavaScriptCore/assembler/MIPSRegisters.h >new file mode 100644 >index 00000000000..a64a39ae2ed >--- /dev/null >+++ b/Source/JavaScriptCore/assembler/MIPSRegisters.h >@@ -0,0 +1,147 @@ >+/* >+ * Copyright (C) 2019 Metrological Group B.V. >+ * Copyright (C) 2019 Igalia S.L. >+ * >+ * Redistribution and use in source and binary forms, with or without >+ * modification, are permitted provided that the following conditions >+ * are met: >+ * 1. Redistributions of source code must retain the above copyright >+ * notice, this list of conditions and the following disclaimer. >+ * 2. Redistributions in binary form must reproduce the above copyright >+ * notice, this list of conditions and the following disclaimer in the >+ * documentation and/or other materials provided with the distribution. >+ * >+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. ``AS IS'' AND ANY >+ * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE >+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR >+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR >+ * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, >+ * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, >+ * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR >+ * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY >+ * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT >+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE >+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. >+ */ >+ >+#pragma once >+ >+#define REGISTER_NAMESPACE MIPSRegisters >+#define n(id) id, REGISTER_NAMESPACE::id >+#define nn(id) id, REGISTER_NAMESPACE::id, #id >+ >+#define FOR_EACH_REGISTER(macro) \ >+ FOR_EACH_GP_REGISTER(macro) \ >+ FOR_EACH_FP_REGISTER(macro) >+ >+#define macro_forward(macro, ...) macro(__VA_ARGS__) >+ >+#define FOR_EACH_GP_REGISTER(macro) \ >+ macro_forward(macro, n(r0), "zero", 0, 0) \ >+ macro_forward(macro, n(r1), "at", 0, 0) \ >+ macro_forward(macro, n(r2), "v0", 0, 0) \ >+ macro_forward(macro, n(r3), "v1", 0, 0) \ >+ macro_forward(macro, n(r4), "a0", 0, 0) \ >+ macro_forward(macro, n(r5), "a1", 0, 0) \ >+ macro_forward(macro, n(r6), "a2", 0, 0) \ >+ macro_forward(macro, n(r7), "a3", 0, 0) \ >+ macro_forward(macro, n(r8), "t0", 0, 0) \ >+ macro_forward(macro, n(r9), "t1", 0, 0) \ >+ macro_forward(macro, n(r10), "t2", 0, 0) \ >+ macro_forward(macro, n(r11), "t3", 0, 0) \ >+ macro_forward(macro, n(r12), "t4", 0, 0) \ >+ macro_forward(macro, n(r13), "t5", 0, 0) \ >+ macro_forward(macro, n(r14), "t6", 0, 0) \ >+ macro_forward(macro, n(r15), "t7", 0, 0) \ >+ macro_forward(macro, n(r16), "s0", 0, 1) \ >+ macro_forward(macro, n(r17), "s1", 0, 0) \ >+ macro_forward(macro, n(r18), "s2", 0, 0) \ >+ macro_forward(macro, n(r19), "s3", 0, 0) \ >+ macro_forward(macro, n(r20), "s4", 0, 0) \ >+ macro_forward(macro, n(r21), "s5", 0, 0) \ >+ macro_forward(macro, n(r22), "s6", 0, 0) \ >+ macro_forward(macro, n(r23), "s7", 0, 0) \ >+ macro_forward(macro, n(r24), "t8", 0, 0) \ >+ macro_forward(macro, n(r25), "t9", 0, 0) \ >+ macro_forward(macro, n(r26), "k0", 0, 0) \ >+ macro_forward(macro, n(r27), "k1", 0, 0) \ >+ macro_forward(macro, n(r28), "gp", 0, 0) \ >+ macro_forward(macro, n(r29), "sp", 0, 0) \ >+ macro_forward(macro, n(r30), "fp", 0, 0) \ >+ macro_forward(macro, n(r31), "ra", 0, 0) >+ >+#define FOR_EACH_REGISTER_ALIAS(macro) \ >+ macro(zero, r0) \ >+ macro(at, r1) \ >+ macro(v0, r2) \ >+ macro(v1, r3) \ >+ macro(a0, r4) \ >+ macro(a1, r5) \ >+ macro(a2, r6) \ >+ macro(a3, r7) \ >+ macro(t0, r8) \ >+ macro(t1, r9) \ >+ macro(t2, r10) \ >+ macro(t3, r11) \ >+ macro(t4, r12) \ >+ macro(t5, r13) \ >+ macro(t6, r14) \ >+ macro(t7, r15) \ >+ macro(s0, r16) \ >+ macro(s1, r17) \ >+ macro(s2, r18) \ >+ macro(s3, r19) \ >+ macro(s4, r20) \ >+ macro(s5, r21) \ >+ macro(s6, r22) \ >+ macro(s7, r23) \ >+ macro(t8, r24) \ >+ macro(t9, r25) \ >+ macro(k0, r26) \ >+ macro(k1, r27) \ >+ macro(gp, r28) \ >+ macro(sp, r29) \ >+ macro(fp, r30) \ >+ macro(ra, r31) >+ >+#define FOR_EACH_SP_REGISTER(macro) \ >+ macro_forward(macro, nn(fir), 0) \ >+ macro_forward(macro, nn(fccr), 25) \ >+ macro_forward(macro, nn(fexr), 26) \ >+ macro_forward(macro, nn(fenr), 28) \ >+ macro_forward(macro, nn(fcsr), 31) \ >+ macro_forward(macro, nn(pc), 32) >+ >+#define FOR_EACH_FP_REGISTER(macro) \ >+ macro_forward(macro, nn(f0), 0, 0) \ >+ macro_forward(macro, nn(f1), 0, 0) \ >+ macro_forward(macro, nn(f2), 0, 0) \ >+ macro_forward(macro, nn(f3), 0, 0) \ >+ macro_forward(macro, nn(f4), 0, 0) \ >+ macro_forward(macro, nn(f5), 0, 0) \ >+ macro_forward(macro, nn(f6), 0, 0) \ >+ macro_forward(macro, nn(f7), 0, 0) \ >+ macro_forward(macro, nn(f8), 0, 0) \ >+ macro_forward(macro, nn(f9), 0, 0) \ >+ macro_forward(macro, nn(f10), 0, 0) \ >+ macro_forward(macro, nn(f11), 0, 0) \ >+ macro_forward(macro, nn(f12), 0, 0) \ >+ macro_forward(macro, nn(f13), 0, 0) \ >+ macro_forward(macro, nn(f14), 0, 0) \ >+ macro_forward(macro, nn(f15), 0, 0) \ >+ macro_forward(macro, nn(f16), 0, 0) \ >+ macro_forward(macro, nn(f17), 0, 0) \ >+ macro_forward(macro, nn(f18), 0, 0) \ >+ macro_forward(macro, nn(f19), 0, 0) \ >+ macro_forward(macro, nn(f20), 0, 0) \ >+ macro_forward(macro, nn(f21), 0, 0) \ >+ macro_forward(macro, nn(f22), 0, 0) \ >+ macro_forward(macro, nn(f23), 0, 0) \ >+ macro_forward(macro, nn(f24), 0, 0) \ >+ macro_forward(macro, nn(f25), 0, 0) \ >+ macro_forward(macro, nn(f26), 0, 0) \ >+ macro_forward(macro, nn(f27), 0, 0) \ >+ macro_forward(macro, nn(f28), 0, 0) \ >+ macro_forward(macro, nn(f29), 0, 0) \ >+ macro_forward(macro, nn(f30), 0, 0) \ >+ macro_forward(macro, nn(f31), 0, 0) >diff --git a/Source/JavaScriptCore/assembler/RegisterInfo.h b/Source/JavaScriptCore/assembler/RegisterInfo.h >new file mode 100644 >index 00000000000..a44c4c2b7bd >--- /dev/null >+++ b/Source/JavaScriptCore/assembler/RegisterInfo.h >@@ -0,0 +1,63 @@ >+/* >+ * Copyright (C) 2019 Metrological Group B.V. >+ * Copyright (C) 2019 Igalia S.L. >+ * >+ * Redistribution and use in source and binary forms, with or without >+ * modification, are permitted provided that the following conditions >+ * are met: >+ * 1. Redistributions of source code must retain the above copyright >+ * notice, this list of conditions and the following disclaimer. >+ * 2. Redistributions in binary form must reproduce the above copyright >+ * notice, this list of conditions and the following disclaimer in the >+ * documentation and/or other materials provided with the distribution. >+ * >+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. ``AS IS'' AND ANY >+ * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE >+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR >+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR >+ * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, >+ * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, >+ * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR >+ * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY >+ * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT >+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE >+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. >+ */ >+ >+#pragma once >+ >+#include <wtf/Assertions.h> >+ >+/* This file serves as the platform independent redirection header for >+ * platform dependent register information. Each architecture has its own header. >+ * >+ * Each header defines a few important macros that are used in a platform independent >+ * way - see for example jit/RegisterSet.cpp. >+ * - FOR_EACH_GP_REGISTER which lists all available general purpose registers. >+ * - FOR_EACH_FP_REGISTER which lists all available floating point registers. >+ * these take themselves a macro that can filter through the available information >+ * spread accross the five macro arguments. >+ * = 1. id: is an identifier used to specify the register (for example, as an enumerator >+ * in an enum); >+ * = 2. nsid: is an identifier specifying the full namespace identifier literal for id; >+ * = 3. name: is a string constant specifying the name of the identifier; >+ * = 4. isReserved: a boolean (usually 0/1) specifying if this is a reserved register; >+ * = 5. isCalleeSaved: a boolean (usually 0/1) specifying if this is a callee saved register; >+ * >+ * - A few other platform dependent macros can be specified to be used in platform >+ * dependent files (for example assembler X86Assembler.h). >+ */ >+ >+#if CPU(X86) >+# include "X86Registers.h" >+#elif CPU(X86_64) >+# include "X86_64Registers.h" >+#elif CPU(MIPS) >+# include "MIPSRegisters.h" >+#elif CPU(ARM_THUMB2) >+# include "ARMv7Registers.h" >+#elif CPU(ARM64) >+# include "ARM64Registers.h" >+#else >+ UNREACHABLE_FOR_PLATFORM(); >+#endif >diff --git a/Source/JavaScriptCore/assembler/X86Assembler.h b/Source/JavaScriptCore/assembler/X86Assembler.h >index 06cb9e171af..5a6d699f3a5 100644 >--- a/Source/JavaScriptCore/assembler/X86Assembler.h >+++ b/Source/JavaScriptCore/assembler/X86Assembler.h >@@ -30,6 +30,7 @@ > #include "AssemblerBuffer.h" > #include "AssemblerCommon.h" > #include "JITCompilationEffort.h" >+#include "RegisterInfo.h" > #include <limits.h> > #include <stdint.h> > #include <wtf/Assertions.h> >@@ -39,6 +40,8 @@ namespace JSC { > > inline bool CAN_SIGN_EXTEND_8_32(int32_t value) { return value == (int32_t)(signed char)value; } > >+// If modifying the namespace name, please be sure to modify it as well >+// in X86Registers.h and X86_64Registers.h on the macro definition of REGISTER_NAMESPACE > namespace X86Registers { > > #if COMPILER(MSVC) >@@ -47,56 +50,25 @@ namespace X86Registers { > #define JSC_X86_ASM_REGISTER_ID_ENUM_BASE_TYPE : int8_t > #endif > >+#define REGISTER_ID(id, nsid, name, res, cs) id, >+ > typedef enum JSC_X86_ASM_REGISTER_ID_ENUM_BASE_TYPE { >- eax, >- ecx, >- edx, >- ebx, >- esp, >- ebp, >- esi, >- edi, >-#if CPU(X86_64) >- r8, >- r9, >- r10, >- r11, >- r12, >- r13, >- r14, >- r15, >-#endif >+ FOR_EACH_GP_REGISTER(REGISTER_ID) > InvalidGPRReg = -1, > } RegisterID; > > typedef enum JSC_X86_ASM_REGISTER_ID_ENUM_BASE_TYPE { >- eip, >- eflags >+ FOR_EACH_SP_REGISTER(REGISTER_ID) > } SPRegisterID; > > typedef enum JSC_X86_ASM_REGISTER_ID_ENUM_BASE_TYPE { >- xmm0, >- xmm1, >- xmm2, >- xmm3, >- xmm4, >- xmm5, >- xmm6, >- xmm7, >-#if CPU(X86_64) >- xmm8, >- xmm9, >- xmm10, >- xmm11, >- xmm12, >- xmm13, >- xmm14, >- xmm15, >-#endif >+ FOR_EACH_FP_REGISTER(REGISTER_ID) > InvalidFPRReg = -1, > } XMMRegisterID; > >-} // namespace X86Register >+#undef REGISTER_ID >+ >+} // namespace X86Registers > > class X86Assembler { > public: >@@ -137,15 +109,9 @@ public: > { > ASSERT(id >= firstRegister() && id <= lastRegister()); > static const char* const nameForRegister[numberOfRegisters()] = { >-#if CPU(X86_64) >- "rax", "rcx", "rdx", "rbx", >- "rsp", "rbp", "rsi", "rdi", >- "r8", "r9", "r10", "r11", >- "r12", "r13", "r14", "r15" >-#else >- "eax", "ecx", "edx", "ebx", >- "esp", "ebp", "esi", "edi", >-#endif >+#define REGISTER_NAME(id, nsid, name, res, cs) name, >+ FOR_EACH_GP_REGISTER(REGISTER_NAME) >+#undef REGISTER_NAME > }; > return nameForRegister[id]; > } >@@ -154,11 +120,9 @@ public: > { > ASSERT(id >= firstSPRegister() && id <= lastSPRegister()); > static const char* const nameForRegister[numberOfSPRegisters()] = { >-#if CPU(X86_64) >- "rip", "rflags" >-#else >- "eip", "eflags" >-#endif >+#define REGISTER_NAME(id, nsid, name, res, cs) name, >+ FOR_EACH_SP_REGISTER(REGISTER_NAME) >+#undef REGISTER_NAME > }; > return nameForRegister[id]; > } >@@ -167,12 +131,9 @@ public: > { > ASSERT(reg >= firstFPRegister() && reg <= lastFPRegister()); > static const char* const nameForRegister[numberOfFPRegisters()] = { >- "xmm0", "xmm1", "xmm2", "xmm3", >- "xmm4", "xmm5", "xmm6", "xmm7", >-#if CPU(X86_64) >- "xmm8", "xmm9", "xmm10", "xmm11", >- "xmm12", "xmm13", "xmm14", "xmm15" >-#endif >+#define REGISTER_NAME(id, nsid, name, res, cs) name, >+ FOR_EACH_FP_REGISTER(REGISTER_NAME) >+#undef REGISTER_NAME > }; > return nameForRegister[reg]; > } >diff --git a/Source/JavaScriptCore/assembler/X86Registers.h b/Source/JavaScriptCore/assembler/X86Registers.h >new file mode 100644 >index 00000000000..7cd88d81585 >--- /dev/null >+++ b/Source/JavaScriptCore/assembler/X86Registers.h >@@ -0,0 +1,80 @@ >+/* >+ * Copyright (C) 2019 Metrological Group B.V. >+ * Copyright (C) 2019 Igalia S.L. >+ * >+ * Redistribution and use in source and binary forms, with or without >+ * modification, are permitted provided that the following conditions >+ * are met: >+ * 1. Redistributions of source code must retain the above copyright >+ * notice, this list of conditions and the following disclaimer. >+ * 2. Redistributions in binary form must reproduce the above copyright >+ * notice, this list of conditions and the following disclaimer in the >+ * documentation and/or other materials provided with the distribution. >+ * >+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. ``AS IS'' AND ANY >+ * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE >+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR >+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR >+ * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, >+ * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, >+ * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR >+ * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY >+ * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT >+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE >+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. >+ */ >+ >+#pragma once >+ >+#include <wtf/Platform.h> >+ >+#define REGISTER_NAMESPACE X86Registers >+#define n(id) id, REGISTER_NAMESPACE::id >+#define nn(id) id, REGISTER_NAMESPACE::id, #id >+ >+#define FOR_EACH_REGISTER(macro) \ >+ FOR_EACH_GP_REGISTER(macro) \ >+ FOR_EACH_FP_REGISTER(macro) >+ >+#define macro_forward(macro, ...) macro(__VA_ARGS__) >+ >+#if !OS(WINDOWS) >+ >+#define FOR_EACH_GP_REGISTER(macro) \ >+ macro_forward(macro, n(eax), "rax", 0, 0) \ >+ macro_forward(macro, n(ecx), "rcx", 0, 0) \ >+ macro_forward(macro, n(edx), "rdx", 0, 0) \ >+ macro_forward(macro, n(ebx), "rbx", 0, 1) \ >+ macro_forward(macro, n(esp), "rsp", 0, 0) \ >+ macro_forward(macro, n(ebp), "rbp", 0, 1) \ >+ macro_forward(macro, n(esi), "rsi", 0, 0) \ >+ macro_forward(macro, n(edi), "rdi", 0, 0) >+ >+#else // OS(WINDOWS) >+ >+#define FOR_EACH_GP_REGISTER(macro) \ >+ macro_forward(macro, n(eax), "rax", 0, 0) \ >+ macro_forward(macro, n(ecx), "rcx", 0, 0) \ >+ macro_forward(macro, n(edx), "rdx", 0, 0) \ >+ macro_forward(macro, n(ebx), "rbx", 0, 1) \ >+ macro_forward(macro, n(esp), "rsp", 0, 0) \ >+ macro_forward(macro, n(ebp), "rbp", 0, 1) \ >+ macro_forward(macro, n(esi), "rsi", 0, 1) \ >+ macro_forward(macro, n(edi), "rdi", 0, 1) >+ >+#endif // !OS(WINDOWS) >+ >+#define FOR_EACH_FP_REGISTER(macro) \ >+ macro_forward(macro, nn(xmm0), 0, 0) \ >+ macro_forward(macro, nn(xmm1), 0, 0) \ >+ macro_forward(macro, nn(xmm2), 0, 0) \ >+ macro_forward(macro, nn(xmm3), 0, 0) \ >+ macro_forward(macro, nn(xmm4), 0, 0) \ >+ macro_forward(macro, nn(xmm5), 0, 0) \ >+ macro_forward(macro, nn(xmm6), 0, 0) \ >+ macro_forward(macro, nn(xmm7), 0, 0) >+ >+#define FOR_EACH_SP_REGISTER(macro) \ >+ macro_forward(macro, nn(eip), 0, 0) \ >+ macro_forward(macro, nn(eflags), 0, 0) >+ >diff --git a/Source/JavaScriptCore/assembler/X86_64Registers.h b/Source/JavaScriptCore/assembler/X86_64Registers.h >new file mode 100644 >index 00000000000..aaf3d9c49d3 >--- /dev/null >+++ b/Source/JavaScriptCore/assembler/X86_64Registers.h >@@ -0,0 +1,107 @@ >+/* >+ * Copyright (C) 2019 Metrological Group B.V. >+ * Copyright (C) 2019 Igalia S.L. >+ * >+ * Redistribution and use in source and binary forms, with or without >+ * modification, are permitted provided that the following conditions >+ * are met: >+ * 1. Redistributions of source code must retain the above copyright >+ * notice, this list of conditions and the following disclaimer. >+ * 2. Redistributions in binary form must reproduce the above copyright >+ * notice, this list of conditions and the following disclaimer in the >+ * documentation and/or other materials provided with the distribution. >+ * >+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. ``AS IS'' AND ANY >+ * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE >+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR >+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR >+ * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, >+ * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, >+ * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR >+ * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY >+ * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT >+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE >+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. >+ */ >+ >+#pragma once >+ >+#include <wtf/Compiler.h> >+#include <wtf/Platform.h> >+ >+#define REGISTER_NAMESPACE X86Registers >+#define n(id) id, REGISTER_NAMESPACE::id >+#define nn(id) id, REGISTER_NAMESPACE::id, #id >+ >+/* We require an exception for MSVC due to the way it's currently handling __VA_ARGS__ >+ See https://bugs.webkit.org/show_bug.cgi?id=198778 */ >+#if COMPILER(MSVC) >+#define EXPAND_ARGS(x) x >+#define macro_forward(macro, ...) EXPAND_ARGS(macro(__VA_ARGS__)) >+#else >+#define macro_forward(macro, ...) macro(__VA_ARGS__) >+#endif >+ >+#if !OS(WINDOWS) >+ >+#define FOR_EACH_GP_REGISTER(macro) \ >+ macro_forward(macro, n(eax), "rax", 0, 0) \ >+ macro_forward(macro, n(ecx), "rcx", 0, 0) \ >+ macro_forward(macro, n(edx), "rdx", 0, 0) \ >+ macro_forward(macro, n(ebx), "rbx", 0, 1) \ >+ macro_forward(macro, n(esp), "rsp", 0, 0) \ >+ macro_forward(macro, n(ebp), "rbp", 0, 1) \ >+ macro_forward(macro, n(esi), "rsi", 0, 0) \ >+ macro_forward(macro, n(edi), "rdi", 0, 0) \ >+ macro_forward(macro, nn(r8), 0, 0) \ >+ macro_forward(macro, nn(r9), 0, 0) \ >+ macro_forward(macro, nn(r10), 0, 0) \ >+ macro_forward(macro, nn(r11), 0, 0) \ >+ macro_forward(macro, nn(r12), 0, 1) \ >+ macro_forward(macro, nn(r13), 0, 1) \ >+ macro_forward(macro, nn(r14), 0, 1) \ >+ macro_forward(macro, nn(r15), 0, 1) >+ >+#else // OS(WINDOWS) >+ >+#define FOR_EACH_GP_REGISTER(macro) \ >+ macro_forward(macro, n(eax), "rax", 0, 0) \ >+ macro_forward(macro, n(ecx), "rcx", 0, 0) \ >+ macro_forward(macro, n(edx), "rdx", 0, 0) \ >+ macro_forward(macro, n(ebx), "rbx", 0, 1) \ >+ macro_forward(macro, n(esp), "rsp", 0, 0) \ >+ macro_forward(macro, n(ebp), "rbp", 0, 1) \ >+ macro_forward(macro, n(esi), "rsi", 0, 1) \ >+ macro_forward(macro, n(edi), "rdi", 0, 1) \ >+ macro_forward(macro, nn(r8), 0, 0) \ >+ macro_forward(macro, nn(r9), 0, 0) \ >+ macro_forward(macro, nn(r10), 0, 0) \ >+ macro_forward(macro, nn(r11), 0, 0) \ >+ macro_forward(macro, nn(r12), 0, 1) \ >+ macro_forward(macro, nn(r13), 0, 1) \ >+ macro_forward(macro, nn(r14), 0, 1) \ >+ macro_forward(macro, nn(r15), 0, 1) >+ >+#endif // !OS(WINDOWS) >+ >+#define FOR_EACH_FP_REGISTER(macro) \ >+ macro_forward(macro, nn(xmm0), 0, 0) \ >+ macro_forward(macro, nn(xmm1), 0, 0) \ >+ macro_forward(macro, nn(xmm2), 0, 0) \ >+ macro_forward(macro, nn(xmm3), 0, 0) \ >+ macro_forward(macro, nn(xmm4), 0, 0) \ >+ macro_forward(macro, nn(xmm5), 0, 0) \ >+ macro_forward(macro, nn(xmm6), 0, 0) \ >+ macro_forward(macro, nn(xmm7), 0, 0) \ >+ macro_forward(macro, nn(xmm8), 0, 0) \ >+ macro_forward(macro, nn(xmm9), 0, 0) \ >+ macro_forward(macro, nn(xmm10), 0, 0) \ >+ macro_forward(macro, nn(xmm11), 0, 0) \ >+ macro_forward(macro, nn(xmm12), 0, 0) \ >+ macro_forward(macro, nn(xmm13), 0, 0) \ >+ macro_forward(macro, nn(xmm14), 0, 0) \ >+ macro_forward(macro, nn(xmm15), 0, 0) >+ >+#define FOR_EACH_SP_REGISTER(macro) \ >+ macro_forward(macro, nn(eip), 0, 0) \ >+ macro_forward(macro, nn(eflags), 0, 0) >diff --git a/Source/JavaScriptCore/jit/GPRInfo.h b/Source/JavaScriptCore/jit/GPRInfo.h >index 6bcf676f3c5..a3f87554579 100644 >--- a/Source/JavaScriptCore/jit/GPRInfo.h >+++ b/Source/JavaScriptCore/jit/GPRInfo.h >@@ -33,7 +33,7 @@ namespace JSC { > > enum NoResultTag { NoResult }; > >-// We use the same conventions in the basline JIT as in the LLint. If you >+// We use the same conventions in the baseline JIT as in the LLint. If you > // change mappings in the GPRInfo, you should change them in the offlineasm > // compiler adequately. The register naming conventions are described at the > // top of the LowLevelInterpreter.asm file. >diff --git a/Source/JavaScriptCore/jit/RegisterSet.cpp b/Source/JavaScriptCore/jit/RegisterSet.cpp >index ed294cb2d9c..c4026c43466 100644 >--- a/Source/JavaScriptCore/jit/RegisterSet.cpp >+++ b/Source/JavaScriptCore/jit/RegisterSet.cpp >@@ -32,6 +32,7 @@ > #include "JSCInlines.h" > #include "MacroAssembler.h" > #include "RegisterAtOffsetList.h" >+#include "assembler/RegisterInfo.h" > #include <wtf/CommaPrinter.h> > > namespace JSC { >@@ -45,17 +46,14 @@ RegisterSet RegisterSet::stackRegisters() > > RegisterSet RegisterSet::reservedHardwareRegisters() > { >-#if CPU(ARM64) >-#if PLATFORM(IOS_FAMILY) >- return RegisterSet(ARM64Registers::x18, ARM64Registers::lr); >-#else >- return RegisterSet(ARM64Registers::lr); >-#endif // PLATFORM(IOS_FAMILY) >-#elif CPU(ARM_THUMB2) >- return RegisterSet(ARMRegisters::lr, ARMRegisters::pc); >-#else >- return { }; >-#endif >+ RegisterSet result; >+ >+#define SET_IF_RESERVED(id, nsid, name, isReserved, isCalleeSaved) if (isReserved) result.set(nsid); >+ FOR_EACH_GP_REGISTER(SET_IF_RESERVED) >+ FOR_EACH_FP_REGISTER(SET_IF_RESERVED) >+#undef SET_IF_RESERVED >+ >+ return result; > } > > RegisterSet RegisterSet::runtimeTagRegisters() >@@ -111,53 +109,14 @@ RegisterSet RegisterSet::macroScratchRegisters() > RegisterSet RegisterSet::calleeSaveRegisters() > { > RegisterSet result; >-#if CPU(X86) >- result.set(X86Registers::ebx); >- result.set(X86Registers::ebp); >- result.set(X86Registers::edi); >- result.set(X86Registers::esi); >-#elif CPU(X86_64) >- result.set(X86Registers::ebx); >- result.set(X86Registers::ebp); >-#if OS(WINDOWS) >- result.set(X86Registers::edi); >- result.set(X86Registers::esi); >-#endif >- result.set(X86Registers::r12); >- result.set(X86Registers::r13); >- result.set(X86Registers::r14); >- result.set(X86Registers::r15); >-#elif CPU(ARM_THUMB2) >- result.set(ARMRegisters::r4); >- result.set(ARMRegisters::r5); >- result.set(ARMRegisters::r6); >- result.set(ARMRegisters::r8); >-#if !PLATFORM(IOS_FAMILY) >- result.set(ARMRegisters::r9); >-#endif >- result.set(ARMRegisters::r10); >- result.set(ARMRegisters::r11); >-#elif CPU(ARM64) >- // We don't include LR in the set of callee-save registers even though it technically belongs >- // there. This is because we use this set to describe the set of registers that need to be saved >- // beyond what you would save by the platform-agnostic "preserve return address" and "restore >- // return address" operations in CCallHelpers. >- for ( >- ARM64Registers::RegisterID reg = ARM64Registers::x19; >- reg <= ARM64Registers::x28; >- reg = static_cast<ARM64Registers::RegisterID>(reg + 1)) >- result.set(reg); >- result.set(ARM64Registers::fp); >- for ( >- ARM64Registers::FPRegisterID reg = ARM64Registers::q8; >- reg <= ARM64Registers::q15; >- reg = static_cast<ARM64Registers::FPRegisterID>(reg + 1)) >- result.set(reg); >-#elif CPU(MIPS) >- result.set(MIPSRegisters::s0); >-#else >- UNREACHABLE_FOR_PLATFORM(); >-#endif >+ >+#define SET_IF_CALLEESAVED(id, nsid, name, isReserved, isCalleeSaved) \ >+ if (isCalleeSaved) \ >+ result.set(nsid); >+ FOR_EACH_GP_REGISTER(SET_IF_CALLEESAVED) >+ FOR_EACH_FP_REGISTER(SET_IF_CALLEESAVED) >+#undef SET_IF_CALLEESAVED >+ > return result; > } >
<b>You cannot view the attachment while viewing its details because your browser does not support IFRAMEs. <a href="attachment.cgi?id=371956">View the attachment on a separate page</a>.</b>
View Attachment As Diff
View Attachment As Raw
Flags:
saam
:
review-
Actions:
View
|
Formatted Diff
|
Diff
Attachments on
bug 198604
:
371485
|
371498
|
371525
|
371526
|
371583
|
371727
|
371729
|
371733
|
371940
|
371943
|
371945
|
371956
|
372029
|
372049
|
372051
|
372912
|
373008
|
373385