AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

[turbofan] Use NumAllocatableRegisters() instead of kMaxNumAllocatableRegisters.

Browse Source 
Use NumAllocatableRegisters() instead of kMaxNumAllocatableRegisters in
ArchDefaultRegisterConfiguration::ArchDefaultRegisterConfiguration()
otherwise the compiler DCHECKS if triggered during snapshot builds on
Arm where only 16 DoubleRegisters are avaiable instead of 32.

BUG=v8:4280
LOG=N

Review URL: https://codereview.chromium.org/1230473006

Cr-Commit-Position: refs/heads/master@{#29548}
This commit is contained in:
rmcilroy 2015-07-09 02:14:32 -07:00 committed by Commit bot
parent 6802c4b921
commit a4584c07e7
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/compiler/register-configuration.cc
View File

@ -25,7 +25,7 @@ class ArchDefaultRegisterConfiguration : public RegisterConfiguration {
1,
1,
#else
DoubleRegister::kMaxNumAllocatableRegisters,
DoubleRegister::NumAllocatableRegisters(),
DoubleRegister::NumAllocatableAliasedRegisters(),
#endif
general_register_name_table_,
@ -35,7 +35,9 @@ class ArchDefaultRegisterConfiguration : public RegisterConfiguration {
for (int i = 0; i < Register::kMaxNumAllocatableRegisters; ++i) {
general_register_name_table_[i] = Register::AllocationIndexToString(i);
}
for (int i = 0; i < DoubleRegister::kMaxNumAllocatableRegisters; ++i) {
DCHECK_GE(DoubleRegister::kMaxNumAllocatableRegisters,
DoubleRegister::NumAllocatableRegisters());
for (int i = 0; i < DoubleRegister::NumAllocatableRegisters(); ++i) {
double_register_name_table_[i] =
DoubleRegister::AllocationIndexToString(i);
}