- 05 Aug, 2016 3 commits
-
-
machenbach authored
BUG=v8:5238 NOTREECHECKS=true Review-Url: https://codereview.chromium.org/2214283003 Cr-Commit-Position: refs/heads/master@{#38381}
-
machenbach authored
This makes test outcomes immutable. This has two benefits: 1) It's more robust. Using the |= operator by mistake wouldn't lead to unwanted mutation. 2) It's faster as now the set reference can be copied again when copying test cases. Test case copying happens an order of magnitude more often than rules do apply. BUG=v8:5238 Review-Url: https://codereview.chromium.org/2212333002 Cr-Commit-Position: refs/heads/master@{#38373}
-
machenbach authored
BUG=v8:5238 Review-Url: https://codereview.chromium.org/2216293002 Cr-Commit-Position: refs/heads/master@{#38362}
-
- 04 Aug, 2016 2 commits
-
-
machenbach authored
This adds the possibility to address test cases in the status file with the variant under which the test is running. This is only allowed in top-level sections. Example: [{ 'test-case': [PASS, SLOW], }] ['variant == foo', { 'test-case': [FAIL], }] The test case "test-case" is marked as slow in all variants. Additionally, in variant foo, it'll be expected to fail. This CL also exemplifies the new feature with test cases running under the ignition_turbofan variant. The corresponding legacy flag is deprecated. BUG=v8:5238 Review-Url: https://codereview.chromium.org/2203013002 Cr-Commit-Position: refs/heads/master@{#38342}
-
hablich authored
Ignition benchmarks will be switched to --ignition-staging. TBR=machenbach@chromium.org NOTRY=true NOTREECHECKS=true NOPRESUBMIT=true Review-Url: https://codereview.chromium.org/2210873002 Cr-Commit-Position: refs/heads/master@{#38322}
-
- 03 Aug, 2016 1 commit
-
-
jochen authored
This will allow for the background parser to parse inner functions BUG=v8:5215 R=marja@chromium.org,verwaest@chromium.org Review-Url: https://codereview.chromium.org/2198043002 Cr-Commit-Position: refs/heads/master@{#38291}
-
- 02 Aug, 2016 5 commits
-
-
machenbach authored
This prepares for adding the ignition_staging variant to the default variant set. BUG=v8:5238 Review-Url: https://codereview.chromium.org/2205883002 Cr-Commit-Position: refs/heads/master@{#38257}
-
machenbach authored
This is not referenced anymore by the infrastructure after: https://codereview.chromium.org/2202113003/ BUG=v8:5238 Review-Url: https://codereview.chromium.org/2203023003 Cr-Commit-Position: refs/heads/master@{#38254}
-
machenbach authored
It should be enough to specify --variants=... with the desired set of variants. The stress/nostress shortcuts might get confusing soon, as we intend to move the stress option out of the developer default set. BUG=v8:5238 Review-Url: https://codereview.chromium.org/2195373002 Cr-Commit-Position: refs/heads/master@{#38248}
-
machenbach authored
Without this fix, two rules [PASS, MODIFIER] and [FAIL] would be joined as [PASS, MODIFIER, FAIL], undermining the intention of the second rule to mark a test as failing. BUG=v8:4680,v8:4900 Review-Url: https://codereview.chromium.org/2201033002 Cr-Commit-Position: refs/heads/master@{#38238}
-
Michael Achenbach authored
Cr-Commit-Position: refs/heads/master@{#38225}
-
- 01 Aug, 2016 4 commits
-
-
machenbach authored
This introduces aliases for testing variants that can be combined with other variant names. E.g. --variants=dev,foo would run the three developer default variants and variant foo. We'll have three stages: "dev" for variants to be run by default on developer workstations, "more" for additional variants, executed on all bots, "extra" for additional variants executed on a subset of bots (e.g. not on very slow or otherwise resource-limited bots). BUG=v8:5238 NOTRY=true Review-Url: https://codereview.chromium.org/2196223002 Cr-Commit-Position: refs/heads/master@{#38214}
-
machenbach authored
The flag was introduced for ignition development. It can only be used when running ignition tests in isolation on the bots. The bots only use ignition_turbo in isolation since a while and don't pass the --ignition flag anymore. BUG=v8:5238 Review-Url: https://codereview.chromium.org/2197123002 Cr-Commit-Position: refs/heads/master@{#38206}
-
machenbach authored
First step of deprecating the dedicated ignition alias. Next it will be changed on the bots to use the bot_default suite. Then we'll delete it. BUG=v8:5238 Review-Url: https://codereview.chromium.org/2194153002 Cr-Commit-Position: refs/heads/master@{#38202}
-
yangguo authored
BUG=chromium:580337 R=machenbach@chromium.org Review-Url: https://codereview.chromium.org/2189023002 Cr-Commit-Position: refs/heads/master@{#38198}
-
- 29 Jul, 2016 2 commits
-
-
bgeron authored
NOTRY=true Review-Url: https://codereview.chromium.org/2197593002 Cr-Commit-Position: refs/heads/master@{#38178}
-
machenbach authored
Required for https://codereview.chromium.org/2194823002/ BUG=chromium:631979 NOTRY=true Review-Url: https://codereview.chromium.org/2189323002 Cr-Commit-Position: refs/heads/master@{#38161}
-
- 28 Jul, 2016 5 commits
-
-
ishell authored
[gdb] Define print functions used by gdb macros in the top level namespace to make them always be available. This should fix the following errors that sometimes appear when using V8 gdb macros: Cannot evaluate function -- may be inlined or No symbol "xxx" in namespace "v8::internal". Review-Url: https://codereview.chromium.org/2186293002 Cr-Commit-Position: refs/heads/master@{#38145}
-
Alexander.Gilday2 authored
Change was meant to be in previous patch but was accidentally reverted. README.md updated with new filename, also. BUG= Review-Url: https://codereview.chromium.org/2184223005 Cr-Commit-Position: refs/heads/master@{#38134}
-
machenbach authored
Splitt of CL: https://codereview.chromium.org/2188693002/#ps100001 Switching to GN is blocked on test errors. These files are required for compilation and allow looking into the test errors locally. BUG=chromium:474921 NOTRY=true Review-Url: https://codereview.chromium.org/2190863003 Cr-Commit-Position: refs/heads/master@{#38130}
-
Alexander.Gilday2 authored
perf-turbo.py merges a perf data file and a turbofan trace file into a single json object which can then be piped to a file and uploaded to turbolizer to display the profiling data in the disassembly. With the changes, turbolizer now shows the event counts for instruction in percentage form and with heatmap-stype colouring. Multiple different events can be recorded at once with a new drop-down menu to select which event to view the counts of. The documentation has been updated with instructions. Using the script is optional and turbolizer retains previous functionality if a trace without profiling data is uploaded. BUG=None Review-Url: https://codereview.chromium.org/2174803002 Cr-Commit-Position: refs/heads/master@{#38124}
-
jochen authored
Use it like this: Program received signal SIGTRAP, Trace/breakpoint trap. [Switching to Thread 0x7f50d02c4700 (LWP 8152)] base::debug::(anonymous namespace)::DebugBreak () at ../../base/debug/debugger_posix.cc:230 230 } (gdb) bt at ../../base/debug/debugger_posix.cc:230 ... args_object=0x7f50d02bfe00, isolate=0x2f001c9e2020) at ../../v8/src/builtins/builtins-api.cc:123 ... (gdb) jss (gdb) bt isolate=0x2f001c9e2020, is_construct=false, target=..., receiver=..., argc=1, args=0x7f50d02c0518, new_target=...) at ../../v8/src/execution.cc:111 isolate=0x2f001c9e2020, callable=..., receiver=..., argc=1, argv=0x7f50d02c0518) at ../../v8/src/execution.cc:168 ... R=yangguo@chromium.org BUG= Review-Url: https://codereview.chromium.org/2186013002 Cr-Commit-Position: refs/heads/master@{#38121}
-
- 27 Jul, 2016 2 commits
-
-
qyearsley authored
This is intended to be submitted just after http://crrev.com/2174553002, which renames these builders. BUG=590036 NOTRY=true Review-Url: https://codereview.chromium.org/2181133002 Cr-Commit-Position: refs/heads/master@{#38105}
-
yangguo authored
Review-Url: https://codereview.chromium.org/2175193003 Cr-Commit-Position: refs/heads/master@{#38081}
-
- 26 Jul, 2016 4 commits
-
-
tandrii authored
Revert of [release] Change blink trybot name on v8 roll CLs (patchset #1 id:1 of https://codereview.chromium.org/2185513002/ ) Reason for revert: Revert, because blink tryserver bot rename is reverted. BUG=chromium:631448 Original issue's description: > [release] Change blink trybot name on v8 roll CLs > > BUG=chromium:590036 > NOTRY=true > > Committed: https://crrev.com/a5fae1039409864295b42a6f33cef85ca9396bda > Cr-Commit-Position: refs/heads/master@{#38041} TBR=hablich@chromium.org,machenbach@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:590036 Review-Url: https://codereview.chromium.org/2186593003 Cr-Commit-Position: refs/heads/master@{#38062}
-
bgeron authored
BUG= R=danno Review-Url: https://codereview.chromium.org/2171543004 Cr-Commit-Position: refs/heads/master@{#38045}
-
machenbach authored
BUG=chromium:590036 NOTRY=true Review-Url: https://codereview.chromium.org/2185513002 Cr-Commit-Position: refs/heads/master@{#38041}
-
mstarzinger authored
This flag is aiming at shipping the ability to generate optimized code directly from bytecode (without re-parsing source code). All features needed to ship such a configuration will be staged behind this flag. R=hablich@chromium.org,rmcilroy@chromium.org Review-Url: https://codereview.chromium.org/2174333002 Cr-Commit-Position: refs/heads/master@{#38040}
-
- 25 Jul, 2016 5 commits
-
-
bgeron authored
/ also selects the search box. BUG= Review-Url: https://codereview.chromium.org/2169053002 Cr-Commit-Position: refs/heads/master@{#38027}
-
bgeron authored
BUG= R=danno Review-Url: https://codereview.chromium.org/2169043002 Cr-Commit-Position: refs/heads/master@{#38025}
-
bgeron authored
If you dragged the node out of the bounding box, this commit allows you to see it again after you zoom. The zoom looks jittery, but I think it's better than not being able to see it at all. BUG= R=danno Review-Url: https://codereview.chromium.org/2168713005 Cr-Commit-Position: refs/heads/master@{#38024}
-
bgeron authored
BUG= R=danno Review-Url: https://codereview.chromium.org/2169023002 Cr-Commit-Position: refs/heads/master@{#38004}
-
bgeron authored
- Remove duplicated CSS. - Expand landing area of upload button. - Give the toolbox a transparent white background. - Make the phase-select box look nicer before file load. - Remove the margin at the top of the screen. The height of the margin was exactly the height of span#graph-toolbox, and due to the semantics of position:relative, space was reserved for it at the top of div#middle. BUG= R=danno Review-Url: https://codereview.chromium.org/2164373002 Cr-Commit-Position: refs/heads/master@{#38003}
-
- 22 Jul, 2016 2 commits
-
-
jochen authored
BUG= R=hpayer@chromium.org Review-Url: https://codereview.chromium.org/2171313002 Cr-Commit-Position: refs/heads/master@{#37979}
-
hablich authored
The auto-tag bot removes the need to handle version changes in each merge individually. As a result this 'feature' is removed. BUG=v8:4408 R=machenbach@chromium.org LOG=N NOTRY=true Review-Url: https://codereview.chromium.org/1398033003 Cr-Commit-Position: refs/heads/master@{#37972}
-
- 21 Jul, 2016 2 commits
-
-
mattloring authored
Update the custom objdump script to handle inline comments starting with '--' or ';;'. Load d8 code.asm file if present. BUG= Review-Url: https://codereview.chromium.org/2159103007 Cr-Commit-Position: refs/heads/master@{#37953}
-
machenbach authored
Helper script to generate gn arguments based on common developer defaults or builder configurations. BUG=chromium:625791 NOTRY=true Review-Url: https://codereview.chromium.org/2138693002 Cr-Commit-Position: refs/heads/master@{#37947}
-
- 20 Jul, 2016 1 commit
-
-
mattloring authored
Allows jitted code to be annotated in perf. To use: 1) Report generated code in code-(pid)-1.asm. perf record -- d8 --print-code --redirect-code-traces script.js 2) Run perf report from the directory containing the above code file. perf report --objdump=v8/tools/objdump-v8 This script supports generated code from both crankshaft or turbofan. BUG= Review-Url: https://codereview.chromium.org/2167553002 Cr-Commit-Position: refs/heads/master@{#37913}
-
- 19 Jul, 2016 1 commit
-
-
klaasb authored
This adds an additional column with percentages to the output of bytecode_dispatch_report.py --top-dispatches-for-bytecode. The percentages always represent the relative number of dispatches to the target bytecode to all dispatches from the source bytecode. The additional flag --sort-sources-relative/-r allows sorting the "Top sources of dispatches to" the given bytecode by this column to more easily find bytecodes that significantly often dispatch to the target. BUG=v8:4899 LOG=N Review-Url: https://codereview.chromium.org/2159683003 Cr-Commit-Position: refs/heads/master@{#37873}
-
- 18 Jul, 2016 1 commit
-
-
bangfu.tao authored
BUG= A bug in android-run.py, which caused the android_arm.release.check throws error: data/local/tmp/v8/test/mjsunit/wasm/function-prototype.js:7: Error loading file load("test/mjsunit/wasm/wasm-constants.js"); Review-Url: https://codereview.chromium.org/2070323002 Cr-Commit-Position: refs/heads/master@{#37825}
-