- 17 Jun, 2016 8 commits
-
-
machenbach authored
BUG=chromium:619503 TBR=jochen@chromium.org, vogelheim@chromium.org, dpranke@chromium.org, thakis@chromium.org NOTRY=true Review-Url: https://codereview.chromium.org/2072203002 Cr-Commit-Position: refs/heads/master@{#37052}
-
jgruber authored
This adds a new BUILTIN frame type, which supports variable number of arguments for builtins implemented in hand-written native code (we will extend this mechanism to TurboFan builtins at some point). Convert the Math.max and Math.min builtins to construct a BUILTIN frame if required. This does not yet work for C++ builtins, but that'll be the next step. R=bmeurer@chromium.org, jarin@chromium.org BUG=v8:4815 LOG=n Review-Url: https://codereview.chromium.org/2069423002 Cr-Commit-Position: refs/heads/master@{#37051}
-
machenbach authored
Depends on https://codereview.chromium.org/2071583003/ to skip redundant clang=1 configs. BUG=chromium:616035 NOTRY=true Review-Url: https://codereview.chromium.org/2071793002 Cr-Commit-Position: refs/heads/master@{#37050}
-
mtrofin authored
This avoids introducing an include path outside v8. Instead, the path to where tracing includes are located is provided via a variable. Please refer to https://codereview.chromium.org/2068073002. Once the v8 change makes its way to chromium, we can remove the TODO in this current change. BUG= Review-Url: https://codereview.chromium.org/2061933002 Cr-Commit-Position: refs/heads/master@{#37049}
-
dpranke authored
R=jochen@chromium.org, machenbach@chromium.org, thakis@chromium.org BUG=chromium:619503 Review-Url: https://codereview.chromium.org/2074003002 Cr-Commit-Position: refs/heads/master@{#37048}
-
bmeurer authored
Import base::ieee754::exp() from FreeBSD msun and introduce a Float64Exp TurboFan operator based on that, similar to what we do for Float64Log. Rewrite Math.exp() as TurboFan builtin and use that operator to also inline Math.exp() into optimized TurboFan functions. CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel BUG=v8:3266,v8:3468,v8:3493,v8:5086,v8:5108,chromium:620786 R=mvstanton@chromium.org Committed: https://crrev.com/93e26314afc9da9b5b8bd998688262444ed73260 Review-Url: https://codereview.chromium.org/2077533002 Cr-Original-Commit-Position: refs/heads/master@{#37037} Cr-Commit-Position: refs/heads/master@{#37047}
-
martyn.capewell authored
If (mask >>> s) == 0, ((x & mask) >> s) == 0, so replace the node with zero in MachineOperatorReducer. BUG= Review-Url: https://codereview.chromium.org/2069973002 Cr-Commit-Position: refs/heads/master@{#37046}
-
v8-autoroll authored
Rolling v8/build to 6b34ef332946c38d33d0c08c1915d4b392808ebb Rolling v8/tools/mb to 340ee1614f0ab8aa8b1846d10b26426789dbca0e TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review-Url: https://codereview.chromium.org/2071073003 Cr-Commit-Position: refs/heads/master@{#37045}
-
- 16 Jun, 2016 30 commits
-
-
mtrofin authored
BUG= Review-Url: https://codereview.chromium.org/2077623002 Cr-Commit-Position: refs/heads/master@{#37044}
-
jyan authored
port d9bf520a R=mvstanton@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com, bjaideep@ca.ibm.com BUG=v8:5095 Review-Url: https://codereview.chromium.org/2070893002 Cr-Commit-Position: refs/heads/master@{#37043}
-
mlippautz authored
Revert of [heap] Add page evacuation mode for new->new (patchset #18 id:440001 of https://codereview.chromium.org/1957323003/ ) Reason for revert: Fragmentation of LABs could result in increasing memory usage (pages) instead of shrinking. BUG=chromium:620320 LOG=N Original issue's description: > [heap] Add page evacuation mode for new->new > > Adds an evacuation mode that allows moving pages within new space without > copying objects. > > Basic idea: > a) Move page within new space > b) Sweep page to make iterable and process ArrayBuffers > c) Finish sweep till next scavenge > > Threshold is currently 70% live bytes, i.e., the same threshold we use > to determine fragmented pages. > > BUG=chromium:581412 > LOG=N > CQ_EXTRA_TRYBOTS=tryserver.v8:v8_linux_arm64_gc_stress_dbg,v8_linux_gc_stress_dbg,v8_mac_gc_stress_dbg,v8_linux64_tsan_rel,v8_mac64_asan_rel > > Committed: https://crrev.com/49b23201671b25092a3c22eb85783f39b95a5f87 > Cr-Commit-Position: refs/heads/master@{#36990} TBR=ulan@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=chromium:581412 Review-Url: https://codereview.chromium.org/2063013005 Cr-Commit-Position: refs/heads/master@{#37042}
-
jyan authored
Port 2d1f977c Original commit message: Support for relocatable globals, to facilitate compilation before instantiation. R=mtrofin@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com, bjaideep@ca.ibm.com BUG=v8:5072 LOG=N Review-Url: https://codereview.chromium.org/2071883002 Cr-Commit-Position: refs/heads/master@{#37041}
-
jyan authored
Port d6473f5c Original commit message: See bug description. R=mlippautz@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com, bjaideep@ca.ibm.com BUG=chromium:619382 LOG=N Review-Url: https://codereview.chromium.org/2072723002 Cr-Commit-Position: refs/heads/master@{#37040}
-
machenbach authored
Revert of [builtins] Introduce proper Float64Exp operator. (patchset #5 id:80001 of https://codereview.chromium.org/2077533002/ ) Reason for revert: [Sheriff] Leads to some different rounding as it seems in some audio layout tests. Please rebase upstream first if intended: https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/builds/7508 Original issue's description: > [builtins] Introduce proper Float64Exp operator. > > Import base::ieee754::exp() from FreeBSD msun and introduce a Float64Exp > TurboFan operator based on that, similar to what we do for Float64Log. > Rewrite Math.exp() as TurboFan builtin and use that operator to also > inline Math.exp() into optimized TurboFan functions. > > BUG=v8:3266,v8:3468,v8:3493,v8:5086,v8:5108 > R=mvstanton@chromium.org > > Committed: https://crrev.com/93e26314afc9da9b5b8bd998688262444ed73260 > Cr-Commit-Position: refs/heads/master@{#37037} TBR=mvstanton@chromium.org,ahaas@chromium.org,bmeurer@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:3266,v8:3468,v8:3493,v8:5086,v8:5108 Review-Url: https://codereview.chromium.org/2070813002 Cr-Commit-Position: refs/heads/master@{#37039}
-
clemensh authored
Without this check, instantiation of the wasm module would fail on DefineOwnProperty on the exports object for the duplicate export. Now we detect this as validation error. R=rossberg@chromium.org, titzer@chromium.org, ahaas@chromium.org Review-Url: https://codereview.chromium.org/2065043002 Cr-Commit-Position: refs/heads/master@{#37038}
-
bmeurer authored
Import base::ieee754::exp() from FreeBSD msun and introduce a Float64Exp TurboFan operator based on that, similar to what we do for Float64Log. Rewrite Math.exp() as TurboFan builtin and use that operator to also inline Math.exp() into optimized TurboFan functions. BUG=v8:3266,v8:3468,v8:3493,v8:5086,v8:5108 R=mvstanton@chromium.org Review-Url: https://codereview.chromium.org/2077533002 Cr-Commit-Position: refs/heads/master@{#37037}
-
clemensh authored
We report the byte offset as column number, but devtools assumes them to be 1-based and subtracts one unconditionally before further processing it. It's a bit unfortunate, but because of that we have to just add 1 to the reported column number on the public StackTrace API. R=ahaas@chromium.org, titzer@chromium.org BUG=chromium:613110 Review-Url: https://codereview.chromium.org/2071563002 Cr-Commit-Position: refs/heads/master@{#37036}
-
mvstanton authored
BUG=v8:5095 Review-Url: https://codereview.chromium.org/2063693002 Cr-Commit-Position: refs/heads/master@{#37035}
-
ishell authored
BUG=v8:4915 LOG=Y Review-Url: https://codereview.chromium.org/2072683002 Cr-Commit-Position: refs/heads/master@{#37034}
-
ishell authored
LoadICState was used to hold the TypeofMode flag which is relevant only for LoadGlobalIC. This CL removes usage of this state from LoadIC and KeyedLoadIC and renames the state class to LoadGlobalICState. BUG=chromium:576312 LOG=Y Review-Url: https://codereview.chromium.org/2065373003 Cr-Commit-Position: refs/heads/master@{#37033}
-
Michael Starzinger authored
This is a follow-up to a recent fix to make the exception reporting even more resilient. The original regression test flushed out more issues on different configurations. TBR=yangguo@chromium.org TEST=mjsunit/regress/regress-crbug-620253 BUG=chromium:620253 NOTREECHECKS=true Review URL: https://codereview.chromium.org/2071783002 . Cr-Commit-Position: refs/heads/master@{#37032}
-
mstarzinger authored
This makes sure exception reporting done by the debug shell behaves gracefully even near the stack limit. When line number determination fails we just fallback to not printing source information. R=yangguo@chromium.org TEST=mjsunit/regress/regress-crbug-620253 BUG=chromium:620253 Review-Url: https://codereview.chromium.org/2069543007 Cr-Commit-Position: refs/heads/master@{#37031}
-
ishell authored
BUG=chromium:576312 LOG=Y Review-Url: https://codereview.chromium.org/2062343002 Cr-Commit-Position: refs/heads/master@{#37030}
-
bmeurer authored
Revert of [turbofan] Properly handle dictionary maps in the prototype chain. (patchset #1 id:1 of https://codereview.chromium.org/2067423003/ ) Reason for revert: As discussed offline with Toon, this is not the correct fix here. Original issue's description: > [turbofan] Properly handle dictionary maps in the prototype chain. > > Dictionary prototypes don't have stable maps, but still don't matter for > element access. Generalized the JSNativeContextSpecialization a bit to > handle everything that Crankshaft can handle in this regard. > > R=jarin@chromium.org > BUG=chromium:616709 > > Committed: https://crrev.com/1c7bdc7f6f4d9512f4982590bd949f265ee9c8c3 > Cr-Commit-Position: refs/heads/master@{#37019} TBR=jarin@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:616709 Review-Url: https://codereview.chromium.org/2076493002 Cr-Commit-Position: refs/heads/master@{#37029}
-
clemensh authored
This will be needed during debugging to know function boundaries within the wasm module bytes, and to extract the bytes for a single function to disassemble it. R=titzer@chromium.org BUG=chromium:613110 Review-Url: https://codereview.chromium.org/2055783002 Cr-Commit-Position: refs/heads/master@{#37028}
-
machenbach authored
Those were wrongly translated from gyp with ia32. This should land before renaming v8_target_arch to v8_target_cpu. BUG=chromium:620527 NOTRY=true TBR=vogelheim@chromium.org Review-Url: https://codereview.chromium.org/2065323004 Cr-Commit-Position: refs/heads/master@{#37027}
-
clemensh authored
This will be used for disassembling individual wasm function for showing them in devtools. The PrintAst function now also optionally provides an offset table mapping from byte offset to line and column in the generated text. R=titzer@chromium.org, ahaas@chromium.org BUG=chromium:613110 Review-Url: https://codereview.chromium.org/2050213002 Cr-Commit-Position: refs/heads/master@{#37026}
-
cbruni authored
BUG= Review-Url: https://codereview.chromium.org/2062323003 Cr-Commit-Position: refs/heads/master@{#37025}
-
jarin authored
Review-Url: https://codereview.chromium.org/2035383003 Cr-Commit-Position: refs/heads/master@{#37024}
-
bjaideep authored
Port d6473f5c Original commit message: See bug description. R=mlippautz@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com BUG=chromium:619382 LOG=N Review-Url: https://codereview.chromium.org/2066603007 Cr-Commit-Position: refs/heads/master@{#37023}
-
bjaideep authored
R=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/2069363002 Cr-Commit-Position: refs/heads/master@{#37022}
-
bjaideep authored
Port 2d1f977c Original commit message: Support for relocatable globals, to facilitate compilation before instantiation. R=mtrofin@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com BUG=v8:5072 LOG=N Review-Url: https://codereview.chromium.org/2069163003 Cr-Commit-Position: refs/heads/master@{#37021}
-
bmeurer authored
Those operators don't have JavaScript observable side effects and would otherwise block load elimination and other optimizations. R=jarin@chromium.org Review-Url: https://codereview.chromium.org/2075443003 Cr-Commit-Position: refs/heads/master@{#37020}
-
bmeurer authored
Dictionary prototypes don't have stable maps, but still don't matter for element access. Generalized the JSNativeContextSpecialization a bit to handle everything that Crankshaft can handle in this regard. R=jarin@chromium.org BUG=chromium:616709 Review-Url: https://codereview.chromium.org/2067423003 Cr-Commit-Position: refs/heads/master@{#37019}
-
bmeurer authored
These operators don't need to be part of the control chain. Drive-by-fix: Properly wire the effects during representation selection. R=jarin@chromium.org Review-Url: https://codereview.chromium.org/2073683002 Cr-Commit-Position: refs/heads/master@{#37018}
-
bbudge authored
LOG=N BUG=v8:4124 Review-Url: https://codereview.chromium.org/2050563002 Cr-Commit-Position: refs/heads/master@{#37017}
-
v8-autoroll authored
Rolling v8/build to 76e9bd0241b00735b5bc402c926d7cb141a726ef Rolling v8/buildtools to 3780bc523aad1d68a5bd00e05c453a80b2ba0b35 Rolling v8/tools/clang to 8b0c8f71a4d41c7d2cd0d898f3547e9984f79b5d Rolling v8/tools/mb to bc05800a7cd7833ad7c2d1423cd8b2153470c9bd TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review-Url: https://codereview.chromium.org/2069333002 Cr-Commit-Position: refs/heads/master@{#37016}
-
gsathya authored
BUG=v8:5004 Review-Url: https://codereview.chromium.org/2070213002 Cr-Commit-Position: refs/heads/master@{#37015}
-
- 15 Jun, 2016 2 commits
-
-
jyan authored
Port 3e2d60d8 R=yangguo@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com, bjaideep@ca.ibm.com BUG= LOG=N Review-Url: https://codereview.chromium.org/2069193003 Cr-Commit-Position: refs/heads/master@{#37014}
-
bbudge authored
Review-Url: https://codereview.chromium.org/2054343002 Cr-Commit-Position: refs/heads/master@{#37013}
-