Skip to content

Commit

Permalink
Merge r180667 - Add calleeSaveRegisters() implementation for ARM Trad…
Browse files Browse the repository at this point in the history
…itional

https://bugs.webkit.org/show_bug.cgi?id=141903

Reviewed by Darin Adler.

* jit/RegisterSet.cpp:
(JSC::RegisterSet::calleeSaveRegisters):
  • Loading branch information
ossy-szeged authored and carlosgcampos committed Mar 1, 2015
1 parent cb51fa6 commit 855abb8
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 0 deletions.
10 changes: 10 additions & 0 deletions Source/JavaScriptCore/ChangeLog
@@ -1,3 +1,13 @@
2015-02-26 Csaba Osztrogonác <ossy@webkit.org>

Add calleeSaveRegisters() implementation for ARM Traditional
https://bugs.webkit.org/show_bug.cgi?id=141903

Reviewed by Darin Adler.

* jit/RegisterSet.cpp:
(JSC::RegisterSet::calleeSaveRegisters):

2015-02-25 Benjamin Poulain <bpoulain@apple.com>

CodeBlock crashes when dumping op_push_name_scope
Expand Down
9 changes: 9 additions & 0 deletions Source/JavaScriptCore/jit/RegisterSet.cpp
Expand Up @@ -88,6 +88,15 @@ RegisterSet RegisterSet::calleeSaveRegisters()
result.set(ARMRegisters::r8);
result.set(ARMRegisters::r10);
result.set(ARMRegisters::r11);
#elif CPU(ARM_TRADITIONAL)
result.set(ARMRegisters::r4);
result.set(ARMRegisters::r5);
result.set(ARMRegisters::r6);
result.set(ARMRegisters::r7);
result.set(ARMRegisters::r8);
result.set(ARMRegisters::r9);
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
Expand Down

0 comments on commit 855abb8

Please sign in to comment.