Skip to content

Commit

Permalink
Fix MacroAssemblerARM64E to emit braaz, brabz correctly
Browse files Browse the repository at this point in the history
https://bugs.webkit.org/show_bug.cgi?id=274321
rdar://128282890

Reviewed by Keith Miller and Yusuke Suzuki.

Previously `m_assembler.braaz` would actually emit blraaz, clobbering
the link register.

* Source/JavaScriptCore/assembler/ARM64EAssembler.h:
(JSC::ARM64EAssembler::braaz):
(JSC::ARM64EAssembler::brabz):

Canonical link: https://commits.webkit.org/278924@main
  • Loading branch information
Achierius authored and hyjorc1 committed May 17, 2024
1 parent 6d33df3 commit 924f612
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Source/JavaScriptCore/assembler/ARM64EAssembler.h
Original file line number Diff line number Diff line change
Expand Up @@ -261,12 +261,12 @@ class ARM64EAssembler : public ARM64Assembler {

ALWAYS_INLINE void braaz(RegisterID dest)
{
insn(encodeGroup4(Group4Op::BLRAAZ, dest));
insn(encodeGroup4(Group4Op::BRAAZ, dest));
}

ALWAYS_INLINE void brabz(RegisterID dest)
{
insn(encodeGroup4(Group4Op::BLRABZ, dest));
insn(encodeGroup4(Group4Op::BRABZ, dest));
}

ALWAYS_INLINE void blraaz(RegisterID dest)
Expand Down

0 comments on commit 924f612

Please sign in to comment.