• Clemens Hammacher's avatar
    Revert "Reland "Reland "[code-comments] Put code comments into the code object""" · 8a5c0095
    Clemens Hammacher authored
    This reverts commit 9c0a4858.
    
    Reason for revert: Seems to break nosnap debug: https://ci.chromium.org/p/v8/builders/luci.v8.ci/V8%20Linux%20-%20nosnap%20-%20debug/22228
    
    Original change's description:
    > Reland "Reland "[code-comments] Put code comments into the code object""
    > 
    > This is a reland of ed3d6472
    > 
    > This reland fixes that padding at the end of Wasm instruction streams
    > triggered asserts in the code printer.
    > 
    > Original change's description:
    > > Reland "[code-comments] Put code comments into the code object"
    > >
    > > This is a reland of e774cffe
    > >
    > > This reland disables a test as v8:8548 is blocking it, which was
    > > broken by a recent CL. CQ did not catch this because the merge-base
    > > CQ used did not yet contain the CL that caused v8:8548.
    > >
    > > Original change's description:
    > > > [code-comments] Put code comments into the code object
    > > >
    > > > Code comments in the snapshot can now be enabled with gn
    > > > arg 'v8_enable_snapshot_code_comments'
    > > >
    > > > Bug: v8:7989
    > > > Change-Id: I8bd00cafa63132d00d849394c311ba15e6b6daf3
    > > > Reviewed-on: https://chromium-review.googlesource.com/c/1329173
    > > > Commit-Queue: Sigurd Schneider <sigurds@chromium.org>
    > > > Reviewed-by: Jakob Gruber <jgruber@chromium.org>
    > > > Reviewed-by: Michael Starzinger <mstarzinger@chromium.org>
    > > > Reviewed-by: Michael Stanton <mvstanton@chromium.org>
    > > > Cr-Commit-Position: refs/heads/master@{#58020}
    > >
    > > TBR=mvstanton@chromium.org,mstarzinger@chromium.org,jgruber@chromium.org,tebbi@chromium.org
    > >
    > > Bug: v8:7989, v8:8548
    > > Change-Id: I464fc897205fefdf2dfc2eadc54d699c4e08a0e9
    > > Reviewed-on: https://chromium-review.googlesource.com/c/1361166
    > > Reviewed-by: Sigurd Schneider <sigurds@chromium.org>
    > > Commit-Queue: Sigurd Schneider <sigurds@chromium.org>
    > > Cr-Commit-Position: refs/heads/master@{#58028}
    > 
    > Bug: v8:7989, v8:8548
    > Change-Id: I254f55ff687ad049f8d92b09331ed26a2bd05d7d
    > Reviewed-on: https://chromium-review.googlesource.com/c/1371784
    > Commit-Queue: Sigurd Schneider <sigurds@chromium.org>
    > Reviewed-by: Michael Starzinger <mstarzinger@chromium.org>
    > Reviewed-by: Jakob Gruber <jgruber@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#58221}
    
    TBR=mvstanton@chromium.org,mstarzinger@chromium.org,sigurds@chromium.org,jgruber@chromium.org
    
    Change-Id: I681a3c63120c6ab953bfe9cd2b07bcf560ebfdee
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: v8:7989, v8:8548
    Reviewed-on: https://chromium-review.googlesource.com/c/1375916
    Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
    Reviewed-by: 's avatarClemens Hammacher <clemensh@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#58228}
    8a5c0095
wasm-serialization.cc 22.6 KB