- 22 Jul, 2016 16 commits
-
-
jgruber authored
This will get a new call site in an upcoming commit, move it to String so it's accessible from without runtime-strings.cc. BUG= Review-Url: https://codereview.chromium.org/2176553002 Cr-Commit-Position: refs/heads/master@{#37974}
-
rmcilroy authored
The effect control linearizer runs off-thread and needs the ToNumber call descriptor. Add a hack to create this on the main thread to ensure off-thread's don't race on it's construction. Also add a DCHEK to the CallInterfaceDescriptor to ensure they are always initilaized on the main thread. Review-Url: https://codereview.chromium.org/2168843003 Cr-Commit-Position: refs/heads/master@{#37973}
-
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}
-
bmeurer authored
So far we don't have a useful way to inline Math.max or Math.min in TurboFan optimized code. This adds new operators NumberMax and NumberMin and changes the Float64Max/Float64Min operators to have JavaScript semantics instead of the C++ semantics that it had previously. This also removes support for recognizing the tenary case in the CommonOperatorReducer, since that doesn't seem to have any positive impact (and actually doesn't show up in regular JavaScript, where people use Math.max/Math.min instead). Drive-by-fix: Also nuke the unused Float32Max/Float32Min operators. R=jarin@chromium.org Review-Url: https://codereview.chromium.org/2170343002 Cr-Commit-Position: refs/heads/master@{#37971}
-
dpranke authored
This re-lands r37926 w/ the needed fix for cross-compiles; we can only re-use the default toolchain when the host can actually run it. R=machenbach@chromium.org BUG=629825 Review-Url: https://codereview.chromium.org/2171083003 Cr-Commit-Position: refs/heads/master@{#37970}
-
jgruber authored
This allows us to skip complicated logic for setting the accessors. BUG= Review-Url: https://codereview.chromium.org/2164903004 Cr-Commit-Position: refs/heads/master@{#37969}
-
yangguo authored
NOTRY=true TBR=littledan@chromium.org Review-Url: https://codereview.chromium.org/2171233002 Cr-Commit-Position: refs/heads/master@{#37968}
-
hpayer authored
BUG=630386 LOG=n Review-Url: https://codereview.chromium.org/2160613002 Cr-Commit-Position: refs/heads/master@{#37967}
-
yangguo authored
This is in preparation to implementing exception prediction for async functions. Each handler table entry can now predict "caught", "uncaught", or "promise". The latter indicates that the exception will lead to a promise rejection. To mark the relevant try-catch blocks, we add a new native syntax. try { } %catch (e) { } indicates a TryCatchStatement with the "promise" prediction. The previous implementation of using the function to tell the relevant try-catch apart from inner try-catch blocks will not work for async functions since these can have inner try-catch blocks inside the same function. BUG=v8:5167 Review-Url: https://codereview.chromium.org/2161263003 Cr-Commit-Position: refs/heads/master@{#37966}
-
bmeurer authored
We can compute the absolute integer value w/o any conditional execution by using the bit trick formula let sign = input >> 31 in (input ^ sign) - sign which generates fairly decent code on all supported architectures. R=jarin@chromium.org Review-Url: https://codereview.chromium.org/2169293002 Cr-Commit-Position: refs/heads/master@{#37965}
-
v8-autoroll authored
Rolling v8/build to 5782f1c84fc41934d265f69e5bd61badbf61e5c5 Rolling v8/tools/mb to c0f2da01e7e7e530fcbbf3823b7c7655632f05b1 TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review-Url: https://codereview.chromium.org/2171153002 Cr-Commit-Position: refs/heads/master@{#37964}
-
zhengxing.li authored
port 4b59bf53 (r37934) original commit message: Use the ForInFilterStub directly. Hence we will only jump to the runtime for special receivers (instance_type <= LAST_SPECIAL_RECEIVER_TYPE) and for converting element indices which are not in the string cache. BUG= Review-Url: https://codereview.chromium.org/2176473002 Cr-Commit-Position: refs/heads/master@{#37963}
-
zhengxing.li authored
port 66cb026f (r37929) original commit message: Original message: Calling Runtime::kAbort through a builtin instead of the c-entry stub will allow to generate the call in a background thread, because a builtin provides its own handle, whereas a code stub does not. @v8-mips-ports: Could you take a special look at the padding that is done in MacroAssembler::Abort()? Reason for revert: The reason for reverting is: Blocks roll: https://build.chromium.org/p/client.v8.ports/builders/V8%20Linux%20-%20arm64%20-%20sim%20-%20debug/builds/1622 The problem was that on arm64 the builtin for Abort() contained a call to Abort(). The problem is fixed by using a NoUseRealAbortsScope in the code generation of Abort(). BUG= Review-Url: https://codereview.chromium.org/2172093002 Cr-Commit-Position: refs/heads/master@{#37962}
-
ritesht authored
Revert "[wasm] Adding a convolution matrix filter test to highlight the performance advantages of JITing" GC-Stress asserts in filter-jit. This reverts commit ccfd224e. BUG=v8:5044 R=bradnelson@chromium.org Review-Url: https://codereview.chromium.org/2168343002 Cr-Commit-Position: refs/heads/master@{#37961}
-
bjaideep authored
Port 8aeb7439 Fix for ppc file, using macro functions to xor and add to handle the case when kPrimaryMagic/kSecondaryMagic is > 16bits. 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/2169183002 Cr-Commit-Position: refs/heads/master@{#37960}
-
jwolfe authored
See discussion in https://codereview.chromium.org/2156303002/#msg8 With the new --harmony-function-tostring behavior, these tests would fail without this change. This change makes the tests pass regardless of whether or not --harmony-function-tostring is used. All of these changes are simply inserting a space after the "function" keyword to match the current function toString behavior. When --harmony-function-tostring is enabled, the toString behavior matches the spacing used in the function declaration. With the declaration matching the current formatting, the toString behavior becomes unaffected by --harmony-function-tostring. BUG=v8:4958 LOG=n Review-Url: https://codereview.chromium.org/2161413002 Cr-Commit-Position: refs/heads/master@{#37959}
-
- 21 Jul, 2016 24 commits
-
-
adamk authored
R=littledan@chromium.org Review-Url: https://codereview.chromium.org/2172723003 Cr-Commit-Position: refs/heads/master@{#37958}
-
ritesht authored
This cl also fixes two bugs in the previous code: 1) JITed functions were not allowed access to the heap because the module instance wasn't correctly synthesized. This wasn't discovered in the previous test. 2) Decoding of functions with the JITSingleFunction opcode was off by 1 as the length of the opcode wasn't computed correctly. BUG=5044 Review-Url: https://codereview.chromium.org/2168183002 Cr-Commit-Position: refs/heads/master@{#37957}
-
bjaideep authored
Port e83739c0 Fix applies to PPC/s390 as well. R=jacob.bramley@arm.com, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com BUG=v8:5214 LOG=N Review-Url: https://codereview.chromium.org/2167273003 Cr-Commit-Position: refs/heads/master@{#37956}
-
ivica.bogosavljevic authored
BUG=v8:5213 Review-Url: https://codereview.chromium.org/2163963003 Cr-Commit-Position: refs/heads/master@{#37955}
-
jpp authored
BUG= https://bugs.chromium.org/p/chromium/issues/detail?id=628803 BUG= https://bugs.chromium.org/p/v8/issues/detail?id=4203 TEST= cctest/asmjs/test-asm-typer.cc LOG= N Review-Url: https://codereview.chromium.org/2172603002 Cr-Commit-Position: refs/heads/master@{#37954}
-
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}
-
rmcilroy authored
Review-Url: https://codereview.chromium.org/2168913002 Cr-Commit-Position: refs/heads/master@{#37952}
-
dpranke authored
I had written "mipsel64", not "mips64el". R=machenbach@chromium.org, milko.leporis@imgtec.com BUG=629057 Review-Url: https://codereview.chromium.org/2167873002 Cr-Commit-Position: refs/heads/master@{#37951}
-
jpp authored
BUG= https://bugs.chromium.org/p/chromium/issues/detail?id=628450 BUG= https://bugs.chromium.org/p/v8/issues/detail?id=4203 TEST= cctest/asmjs/test-asm-typer.cc TEST= mjsunit/wasm/* LOG= N Review-Url: https://codereview.chromium.org/2164273002 Cr-Commit-Position: refs/heads/master@{#37950}
-
jgruber authored
We no longer need to prepare the stack overflow error in advance now that Errors are constructed in C++. R=yangguo@chromium.org BUG= Committed: https://crrev.com/ba95d10ccbe13e2fca427228483b045576f2dc4c Review-Url: https://codereview.chromium.org/2161953003 Cr-Original-Commit-Position: refs/heads/master@{#37923} Cr-Commit-Position: refs/heads/master@{#37949}
-
yangguo authored
R=jgruber@chromium.org Review-Url: https://codereview.chromium.org/2168883002 Cr-Commit-Position: refs/heads/master@{#37948}
-
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}
-
ishell authored
BUG=chromium:618701 Review-Url: https://codereview.chromium.org/2167493003 Cr-Commit-Position: refs/heads/master@{#37946}
-
titzer authored
R=ahaas@chromium.org,rossberg@chromium.org,bradnelson@chromium.org BUG= Review-Url: https://codereview.chromium.org/2165633006 Cr-Commit-Position: refs/heads/master@{#37945}
-
bmeurer authored
This is never passed to the Typer, and actually wouldn't work anyways, since we cannot derive any meaningful types for Parameters in JavaScript. R=mstarzinger@chromium.org Review-Url: https://codereview.chromium.org/2171723002 Cr-Commit-Position: refs/heads/master@{#37944}
-
marija.antic authored
Add sign extension for Mips64Shr and Mips64Sar operators. BUG= Review-Url: https://codereview.chromium.org/2154703002 Cr-Commit-Position: refs/heads/master@{#37943}
-
yangguo authored
R=mstarzinger@chromium.org BUG=chromium:629996 Review-Url: https://codereview.chromium.org/2166123003 Cr-Commit-Position: refs/heads/master@{#37942}
-
verwaest authored
Replace the zonelist with a link from a scope to any of its inner scopes, and a link to any sibling scope. This makes scopes that track inner scopes use roughly the same amount of space as previously scopes without inner scopes would use for the empty zonelist (pointer to the memory + length field, which, granted could be slightly smaller on 64bit). BUG=v8:5209 Review-Url: https://codereview.chromium.org/2162143005 Cr-Commit-Position: refs/heads/master@{#37941}
-
cbruni authored
Only start checking if new keys are shadowed after the first prototype has added non-enumerable shadow keys. This helps minimally in some corner cases if there are few enumerable properties on the prototype compared to the receiver. BUG=chromium:628173 Review-Url: https://codereview.chromium.org/2169523002 Cr-Commit-Position: refs/heads/master@{#37940}
-
titzer authored
R=ahaas@chromium.org, rossberg@chromium.org BUG= Review-Url: https://codereview.chromium.org/2170773003 Cr-Commit-Position: refs/heads/master@{#37939}
-
marja authored
It's anyway just the "same" AstNodeFactory (i.e., it's passed the same AstValueFactory), so no need to have several of them for each FunctionState. R=verwaest@chromium.org BUG= Review-Url: https://codereview.chromium.org/2169823002 Cr-Commit-Position: refs/heads/master@{#37938}
-
weiliang.lin authored
BUG= Review-Url: https://codereview.chromium.org/2161513002 Cr-Commit-Position: refs/heads/master@{#37937}
-
machenbach authored
Revert of Fix double-building of v8 in GN builds when setting just v8_target_cpu. (patchset #1 id:1 of https://codereview.chromium.org/2166173002/ ) Reason for revert: Breaks: https://build.chromium.org/p/client.v8.fyi/builders/V8%20Android%20GN%20%28dbg%29/builds/4590 And also the trybot: https://build.chromium.org/p/tryserver.chromium.android/builders/android_clang_dbg_recipe/builds/99806 Original issue's description: > Fix double-building of v8 in GN builds when setting just v8_target_cpu. > > Because of the somewhat strange way default toolchains and custom > toolchains and user-specified arguments work in GN, if you did a v8 > build that just set v8_target_cpu, you could end up building two > identical copies of v8 (see the comments in the change for more). > > This CL identifies that case and fixes it. > > R=machenbach@chromium.org > BUG=629825 > > Committed: https://crrev.com/3536db45c9409c9aadc4eee6004cf337c0588cdb > Cr-Commit-Position: refs/heads/master@{#37926} TBR=dpranke@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=629825 Review-Url: https://codereview.chromium.org/2167113002 Cr-Commit-Position: refs/heads/master@{#37936}
-
bmeurer authored
Use better names for the query methods on the Truncation class, that express more clearly what you intend to query. R=jarin@chromium.org Review-Url: https://codereview.chromium.org/2171703002 Cr-Commit-Position: refs/heads/master@{#37935}
-