Commit 2eb3b53d authored by Clemens Backes's avatar Clemens Backes Committed by Commit Bot

[wasm] Inline two simple macros

The LOAD_RAW and LOAD_RAW_NODE_OFFSET macros just call method on the
GraphAssembler now, hence we can remove the macros and just do the calls
directly. This reduces code complexity.

R=jkummerow@chromium.org, tebbi@chromium.org

Bug: v8:10123
Change-Id: I1d779969a14a52256eecb33007564eb190bdcff2
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2039051
Commit-Queue: Clemens Backes <clemensb@chromium.org>
Reviewed-by: 's avatarJakob Kummerow <jkummerow@chromium.org>
Reviewed-by: 's avatarTobias Tebbi <tebbi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#66149}
parent 37913a2f
......@@ -648,11 +648,20 @@ Node* GraphAssembler::Store(StoreRepresentation rep, Node* object, Node* offset,
effect(), control()));
}
Node* GraphAssembler::Store(StoreRepresentation rep, Node* object, int offset,
Node* value) {
return Store(rep, object, Int32Constant(offset), value);
}
Node* GraphAssembler::Load(MachineType type, Node* object, Node* offset) {
return AddNode(graph()->NewNode(machine()->Load(type), object, offset,
effect(), control()));
}
Node* GraphAssembler::Load(MachineType type, Node* object, int offset) {
return Load(type, object, Int32Constant(offset));
}
Node* GraphAssembler::StoreUnaligned(MachineRepresentation rep, Node* object,
Node* offset, Node* value) {
Operator const* const op =
......
......@@ -309,7 +309,9 @@ class V8_EXPORT_PRIVATE GraphAssembler {
Node* Checkpoint(FrameState frame_state);
Node* Store(StoreRepresentation rep, Node* object, Node* offset, Node* value);
Node* Store(StoreRepresentation rep, Node* object, int offset, Node* value);
Node* Load(MachineType type, Node* object, Node* offset);
Node* Load(MachineType type, Node* object, int offset);
Node* StoreUnaligned(MachineRepresentation rep, Node* object, Node* offset,
Node* value);
......
This diff is collapsed.
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