- 11 Mar, 2016 12 commits
-
-
m0609.shim authored
After histrogram timer added time recaling functionality, some events, e.g. parse, histogram timer generates event log ending with 'MicroSeconds'. Since ProfViz can't recorgnize it, this patch cuts off 'MicroSeconds' postfix. R=vogelheim@chromium.org, yangguo@chromium.org BUG=chromium: LOG=N Review URL: https://codereview.chromium.org/1771293002 Cr-Commit-Position: refs/heads/master@{#34710}
-
danno authored
Review URL: https://codereview.chromium.org/1777013004 Cr-Commit-Position: refs/heads/master@{#34709}
-
mstarzinger authored
This also runs the message test suite against Ignition. By now most of the source positions (and exception messages) are accurate, the failing ones have been blacklisted. R=machenbach@chromium.org Review URL: https://codereview.chromium.org/1783773003 Cr-Commit-Position: refs/heads/master@{#34708}
-
cbruni authored
Avoid creating and initializing timer scopes for builtins as well. BUG=chromium:589679 LOG=N Review URL: https://codereview.chromium.org/1784833002 Cr-Commit-Position: refs/heads/master@{#34707}
-
machenbach authored
Revert of Replace PushIfAbsent by a Stack object and move StringBuilderJoin to JS (patchset #6 id:100001 of https://codereview.chromium.org/1775403008/ ) Reason for revert: [Sheriff] This lets a gc stress test time out: https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20gc%20stress/builds/2337/steps/Mjsunit%20%28flakes%29/logs/regress-542823 The test ran in <2sec before this. Please fix the test as well on a reland. Original issue's description: > Replace PushIfAbsent by a Stack object and move StringBuilderJoin to JS > > This significantly speeds up String(array). > BUG= > > Committed: https://crrev.com/c91faa0b39b62025460eb9f8b578e20d88f3549e > Cr-Commit-Position: refs/heads/master@{#34696} TBR=cbruni@chromium.org,adamk@chromium.org,bmeurer@chromium.org,verwaest@chromium.org # 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/1785003004 Cr-Commit-Position: refs/heads/master@{#34706}
-
yangguo authored
R=jkummerow@chromium.org BUG=v8:4810 LOG=N Review URL: https://codereview.chromium.org/1780173002 Cr-Commit-Position: refs/heads/master@{#34705}
-
v8-autoroll authored
Rolling v8/tools/clang to 96f43fa6c1232981bc401c0b6f2e2697a318356b TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review URL: https://codereview.chromium.org/1781183002 Cr-Commit-Position: refs/heads/master@{#34704}
-
yangguo authored
TBR=littledan@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/1783853003 Cr-Commit-Position: refs/heads/master@{#34703}
-
yangguo authored
R=littledan@chromium.org BUG=v8:4810 LOG=N Review URL: https://codereview.chromium.org/1780183002 Cr-Commit-Position: refs/heads/master@{#34702}
-
ahaas authored
R=binji@chromium.org Review URL: https://codereview.chromium.org/1779123002 Cr-Commit-Position: refs/heads/master@{#34701}
-
zhengxing.li authored
port 5096492f (r34627) original commit message: - Eliminate stubs with a variable number of arguments. (That only worked due to their very limited use. These stubs' interface descriptors were basically lying about their number of args, which will fail when used generically.) - Fix all CallApi*Stubs' interface descriptors to no longer lie about their arguments. - Unify CallApi*Stub, for * in Function, Accessor, FunctionWithFixedArgs. (Since these are now all doing the same thing.) - Rename the unified stub (and interface descriptors) to *ApiCallback*, since that's really what they're doing. - Refuse inlining an API callback if its number of parameters exceeds the supported number of args. BUG= Review URL: https://codereview.chromium.org/1784623006 Cr-Commit-Position: refs/heads/master@{#34700}
-
adamk authored
Now that the destructuring flags are gone, we always call CheckAndRewriteReferenceExpression, so unified the two methods again. Also cleaned up the code within, e.g. removing unnecessary Scanner::Location construction. Review URL: https://codereview.chromium.org/1781933003 Cr-Commit-Position: refs/heads/master@{#34699}
-
- 10 Mar, 2016 28 commits
-
-
littledan authored
- Make separate iterator functions for TypedArrays which do a type check (and in the future should check for detached TypedArrays) - Share the toString method with Arrays BUG=v8:4785 R=adamk LOG=Y Review URL: https://codereview.chromium.org/1780113002 Cr-Commit-Position: refs/heads/master@{#34698}
-
littledan authored
When the TypedArray iterator constructor code path was added, a technically incorrect optimization was added which uses the arraylike code path on array arguments, even though the %ArrayIterator% could have been monkey-patched to give different behavior. A previous patch removed this optimization, and this patch restores it due to the huge performance regression that resulted. The optimization was previously removed in https://codereview.chromium.org/1767893002 BUG=chromium:593644 R=adamk LOG=Y Review URL: https://codereview.chromium.org/1779373002 Cr-Commit-Position: refs/heads/master@{#34697}
-
verwaest authored
This significantly speeds up String(array). BUG= Review URL: https://codereview.chromium.org/1775403008 Cr-Commit-Position: refs/heads/master@{#34696}
-
yangguo authored
R=littledan@chromium.org BUG=v8:4743 LOG=N Review URL: https://codereview.chromium.org/1774623005 Cr-Commit-Position: refs/heads/master@{#34695}
-
mbrandy authored
Port 240b7db9 R=ahaas@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1780283002 Cr-Commit-Position: refs/heads/master@{#34694}
-
littledan authored
This check is guaranteed by the Promise spec and tested by test262 tests. It only has to run for subclasses. This patch adds the check to the Promise code. BUG=v8:4633 R=adamk LOG=Y Review URL: https://codereview.chromium.org/1780823003 Cr-Commit-Position: refs/heads/master@{#34693}
-
adamk authored
These flags have been on by default since version 4.9, which has been in stable Chrome for over a week now, demonstrating that they're here to stay. Also moved the tests out of harmony/ and into es6/. Review URL: https://codereview.chromium.org/1776683003 Cr-Commit-Position: refs/heads/master@{#34692}
-
neis authored
The comment was correct but the code didn't match :( Must not close when .value throws. R=adamk@chromium.org BUG=v8:3566 LOG=n Review URL: https://codereview.chromium.org/1778333002 Cr-Commit-Position: refs/heads/master@{#34691}
-
neis authored
R=littledan@chromium.org BUG= Review URL: https://codereview.chromium.org/1779133002 Cr-Commit-Position: refs/heads/master@{#34690}
-
ssanfilippo authored
LOG=N Review URL: https://codereview.chromium.org/1776853003 Cr-Commit-Position: refs/heads/master@{#34689}
-
kozyatinskiy authored
This method was added as part of DevTools experiment. Experiment UI was removed in https://codereview.chromium.org/201293007 2 years ago. Experiment backend was removed in https://codereview.chromium.org/1785533002/. R=yangguo@chromium.org LOG=N Review URL: https://codereview.chromium.org/1770383006 Cr-Commit-Position: refs/heads/master@{#34688}
-
mlippautz authored
Move evacuation of new and old space pages to the provided framework for parallelization. Drive-by-fix: Remove left overs from POPULAR_PAGE flag. BUG=chromium:524425 LOG=N Review URL: https://codereview.chromium.org/1782043004 Cr-Commit-Position: refs/heads/master@{#34687}
-
jfb authored
Copy-pasta error. R= titzer@chromium.org Review URL: https://codereview.chromium.org/1778383003 Cr-Commit-Position: refs/heads/master@{#34686}
-
Michael Achenbach authored
Cr-Commit-Position: refs/heads/master@{#34685}
-
verwaest authored
Use internal handles as API, and move boilerplate code into the call wrappers. BUG= Review URL: https://codereview.chromium.org/1776913005 Cr-Commit-Position: refs/heads/master@{#34684}
-
ahaas authored
With the original implementation nodes which were not reachable from end could cause the verification to fail. R=titzer@chromium.org, mstarzinger@chromium.org Review URL: https://codereview.chromium.org/1782863002 Cr-Commit-Position: refs/heads/master@{#34683}
-
titzer authored
R=mstarzinger@chromium.org,bradnelson@chromium.org BUG= Review URL: https://codereview.chromium.org/1782613003 Cr-Commit-Position: refs/heads/master@{#34682}
-
mbrandy authored
Correct handling of the bottom frame in DoComputeArgumentsAdaptorFrame. R=ishell@chromium.org, jarin@chromium.org, michael_dawson@ca.ibm.com BUG=v8:4698 LOG=N Review URL: https://codereview.chromium.org/1784633002 Cr-Commit-Position: refs/heads/master@{#34681}
-
cbruni authored
With the flag enables we print the keys found on each prototype and the number of prototypes. BUG= Review URL: https://codereview.chromium.org/1779523002 Cr-Commit-Position: refs/heads/master@{#34680}
-
zhengxing.li authored
Reason for revert: The original x87 related codes of the CL #34627 (https://codereview.chromium.org/1748123003) had a mistake and caused the x87 debug build fail. Need to fix it and add it into the x87 port of the CL #34627 BUG= Review URL: https://codereview.chromium.org/1778293003 Cr-Commit-Position: refs/heads/master@{#34679}
-
machenbach authored
This adds a function to the sancov_merger that merges the output files of one swarming shard in parallel. This will be called from the infrastructure when collecting/merging swarming results. The tool will be called subsequently for each shard. On the first call, the target dir will be empty and the sancov files will just be moved. BUG=chromium:568949 LOG=n NOTRY=true Review URL: https://codereview.chromium.org/1782843002 Cr-Commit-Position: refs/heads/master@{#34678}
-
mstarzinger authored
This is because compiler.cc is awesome. There are cases where we do not yet have a SharedFunctionInfo that can tell us whether we are compiling a generator function, we query the FunctionLiteral instead. R=rmcilroy@chromium.org BUG=v8:4681 LOG=n Review URL: https://codereview.chromium.org/1780193002 Cr-Commit-Position: refs/heads/master@{#34677}
-
rossberg authored
This is old code, and the same pattern appears in other places. I have no idea what the checker's problem is all of a sudden... TBR=machenbach@chromium.org BUG= Review URL: https://codereview.chromium.org/1785693002 Cr-Commit-Position: refs/heads/master@{#34676}
-
mstarzinger authored
This ensures our optimizing compilers as well as the interpreter are never tasked with compiling the generator-resuming builtin methods. The corresponding intrinsics for those methods are not supported and it is not possible to provide a C++ reference implementation for them. We do this by assigning builtin function ids to them that we can recognize during the compiler dispatch. Note that this also affects the interpreter, because methods having a builtin function id assigned are not interpreted ({function_data} field is overlapping). If this ever changes we can still do an early check in the compiler dispatch (similar to the optimizing compilers) easily. This applies to the following methods: - Generator.prototype.next (calls Runtime_GeneratorNext). - Generator.prototype.return (calls Runtime_GeneratorReturn). - Generator.prototype.throw (calls Runtime_GeneratorThrow). R=neis@chromium.org BUG=v8:4681 LOG=n Review URL: https://codereview.chromium.org/1779123003 Cr-Commit-Position: refs/heads/master@{#34675}
-
joransiu authored
Add S390 platform specific \#includes across various common files. Add S390 CPU features to enum. Add S390 implementation to extract sp/fp/pc from signal context. R=danno@chromium.org,jkummerow@chromium.org,jochen@chromium.org,jyan@ca.ibm.com,michael_dawson@ca.ibm.com,mbrandy@us.ibm.com BUG= Review URL: https://codereview.chromium.org/1777593003 Cr-Commit-Position: refs/heads/master@{#34674}
-
balazs.kilvady authored
Make the low level assembler implementation exact and protected to disallow explicit usage. BUG= Review URL: https://codereview.chromium.org/1749263002 Cr-Commit-Position: refs/heads/master@{#34673}
-
mstarzinger authored
This ensures the interpreter is not tasked with compiling generator functions. It currently does not support suspending activations at yielding points, but we still want to be able to activate it for the rest of JavaScript in the meantime. R=rmcilroy@chromium.org BUG=v8:4681 LOG=n Review URL: https://codereview.chromium.org/1782013002 Cr-Commit-Position: refs/heads/master@{#34672}
-
machenbach authored
BUG=chromium:535160, chromium:568949 LOG=n TBR=tandrii@chromium.org, kjellander@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/1777263002 Cr-Commit-Position: refs/heads/master@{#34671}
-