• Clemens Backes's avatar
    [wasm] Remove one {NewNativeModule} method · dde3166b
    Clemens Backes authored
    This makes the {code_size_estimate} computation explicit in the caller,
    and removes one of the two {NewNativeModule} constructors. It turns out
    that the calculation is totally off in the streaming calculation phase,
    since no function bodies have been parsed yet. So all
    {WasmFunction::code} fields are still empty, and we compute an estimate
    that is way too low.
    This CL prepares the actual fix for that (by computing a better estimate
    at specific call sites).
    
    R=ahaas@chromium.org
    
    Bug: v8:9950
    Change-Id: I68a891c97e5f65a9c7e73e21684bdfa7e261e216
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1901273
    Commit-Queue: Clemens Backes <clemensb@chromium.org>
    Reviewed-by: 's avatarAndreas Haas <ahaas@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#64845}
    dde3166b
wasm-serialization.cc 24.1 KB