Michael Achenbach
45768b0f0e
[build] Make inspector the default in gyp
...
Bug: chromium:645890
Change-Id: If34ac1336d0ee3c23e89050aef2cf30b754b67c1
Reviewed-on: https://chromium-review.googlesource.com/461145
Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Reviewed-by: Yang Guo <yangguo@chromium.org>
Reviewed-by: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#44272}
2017-03-30 15:08:31 +00:00
Michael Achenbach
9bc1eb5ece
Enable inspector in gyp
...
BUG=chromium:645890
Change-Id: Iec8f473074a65abe9b4fbc968d5617038120efa5
Reviewed-on: https://chromium-review.googlesource.com/458011
Commit-Queue: Michael Achenbach <machenbach@chromium.org>
Reviewed-by: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#44071}
2017-03-23 16:36:08 +00:00
ulan
e5b152264e
Enable -Wsign-compare compiler warnings.
...
BUG=v8:5614
Review-Url: https://codereview.chromium.org/2493193002
Cr-Commit-Position: refs/heads/master@{#40958}
2016-11-14 10:49:15 +00:00
machenbach
c2c6a9a635
[build] Switch off warning on gcc bots.
...
Switch off maybe-uninitialized warning as it gives often
false positives with optimizations in gcc.
NOTRY=true
TBR=bmeurer@chromium.org , zhengxing.li@intel.com , caitp@igalia.com , ahaas@chromium.org
Committed: https://crrev.com/0ef0d5b2b8cffd1e3647d79c5ba19752bbbc469c
Review-Url: https://codereview.chromium.org/2425913002
Cr-Original-Commit-Position: refs/heads/master@{#40382}
Cr-Commit-Position: refs/heads/master@{#40394}
2016-10-18 10:47:58 +00:00
machenbach
179436cf8c
Revert of [build] Switch off warning on gcc bots. (patchset #2 id:20001 of https://codereview.chromium.org/2425913002/ )
...
Reason for revert:
Seems to be the wrong flag:
https://build.chromium.org/p/client.v8.ports/builders/V8%20Mips%20-%20builder/builds/4709
Original issue's description:
> [build] Switch off warning on gcc bots.
>
> Switch off maybe-uninitialized warning as it gives often
> false positives with optimizations in gcc.
>
> NOTRY=true
> TBR=bmeurer@chromium.org , zhengxing.li@intel.com , caitp@igalia.com , ahaas@chromium.org
>
> Committed: https://crrev.com/0ef0d5b2b8cffd1e3647d79c5ba19752bbbc469c
> Cr-Commit-Position: refs/heads/master@{#40382}
TBR=bmeurer@chromium.org ,zhengxing.li@intel.com,caitp@igalia.com,ahaas@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review-Url: https://codereview.chromium.org/2422103006
Cr-Commit-Position: refs/heads/master@{#40385}
2016-10-18 08:03:09 +00:00
machenbach
0ef0d5b2b8
[build] Switch off warning on gcc bots.
...
Switch off maybe-uninitialized warning as it gives often
false positives with optimizations in gcc.
NOTRY=true
TBR=bmeurer@chromium.org , zhengxing.li@intel.com , caitp@igalia.com , ahaas@chromium.org
Review-Url: https://codereview.chromium.org/2425913002
Cr-Commit-Position: refs/heads/master@{#40382}
2016-10-18 07:54:59 +00:00
machenbach
45fe9019bf
[gn] Switch android bots to gn
...
Also roll build and android_tools, which contains a bump of
the ndk to r12b.
BUG=chromium:629806
Review-Url: https://codereview.chromium.org/2320843003
Cr-Commit-Position: refs/heads/master@{#39269}
2016-09-08 10:31:10 +00:00
dgozman
81b7e77ff0
[inspector] Build inspector under v8_enable_inspector build flag.
...
- inspector becomes a dependency of v8_base;
- generated public protocol files are placed to gen/v8/include/inspector/<Domain.h>;
- added v8_enable_inspector_override to be used in embedders (gn only);
- combined public headers into v8-inspector.h and v8-inspector-protocol.h.
BUG=chromium:635948
Review-Url: https://codereview.chromium.org/2292053003
Cr-Commit-Position: refs/heads/master@{#39226}
2016-09-06 23:26:35 +00:00
hablich
4f8e0fa685
Reland of Fix compiler warnings on "make android_arm" (patchset #1 id:1 of https://codereview.chromium.org/2286163002/ )
...
Reason for revert:
Roll was unstuck before the revert landed => reland
Original issue's description:
> Revert of Fix compiler warnings on "make android_arm" (patchset #1 id:1 of https://codereview.chromium.org/2264283007/ )
>
> Reason for revert:
> Speculative revert because of roll blocker https://build.chromium.org/p/client.v8.ports/builders/V8%20Linux%20-%20arm64%20-%20sim%20-%20debug/builds/2241
>
> Original issue's description:
> > Fix compiler warnings on "make android_arm"
> >
> > Committed: https://crrev.com/3e809a6129d0097529c885579ac46e4acf4e99f6
> > Cr-Commit-Position: refs/heads/master@{#38937}
>
> TBR=bmeurer@chromium.org ,jkummerow@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
>
> Committed: https://crrev.com/d992c1f52f116930239ed90cc033442047e789b4
> Cr-Commit-Position: refs/heads/master@{#38961}
TBR=bmeurer@chromium.org ,jkummerow@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review-Url: https://codereview.chromium.org/2285113002
Cr-Commit-Position: refs/heads/master@{#38962}
2016-08-28 21:03:32 +00:00
hablich
d992c1f52f
Revert of Fix compiler warnings on "make android_arm" (patchset #1 id:1 of https://codereview.chromium.org/2264283007/ )
...
Reason for revert:
Speculative revert because of roll blocker https://build.chromium.org/p/client.v8.ports/builders/V8%20Linux%20-%20arm64%20-%20sim%20-%20debug/builds/2241
Original issue's description:
> Fix compiler warnings on "make android_arm"
>
> Committed: https://crrev.com/3e809a6129d0097529c885579ac46e4acf4e99f6
> Cr-Commit-Position: refs/heads/master@{#38937}
TBR=bmeurer@chromium.org ,jkummerow@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
Review-Url: https://codereview.chromium.org/2286163002
Cr-Commit-Position: refs/heads/master@{#38961}
2016-08-28 20:58:37 +00:00
jkummerow
3e809a6129
Fix compiler warnings on "make android_arm"
...
Review-Url: https://codereview.chromium.org/2264283007
Cr-Commit-Position: refs/heads/master@{#38937}
2016-08-26 09:53:55 +00:00
yangguo
b96823ff29
[inspector] suppress build warnings on windows/clang
...
R=machenbach@chromium.org ,thakis@chromium.org
BUG=v8:5258
Review-Url: https://codereview.chromium.org/2206983002
Cr-Commit-Position: refs/heads/master@{#38343}
2016-08-04 15:43:13 +00:00
machenbach
4289c28fb9
[build] Conditionally print to stdout on Android
...
This adds a gyp/gn variable to control printing to stdout
on Android. This is false by default and true for all
v8 stand-alone android bots.
BUG=chromium:629806
Review-Url: https://codereview.chromium.org/2183063002
Cr-Commit-Position: refs/heads/master@{#38253}
2016-08-02 15:38:04 +00:00
machenbach
5ef69e93a2
[build] Port using prebuilt instrumented libraries for msan
...
BUG=chromium:631979
NOTRY=true
Review-Url: https://codereview.chromium.org/2189343002
Cr-Commit-Position: refs/heads/master@{#38197}
2016-08-01 09:43:12 +00:00
yangguo
3e3ff89d4b
Reland [inspector] add inspector protocol parser test build target.
...
BUG=chromium:580337
Committed: https://crrev.com/b8229434c9fd3bfef81fb77d88779d0e78d06662
Review-Url: https://codereview.chromium.org/2190833004
Cr-Original-Commit-Position: refs/heads/master@{#38135}
Cr-Commit-Position: refs/heads/master@{#38190}
2016-08-01 05:42:37 +00:00
akos.palfi
45d6909e7b
MIPS: Fix mksnapshot on big-endian.
...
Paritally revert standalone.gypi changes in a451bd1a68
and introduce a new separate variable for the mkpeephole.
On big-endian MIPS, qemu is used to build the snapshot,
because there's no simulator support for big-endian MIPS.
BUG=
Committed: https://crrev.com/928d2395c3fdf836cf9961cde96e6b274a6b1e20
Review-Url: https://codereview.chromium.org/2172653002
Cr-Original-Commit-Position: refs/heads/master@{#38103}
Cr-Commit-Position: refs/heads/master@{#38165}
2016-07-29 09:30:54 +00:00
machenbach
224e2656e8
Revert of Reland [inspector] add inspector protocol parser test build target. (patchset #2 id:20001 of https://codereview.chromium.org/2190833004/ )
...
Reason for revert:
Remaining win x64 errors:
https://codereview.chromium.org/2187753004/
Original issue's description:
> Reland [inspector] add inspector protocol parser test build target.
>
> BUG=chromium:580337
>
> Committed: https://crrev.com/b8229434c9fd3bfef81fb77d88779d0e78d06662
> Cr-Commit-Position: refs/heads/master@{#38135}
TBR=yangguo@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:580337
Review-Url: https://codereview.chromium.org/2193143002
Cr-Commit-Position: refs/heads/master@{#38156}
2016-07-29 07:05:15 +00:00
yangguo
b8229434c9
Reland [inspector] add inspector protocol parser test build target.
...
BUG=chromium:580337
Review-Url: https://codereview.chromium.org/2190833004
Cr-Commit-Position: refs/heads/master@{#38135}
2016-07-28 12:41:34 +00:00
machenbach
6b23ce3061
Revert "[inspector] add inspector protocol parser test build target for GYP."
...
Breaks chromium win.
This also includes:
Revert "[gn] Add missing deps to inspector_protocol_parser_test"
Revert "Fix gitignores after moving inspector_protocol"
NOTRY=true
BUG=chromium:580337
TBR=yangguo@chromium.org , jochen@chromium.org , pfeldman@chromium.org
Review-Url: https://codereview.chromium.org/2187263002
Cr-Commit-Position: refs/heads/master@{#38128}
2016-07-28 10:21:48 +00:00
yangguo
f7fa02f8b2
[inspector] add inspector protocol parser test build target for GYP.
...
This adds the build target inspector_protocol_parser_test, which depends
on several more inspector protocol related targets. The resulting
gtest executable runs, but has failures that have to be fixed.
GN build configuration will follow.
A copy of v8-inspector/js_protocol.json has been included to be able to
use it before the actual migration.
BUG=chromium:580337
Review-Url: https://codereview.chromium.org/2184733002
Cr-Commit-Position: refs/heads/master@{#38111}
2016-07-28 06:57:03 +00:00
machenbach
366b00cfbe
Revert of MIPS: Fix mksnapshot on big-endian. (patchset #3 id:40001 of https://codereview.chromium.org/2172653002/ )
...
Reason for revert:
Blocks the roll:
https://codereview.chromium.org/2189443003/
Doesn't work with the last chromium gyp bot.
Original issue's description:
> MIPS: Fix mksnapshot on big-endian.
>
> Paritally revert standalone.gypi changes in a451bd1a68
and introduce a new separate variable for the mkpeephole.
>
> On big-endian MIPS, qemu is used to build the snapshot,
> because there's no simulator support for big-endian MIPS.
>
> BUG=
>
> Committed: https://crrev.com/928d2395c3fdf836cf9961cde96e6b274a6b1e20
> Cr-Commit-Position: refs/heads/master@{#38103}
TBR=oth@chromium.org ,machenbach@google.com,akos.palfi@imgtec.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=
Review-Url: https://codereview.chromium.org/2182633010
Cr-Commit-Position: refs/heads/master@{#38108}
2016-07-27 19:10:45 +00:00
akos.palfi
928d2395c3
MIPS: Fix mksnapshot on big-endian.
...
Paritally revert standalone.gypi changes in a451bd1a68
and introduce a new separate variable for the mkpeephole.
On big-endian MIPS, qemu is used to build the snapshot,
because there's no simulator support for big-endian MIPS.
BUG=
Review-Url: https://codereview.chromium.org/2172653002
Cr-Commit-Position: refs/heads/master@{#38103}
2016-07-27 14:49:43 +00:00
oth
a451bd1a68
Reland "[interpeter] Move to table based peephole optimizer."
...
Original issue's description:
> [interpeter] Move to table based peephole optimizer.
>
> Introduces a lookup table for peephole optimizations.
>
> Fixes some tests using BytecodePeepholeOptimizer::Write() that should
> have been update to use BytecodePeepholeOptimizer::WriteJump().
>
> BUG=v8:4280
> LOG=N
>
> Committed: https://crrev.com/f4234422b93b21a286b0f31799009bcbe8b90b9e
> Cr-Commit-Position: refs/heads/master@{#37819}
BUG=v8:4280
LOG=N
Review-Url: https://codereview.chromium.org/2164583002
Cr-Commit-Position: refs/heads/master@{#37866}
2016-07-19 11:56:33 +00:00
machenbach
a4053e027d
Revert of Add force_dynamic_crt to build as static library but with /MD on windows (patchset #3 id:40001 of https://codereview.chromium.org/2149963002/ )
...
Reason for revert:
Fails gyp build with chromium:
https://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/builds/37051
Blocks roll:
https://codereview.chromium.org/2157903002/
Please add the trybot ios-simulator on reland.
Original issue's description:
> [build] Add force_dynamic_crt option to build a static library with /MD on windows
>
> Adds option to build a V8 library statically, but with the options on
> windows that allows it to be subsequently included in another DLL. On
> Windows this is required for it to correclty link against the correct
> C++ runtime. Require for our Node.js shared library build.
>
> Reference: https://github.com/nodejs/node/pull/7487
>
> BUG=
> R=machenbach@chromium.org , michael_dawson@ca.ibm.com
>
> Committed: https://crrev.com/9cf88c1c364cf76c1e745aa63196768435e8ef5d
> Cr-Commit-Position: refs/heads/master@{#37814}
TBR=michael_dawson@ca.ibm.com ,franzih@chromium.org,sxa@uk.ibm.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=
Review-Url: https://codereview.chromium.org/2155073002
Cr-Commit-Position: refs/heads/master@{#37822}
2016-07-18 09:21:34 +00:00
sxa
9cf88c1c36
[build] Add force_dynamic_crt option to build a static library with /MD on windows
...
Adds option to build a V8 library statically, but with the options on
windows that allows it to be subsequently included in another DLL. On
Windows this is required for it to correclty link against the correct
C++ runtime. Require for our Node.js shared library build.
Reference: https://github.com/nodejs/node/pull/7487
BUG=
R=machenbach@chromium.org , michael_dawson@ca.ibm.com
Review-Url: https://codereview.chromium.org/2149963002
Cr-Commit-Position: refs/heads/master@{#37814}
2016-07-18 07:12:48 +00:00
ssanfilippo
8465244e79
Broaden the condition under which gold is used as linker.
...
LOG=N
Review-Url: https://codereview.chromium.org/2121123002
Cr-Commit-Position: refs/heads/master@{#37521}
2016-07-05 10:28:17 +00:00
thakis
8c8a9f18fb
v8 clang/win: Stop passing /FIIntrin.h
...
See https://codereview.chromium.org/2076483002 for a lengthy reasoning.
The two mysterious files called out there were in boringssl and nacl,
so for v8 this should be a complete no-op.
BUG=chromium:592745
Review-Url: https://codereview.chromium.org/2099843002
Cr-Commit-Position: refs/heads/master@{#37272}
2016-06-27 07:34:35 +00:00
machenbach
77af83ba18
[build] Use icu data file by default
...
BUG=chromium:616033
LOG=y
Review-Url: https://codereview.chromium.org/2049743002
Cr-Commit-Position: refs/heads/master@{#36862}
2016-06-09 13:24:58 +00:00
machenbach
c4fab3ec25
[build] Use sysroot for linux compilation with clang
...
This ports the configuration for using a sysroot from
chromium's common.gypi.
This is restricted to clang only.
BUG=chromium:474921, chromium:616032
LOG=y
Committed: https://crrev.com/c47d3c63695d2999752f5fc61ff171f3ff100553
Review-Url: https://codereview.chromium.org/2028623002
Cr-Original-Commit-Position: refs/heads/master@{#36729}
Cr-Commit-Position: refs/heads/master@{#36830}
2016-06-08 15:27:59 +00:00
machenbach
0e2d92967d
Revert of [build] Use sysroot for linux compilation with clang (patchset #5 id:80001 of https://codereview.chromium.org/2028623002/ )
...
Reason for revert:
sysroot logic breaks "make arm"
Original issue's description:
> [build] Use sysroot for linux compilation with clang
>
> This ports the configuration for using a sysroot from
> chromium's common.gypi.
>
> This is restricted to clang only.
>
> BUG=chromium:474921, chromium:616032
> LOG=y
>
> Committed: https://crrev.com/c47d3c63695d2999752f5fc61ff171f3ff100553
> Cr-Commit-Position: refs/heads/master@{#36729}
TBR=jochen@chromium.org ,vogelheim@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:474921, chromium:616032
Review-Url: https://codereview.chromium.org/2042693003
Cr-Commit-Position: refs/heads/master@{#36741}
2016-06-06 11:04:26 +00:00
machenbach
46d8293f50
Revert of [build] Fix default for target_arch (patchset #1 id:1 of https://codereview.chromium.org/2040803005/ )
...
Reason for revert:
sysroot logic breaks "make arm"
Original issue's description:
> [build] Fix default for target_arch
>
> Fix after https://codereview.chromium.org/2028623002/
>
> BUG=chromium:474921, chromium:616032
> TBR=vogelheim@chromium.org , jochen@chromium.org
> NOTRY=true
>
> Committed: https://crrev.com/9dcb67dd174fefc98f9c964ef54fed8fe6feb224
> Cr-Commit-Position: refs/heads/master@{#36733}
TBR=vogelheim@chromium.org ,jochen@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:474921, chromium:616032
Review-Url: https://codereview.chromium.org/2039853002
Cr-Commit-Position: refs/heads/master@{#36740}
2016-06-06 11:01:42 +00:00
machenbach
9dcb67dd17
[build] Fix default for target_arch
...
Fix after https://codereview.chromium.org/2028623002/
BUG=chromium:474921, chromium:616032
TBR=vogelheim@chromium.org , jochen@chromium.org
NOTRY=true
Review-Url: https://codereview.chromium.org/2040803005
Cr-Commit-Position: refs/heads/master@{#36733}
2016-06-06 09:12:43 +00:00
machenbach
8ee22392f8
[build] Roll android_tools to 04c2c06
...
This ports:
https://codereview.chromium.org/1991343002
BUG=chromium:599327
Review-Url: https://codereview.chromium.org/2040913002
Cr-Commit-Position: refs/heads/master@{#36730}
2016-06-06 08:48:59 +00:00
machenbach
c47d3c6369
[build] Use sysroot for linux compilation with clang
...
This ports the configuration for using a sysroot from
chromium's common.gypi.
This is restricted to clang only.
BUG=chromium:474921, chromium:616032
LOG=y
Review-Url: https://codereview.chromium.org/2028623002
Cr-Commit-Position: refs/heads/master@{#36729}
2016-06-06 08:41:03 +00:00
jochen
09d90e4e70
Reland "Move (hopefully) remaining isolate related variables to toolchain.gypi"
...
BUG=chromium:609107
R=machenbach@chromium.org
Review-Url: https://codereview.chromium.org/2032733003
Cr-Commit-Position: refs/heads/master@{#36677}
2016-06-02 09:55:44 +00:00
jochen
a8f57df4f4
Remove has_valgrind.py - the bots now set this flag explicitly
...
BUG=none
R=machenbach@chromium.org
NOTRY=true
Review-Url: https://codereview.chromium.org/2034643002
Cr-Commit-Position: refs/heads/master@{#36673}
2016-06-02 09:02:56 +00:00
machenbach
dab67f0331
Revert of Move (hopefully) remaining isolate related variables to toolchain.gypi (patchset #1 id:1 of https://codereview.chromium.org/2027213002/ )
...
Reason for revert:
Fails:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/builds/7182/steps/gclient%20runhooks/logs/stdio
Original issue's description:
> Move (hopefully) remaining isolate related variables to toolchain.gypi
>
> BUG=chromium:609107
> R=machenbach@chromium.org
>
> Committed: https://crrev.com/a68a1eb38dda0a1eba3b7644a172d2f85e9c7abe
> Cr-Commit-Position: refs/heads/master@{#36664}
TBR=jochen@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:609107
Review-Url: https://codereview.chromium.org/2038463002
Cr-Commit-Position: refs/heads/master@{#36665}
2016-06-02 07:10:36 +00:00
jochen
a68a1eb38d
Move (hopefully) remaining isolate related variables to toolchain.gypi
...
BUG=chromium:609107
R=machenbach@chromium.org
Review-Url: https://codereview.chromium.org/2027213002
Cr-Commit-Position: refs/heads/master@{#36664}
2016-06-02 06:52:56 +00:00
jochen
34fbcd70ae
Move gcmole to toolchain.gypi
...
it's referenced from isolate.gypi which in turn is used in
non-standalone builds as well.
BUG=chromium:609107
R=machenbach@chromium.org
Review-Url: https://codereview.chromium.org/2023783003
Cr-Commit-Position: refs/heads/master@{#36591}
2016-05-30 14:33:17 +00:00
machenbach
db5246fadf
[build] Enable -Wformat-pedantic compiler warning
...
For cross-compiler-compatibility and standards compliance %p
requires a void*, rather than any pointer type.
This was fixed in:
https://codereview.chromium.org/2001073002/
BUG=chromium:474921
Review-Url: https://codereview.chromium.org/2011483002
Cr-Commit-Position: refs/heads/master@{#36473}
2016-05-24 12:23:27 +00:00
machenbach
daae44f69b
[build] Enable -Wlong-long compiler warning.
...
BUG=chromium:474921
Review-Url: https://codereview.chromium.org/2004063003
Cr-Commit-Position: refs/heads/master@{#36471}
2016-05-24 11:53:44 +00:00
jkummerow
148e7076ca
Turn on -Wmissing-field-initializers on Linux.
...
Because not initializing fields can be, you know, dangerous.
Review-Url: https://codereview.chromium.org/1952703002
Cr-Commit-Position: refs/heads/master@{#36071}
2016-05-06 10:20:30 +00:00
machenbach
3bf44848c3
[gn] Move build to gypfiles
...
This prepares for pulling chromium's build as dependency for
gn. After this, the files in build and gypfiles need to stay
in sync until chromium is updated.
BUG=chromium:474921
LOG=n
Review-Url: https://codereview.chromium.org/1848553003
Cr-Commit-Position: refs/heads/master@{#35898}
2016-04-29 10:11:11 +00:00