Fix build on unified-X platform 33/307833/2 accepted/tizen/unified/20240315.155154 accepted/tizen/unified/x/20240315.081636 submit/tizen/20240314.160014
authorDaniil Ruban <intx82@gmail.com>
Wed, 13 Mar 2024 11:57:27 +0000 (12:57 +0100)
committerDae-Hyun Ko <dhyuna.ko@samsung.com>
Thu, 14 Mar 2024 12:12:17 +0000 (12:12 +0000)
- Disable v8_enable_static_roots according to 'https://groups.google.com/g/v8-reviews/c/GIhTcy8qMe4?pli=1'
- Fix minor changes with std::move and lvalues in v8/src/base
- Move -fconcepts flag to 'cflags_cc' instead of 'cflags'

Change-Id: Ib82c8333750d86a0ccc3a01f95c13c9d5d4d5a2d
Signed-off-by: Daniil Ruban <intx82@gmail.com>
build/config/compiler/BUILD.gn
v8/BUILD.gn
v8/src/base/optional.h

index 812f071..9e7ff32 100644 (file)
@@ -510,7 +510,8 @@ config("compiler") {
     if (!is_clang) {
       # Use pipes for communicating between sub-processes. Faster.
       # (This flag doesn't do anything with Clang.)
-      cflags += [ "-pipe", "-fconcepts", "-flax-vector-conversions" ]
+      cflags += [ "-pipe", "-flax-vector-conversions" ]
+      cflags_cc += [ "-fconcepts" ]
     }
 
     ldflags += [
index 6d10a4e..f44bb58 100644 (file)
@@ -131,7 +131,7 @@ declare_args() {
   v8_enable_snapshot_native_code_counters = ""
 
   # Use pre-generated static root pointer values from static-roots.h.
-  v8_enable_static_roots = ""
+  v8_enable_static_roots = false
 
   # Mode used by gen-static-roots.py to have a heap layout which is identical
   # to when v8_enable_static_roots is enabled.
index d24a686..d34fc84 100644 (file)
@@ -632,7 +632,7 @@ class OPTIONAL_DECLSPEC_EMPTY_BASES Optional
                   "U must be convertible to T");
     return storage_.is_populated_
                ? std::move(storage_.value_)
-               : static_cast<T>(std::forward<U>(default_value));
+               : default_value;
   }
 
   void swap(Optional& other) {