• Simon Zünd's avatar
    [stack-trace] Change column number for wasm frames to module offset · f16f0bcc
    Simon Zünd authored
    The CL https://crrev.com/c/1646846 changed column numbers for Wasm
    frames in Error.stack traces. Instead of using the offset relative to
    the beginning of the function, the absolute offset inside the module
    is displayed as hex.
    
    This CL propagates that change to the StackTrace C++ API, so
    StackFrame::GetColumn() also returns the absolute offset. Note that the
    StackFrame API historically uses "0" to signal "no information", so the
    line and column numbers for Wasm frames are also adjusted to 1-based,
    even though they signify function index and absolute offset
    into the module.
    
    This CL does not touch Script::PositionInfo.column. That field still
    contains the offset relative to the function start.
    
    Bug: v8:8742
    Change-Id: If4fd37fa681c7ebd0823ce0d95eccc1335c35272
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1655300
    Commit-Queue: Simon Zünd <szuend@chromium.org>
    Reviewed-by: 's avatarBen Titzer <titzer@chromium.org>
    Reviewed-by: 's avatarMichael Lippautz <mlippautz@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#62171}
    f16f0bcc
Name
Last commit
Last update
benchmarks Loading commit data...
build_overrides Loading commit data...
custom_deps Loading commit data...
docs Loading commit data...
gni Loading commit data...
include Loading commit data...
infra Loading commit data...
samples Loading commit data...
src Loading commit data...
test Loading commit data...
testing Loading commit data...
third_party Loading commit data...
tools Loading commit data...
.clang-format Loading commit data...
.clang-tidy Loading commit data...
.editorconfig Loading commit data...
.git-blame-ignore-revs Loading commit data...
.gitattributes Loading commit data...
.gitignore Loading commit data...
.gn Loading commit data...
.vpython Loading commit data...
.ycm_extra_conf.py Loading commit data...
AUTHORS Loading commit data...
BUILD.gn Loading commit data...
CODE_OF_CONDUCT.md Loading commit data...
COMMON_OWNERS Loading commit data...
ChangeLog Loading commit data...
DEPS Loading commit data...
ENG_REVIEW_OWNERS Loading commit data...
INFRA_OWNERS Loading commit data...
INTL_OWNERS Loading commit data...
LICENSE Loading commit data...
LICENSE.fdlibm Loading commit data...
LICENSE.strongtalk Loading commit data...
LICENSE.v8 Loading commit data...
LICENSE.valgrind Loading commit data...
MIPS_OWNERS Loading commit data...
OWNERS Loading commit data...
PPC_OWNERS Loading commit data...
PRESUBMIT.py Loading commit data...
README.md Loading commit data...
S390_OWNERS Loading commit data...
WATCHLISTS Loading commit data...
codereview.settings Loading commit data...