• Clemens Hammacher's avatar
    [wasm] [decoder] Merge checked_read_leb and consume_leb · 02b4d0e6
    Clemens Hammacher authored
    Both methods decoded a LEB128 encoded integer, but only consume_leb
    incremented the pc pointer accordingly.
    This CL implements consume_leb by using checked_read_leb.
    
    It also refactors a few things:
    1) It removes error_pt, which was only avaible in checked_read_leb.
    2) It renames the error method to errorf, since it receives a format
       string. This also avoids a name clash.
    3) It implements sign extension directly in checked_read_leb instead of
       doing this in the caller.
    
    R=ahaas@chromium.org
    BUG=v8:5822
    
    Change-Id: I8058f57418493861e5df26d4949041f6766d5138
    Reviewed-on: https://chromium-review.googlesource.com/467150
    Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
    Reviewed-by: 's avatarAndreas Haas <ahaas@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#44405}
    02b4d0e6
Name
Last commit
Last update
..
OWNERS Loading commit data...
test-run-wasm-64.cc Loading commit data...
test-run-wasm-asmjs.cc Loading commit data...
test-run-wasm-interpreter.cc Loading commit data...
test-run-wasm-js.cc Loading commit data...
test-run-wasm-module.cc Loading commit data...
test-run-wasm-relocation.cc Loading commit data...
test-run-wasm-simd.cc Loading commit data...
test-run-wasm.cc Loading commit data...
test-wasm-breakpoints.cc Loading commit data...
test-wasm-interpreter-entry.cc Loading commit data...
test-wasm-stack.cc Loading commit data...
test-wasm-trap-position.cc Loading commit data...
wasm-run-utils.h Loading commit data...