Commit 5a29432d authored by Clemens Hammacher's avatar Clemens Hammacher Committed by Commit Bot

[liftoff] [cleanup] Construct RegLists using symbolic names

Now that https://crrev.com/c/728026 has landed, we can construct the
constexpr RegLists using symbolic register names instead of hard-coding
the register codes.

R=titzer@chromium.org

Bug: v8:6600
Change-Id: I21e46aeb5e8598a56f641341bcd7cf718fe4fbf9
Reviewed-on: https://chromium-review.googlesource.com/735548Reviewed-by: 's avatarBen Titzer <titzer@chromium.org>
Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#48906}
parent 49e87d2f
...@@ -13,12 +13,8 @@ namespace wasm { ...@@ -13,12 +13,8 @@ namespace wasm {
static constexpr bool kLiftoffAssemblerImplementedOnThisPlatform = true; static constexpr bool kLiftoffAssemblerImplementedOnThisPlatform = true;
static constexpr RegList kLiftoffAssemblerGpCacheRegs = 1 << 0 | // eax static constexpr RegList kLiftoffAssemblerGpCacheRegs =
1 << 1 | // ecx Register::ListOf<eax, ecx, edx, ebx, esi, edi>();
1 << 2 | // edx
1 << 3 | // ebx
1 << 6 | // esi
1 << 7; // edi
} // namespace wasm } // namespace wasm
} // namespace internal } // namespace internal
......
...@@ -13,12 +13,8 @@ namespace wasm { ...@@ -13,12 +13,8 @@ namespace wasm {
static constexpr bool kLiftoffAssemblerImplementedOnThisPlatform = true; static constexpr bool kLiftoffAssemblerImplementedOnThisPlatform = true;
static constexpr RegList kLiftoffAssemblerGpCacheRegs = 1 << 0 | // rax static constexpr RegList kLiftoffAssemblerGpCacheRegs =
1 << 1 | // rcx Register::ListOf<rax, rcx, rdx, rbx, rsi, rdi>();
1 << 2 | // rdx
1 << 3 | // rbx
1 << 6 | // rsi
1 << 7; // rdi
} // namespace wasm } // namespace wasm
} // namespace internal } // namespace internal
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment