- 09 Nov, 2015 17 commits
-
-
mstarzinger authored
This redcues the noise created by --trace-turbo-inlining when there actually are no candidates being processed. R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/1410723004 Cr-Commit-Position: refs/heads/master@{#31885}
-
zhengxing.li authored
port 7c3396d0 (r31871) original commit message: Introduce receiver conversion mode specialization for the Call and CallFunction builtins, so we can specialize the builtin functionality (actually an optimization only) based on static information from the callsite (this is basically a superset of the optimizations that were available with the CallFunctionStub and CallICStub, except that these optimizations are correct now). This fixes a regression introduced by the removal of CallFunctionStub, for programs that call a lot. BUG= Review URL: https://codereview.chromium.org/1431133002 Cr-Commit-Position: refs/heads/master@{#31884}
-
jacob.bramley authored
BUG= Review URL: https://codereview.chromium.org/1429063003 Cr-Commit-Position: refs/heads/master@{#31883}
-
caitpotter88 authored
Remove some non-standard code that doesn't do anything anyways. While FireFox uses this to set the default value for the multiline flag, it is nonstandard and slated for removal. The matching behaviour has never been implemented in either JSC or V8, so there is little web-compat risk. The only possible risk could be someone depending on the ToBoolean() behaviour of the flag, but this seems unlikely. BUG=v8:3870 LOG=N R=adamk@chromium.org, littledan@chromium.org, yangguo@chromium.org CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1417733012 Cr-Commit-Position: refs/heads/master@{#31882}
-
Michael Achenbach authored
Cr-Commit-Position: refs/heads/master@{#31881}
-
bmeurer authored
R=jarin@chromium.org Review URL: https://codereview.chromium.org/1413783010 Cr-Commit-Position: refs/heads/master@{#31880}
-
mstarzinger authored
This switches loading and storing of the message object within the Isolate to use JavaScript operators built by the JSOperatorBuilder instead of machine operators. This is a preparation for a stricter representation selection for loads and stores. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1412443010 Cr-Commit-Position: refs/heads/master@{#31879}
-
balazs.kilvady authored
BUG= Review URL: https://codereview.chromium.org/1431813002 Cr-Commit-Position: refs/heads/master@{#31878}
-
ulan authored
BUG=chromium:553051 LOG=NO Review URL: https://codereview.chromium.org/1433663002 Cr-Commit-Position: refs/heads/master@{#31877}
-
mstarzinger authored
This separates the post-processing step for optimized code maps out of the CodeFlusher. It uses the complete SharedFunctionInfo::Iterator to visit all candidates instead of gathering candidates during marking. Gathering candidates during marking no longer makes sense, now that the majority of SharedFunctionInfo objects will hold such an optimized code map. Also it reduces complexity of the implementation. Also conflating this mechanism with "code flushing" was confusing. This reverts commit 7f1fb29f. R=ulan@chromium.org Review URL: https://codereview.chromium.org/1418453008 Cr-Commit-Position: refs/heads/master@{#31876}
-
bmeurer authored
Introduce Reducer::Finalize, which get's called by the GraphReducer once all reductions are done, and use this to implement full inlining as part of the regular reducer fixpoint. R=jarin@chromium.org BUG=v8:4493 LOG=n Review URL: https://codereview.chromium.org/1419373012 Cr-Commit-Position: refs/heads/master@{#31875}
-
jacob.bramley authored
BUG= Review URL: https://codereview.chromium.org/1414283009 Cr-Commit-Position: refs/heads/master@{#31874}
-
bmeurer authored
The %StringCharFromCode and %CharFromCode runtime function perform exactly the same task, so we need only one of them. R=yangguo@chromium.org Review URL: https://codereview.chromium.org/1417743007 Cr-Commit-Position: refs/heads/master@{#31873}
-
yangguo authored
R=bmeurer@chromium.org BUG=v8:4536 LOG=N Review URL: https://codereview.chromium.org/1427743008 Cr-Commit-Position: refs/heads/master@{#31872}
-
bmeurer authored
Introduce receiver conversion mode specialization for the Call and CallFunction builtins, so we can specialize the builtin functionality (actually an optimization only) based on static information from the callsite (this is basically a superset of the optimizations that were available with the CallFunctionStub and CallICStub, except that these optimizations are correct now). This fixes a regression introduced by the removal of CallFunctionStub, for programs that call a lot. R=yangguo@chromium.org BUG=chromium:552244 LOG=n Review URL: https://codereview.chromium.org/1436493002 Cr-Commit-Position: refs/heads/master@{#31871}
-
jarin authored
Review URL: https://codereview.chromium.org/1425143008 Cr-Commit-Position: refs/heads/master@{#31870}
-
mtrofin authored
While working on frame elision, I wanted to disassemble codegen in the debugger, as the code generation is progressing. I discovered we had a "Print" member on the x64 assembler, without any implementation. I pulled it up to AssemblerBase and gave it an implementation that should work for the other architectures. Also checked that ia32, x87, arm and arm64 assemblers didn't have such an implementation - free Print. Arm64 has a naming conflict with the v8::internal::Disassembler. I renamed the arm64 type with a more specific name. Opportunistically fixed a bug in the name converter. This debug-time printer doesn't provide a Code object, which should be OK with the name converters, by the looks of other APIs there. All this means is that when using the Print() API, we just get addresses dumped without any context (like what this address may be - a stub maybe, etc). This seems fine for the scenario. There may be other places that assume a Code object. Since this is a diagnostics-only scenario, for codegen developers, I feel it is reasonable to fix such other places as we find them. Review URL: https://codereview.chromium.org/1431933003 Cr-Commit-Position: refs/heads/master@{#31869}
-
- 08 Nov, 2015 1 commit
-
-
https://codereview.chromium.org/1422793004/pkotwicz authored
BUG=chromium:547162 LOG=Y Review URL: https://codereview.chromium.org/1412933009 Cr-Commit-Position: refs/heads/master@{#31868}
-
- 07 Nov, 2015 2 commits
-
-
ishell authored
JSRegExp's properties backing stores must not be shared. BUG=chromium:548580 LOG=N Review URL: https://codereview.chromium.org/1429743006 Cr-Commit-Position: refs/heads/master@{#31867}
-
martyn.capewell authored
Use compare-negate instruction if the right-hand input to a compare is a negate operation. BUG= Review URL: https://codereview.chromium.org/1410123009 Cr-Commit-Position: refs/heads/master@{#31866}
-
- 06 Nov, 2015 18 commits
-
-
rmcilroy authored
IC::GetSharedFunctionInfo get's the JS frame which called the IC. When the function is running in the interpreter, there is a Bytecode handler stub frame between the IC and the JS frame. Modify IC::GetSharedFunctionInfo to handle this. BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1420283020 Cr-Commit-Position: refs/heads/master@{#31865}
-
rmcilroy authored
Adds a blacklist of tests which are currently unsupported or broken in Ignition to the mjsunit and test262 test status. Also removes --ignition-script-filter flag, and adds a --ignition_fallback_on_eval_and_catch flag which fallsback to fullcodegen for functions which call eval or contain a catch block. BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1420963009 Cr-Commit-Position: refs/heads/master@{#31864}
-
adamk authored
This allows testing the configuration where --harmony-default-parameters is passed without --harmony-destructuring. R=rossberg@chromium.org BUG=v8:2160 LOG=n Review URL: https://codereview.chromium.org/1410023015 Cr-Commit-Position: refs/heads/master@{#31863}
-
rmcilroy authored
BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1419273008 Cr-Commit-Position: refs/heads/master@{#31862}
-
yangguo authored
R=ulan@chromium.org Review URL: https://codereview.chromium.org/1408003012 Cr-Commit-Position: refs/heads/master@{#31861}
-
mstarzinger authored
Now that the VisitCode visitor is actually marking through to inlined code objects (as opposed to the VisitJSFunction visitor), we can make this helper method private again. R=ulan@chromium.org Review URL: https://codereview.chromium.org/1422853008 Cr-Commit-Position: refs/heads/master@{#31860}
-
rmcilroy authored
BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1406253008 Cr-Commit-Position: refs/heads/master@{#31859}
-
ahaas authored
R=titzer@chromium.org Review URL: https://codereview.chromium.org/1413463009 Cr-Commit-Position: refs/heads/master@{#31858}
-
mstarzinger authored
This fixes a corner case where optimized code could still be reachable through the optimized code map, even though unoptimized code for any inlined function might have long been flushed. R=ulan@chromium.org TEST=cctest/test-heap/Regress513496 BUG=chromium:513496 LOG=n Review URL: https://codereview.chromium.org/1415683011 Cr-Commit-Position: refs/heads/master@{#31857}
-
bmeurer authored
We forgot to add the number of arguments parameter in JSFrameSpecialization, which was added before the context. R=jarin@chromium.org BUG=chromium:552304 LOG=n Review URL: https://codereview.chromium.org/1429233004 Cr-Commit-Position: refs/heads/master@{#31856}
-
hablich authored
Revert of [turbofan] Enable general purpose inlining. (patchset #2 id:20001 of https://codereview.chromium.org/1416873007/ ) Reason for revert: As discussed with Benedikt: Disable --turbo-inlining again because it tanks Jetstream. Original issue's description: > [turbofan] Enable general purpose inlining. > > R=bmeurer@chromium.org,hablich@chromium.org > BUG=v8:4493 > LOG=n > > Committed: https://crrev.com/161a0462fb0e79f155f3ea085ba774dad99539cb > Cr-Commit-Position: refs/heads/master@{#31634} TBR=bmeurer@chromium.org,mstarzinger@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:4493 Review URL: https://codereview.chromium.org/1409153008 Cr-Commit-Position: refs/heads/master@{#31855}
-
rmcilroy authored
Adds a test that the receiver for sloppy mode functions is replaced with the global proxy when called with an undefined receiever. BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1410113008 Cr-Commit-Position: refs/heads/master@{#31854}
-
ulan authored
if that is responsible for memory regression in UMA. Related CLs: crrev.com/1420363004, crrev.com/1423453003 TBR=hpayer@chromium.org BUG=chromium:552305 LOG=NO Review URL: https://codereview.chromium.org/1420283021 Cr-Commit-Position: refs/heads/master@{#31853}
-
mstarzinger authored
This avoids the need to clear optimized code maps within the GC by just preventing entries being added in the first place, whenever a snapshot is being constructed. The main goal here is to simplify the logic in the already complex visitor for our shared function info objects. R=yangguo@chromium.org Review URL: https://codereview.chromium.org/1411513005 Cr-Commit-Position: refs/heads/master@{#31852}
-
hablich authored
This will be used as a data source for an UMA histogram. LOG=N BUG=chromium:546603 R=jochen@chromium.org,yangguo@chromium.org Review URL: https://codereview.chromium.org/1413503007 Cr-Commit-Position: refs/heads/master@{#31851}
-
ahaas authored
This change does not change how the BufferedRawMachineAssemblerTester is used in tests, but it makes its construction in other constructors (e.g. in the WasmRunner) cleaner. R=titzer@chromium.org Review URL: https://codereview.chromium.org/1414203009 Cr-Commit-Position: refs/heads/master@{#31850}
-
zhengxing.li authored
port 4ab1b05d (r31826). original commit message: Depends on https://codereview.chromium.org/1407373007/ After that CL, all receiver conversion is handled by the call builtin. BUG= Review URL: https://codereview.chromium.org/1419563006 Cr-Commit-Position: refs/heads/master@{#31849}
-
zhengxing.li authored
port 44c44521 (r31823). original commit message: This fixes receiver conversion since the Call builtin does it correctly. BUG= Review URL: https://codereview.chromium.org/1416673009 Cr-Commit-Position: refs/heads/master@{#31848}
-
- 05 Nov, 2015 2 commits
-
-
paul.lind authored
Use 64-bit load of receiver in 4ab1b05d. BUG= Review URL: https://codereview.chromium.org/1412693003 Cr-Commit-Position: refs/heads/master@{#31847}
-
adamk authored
It was originally shipped in https://crrev.com/eef2b9b09723ba1dae3ec0172341e93e9030ada0, but was reverted due to poor interaction with Blink. That interaction seems to be fixed thanks to changes to the V8 API and to @@toStringTag handling on access-checked objects. BUG=v8:3502 LOG=y CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.v8:v8_linux_nosnap_rel;tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1406293011 Cr-Commit-Position: refs/heads/master@{#31846}
-