|
@@ -1,382 +0,0 @@
|
|
|
-From 05f6ba814422a392d59037ebe4412168da0e44db Mon Sep 17 00:00:00 2001
|
|
|
-From: Mark Lam <mark.lam@apple.com>
|
|
|
-Date: Tue, 15 Jun 2021 01:04:01 +0000
|
|
|
-Subject: [PATCH] Add ldp and stp support for FP registers, plus some bug
|
|
|
- fixes. https://bugs.webkit.org/show_bug.cgi?id=226998 rdar://79313717
|
|
|
-
|
|
|
-Reviewed by Robin Morisset.
|
|
|
-
|
|
|
-This patch does the following:
|
|
|
-1. Add ldp and stp support for FP registers.
|
|
|
- This simply entails providing wrappers that take FPRegisterID and passing true
|
|
|
- for the V bit to the underlying loadStoreRegisterPairXXX encoding function.
|
|
|
- V is for vector (aka floating point). This will cause bit 26 in the instruction
|
|
|
- to be set indicating that it's loading / storing floating point registers.
|
|
|
-
|
|
|
-2. Add ARM64 disassembler support ldp and stp for FP registers.
|
|
|
- This includes fixing A64DOpcodeLoadStoreRegisterPair::mask to not exclude the
|
|
|
- FP versions of the instructions.
|
|
|
-
|
|
|
-3. Add ARM64Assembler query methods for determining if an immediate is encodable
|
|
|
- as the signed 12 bit immediate of ldp and stp instructions.
|
|
|
-
|
|
|
-4. Fix ldp and stp offset form to take an int instead of an unsigned. The
|
|
|
- immediate it takes is a 12-bit signed int, not unsigned.
|
|
|
-
|
|
|
-5. In loadStoreRegisterPairXXX encoding functions used by the forms of ldp and stp,
|
|
|
- RELEASE_ASSERT that the passed in immediate is encodable. Unlike ldur / stur,
|
|
|
- there is no form of ldp / stp that takes the offset in a register that can be
|
|
|
- used as a fail over. Hence, if the immediate is not encodable, this is a
|
|
|
- non-recoverable event. The client is responsible for ensuring that the offset
|
|
|
- is encodable.
|
|
|
-
|
|
|
-6. Added some testmasm tests for testing the offset form (as opposed to PreIndex
|
|
|
- and PostIndex forms) of ldp and stp. We currently only use the offset form
|
|
|
- in our JITs.
|
|
|
-
|
|
|
-* assembler/ARM64Assembler.h:
|
|
|
-(JSC::ARM64Assembler::isValidLDPImm):
|
|
|
-(JSC::ARM64Assembler::isValidLDPFPImm):
|
|
|
-(JSC::ARM64Assembler::ldp):
|
|
|
-(JSC::ARM64Assembler::ldnp):
|
|
|
-(JSC::ARM64Assembler::isValidSTPImm):
|
|
|
-(JSC::ARM64Assembler::isValidSTPFPImm):
|
|
|
-(JSC::ARM64Assembler::stp):
|
|
|
-(JSC::ARM64Assembler::stnp):
|
|
|
-(JSC::ARM64Assembler::loadStoreRegisterPairPostIndex):
|
|
|
-(JSC::ARM64Assembler::loadStoreRegisterPairPreIndex):
|
|
|
-(JSC::ARM64Assembler::loadStoreRegisterPairOffset):
|
|
|
-(JSC::ARM64Assembler::loadStoreRegisterPairNonTemporal):
|
|
|
-* assembler/AssemblerCommon.h:
|
|
|
-(JSC::isValidSignedImm7):
|
|
|
-* assembler/MacroAssemblerARM64.h:
|
|
|
-(JSC::MacroAssemblerARM64::loadPair64):
|
|
|
-(JSC::MacroAssemblerARM64::storePair64):
|
|
|
-* assembler/testmasm.cpp:
|
|
|
-(JSC::testLoadStorePair64Int64):
|
|
|
-(JSC::testLoadStorePair64Double):
|
|
|
-* disassembler/ARM64/A64DOpcode.cpp:
|
|
|
-(JSC::ARM64Disassembler::A64DOpcodeLoadStoreRegisterPair::format):
|
|
|
-* disassembler/ARM64/A64DOpcode.h:
|
|
|
-
|
|
|
-
|
|
|
-
|
|
|
-Canonical link: https://commits.webkit.org/238801@main
|
|
|
-git-svn-id: https://svn.webkit.org/repository/webkit/trunk@278856 268f45cc-cd09-0410-ab3c-d52691b4dbfc
|
|
|
-
|
|
|
-Signed-off-by: James Hilliard <james.hilliard1@gmail.com>
|
|
|
-[james.hilliard1@gmail.com: backport from upstream commit
|
|
|
-05f6ba814422a392d59037ebe4412168da0e44db]
|
|
|
----
|
|
|
- Source/JavaScriptCore/ChangeLog | 61 +++
|
|
|
- .../JavaScriptCore/assembler/ARM64Assembler.h | 104 ++++-
|
|
|
- .../assembler/AssemblerCommon.h | 11 +-
|
|
|
- .../assembler/MacroAssemblerARM64.h | 20 +
|
|
|
- Source/JavaScriptCore/assembler/testmasm.cpp | 437 ++++++++++++++++++
|
|
|
- .../disassembler/ARM64/A64DOpcode.cpp | 8 +-
|
|
|
- .../disassembler/ARM64/A64DOpcode.h | 4 +-
|
|
|
- 7 files changed, 630 insertions(+), 15 deletions(-)
|
|
|
-
|
|
|
-diff --git a/Source/JavaScriptCore/assembler/ARM64Assembler.h b/Source/JavaScriptCore/assembler/ARM64Assembler.h
|
|
|
-index 2cc53c8ccda5..758cbe402779 100644
|
|
|
---- a/Source/JavaScriptCore/assembler/ARM64Assembler.h
|
|
|
-+++ b/Source/JavaScriptCore/assembler/ARM64Assembler.h
|
|
|
-@@ -1,5 +1,5 @@
|
|
|
- /*
|
|
|
-- * Copyright (C) 2012-2020 Apple Inc. All rights reserved.
|
|
|
-+ * Copyright (C) 2012-2021 Apple Inc. All rights reserved.
|
|
|
- *
|
|
|
- * Redistribution and use in source and binary forms, with or without
|
|
|
- * modification, are permitted provided that the following conditions
|
|
|
-@@ -1114,6 +1114,20 @@ public:
|
|
|
- insn(0x0);
|
|
|
- }
|
|
|
-
|
|
|
-+ template<int datasize>
|
|
|
-+ ALWAYS_INLINE static bool isValidLDPImm(int immediate)
|
|
|
-+ {
|
|
|
-+ unsigned immedShiftAmount = memPairOffsetShift(false, MEMPAIROPSIZE_INT(datasize));
|
|
|
-+ return isValidSignedImm7(immediate, immedShiftAmount);
|
|
|
-+ }
|
|
|
-+
|
|
|
-+ template<int datasize>
|
|
|
-+ ALWAYS_INLINE static bool isValidLDPFPImm(int immediate)
|
|
|
-+ {
|
|
|
-+ unsigned immedShiftAmount = memPairOffsetShift(true, MEMPAIROPSIZE_FP(datasize));
|
|
|
-+ return isValidSignedImm7(immediate, immedShiftAmount);
|
|
|
-+ }
|
|
|
-+
|
|
|
- template<int datasize>
|
|
|
- ALWAYS_INLINE void ldp(RegisterID rt, RegisterID rt2, RegisterID rn, PairPostIndex simm)
|
|
|
- {
|
|
|
-@@ -1129,17 +1143,45 @@ public:
|
|
|
- }
|
|
|
-
|
|
|
- template<int datasize>
|
|
|
-- ALWAYS_INLINE void ldp(RegisterID rt, RegisterID rt2, RegisterID rn, unsigned pimm = 0)
|
|
|
-+ ALWAYS_INLINE void ldp(RegisterID rt, RegisterID rt2, RegisterID rn, int simm = 0)
|
|
|
-+ {
|
|
|
-+ CHECK_DATASIZE();
|
|
|
-+ insn(loadStoreRegisterPairOffset(MEMPAIROPSIZE_INT(datasize), false, MemOp_LOAD, simm, rn, rt, rt2));
|
|
|
-+ }
|
|
|
-+
|
|
|
-+ template<int datasize>
|
|
|
-+ ALWAYS_INLINE void ldnp(RegisterID rt, RegisterID rt2, RegisterID rn, int simm = 0)
|
|
|
-+ {
|
|
|
-+ CHECK_DATASIZE();
|
|
|
-+ insn(loadStoreRegisterPairNonTemporal(MEMPAIROPSIZE_INT(datasize), false, MemOp_LOAD, simm, rn, rt, rt2));
|
|
|
-+ }
|
|
|
-+
|
|
|
-+ template<int datasize>
|
|
|
-+ ALWAYS_INLINE void ldp(FPRegisterID rt, FPRegisterID rt2, RegisterID rn, PairPostIndex simm)
|
|
|
-+ {
|
|
|
-+ CHECK_DATASIZE();
|
|
|
-+ insn(loadStoreRegisterPairPostIndex(MEMPAIROPSIZE_FP(datasize), true, MemOp_LOAD, simm, rn, rt, rt2));
|
|
|
-+ }
|
|
|
-+
|
|
|
-+ template<int datasize>
|
|
|
-+ ALWAYS_INLINE void ldp(FPRegisterID rt, FPRegisterID rt2, RegisterID rn, PairPreIndex simm)
|
|
|
-+ {
|
|
|
-+ CHECK_DATASIZE();
|
|
|
-+ insn(loadStoreRegisterPairPreIndex(MEMPAIROPSIZE_FP(datasize), true, MemOp_LOAD, simm, rn, rt, rt2));
|
|
|
-+ }
|
|
|
-+
|
|
|
-+ template<int datasize>
|
|
|
-+ ALWAYS_INLINE void ldp(FPRegisterID rt, FPRegisterID rt2, RegisterID rn, int simm = 0)
|
|
|
- {
|
|
|
- CHECK_DATASIZE();
|
|
|
-- insn(loadStoreRegisterPairOffset(MEMPAIROPSIZE_INT(datasize), false, MemOp_LOAD, pimm, rn, rt, rt2));
|
|
|
-+ insn(loadStoreRegisterPairOffset(MEMPAIROPSIZE_FP(datasize), true, MemOp_LOAD, simm, rn, rt, rt2));
|
|
|
- }
|
|
|
-
|
|
|
- template<int datasize>
|
|
|
-- ALWAYS_INLINE void ldnp(RegisterID rt, RegisterID rt2, RegisterID rn, unsigned pimm = 0)
|
|
|
-+ ALWAYS_INLINE void ldnp(FPRegisterID rt, FPRegisterID rt2, RegisterID rn, int simm = 0)
|
|
|
- {
|
|
|
- CHECK_DATASIZE();
|
|
|
-- insn(loadStoreRegisterPairNonTemporal(MEMPAIROPSIZE_INT(datasize), false, MemOp_LOAD, pimm, rn, rt, rt2));
|
|
|
-+ insn(loadStoreRegisterPairNonTemporal(MEMPAIROPSIZE_FP(datasize), true, MemOp_LOAD, simm, rn, rt, rt2));
|
|
|
- }
|
|
|
-
|
|
|
- template<int datasize>
|
|
|
-@@ -1743,6 +1785,18 @@ public:
|
|
|
- smaddl(rd, rn, rm, ARM64Registers::zr);
|
|
|
- }
|
|
|
-
|
|
|
-+ template<int datasize>
|
|
|
-+ ALWAYS_INLINE static bool isValidSTPImm(int immediate)
|
|
|
-+ {
|
|
|
-+ return isValidLDPImm<datasize>(immediate);
|
|
|
-+ }
|
|
|
-+
|
|
|
-+ template<int datasize>
|
|
|
-+ ALWAYS_INLINE static bool isValidSTPFPImm(int immediate)
|
|
|
-+ {
|
|
|
-+ return isValidLDPFPImm<datasize>(immediate);
|
|
|
-+ }
|
|
|
-+
|
|
|
- template<int datasize>
|
|
|
- ALWAYS_INLINE void stp(RegisterID rt, RegisterID rt2, RegisterID rn, PairPostIndex simm)
|
|
|
- {
|
|
|
-@@ -1758,17 +1812,45 @@ public:
|
|
|
- }
|
|
|
-
|
|
|
- template<int datasize>
|
|
|
-- ALWAYS_INLINE void stp(RegisterID rt, RegisterID rt2, RegisterID rn, unsigned pimm = 0)
|
|
|
-+ ALWAYS_INLINE void stp(RegisterID rt, RegisterID rt2, RegisterID rn, int simm = 0)
|
|
|
-+ {
|
|
|
-+ CHECK_DATASIZE();
|
|
|
-+ insn(loadStoreRegisterPairOffset(MEMPAIROPSIZE_INT(datasize), false, MemOp_STORE, simm, rn, rt, rt2));
|
|
|
-+ }
|
|
|
-+
|
|
|
-+ template<int datasize>
|
|
|
-+ ALWAYS_INLINE void stnp(RegisterID rt, RegisterID rt2, RegisterID rn, int simm = 0)
|
|
|
-+ {
|
|
|
-+ CHECK_DATASIZE();
|
|
|
-+ insn(loadStoreRegisterPairNonTemporal(MEMPAIROPSIZE_INT(datasize), false, MemOp_STORE, simm, rn, rt, rt2));
|
|
|
-+ }
|
|
|
-+
|
|
|
-+ template<int datasize>
|
|
|
-+ ALWAYS_INLINE void stp(FPRegisterID rt, FPRegisterID rt2, RegisterID rn, PairPostIndex simm)
|
|
|
-+ {
|
|
|
-+ CHECK_DATASIZE();
|
|
|
-+ insn(loadStoreRegisterPairPostIndex(MEMPAIROPSIZE_FP(datasize), true, MemOp_STORE, simm, rn, rt, rt2));
|
|
|
-+ }
|
|
|
-+
|
|
|
-+ template<int datasize>
|
|
|
-+ ALWAYS_INLINE void stp(FPRegisterID rt, FPRegisterID rt2, RegisterID rn, PairPreIndex simm)
|
|
|
-+ {
|
|
|
-+ CHECK_DATASIZE();
|
|
|
-+ insn(loadStoreRegisterPairPreIndex(MEMPAIROPSIZE_FP(datasize), true, MemOp_STORE, simm, rn, rt, rt2));
|
|
|
-+ }
|
|
|
-+
|
|
|
-+ template<int datasize>
|
|
|
-+ ALWAYS_INLINE void stp(FPRegisterID rt, FPRegisterID rt2, RegisterID rn, int simm = 0)
|
|
|
- {
|
|
|
- CHECK_DATASIZE();
|
|
|
-- insn(loadStoreRegisterPairOffset(MEMPAIROPSIZE_INT(datasize), false, MemOp_STORE, pimm, rn, rt, rt2));
|
|
|
-+ insn(loadStoreRegisterPairOffset(MEMPAIROPSIZE_FP(datasize), true, MemOp_STORE, simm, rn, rt, rt2));
|
|
|
- }
|
|
|
-
|
|
|
- template<int datasize>
|
|
|
-- ALWAYS_INLINE void stnp(RegisterID rt, RegisterID rt2, RegisterID rn, unsigned pimm = 0)
|
|
|
-+ ALWAYS_INLINE void stnp(FPRegisterID rt, FPRegisterID rt2, RegisterID rn, int simm = 0)
|
|
|
- {
|
|
|
- CHECK_DATASIZE();
|
|
|
-- insn(loadStoreRegisterPairNonTemporal(MEMPAIROPSIZE_INT(datasize), false, MemOp_STORE, pimm, rn, rt, rt2));
|
|
|
-+ insn(loadStoreRegisterPairNonTemporal(MEMPAIROPSIZE_FP(datasize), true, MemOp_STORE, simm, rn, rt, rt2));
|
|
|
- }
|
|
|
-
|
|
|
- template<int datasize>
|
|
|
-@@ -3544,6 +3626,7 @@ protected:
|
|
|
- ASSERT(opc == (opc & 1)); // Only load or store, load signed 64 is handled via size.
|
|
|
- ASSERT(V || (size != MemPairOp_LoadSigned_32) || (opc == MemOp_LOAD)); // There isn't an integer store signed.
|
|
|
- unsigned immedShiftAmount = memPairOffsetShift(V, size);
|
|
|
-+ RELEASE_ASSERT(isValidSignedImm7(immediate, immedShiftAmount));
|
|
|
- int imm7 = immediate >> immedShiftAmount;
|
|
|
- ASSERT((imm7 << immedShiftAmount) == immediate && isInt<7>(imm7));
|
|
|
- return (0x28800000 | size << 30 | V << 26 | opc << 22 | (imm7 & 0x7f) << 15 | rt2 << 10 | xOrSp(rn) << 5 | rt);
|
|
|
-@@ -3575,6 +3658,7 @@ protected:
|
|
|
- ASSERT(opc == (opc & 1)); // Only load or store, load signed 64 is handled via size.
|
|
|
- ASSERT(V || (size != MemPairOp_LoadSigned_32) || (opc == MemOp_LOAD)); // There isn't an integer store signed.
|
|
|
- unsigned immedShiftAmount = memPairOffsetShift(V, size);
|
|
|
-+ RELEASE_ASSERT(isValidSignedImm7(immediate, immedShiftAmount));
|
|
|
- int imm7 = immediate >> immedShiftAmount;
|
|
|
- ASSERT((imm7 << immedShiftAmount) == immediate && isInt<7>(imm7));
|
|
|
- return (0x29800000 | size << 30 | V << 26 | opc << 22 | (imm7 & 0x7f) << 15 | rt2 << 10 | xOrSp(rn) << 5 | rt);
|
|
|
-@@ -3592,6 +3676,7 @@ protected:
|
|
|
- ASSERT(opc == (opc & 1)); // Only load or store, load signed 64 is handled via size.
|
|
|
- ASSERT(V || (size != MemPairOp_LoadSigned_32) || (opc == MemOp_LOAD)); // There isn't an integer store signed.
|
|
|
- unsigned immedShiftAmount = memPairOffsetShift(V, size);
|
|
|
-+ RELEASE_ASSERT(isValidSignedImm7(immediate, immedShiftAmount));
|
|
|
- int imm7 = immediate >> immedShiftAmount;
|
|
|
- ASSERT((imm7 << immedShiftAmount) == immediate && isInt<7>(imm7));
|
|
|
- return (0x29000000 | size << 30 | V << 26 | opc << 22 | (imm7 & 0x7f) << 15 | rt2 << 10 | xOrSp(rn) << 5 | rt);
|
|
|
-@@ -3609,6 +3694,7 @@ protected:
|
|
|
- ASSERT(opc == (opc & 1)); // Only load or store, load signed 64 is handled via size.
|
|
|
- ASSERT(V || (size != MemPairOp_LoadSigned_32) || (opc == MemOp_LOAD)); // There isn't an integer store signed.
|
|
|
- unsigned immedShiftAmount = memPairOffsetShift(V, size);
|
|
|
-+ RELEASE_ASSERT(isValidSignedImm7(immediate, immedShiftAmount));
|
|
|
- int imm7 = immediate >> immedShiftAmount;
|
|
|
- ASSERT((imm7 << immedShiftAmount) == immediate && isInt<7>(imm7));
|
|
|
- return (0x28000000 | size << 30 | V << 26 | opc << 22 | (imm7 & 0x7f) << 15 | rt2 << 10 | xOrSp(rn) << 5 | rt);
|
|
|
-diff --git a/Source/JavaScriptCore/assembler/AssemblerCommon.h b/Source/JavaScriptCore/assembler/AssemblerCommon.h
|
|
|
-index a594823d6a4d..2e50ffdbc82a 100644
|
|
|
---- a/Source/JavaScriptCore/assembler/AssemblerCommon.h
|
|
|
-+++ b/Source/JavaScriptCore/assembler/AssemblerCommon.h
|
|
|
-@@ -1,5 +1,5 @@
|
|
|
- /*
|
|
|
-- * Copyright (C) 2012-2019 Apple Inc. All rights reserved.
|
|
|
-+ * Copyright (C) 2012-2021 Apple Inc. All rights reserved.
|
|
|
- *
|
|
|
- * Redistribution and use in source and binary forms, with or without
|
|
|
- * modification, are permitted provided that the following conditions
|
|
|
-@@ -74,6 +74,15 @@ ALWAYS_INLINE bool isValidSignedImm9(int32_t value)
|
|
|
- return isInt9(value);
|
|
|
- }
|
|
|
-
|
|
|
-+ALWAYS_INLINE bool isValidSignedImm7(int32_t value, int alignmentShiftAmount)
|
|
|
-+{
|
|
|
-+ constexpr int32_t disallowedHighBits = 32 - 7;
|
|
|
-+ int32_t shiftedValue = value >> alignmentShiftAmount;
|
|
|
-+ bool fitsIn7Bits = shiftedValue == ((shiftedValue << disallowedHighBits) >> disallowedHighBits);
|
|
|
-+ bool hasCorrectAlignment = value == (shiftedValue << alignmentShiftAmount);
|
|
|
-+ return fitsIn7Bits && hasCorrectAlignment;
|
|
|
-+}
|
|
|
-+
|
|
|
- class ARM64LogicalImmediate {
|
|
|
- public:
|
|
|
- static ARM64LogicalImmediate create32(uint32_t value)
|
|
|
-diff --git a/Source/JavaScriptCore/assembler/MacroAssemblerARM64.h b/Source/JavaScriptCore/assembler/MacroAssemblerARM64.h
|
|
|
-index f86aec1c5400..14e477fde3b8 100644
|
|
|
---- a/Source/JavaScriptCore/assembler/MacroAssemblerARM64.h
|
|
|
-+++ b/Source/JavaScriptCore/assembler/MacroAssemblerARM64.h
|
|
|
-@@ -1244,6 +1244,16 @@ public:
|
|
|
- m_assembler.ldnp<64>(dest1, dest2, src, offset.m_value);
|
|
|
- }
|
|
|
-
|
|
|
-+ void loadPair64(RegisterID src, FPRegisterID dest1, FPRegisterID dest2)
|
|
|
-+ {
|
|
|
-+ loadPair64(src, TrustedImm32(0), dest1, dest2);
|
|
|
-+ }
|
|
|
-+
|
|
|
-+ void loadPair64(RegisterID src, TrustedImm32 offset, FPRegisterID dest1, FPRegisterID dest2)
|
|
|
-+ {
|
|
|
-+ m_assembler.ldp<64>(dest1, dest2, src, offset.m_value);
|
|
|
-+ }
|
|
|
-+
|
|
|
- void abortWithReason(AbortReason reason)
|
|
|
- {
|
|
|
- // It is safe to use dataTempRegister directly since this is a crashing JIT Assert.
|
|
|
-@@ -1568,6 +1578,16 @@ public:
|
|
|
- m_assembler.stnp<64>(src1, src2, dest, offset.m_value);
|
|
|
- }
|
|
|
-
|
|
|
-+ void storePair64(FPRegisterID src1, FPRegisterID src2, RegisterID dest)
|
|
|
-+ {
|
|
|
-+ storePair64(src1, src2, dest, TrustedImm32(0));
|
|
|
-+ }
|
|
|
-+
|
|
|
-+ void storePair64(FPRegisterID src1, FPRegisterID src2, RegisterID dest, TrustedImm32 offset)
|
|
|
-+ {
|
|
|
-+ m_assembler.stp<64>(src1, src2, dest, offset.m_value);
|
|
|
-+ }
|
|
|
-+
|
|
|
- void store32(RegisterID src, ImplicitAddress address)
|
|
|
- {
|
|
|
- if (tryStoreWithOffset<32>(src, address.base, address.offset))
|
|
|
-diff --git a/Source/JavaScriptCore/disassembler/ARM64/A64DOpcode.cpp b/Source/JavaScriptCore/disassembler/ARM64/A64DOpcode.cpp
|
|
|
-index 247c79dcb428..dfe09b671470 100644
|
|
|
---- a/Source/JavaScriptCore/disassembler/ARM64/A64DOpcode.cpp
|
|
|
-+++ b/Source/JavaScriptCore/disassembler/ARM64/A64DOpcode.cpp
|
|
|
-@@ -1,5 +1,5 @@
|
|
|
- /*
|
|
|
-- * Copyright (C) 2012, 2016 Apple Inc. All rights reserved.
|
|
|
-+ * Copyright (C) 2012-2021 Apple Inc. All rights reserved.
|
|
|
- *
|
|
|
- * Redistribution and use in source and binary forms, with or without
|
|
|
- * modification, are permitted provided that the following conditions
|
|
|
-@@ -72,6 +72,8 @@ static const OpcodeGroupInitializer opcodeGroupList[] = {
|
|
|
- OPCODE_GROUP_ENTRY(0x0a, A64DOpcodeLogicalShiftedRegister),
|
|
|
- OPCODE_GROUP_ENTRY(0x0b, A64DOpcodeAddSubtractExtendedRegister),
|
|
|
- OPCODE_GROUP_ENTRY(0x0b, A64DOpcodeAddSubtractShiftedRegister),
|
|
|
-+ OPCODE_GROUP_ENTRY(0x0c, A64DOpcodeLoadStoreRegisterPair),
|
|
|
-+ OPCODE_GROUP_ENTRY(0x0d, A64DOpcodeLoadStoreRegisterPair),
|
|
|
- OPCODE_GROUP_ENTRY(0x11, A64DOpcodeAddSubtractImmediate),
|
|
|
- OPCODE_GROUP_ENTRY(0x12, A64DOpcodeMoveWide),
|
|
|
- OPCODE_GROUP_ENTRY(0x12, A64DOpcodeLogicalImmediate),
|
|
|
-@@ -1363,9 +1365,9 @@ const char* A64DOpcodeLoadStoreRegisterPair::format()
|
|
|
- appendInstructionName(thisOpName);
|
|
|
- unsigned offsetShift;
|
|
|
- if (vBit()) {
|
|
|
-- appendFPRegisterName(rt(), size());
|
|
|
-+ appendFPRegisterName(rt(), size() + 2);
|
|
|
- appendSeparator();
|
|
|
-- appendFPRegisterName(rt2(), size());
|
|
|
-+ appendFPRegisterName(rt2(), size() + 2);
|
|
|
- offsetShift = size() + 2;
|
|
|
- } else {
|
|
|
- if (!lBit())
|
|
|
-diff --git a/Source/JavaScriptCore/disassembler/ARM64/A64DOpcode.h b/Source/JavaScriptCore/disassembler/ARM64/A64DOpcode.h
|
|
|
-index e071babb8e01..fd9db7cae58e 100644
|
|
|
---- a/Source/JavaScriptCore/disassembler/ARM64/A64DOpcode.h
|
|
|
-+++ b/Source/JavaScriptCore/disassembler/ARM64/A64DOpcode.h
|
|
|
-@@ -1,5 +1,5 @@
|
|
|
- /*
|
|
|
-- * Copyright (C) 2012-2019 Apple Inc. All rights reserved.
|
|
|
-+ * Copyright (C) 2012-2021 Apple Inc. All rights reserved.
|
|
|
- *
|
|
|
- * Redistribution and use in source and binary forms, with or without
|
|
|
- * modification, are permitted provided that the following conditions
|
|
|
-@@ -787,7 +787,7 @@ public:
|
|
|
-
|
|
|
- class A64DOpcodeLoadStoreRegisterPair : public A64DOpcodeLoadStore {
|
|
|
- public:
|
|
|
-- static constexpr uint32_t mask = 0x3a000000;
|
|
|
-+ static constexpr uint32_t mask = 0x38000000;
|
|
|
- static constexpr uint32_t pattern = 0x28000000;
|
|
|
-
|
|
|
- DEFINE_STATIC_FORMAT(A64DOpcodeLoadStoreRegisterPair, thisObj);
|
|
|
---
|
|
|
-2.25.1
|
|
|
-
|