- 16 Aug, 2016 23 commits
-
-
bjaideep authored
Port d941b52d Original commit message: These new representations aren't used yet. R=mvstanton@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/2246213004 Cr-Commit-Position: refs/heads/master@{#38664}
-
rmcilroy authored
Revert of [Interpreter] Introduce InterpreterCompilationJob (patchset #7 id:140001 of https://codereview.chromium.org/2240463002/ ) Reason for revert: Failing on Win64 bot: https://build.chromium.org/p/client.v8/builders/V8%20Win64/builds/12061/steps/Check/logs/regress-635429 Original issue's description: > [Interpreter] Introduce InterpreterCompilationJob > > Adds InterpreterCompilationJob as a sub-class of > CompilationJob, to enable off-thread bytecode > generation. Currently only used in > Interpreter::MakeBytecode. > > As part of this change, CompilationJob is modified > to make it less specific to optimized compilation, > renaming the phases as follows: > - CreateGraph -> PrepareJob > - OptimizeGraph -> ExecuteJob > - GenerateCode -> FinalizeJob > > RegisterWeakObjectsInOptimizedCode is also moved out > of CompilationJob and instead becomes a static function > on Compiler. > > BUG=v8:5203 > > Committed: https://crrev.com/1fb6a7e697e8bc5b4af51647553741f966e00cdc > Cr-Commit-Position: refs/heads/master@{#38662} TBR=mstarzinger@chromium.org,jkummerow@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:5203 Review-Url: https://codereview.chromium.org/2249153002 Cr-Commit-Position: refs/heads/master@{#38663}
-
rmcilroy authored
Adds InterpreterCompilationJob as a sub-class of CompilationJob, to enable off-thread bytecode generation. Currently only used in Interpreter::MakeBytecode. As part of this change, CompilationJob is modified to make it less specific to optimized compilation, renaming the phases as follows: - CreateGraph -> PrepareJob - OptimizeGraph -> ExecuteJob - GenerateCode -> FinalizeJob RegisterWeakObjectsInOptimizedCode is also moved out of CompilationJob and instead becomes a static function on Compiler. BUG=v8:5203 Review-Url: https://codereview.chromium.org/2240463002 Cr-Commit-Position: refs/heads/master@{#38662}
-
verwaest authored
BUG=v8:5209 Review-Url: https://codereview.chromium.org/2244233003 Cr-Commit-Position: refs/heads/master@{#38661}
-
ivica.bogosavljevic authored
Port 77c9cb83. Original commit message: This commit fixes wasm little-endian load issue on big-endian platform by introducing reverse byte operation immediately after a load. BUG= Review-Url: https://codereview.chromium.org/2235703002 Cr-Commit-Position: refs/heads/master@{#38660}
-
verwaest authored
This reduces sizeof(Variable) from 64 to 40 on x64 BUG=v8:5209 Review-Url: https://codereview.chromium.org/2253513002 Cr-Commit-Position: refs/heads/master@{#38659}
-
titzer authored
R=ahaas@chromium.org BUG= Review-Url: https://codereview.chromium.org/2253543003 Cr-Commit-Position: refs/heads/master@{#38658}
-
mvstanton authored
These new representations aren't used yet. BUG= Review-Url: https://codereview.chromium.org/2216383002 Cr-Commit-Position: refs/heads/master@{#38657}
-
verwaest authored
BUG= Review-Url: https://codereview.chromium.org/2253503002 Cr-Commit-Position: refs/heads/master@{#38656}
-
marja authored
There was a weird check in AllocateNonParameterLocal which looked like ".result" was treated differently from other temporaries. This couldn't be generalized to other temporaries, since some temporaries were both in temps_ and params_ (and some, like ".result" would be only in params_). Side product: Don't use AstValueFactory in scope analysis. It's unnecessary (was only used for some DCHECKs which can be replaced with more general checks). This change also ensures we don't create new values during scope analysis. BUG=v8:5209 Review-Url: https://codereview.chromium.org/2242783002 Cr-Commit-Position: refs/heads/master@{#38655}
-
mic.besace authored
Port d61a5c37 Original commit message: As a first step I uncommit the memory on the main thread. Also to measure impact and stability of that optimization. In a follow-up CL, the uncommitting should be moved on the concurrent thread. R=jochen@chromium.org, hpayer@chromium.org BUG= Review-Url: https://codereview.chromium.org/2245703002 Cr-Commit-Position: refs/heads/master@{#38654}
-
mlippautz authored
Slots in ConsString/SlicedString can point to an evacutaion candidate. The MakeExternal function makes in-place conversion to external string. After the conversion we can have a recorded slot containing an external pointer. As long as the external pointer is aligned, this is not a problem. We clear the recorded slots to fix verify-heap checks. BUG=chromium:631969 LOG=NO Finalizing CL: https://codereview.chromium.org/2199863002/ Review-Url: https://codereview.chromium.org/2242183003 Cr-Commit-Position: refs/heads/master@{#38653}
-
mlippautz authored
BUG= Review-Url: https://codereview.chromium.org/2241503002 Cr-Commit-Position: refs/heads/master@{#38652}
-
klaasb authored
Allows us to create a corresponding TurboFan node, so TF can optimize it. BUG=v8:4280 LOG=n Review-Url: https://codereview.chromium.org/2248633002 Cr-Commit-Position: refs/heads/master@{#38651}
-
verwaest authored
This moves language-mode initialization for MODULE_SCOPE from Scope::SetDefaults to the single constructor that needs to do it. The CL additionally makes it more obvious that scope-info-backed scopes always initialize language-mode unless WITH_SCOPE. BUG= Review-Url: https://codereview.chromium.org/2251683002 Cr-Commit-Position: refs/heads/master@{#38650}
-
bgeron authored
[turbofan] Instead of using properties, use a whitelist for store-store elimination. Make --turbo imply --turbo-store-elimination, so it is tested by ClusterFuzz. This should fix bug v8:5280. BUG=v8:5280 Review-Url: https://codereview.chromium.org/2236443006 Cr-Commit-Position: refs/heads/master@{#38649}
-
mstarzinger authored
This stages the --ignition-preserve-bytecode flag which preserves the bytecode even when switching to baseline code. It is now implied by the combined --ignition-staging flag. R=rmcilroy@chromium.org Review-Url: https://codereview.chromium.org/2244303003 Cr-Commit-Position: refs/heads/master@{#38648}
-
hablich authored
BUG=v8:5291 R=titzer@chromium.org Review-Url: https://codereview.chromium.org/2241203004 Cr-Commit-Position: refs/heads/master@{#38647}
-
verwaest authored
BUG=v8:5209 Review-Url: https://codereview.chromium.org/2247073003 Cr-Commit-Position: refs/heads/master@{#38646}
-
jgruber authored
Simple stack traces are captured through Isolate::CaptureSimpleStackTrace. Captured frames are stored in a FixedArray, which in turn is stored as a property (using a private symbol) on the error object itself. Actual formatting of the textual stack trace is done lazily when the user reads the stack property of the error object. This would involve many conversions back and forth between index-encoded raw data (receiver, function, offset and code), JS CallSite objects, and C++ CallSite objects. This commit refactors the C++ CallSite class into a Struct class called StackTraceFrame, which is the new single point of truth frame information. Isolate::CaptureSimpleStackTrace stores an array of StackTraceFrames, and JS CallSite objects (now created only when the user specifies custom stack trace formatting through Error.prepareStackTrace) internally only store a reference to a StackTraceFrame. BUG= Review-Url: https://codereview.chromium.org/2230953002 Cr-Commit-Position: refs/heads/master@{#38645}
-
mtrofin authored
A test exercising the public APIs for wasm serialization and simulates the serialization scenario - serialize in one isolate, deserialize in another. BUG=v8:5072 Review-Url: https://codereview.chromium.org/2249973002 Cr-Commit-Position: refs/heads/master@{#38644}
-
verwaest authored
Follow-up cleanup to https://codereview.chromium.org/2231813003 BUG= Review-Url: https://codereview.chromium.org/2237873002 Cr-Commit-Position: refs/heads/master@{#38643}
-
v8-autoroll authored
Rolling v8/build to f5aadfbec4267ecafa3200c571131c84d1ced4be Rolling v8/third_party/WebKit/Source/platform/inspector_protocol to e17560795294ff0c23bdae7665ec71c95a75561e Rolling v8/tools/clang to ad9e1e262ff737d779bb2e5b49e3987547f05694 Rolling v8/tools/mb to e89c543a2ab08d70b876130ac83c04a92daffeaf TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review-Url: https://codereview.chromium.org/2249993002 Cr-Commit-Position: refs/heads/master@{#38642}
-
- 15 Aug, 2016 12 commits
-
-
adamk authored
R=verwaest@chromium.org Review-Url: https://codereview.chromium.org/2248813002 Cr-Commit-Position: refs/heads/master@{#38641}
-
jyan authored
R=joransiu@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com, bjaideep@ca.ibm.com BUG= Review-Url: https://codereview.chromium.org/2244373002 Cr-Commit-Position: refs/heads/master@{#38640}
-
gsathya authored
Review-Url: https://codereview.chromium.org/2246963002 Cr-Commit-Position: refs/heads/master@{#38639}
-
adamk authored
The structure of this code changed in https://codereview.chromium.org/2201193004, but the structural changes were not essential to that CL's purpose (which was to use Variable::binding_needs_init() consistently to decide whether to hole-check). Now the code should appear as it did before that change, but with the addition of binding_needs_init() checks. R=bmeurer@chromium.org, mstarzinger@chromium.org Review-Url: https://codereview.chromium.org/2237933002 Cr-Commit-Position: refs/heads/master@{#38638}
-
mattloring authored
Removes dependence on InnerArrayIndexOf/InnerArrayLastIndexOf and reduces type polution caused by sharing these functions between standard and typed arrays. BUG= Review-Url: https://codereview.chromium.org/2243523002 Cr-Commit-Position: refs/heads/master@{#38637}
-
jbroman authored
This includes UTF-8 strings and two-byte strings, both length-delimited (in bytes, not characters). Two-byte strings are written/read in host byte order. BUG=chromium:148757 Review-Url: https://codereview.chromium.org/2245753002 Cr-Commit-Position: refs/heads/master@{#38636}
-
jyan authored
Add an extra paramter to disable scale on BaseWithIndexAndDisplacementMatcher. R=bmeurer@chromium.org, epertoso@chromium.org, jarin@chromium.org, mstarzinger@chromium.org, mtrofin@chromium.org, titzer@chromium.org BUG= Review-Url: https://codereview.chromium.org/2239813002 Cr-Commit-Position: refs/heads/master@{#38635}
-
klaasb authored
Adds TestResultScope and uses it to directly jump/fall through to the correct branch in expressions used as branch conditions. Should enable nicer TurboFan-graphs for easier control-flow transformations in the future. BUG=v8:4280 LOG=n Review-Url: https://codereview.chromium.org/2242463002 Cr-Commit-Position: refs/heads/master@{#38634}
-
rmcilroy authored
Removes Variable::is_possibly_eval() and instead stores whether a call is possibly eval in the Call node's bitfield. Also removes HandleDereferenceMode since it's no longer used. BUG=v8:5203 Review-Url: https://codereview.chromium.org/2242583003 Cr-Commit-Position: refs/heads/master@{#38633}
-
baptiste.afsa authored
R=jarin@chromium.org Review-Url: https://codereview.chromium.org/2193063003 Cr-Commit-Position: refs/heads/master@{#38632}
-
v8-autoroll authored
Rolling v8/third_party/WebKit/Source/platform/inspector_protocol to ac24d974a70c8611d2837e183d6cf99f39fb0410 TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review-Url: https://codereview.chromium.org/2244173002 Cr-Commit-Position: refs/heads/master@{#38631}
-
jbroman authored
This includes unsigned integers (encoded as base-128 varints), signed integers (ZigZag-encoded, then varint-encoded) and doubles (written in host byte order). BUG=chromium:148757 Review-Url: https://codereview.chromium.org/2232323004 Cr-Commit-Position: refs/heads/master@{#38630}
-
- 14 Aug, 2016 1 commit
-
-
v8-autoroll authored
Rolling v8/build to 4155375bddb65fe3d2dbc42ab0d64c4d72527165 Rolling v8/third_party/WebKit/Source/platform/inspector_protocol to 9d440c96636c5a41ce3e40f1924fe41dd2694f51 TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review-Url: https://codereview.chromium.org/2244113002 Cr-Commit-Position: refs/heads/master@{#38629}
-
- 13 Aug, 2016 1 commit
-
-
v8-autoroll authored
Rolling v8/build to 45574dce74fca42e485fbc5cd78bd24bcfeb905f Rolling v8/buildtools to adb8bf4e8fc92aa1717bf151b862d58e6f27c4f2 Rolling v8/tools/clang to 6d377a47e9c668c7550d17a7d4e6ba9f5931703a Rolling v8/tools/mb to c78da3f5bccc979b35907c4cbf937aa5187e41fa TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review-Url: https://codereview.chromium.org/2240213004 Cr-Commit-Position: refs/heads/master@{#38628}
-
- 12 Aug, 2016 3 commits
-
-
caitp authored
No longer include the "async" keyword, or an async arrow function's single identifier parameter as part of its inferred name. BUG=v8:5281, v8:4483 R=adamk@chromium.org, littledan@chromium.org, marja@chromium.org Review-Url: https://codereview.chromium.org/2235423003 Cr-Commit-Position: refs/heads/master@{#38627}
-
jshin authored
Throw 'Range Error: invalid string length' when the result of case mapping is longer than the max string length (kMaxLength in objects.h = 1 << 28 - 16). This is for case mapping with ICU. A new test (case-mapping-slow.js) is added with PASS,SLOW. It's configured to skip unless arch=x64 and mode=release and not on simulator. This is a reattempt to land https://codereview.chromium.org/2236593002 that was reverted. BUG=v8:5271 TEST=intl/general/case-mapping-slow.js with --icu_case_mapping Review-Url: https://codereview.chromium.org/2236963003 Cr-Commit-Position: refs/heads/master@{#38626}
-
jbroman authored
BUG=chromium:148757 Review-Url: https://codereview.chromium.org/2232243003 Cr-Commit-Position: refs/heads/master@{#38625}
-