• Clemens Backes's avatar
    Revert "Reland "[wasm][test] Fix test expectation"" · 2872775f
    Clemens Backes authored
    This reverts commit a55c82d4.
    
    Reason for revert: Fails on UBSan: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20-%20cfi/31712/overview
    
    Original change's description:
    > Reland "[wasm][test] Fix test expectation"
    >
    > This is a reland of 6f9cde1e, with
    > special handling for MSan as well.
    >
    > Original change's description:
    > > [wasm][test] Fix test expectation
    > >
    > > In the mprotect case, there could be one or multiple succeeding writes
    > > until we finally crash. Thus do not check that we never successfully
    > > write, but just check that the last printed statement is *before* a
    > > write.
    > >
    > > R=jkummerow@chromium.org
    > >
    > > Bug: v8:12226
    > > Change-Id: I04209691a9320a9b29dd0ec364539e062ad2dc03
    > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3160343
    > > Commit-Queue: Clemens Backes <clemensb@chromium.org>
    > > Reviewed-by: Jakob Kummerow <jkummerow@chromium.org>
    > > Cr-Commit-Position: refs/heads/main@{#76829}
    >
    > Bug: v8:12226
    > Cq-Include-Trybots: luci.v8.try:v8_linux64_msan_rel_ng
    > Change-Id: I85ca98be43fc1d933d39a4602194e1771c33007c
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3162037
    > Reviewed-by: Jakob Kummerow <jkummerow@chromium.org>
    > Commit-Queue: Clemens Backes <clemensb@chromium.org>
    > Cr-Commit-Position: refs/heads/main@{#76839}
    
    Bug: v8:12226
    Change-Id: Ifb0f4b7e87c9c54271f7f3de29b1f1fc6e867f3f
    Cq-Include-Trybots: luci.v8.try:v8_linux64_msan_rel_ng
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3162040
    Auto-Submit: Clemens Backes <clemensb@chromium.org>
    Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Cr-Commit-Position: refs/heads/main@{#76840}
    2872775f
memory-protection-unittest.cc 11.8 KB