• Leszek Swirski's avatar
    Revert "[iwyu] Remove sfi-inl.h -> wasm include" · d850f7ea
    Leszek Swirski authored
    This reverts commit 9d18a7fd.
    
    Reason for revert: Breaks build https://ci.chromium.org/p/v8/builders/luci.v8.ci/V8%20Linux%20-%20noi18n%20-%20debug/21856
    
    Original change's description:
    > [iwyu] Remove sfi-inl.h -> wasm include
    > 
    > This significantly reduces the build time when modifying wasm
    > files: before touching all wasm headers required 684 steps to
    > rebuild, now it's 216.
    > 
    > BUG=v8:7754,v8:7490
    > 
    > Change-Id: Id7ff6f9063168556daad4840ee614cf68144cdb2
    > Reviewed-on: https://chromium-review.googlesource.com/1145264
    > Commit-Queue: Marja Hölttä <marja@chromium.org>
    > Reviewed-by: Jakob Gruber <jgruber@chromium.org>
    > Reviewed-by: Ulan Degenbaev <ulan@chromium.org>
    > Reviewed-by: Tobias Tebbi <tebbi@chromium.org>
    > Reviewed-by: Toon Verwaest <verwaest@chromium.org>
    > Reviewed-by: Benedikt Meurer <bmeurer@chromium.org>
    > Reviewed-by: Clemens Hammacher <clemensh@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#54681}
    
    TBR=ulan@chromium.org,marja@chromium.org,titzer@chromium.org,jgruber@chromium.org,clemensh@chromium.org,tebbi@chromium.org,bmeurer@chromium.org,verwaest@chromium.org
    
    Change-Id: I3b4087916f65b16db75974dba58914c8ea377a08
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: v8:7754, v8:7490
    Reviewed-on: https://chromium-review.googlesource.com/1149920Reviewed-by: 's avatarLeszek Swirski <leszeks@chromium.org>
    Commit-Queue: Leszek Swirski <leszeks@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#54683}
    d850f7ea
builtins-dataview.cc 4.3 KB