- 14 Aug, 2015 1 commit
-
-
vogelheim authored
- Make the API look like v8::V8::InitializeICU. (That is: A static method call, not an object to be created on the stack.) - Fix path separator on Windows, by calling base::OS::isPathSeparator. - Move into API, so that it can be called by hello-world & friends. - Actually call it from hello-world and friends. R=jochen@chromium.org BUG= Review URL: https://codereview.chromium.org/1292053002 Cr-Commit-Position: refs/heads/master@{#30174}
-
- 13 Aug, 2015 1 commit
-
-
yangguo authored
Review URL: https://codereview.chromium.org/1285163003 Cr-Commit-Position: refs/heads/master@{#30144}
-
- 12 Aug, 2015 2 commits
-
-
yangguo authored
R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/1282793002 Cr-Commit-Position: refs/heads/master@{#30136}
-
mstarzinger authored
This tries to remove includes of "-inl.h" headers from normal ".h" headers, thereby reducing the chance of any cyclic dependencies and decreasing the average size of our compilation units. Note that this change still leaves 7 violations of that rule in the code. However there now is the "tools/check-inline-includes.sh" tool detecting such violations. R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/1283033003 Cr-Commit-Position: refs/heads/master@{#30125}
-
- 11 Aug, 2015 1 commit
-
-
adamk authored
The previous approach to this (in 1b1de2d2) did not work well with bots that intentionally run no tests. But I think this warning message is still useful for developers. Review URL: https://codereview.chromium.org/1281313004 Cr-Commit-Position: refs/heads/master@{#30116}
-
- 10 Aug, 2015 3 commits
-
-
machenbach authored
Revert of Make run-tests.py warn when it's not testing anything (patchset #1 id:1 of https://codereview.chromium.org/1283513003/ ) Reason for revert: Some bots that don't run tests (by design) don't return the test results json anymore which makes the infrastructure side fail now. Original issue's description: > Make run-tests.py warn when it's not testing anything > > I've often been confused by an "all passed" run of the test script > when it turned out that either all the tests I cared about were skipped > or, more likely, I mistyped the name(s) of the tests I wanted to run. > This patch aims to fix that (and gives a useful diagnostic for the > "all matched tests were skipped" case). > > R=machenbach@chromium.org > NOTRY=true > > Committed: https://crrev.com/1b1de2d221a0ed23e529e91bf9fa3dc287acb2ca > Cr-Commit-Position: refs/heads/master@{#30095} TBR=jkummerow@chromium.org,adamk@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1283593005 Cr-Commit-Position: refs/heads/master@{#30097}
-
adamk authored
I've often been confused by an "all passed" run of the test script when it turned out that either all the tests I cared about were skipped or, more likely, I mistyped the name(s) of the tests I wanted to run. This patch aims to fix that (and gives a useful diagnostic for the "all matched tests were skipped" case). R=machenbach@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/1283513003 Cr-Commit-Position: refs/heads/master@{#30095}
-
adamk authored
R=machenbach@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/1279613005 Cr-Commit-Position: refs/heads/master@{#30094}
-
- 07 Aug, 2015 2 commits
-
-
titzer authored
[turbofan] Remove architecture-specific linkage files and LinkageTraits. Use macro-assembler-defined constants. R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1272883003 Cr-Commit-Position: refs/heads/master@{#30063}
-
machenbach authored
BUG=chromium:511215 LOG=n Review URL: https://codereview.chromium.org/1276853002 Cr-Commit-Position: refs/heads/master@{#30057}
-
- 06 Aug, 2015 3 commits
-
-
Michael Achenbach authored
Cr-Commit-Position: refs/heads/master@{#30048}
-
mstarzinger authored
R=titzer@chromium.org Review URL: https://codereview.chromium.org/1276843004 Cr-Commit-Position: refs/heads/master@{#30046}
-
machenbach authored
Use the same path separator for test cases on the cmd-line on all platforms as in testcase.GetLabel, which is used to report failures. BUG=chromium:511215 LOG=n NOTRY=true Review URL: https://codereview.chromium.org/1281453003 Cr-Commit-Position: refs/heads/master@{#30045}
-
- 05 Aug, 2015 1 commit
-
-
yangguo authored
This is only an estimate since it counts objects that could be shared, for example strings, cow arrays, heap numbers, etc. It however ignores objects that could be shared, but may only be used by the context to be measured, for example shared function infos, script objects, scope infos, etc. R=jochen@chromium.org Review URL: https://codereview.chromium.org/1268333004 Cr-Commit-Position: refs/heads/master@{#30029}
-
- 04 Aug, 2015 3 commits
-
-
adamk authored
To avoid tanking context startup performance, only the actual installation of the JS-exposed API is flag-guarded. The remainder of the implementation still resides in the snapshot. Review URL: https://codereview.chromium.org/1257063003 Cr-Commit-Position: refs/heads/master@{#30017}
-
mtrofin authored
Setting up the stage for heuristics that preprocess live ranges before register allocation, and are independent of register allocation - e.g. the deferred blocks heuristic, or the split at call sites heuristic. Added a separate flag for this, since we intend to enable it for the linear allocator as well. Currently, the option is "on" for greedy, as a point in time to enable its testing (through the greedy allocator bots). BUG= Review URL: https://codereview.chromium.org/1256313003 Cr-Commit-Position: refs/heads/master@{#30005}
-
yangguo authored
This is a pure refactor and does not change functionality. R=ulan@chromium.org Review URL: https://codereview.chromium.org/1264993002 Cr-Commit-Position: refs/heads/master@{#29995}
-
- 03 Aug, 2015 1 commit
-
-
oth authored
The BytecodeArrayBuilder has responsibility for emitting the BytecodeArray. It will be used by the AST walker. Bytecode now uses an accumulator plus registers rather being pure register based. Update BytecodeArray::Disassemble to print operand information. BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1266713004 Cr-Commit-Position: refs/heads/master@{#29970}
-
- 31 Jul, 2015 1 commit
-
-
yangguo authored
R=cbruni@chromium.org Review URL: https://codereview.chromium.org/1265923002 Cr-Commit-Position: refs/heads/master@{#29951}
-
- 30 Jul, 2015 1 commit
-
-
titzer authored
This is the first step in cutting the Gordian linkage/linkage-impl knot. This basically changes the axis along which we organize call descriptor building logic from having platform-specific files dedicated to all call descriptor types to having call-descriptor-type-specific files that have The next step is to factor the JS, code stub, and runtime call descriptors similarly, dumping them into: compiler/js-linkage.cc compiler/runtime-linkage.cc compiler/code-stub-linkage.cc or, alternatively, all of them just into compiler/js-linkage.cc. This also anticipates a wasm-linkage.cc file in the future. R=bmeurer@chromium.org,danno@chromium.org BUG= Review URL: https://codereview.chromium.org/1266603002 Cr-Commit-Position: refs/heads/master@{#29931}
-
- 29 Jul, 2015 2 commits
-
-
machenbach authored
BUG=chromium:511215 NOTREECHECKS=true NOTRY=true LOG=n TBR=jkummerow@chromium.org Review URL: https://codereview.chromium.org/1262113002 Cr-Commit-Position: refs/heads/master@{#29900}
-
machenbach authored
This allows variants to be named on test failures (follow up) and then to be used in the test runner for a repro. This also speeds up variant iteration for test262 and fixes a bug with variants for benchmarks. BUG=chromium:511215 NOTREECHECKS=true LOG=n Review URL: https://codereview.chromium.org/1245623005 Cr-Commit-Position: refs/heads/master@{#29899}
-
- 27 Jul, 2015 1 commit
-
-
machenbach authored
BUG=chromium:267773 LOG=n NOTRY=true Review URL: https://codereview.chromium.org/1255183002 Cr-Commit-Position: refs/heads/master@{#29875}
-
- 24 Jul, 2015 4 commits
-
-
oth authored
BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1230753004 Cr-Commit-Position: refs/heads/master@{#29843}
-
yangguo authored
R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/1248443003 Cr-Commit-Position: refs/heads/master@{#29840}
-
Michael Achenbach authored
Cr-Commit-Position: refs/heads/master@{#29833}
-
Michael Achenbach authored
Cr-Commit-Position: refs/heads/master@{#29832}
-
- 23 Jul, 2015 3 commits
-
-
rmcilroy authored
Adds basic support for generation of interpreter bytecode handler code snippets. The InterpreterAssembler class exposes a set of low level, interpreter specific operations which can be used to build a Turbofan graph. The Interpreter class generates a bytecode handler snippet for each bytecode by assembling operations using an InterpreterAssembler. Currently only two simple bytecodes are supported: LoadLiteral0 and Return. BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1239793002 Cr-Commit-Position: refs/heads/master@{#29814}
-
machenbach authored
There are many test names in the v8 code base that prefix others, which makes it hard to only run those tests. BUG=chromium:511215 LOG=n NOTRY=true Review URL: https://codereview.chromium.org/1251363002 Cr-Commit-Position: refs/heads/master@{#29809}
-
mstarzinger authored
Using the GraphBuilder base class forces each node creation to go through a virtual function dispatch just for the sake of saving the duplication of the NewNode helper methods. In total that added up to saving minus (sic!) six lines of code. R=titzer@chromium.org Review URL: https://codereview.chromium.org/1252093002 Cr-Commit-Position: refs/heads/master@{#29799}
-
- 22 Jul, 2015 1 commit
-
-
jochen authored
The layout of fixed typed array base is then capable of handling external typed arrays as well. In a follow-up CL, I'll delete external typed arrays, and use fixed typed array base instead BUG=v8:3996 R=jarin@chromium.org,mstarzinger@chromium.org LOG=n Review URL: https://codereview.chromium.org/1248483007 Cr-Commit-Position: refs/heads/master@{#29786}
-
- 21 Jul, 2015 1 commit
-
-
aperez authored
The gdb-v8-support.py script uses the "re" module to match regular expression, but it does not import it, resulting in an error when loading the script from GDB. This patch adds the missing import. BUG= Review URL: https://codereview.chromium.org/1245713002 Cr-Commit-Position: refs/heads/master@{#29770}
-
- 20 Jul, 2015 2 commits
-
-
danno authored
In many cases, the context that TurboFan's ASTGraphBuilder or subsequent reduction operations attaches to nodes does not need to be that exact context, but rather only needs to be one with the same native context, because it is used internally only to fetch the native context, e.g. for creating and throwing exceptions. This reducer recognizes common cases where the context that is specified for a node can be relaxed to a canonical, less specific one. This relaxed context can either be the enclosing function's context or a specific Module or Script context that is explicitly created within the function. This optimization is especially important for TurboFan-generated code stubs which use context specialization and inlining to generate optimal code. Without context relaxation, many extraneous moves are generated to pass exactly the right context to internal functions like ToNumber and AllocateHeapNumber, which only need the native context. By turning context relaxation on, these moves disappear because all these common internal context uses are unified to the context passed into the stub function, which is typically already in the correct context register and remains there for short stubs. It also eliminates the explicit use of a specialized context constant in the code stub in these cases, which could cause memory leaks. Review URL: https://codereview.chromium.org/1244583003 Cr-Commit-Position: refs/heads/master@{#29763}
-
epertoso authored
R=jochen@chromium.org BUG=511204 LOG=y Review URL: https://codereview.chromium.org/1246603002 Cr-Commit-Position: refs/heads/master@{#29759}
-
- 17 Jul, 2015 1 commit
-
-
binji authored
BUG=chromium:497295 R=jarin@chromium.org LOG=n Review URL: https://codereview.chromium.org/1208933006 Cr-Commit-Position: refs/heads/master@{#29736}
-
- 16 Jul, 2015 2 commits
-
-
bbudge authored
This CL exposes the constructor function, defines type related information, and implements value type semantics. It also refactors test/mjsunit/samevalue.js to test SameValue and SameValueZero. TEST=test/mjsunit/harmony/simd.js, test/cctest/test-simd.cc LOG=Y BUG=v8:4124 Committed: https://crrev.com/e5ed3bee99807c502fa7d7a367ec401e16d3f773 Cr-Commit-Position: refs/heads/master@{#29689} Review URL: https://codereview.chromium.org/1219943002 Cr-Commit-Position: refs/heads/master@{#29712}
-
hablich authored
Revert of Expose SIMD.Float32x4 type to Javascript. (patchset #14 id:450001 of https://codereview.chromium.org/1219943002/) Reason for revert: Seems to brake the latest roll into Chromium: http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_compile_dbg_ng/builds/59796/steps/compile%20%28with%20patch%29/logs/stdio Original issue's description: > Expose SIMD.Float32x4 type to Javascript. > This CL exposes the constructor function, defines type related > information, and implements value type semantics. > It also refactors test/mjsunit/samevalue.js to test SameValue and SameValueZero. > > TEST=test/mjsunit/harmony/simd.js, test/cctest/test-simd.cc > > LOG=Y > BUG=v8:4124 > > Committed: https://crrev.com/e5ed3bee99807c502fa7d7a367ec401e16d3f773 > Cr-Commit-Position: refs/heads/master@{#29689} TBR=rossberg@chromium.org,littledan@chromium.org,martyn.capewell@arm.com,bbudge@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:4124 Review URL: https://codereview.chromium.org/1241533004 Cr-Commit-Position: refs/heads/master@{#29701}
-
- 15 Jul, 2015 2 commits
-
-
bbudge authored
This CL exposes the constructor function, defines type related information, and implements value type semantics. It also refactors test/mjsunit/samevalue.js to test SameValue and SameValueZero. TEST=test/mjsunit/harmony/simd.js, test/cctest/test-simd.cc LOG=Y BUG=v8:4124 Review URL: https://codereview.chromium.org/1219943002 Cr-Commit-Position: refs/heads/master@{#29689}
-
mvstanton authored
Gdb macro jfv on an object will print it as a feedback vector. Printouts look like this: DebugPrint: 0x5dc0d2ad: [TypeFeedbackVector] - length: 12 - ics with type info: 3 - generic ics: 0 ICSlot 0 CALL_IC MONOMORPHIC [4]: 0x5dc0d365 WeakCell for 0x5dc0cd69 <JS Function foo (SharedFunctionInfo 0x5dc0cb0d)> [5]: 0x4203c4c1 <Code: HANDLER> ICSlot 1 LOAD_IC MONOMORPHIC [6]: 0x5dc0d1f5 WeakCell for 0x3a710481 <Map(FAST_HOLEY_SMI_ELEMENTS)> [7]: 0x4203a1c1 <Code: HANDLER> ICSlot 2 LOAD_IC UNINITIALIZED [8]: 0x3060d045 <Symbol: 711234650 <String[20]: uninitialized_symbol>> [9]: 0x3060d045 <Symbol: 711234650 <String[20]: uninitialized_symbol>> ICSlot 3 LOAD_IC MONOMORPHIC [10]: 0x5dc0d3b5 WeakCell for 0x3a710d71 <Map(FAST_HOLEY_ELEMENTS)> [11]: 0x4202af01 <Code: HANDLER> BUG= Review URL: https://codereview.chromium.org/1225403005 Cr-Commit-Position: refs/heads/master@{#29679}
-
- 13 Jul, 2015 1 commit
-
-
Michael Achenbach authored
Cr-Commit-Position: refs/heads/master@{#29612}
-