Skip to content

deps: V8: backports for gcc 12

Rodrigo Muino Tomonari requested to merge github/fork/richardlau/v8-gcc12 into main

Original commit message:

Fix build with older versions of GCC.

This ports the change from bazel on v8:
https://crrev.com/c/3368869

Compilation errors started showing after this CL:
https://crrev.com/c/5199515

Change-Id: I8c161a0d9ad5c04d452c444ef4feafae2ef4f6db
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/5280535
Reviewed-by: Leszek Swirski <leszeks@chromium.org>
Commit-Queue: Milad Farazmand <mfarazma@redhat.com>
Cr-Commit-Position: refs/heads/main@{#92252}

Refs: https://github.com/v8/v8/commit/f8d5e576b814c92c39ec0cea80c21e4162270e12

Original commit message:

Fix build with gcc12

- A number of erroneous flags have been added to BUILD.gn
- wasm-init-expr.cc is creating an 8 byte buffer witch may be
  much smaller than max size_t output. We also need to make room
  for the `f` character and the terminating null character
- inspector_protocol currently generates the following error
   ```
   error: loop variable ‘json_in’ of type ‘const std::string&’ {aka
   ‘const std::__cxx11::basic_string<char>&’} binds to a temporary
   constructed from type ‘const char* const’
   ```

Change-Id: I1139899b2664e47d01ebc44f2e972fc4c0ec212d
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/5331756
Reviewed-by: Matthias Liedtke <mliedtke@chromium.org>
Commit-Queue: Milad Farazmand <mfarazma@redhat.com>
Reviewed-by: Toon Verwaest <verwaest@chromium.org>
Cr-Commit-Position: refs/heads/main@{#92615}

Refs: https://github.com/v8/v8/commit/c4be0a97f981ce08bad854684c941e4c98647025

Merge request reports