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

[wasm-simd] LoadTransform is always a SIMD operation

Browse Source 
LoadTransform is always a SIMD operation (it always results in a v128),
so it should unconditionally set has_simd_.

Bug: chromium:1137583
Change-Id: I8496e787c89edec734f4bbfd16dd8b5995fab98a
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2472638
Reviewed-by: Bill Budge <bbudge@chromium.org>
Commit-Queue: Zhi An Ng <zhin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#70518}
This commit is contained in:
Ng Zhi An 2020-10-14 13:16:35 -07:00 committed by Commit Bot
parent 8387acfab2
commit 752aba9dce
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/compiler/wasm-compiler.cc
View File

@ -4016,9 +4016,7 @@ Node* WasmGraphBuilder::LoadTransform(wasm::ValueType type, MachineType memtype,
Node* index, uint64_t offset,
uint32_t alignment,
wasm::WasmCodePosition position) {
if (memtype.representation() == MachineRepresentation::kSimd128) {
has_simd_ = true;
}
has_simd_ = true;
Node* load;
// {offset} is validated to be within uintptr_t range in {BoundsCheckMem}.