- 03 Jun, 2016 23 commits
-
-
machenbach authored
This adds corresponding *_run targets for all swarming isolate actions existing in gyp. This also wires all targets together under gn_all. BUG=chromium:474921 NOTRY=true Review-Url: https://codereview.chromium.org/2033813004 Cr-Commit-Position: refs/heads/master@{#36714}
-
machenbach authored
This adds the v8-side fuzzer executables for smoke testing. This also renames the old gyp targets to stay consistent with chromium. Naming convention for type X after the rename: library: X_fuzzer (gn), X_fuzzer_lib (gyp) executable v8: v8_simple_X_fuzzer executable chromium: v8_X_fuzzer BUG=chromium:474921 Review-Url: https://codereview.chromium.org/2032363002 Cr-Commit-Position: refs/heads/master@{#36713}
-
bbudge authored
- Adds names for float registers to RegisterConfiguration and uses them when we have the MachineRepresentation. LOG=N BUG=v8:4124 Review-Url: https://codereview.chromium.org/2030143002 Cr-Commit-Position: refs/heads/master@{#36712}
-
yangguo authored
R=littledan@chromium.org Review-Url: https://codereview.chromium.org/2021013002 Cr-Commit-Position: refs/heads/master@{#36711}
-
mstarzinger authored
This implements propagation of frame states from checkpoints to all deoptimization points inserted by the EffectControlLinearizer. It also allows us to remove the eager frame state input from all the checked conversion operations. R=jarin@chromium.org BUG=v8:5021 Review-Url: https://codereview.chromium.org/2033143002 Cr-Commit-Position: refs/heads/master@{#36710}
-
ahaas authored
R=titzer@chromium.org, bmeurer@chromium.org BUG=v8:5065 Review-Url: https://codereview.chromium.org/2036143002 Cr-Commit-Position: refs/heads/master@{#36709}
-
rmcilroy authored
BUG=v8:4280 NOTRY=true Review-Url: https://codereview.chromium.org/2033393002 Cr-Commit-Position: refs/heads/master@{#36708}
-
rmcilroy authored
Add intrinsics for IsSmi, IsTypedArray, IsRegExp and IsJSProxy, all of which are intrinsics in Full-Codegen. BUG=v8:4280 LOG=N Review-Url: https://codereview.chromium.org/2034493002 Cr-Commit-Position: refs/heads/master@{#36707}
-
rmcilroy authored
BUG=v8:4751 TBR=machenbach@chromium.org Review-Url: https://codereview.chromium.org/2033773003 Cr-Commit-Position: refs/heads/master@{#36706}
-
mstarzinger authored
These speculative binary operators are simplified operators and should not need a frame state themselves. These eager bailout points can by now be found via checkpoints in the graph, whereas frame states attached to nodes directly should always represent lazy bailout points. R=jarin@chromium.org BUG=v8:5021 Review-Url: https://codereview.chromium.org/2037673002 Cr-Commit-Position: refs/heads/master@{#36705}
-
zhengxing.li authored
port 817b59c8 (r36654) original commit message: Adds instructions for ARM to push floats. BUG= Review-Url: https://codereview.chromium.org/2036773004 Cr-Commit-Position: refs/heads/master@{#36704}
-
bmeurer authored
Introduce a dedicated Float64Log machine operator, that is either implemented by a direct C call or by platform specific code, i.e. using the FPU on x64 and ia32. This operator is used to implement Math.log as a proper TurboFan builtin on top of the CodeStubAssembler. Also introduce a NumberLog simplified operator on top of Float64Log and use that for the fast inline path of Math.log inside TurboFan optimized code. BUG=v8:5065 Review-Url: https://codereview.chromium.org/2029413005 Cr-Commit-Position: refs/heads/master@{#36703}
-
zhengxing.li authored
port 471893cc (r36649) original commit message: GenerateSmiToDouble on ia32 assumes that it is called from a JSFrame and can restore the context from the StandardFrameConstants::kContextObject. In the case of the interpreter it is called from a interpreter handler stub frame which doesn't push the context onto it's frame. Instead, push and pop esi to explicitly restore it correctly. BUG= Review-Url: https://codereview.chromium.org/2036083003 Cr-Commit-Position: refs/heads/master@{#36702}
-
zhengxing.li authored
port 63ea3a50 (r36599) original commit message: Previously, we used the lowest bit for something else. BUG= Review-Url: https://codereview.chromium.org/2032063003 Cr-Commit-Position: refs/heads/master@{#36701}
-
zhengxing.li authored
port 56d90782 (r36597) original commit message: In Crankshaft, we would install special ICs that didn't need a vector and slot in the MEGAMORPHIC case. This optimization limits our hand against future improvements. BUG= Review-Url: https://codereview.chromium.org/2030303002 Cr-Commit-Position: refs/heads/master@{#36700}
-
mstarzinger authored
This removes the frame state input representing the before-state from nodes performing property accesses. These frame states can by now be found via checkpoints in the graph. R=jarin@chromium.org BUG=v8:5021 Review-Url: https://codereview.chromium.org/2034673002 Cr-Commit-Position: refs/heads/master@{#36699}
-
Miran.Karic authored
A value in unittests/Bytecodes.DecodeBytecodeAndOperands was 16 bit instead of 8 bit. This was causing test to fail on big endian machines. BUG= Review-Url: https://codereview.chromium.org/2030063002 Cr-Commit-Position: refs/heads/master@{#36698}
-
zhengxing.li authored
x87 FPU converts the SNaN to QNaN automatically when loading SNaN from memmory. This function caused v8 x87 port can't distinguish the Hole NaN (V8 used SNaN for it) from Javascript visible NaNs (V8 used QNaN for it). Many test cases failed in this function for v8 x87 port. It's a big effort to refactor all code of x87 FPU loads value from memmory to fix this issue. So here's a temporary workaround for it, what's this CL does are: 1. Removed all previous x87 workaround of this issue. 2. Used SNaN of MIPS which is a not used QNaN in v8 x87 port as the Hole NaN for v8 x87 port. 3. This CL is only local to x87 port. BUG= Review-Url: https://codereview.chromium.org/2033133004 Cr-Commit-Position: refs/heads/master@{#36697}
-
ahaas authored
We need the host address in another CL to clear invalid slots. R=ulan@chromium.org Review-Url: https://codereview.chromium.org/2025833002 Cr-Commit-Position: refs/heads/master@{#36696}
-
brucedawson authored
VC++ complains about truncation of integer constants despite use of static_cast. This isn't very helpful as it gives no way of suppressing the warning in code, so this change suppresses it on the command line. Additionally, the linker complains about importing of locally defined functions in component builds. Until this is fixed the warnings should be suppressed. NOTRY=true Review-Url: https://codereview.chromium.org/2028353004 Cr-Commit-Position: refs/heads/master@{#36695}
-
machenbach authored
BUG=chromium:474921 Review-Url: https://codereview.chromium.org/2034633004 Cr-Commit-Position: refs/heads/master@{#36694}
-
ivica.bogosavljevic authored
that do not support unaligned access. This test fails because WasmGraphBuilder::BuildCFuncInstruction allocates space for doubles using StackSlot turbofan operator, but this space is not guaranteed to be 8 bytes aligned if SP itself is not 8 bytes aligned (which is the case on 32-bit architectures). BUG=mjsunit/wasm/embenchen/lua_binarytrees Review-Url: https://codereview.chromium.org/2034523003 Cr-Commit-Position: refs/heads/master@{#36693}
-
v8-autoroll authored
Rolling v8/build to de44ba22d7b4cd7c4285be52a4491f19a9f6c864 Rolling v8/tools/clang to 749adbcdf8277f1b305ee55f921d9246d4aedd45 TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review-Url: https://codereview.chromium.org/2037593003 Cr-Commit-Position: refs/heads/master@{#36692}
-
- 02 Jun, 2016 17 commits
-
-
jyan authored
R=joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com, bjaideep@ca.ibm.com BUG= Review-Url: https://codereview.chromium.org/2036523003 Cr-Commit-Position: refs/heads/master@{#36691}
-
danno authored
Previously, CodeAssembler Variables declared in an explicit C++ scope would continue to be merged into future labels beyond that scope, causing asserts. This CL ensures that Variables are properly ignored when they go out of scope. Review-Url: https://codereview.chromium.org/2035683002 Cr-Commit-Position: refs/heads/master@{#36690}
-
danno authored
Previously, turbofan selected the gap use from first predecessor block when hinting a phi, unless that block was deferred, in which case the gap move from the first non-deferred predecessor block was chosen. This strategy didn't guarantee that an important invariant was maintained: the predecessor blocks chosen for hinting phis must preceed the phi's block in the rpo ordering. In most cases the strategy worked, since graphs generated by the AstGraphBuilder and existing stubs just happened to always generate schedules where this rpo ordering property for the first predecessor block, but it is quite possible to generate a code stub by hand that doesn't have this property (see included test case). After this CL, the allocator chooses either the the first non-deferred "rpo-preceeding" block to be the hinting block, or the first deferred "rpo-preceeding" block if that doesn't exist. In all previously-existing code, this behavior is the same as the original algorithm, but has the benefit of not failing in the register allocator in hand-crafted stubs where all the "rpo-preceeding" predecessors are all in deferred code. Review-Url: https://codereview.chromium.org/2030463003 Cr-Commit-Position: refs/heads/master@{#36689}
-
gsathya authored
The PromiseSet operation is called with two types of promises 1) A newly created promise object with no properties 2) Promise object with callbacks and other properties PromiseSet is called with the first type of promise (with no properties) from multiple call sites. PromiseSet is called with the second type of promise object only from FulfillPromise. Furthermore, this call only sets the value and status of the promise, the rest of the values are reset to UNDEFINED (which isn't necessary). This patch inlines the calls to set the value and status of the promise in FulfillPromise, instead of calling out to PromiseSet. This patch also reduces the number of symbol lookups, as we only set the value and status of the promise, and don't change the callback or deferred symbols. This patch results in a performance improvement of 2.8% over 5 runs in the bluebird benchmark. BUG=v8:5046 Review-Url: https://codereview.chromium.org/2025073002 Cr-Commit-Position: refs/heads/master@{#36688}
-
bjaideep authored
Port a2fef3af Original commit message: This CL introduces a DECLARE_DEFAULT_DESCRIPTOR macro that helps defining a CallInterfaceDescriptor in a cases where it is not important which registers to use for passing arguments. One can use such descriptors for new TurboFan stubs. HasPropertyDescriptor now uses the new machinery. R=ishell@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com BUG= LOG=N Review-Url: https://codereview.chromium.org/2034773002 Cr-Commit-Position: refs/heads/master@{#36687}
-
ishell authored
[stubs] Extend HasProperty stub with dictionary-mode, string wrapper and double-elements objects support. This CL also replaces some Branch() usages with GotoIf/GotoUnless. (This is a reland after fixing issues that prevented this CL from landing in other CLs). BUG=v8:2743 LOG=Y Committed: https://crrev.com/24066b6df4259b302edfa1db884c479008776a7e Cr-Commit-Position: refs/heads/master@{#36657} Review-Url: https://codereview.chromium.org/1995453002 Cr-Commit-Position: refs/heads/master@{#36686}
-
jyan authored
R=joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com, bjaideep@ca.ibm.com BUG= Review-Url: https://codereview.chromium.org/2033103002 Cr-Commit-Position: refs/heads/master@{#36685}
-
ishell authored
... to ensure that the zone is properly shut down. Review-Url: https://codereview.chromium.org/2034463003 Cr-Commit-Position: refs/heads/master@{#36684}
-
ishell authored
... since CodeStubAssembler does not belong to v8::internal::compiler namespace anymore. Review-Url: https://codereview.chromium.org/2035533003 Cr-Commit-Position: refs/heads/master@{#36683}
-
bjaideep authored
Some system header files on AIX include inttypes.h without defining __STDC_FORMAT_MACROS and therefore the printf format specifier macro (for eg. PRId64) doesn't get defined as they are guarded with __STDC_FORMAT_MACROS macro on AIX. This error showed up recently when the format specifier was used in wasm-interpreter.cc, where a AIX system header file is included which also includes inttypes.h without defining __STDC_FORMAT_MACROS. R=ahaas@chromium.org, titzer@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com BUG= LOG=N Review-Url: https://codereview.chromium.org/2033483002 Cr-Commit-Position: refs/heads/master@{#36682}
-
jkummerow authored
Most maps have a small code cache (often only one entry), so this patch optimizes memory consumption of such cases by using plain FixedArrays, only switching to CodeCacheHashTables when the number of cached entries gets so large that linear-scan lookups get too slow. On loading inbox.google.com, this gets the aggregate size of all maps' code caches (there are about 13,600 of them) from 4300 KB to 970 KB. Review-Url: https://codereview.chromium.org/2021373002 Cr-Commit-Position: refs/heads/master@{#36681}
-
mvstanton authored
It should work just the same as Compile() and CompileOptimized(). R=mstarzinger@chromium.org BUG= Review-Url: https://codereview.chromium.org/2033053002 Cr-Commit-Position: refs/heads/master@{#36680}
-
mstarzinger authored
This removes the frame state input representing the before-state from nodes having the {JSCallRuntime} operator. These frame states can by now be found via checkpoints in the graph. It also makes the predicate for runtime function ids (i.e. {Linkage::NeedsFrameStateInput}) binary. R=jarin@chromium.org BUG=v8:5021 Review-Url: https://codereview.chromium.org/2018353004 Cr-Commit-Position: refs/heads/master@{#36679}
-
mythria authored
Object statistics were collected during the mark compact phase. If an incremental marking happened before mark compact phase then most of the objects are already visited and hence this phase does not collect accurate statistics. This cl updates incremental marking pass to collect object statistics along with mark compact phase. BUG= Review-Url: https://codereview.chromium.org/1943423002 Cr-Commit-Position: refs/heads/master@{#36678}
-
jochen authored
BUG=chromium:609107 R=machenbach@chromium.org Review-Url: https://codereview.chromium.org/2032733003 Cr-Commit-Position: refs/heads/master@{#36677}
-
neis authored
In one corner case, we incorrectly returned a value without first wrapping it in an iterator result object. R=littledan@chromium.org BUG=v8:5057 Review-Url: https://codereview.chromium.org/2034653002 Cr-Commit-Position: refs/heads/master@{#36676}
-
ishell authored
This CL introduces a DECLARE_DEFAULT_DESCRIPTOR macro that helps defining a CallInterfaceDescriptor in a cases where it is not important which registers to use for passing arguments. One can use such descriptors for new TurboFan stubs. HasPropertyDescriptor now uses the new machinery. Review-Url: https://codereview.chromium.org/2002143002 Cr-Commit-Position: refs/heads/master@{#36675}
-