- 26 Jul, 2016 1 commit
-
-
ishell authored
This is a first step towards a perfect world where a call interface descriptor is the only place that defines calling convention for a particular code stub. Review-Url: https://codereview.chromium.org/2172223002 Cr-Commit-Position: refs/heads/master@{#38059}
-
- 01 Dec, 2015 1 commit
-
-
mtrofin authored
Node has an operator<<, however, constructing an ostream in a debugger session is non-trivial, hence this method. BUG= Review URL: https://codereview.chromium.org/1488953003 Cr-Commit-Position: refs/heads/master@{#32439}
-
- 24 Sep, 2015 1 commit
-
-
mstarzinger authored
This introduces the NodeProperties::ChangeOp helper which guards node operator changes so that additional checking can be done without any additional dependencies being pulled into the Node class. For now only the input count is checked, but additional checking might follow. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1366753003 Cr-Commit-Position: refs/heads/master@{#30916}
-
- 16 Sep, 2015 1 commit
-
-
jarin authored
Review URL: https://codereview.chromium.org/1348073002 Cr-Commit-Position: refs/heads/master@{#30764}
-
- 18 Aug, 2015 1 commit
-
-
jarin authored
BUG= Review URL: https://codereview.chromium.org/1294913003 Cr-Commit-Position: refs/heads/master@{#30212}
-
- 13 Aug, 2015 1 commit
-
-
mstarzinger authored
This tries to remove includes of "-inl.h" headers from normal ".h" headers, thereby reducing the chance of any cyclic dependencies and decreasing the average size of our compilation units. Note that this change still leaves 5 violations of that rule in the code. It only tackles "node.h" including "types-inl.h". R=titzer@chromium.org Review URL: https://codereview.chromium.org/1288053004 Cr-Commit-Position: refs/heads/master@{#30161}
-
- 19 Jun, 2015 1 commit
-
-
bmeurer authored
R=jarin@chromium.org Review URL: https://codereview.chromium.org/1196613003 Cr-Commit-Position: refs/heads/master@{#29149}
-
- 12 Jun, 2015 1 commit
-
-
bmeurer authored
Up until now we used int32_t for NodeId, but that was not ideal because negative values are invalid for NodeId and we use it as an array index for example in the NodeMarker class, where C++ compilers on x64 have to generate code that does proper sign extension for the indices, which is completely unnecessary. R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/1178403004 Cr-Commit-Position: refs/heads/master@{#28997}
-
- 22 May, 2015 1 commit
-
-
titzer authored
This reduces the storage per-Node storage from 7 words to 6 and per-edge storage from 6 words to 4. On average this is about 10%-15% space savings over the whole graph. Remove the use of std::deque as the out-of-line storage for inputs. Reduce size of Use links and use pointer arithmetic to find Node from Use. R=mstarzinger@chromium.org,jarin@chromium.org BUG= Review URL: https://codereview.chromium.org/1150923003 Cr-Commit-Position: refs/heads/master@{#28583}
-
- 21 May, 2015 1 commit
-
-
titzer authored
BUG= Review URL: https://codereview.chromium.org/1149563004 Cr-Commit-Position: refs/heads/master@{#28540}
-
- 18 May, 2015 1 commit
-
-
titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1132353004 Cr-Commit-Position: refs/heads/master@{#28452}
-
- 05 May, 2015 1 commit
-
-
bmeurer authored
Tail calls are matched on the graph, with a dedicated tail call optimization that is actually testable. The instruction selection can still fall back to a regular if the platform constraints don't allow to emit a tail call (i.e. the return locations of caller and callee differ or the callee takes non-register parameters, which is a restriction that will be removed in the future). Also explicitly limit tail call optimization to stubs for now and drop the global flag. BUG=v8:4076 LOG=n Review URL: https://codereview.chromium.org/1114163005 Cr-Commit-Position: refs/heads/master@{#28219}
-
- 20 Apr, 2015 1 commit
-
-
Ross McIlroy authored
R=jochen@chromium.org Review URL: https://codereview.chromium.org/1088993003 Cr-Commit-Position: refs/heads/master@{#27937}
-
- 24 Mar, 2015 1 commit
-
-
titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1032553006 Cr-Commit-Position: refs/heads/master@{#27401}
-
- 19 Mar, 2015 1 commit
-
-
titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1022783002 Cr-Commit-Position: refs/heads/master@{#27319}
-
- 20 Feb, 2015 1 commit
-
-
bmeurer authored
R=titzer@chromium.org Review URL: https://codereview.chromium.org/940403002 Cr-Commit-Position: refs/heads/master@{#26773}
-
- 20 Jan, 2015 1 commit
-
-
titzer authored
BUG= Review URL: https://codereview.chromium.org/816053002 Cr-Commit-Position: refs/heads/master@{#26149}
-
- 16 Jan, 2015 4 commits
-
-
machenbach authored
Revert of Revert of [turbofan] Improve memory layout of Node class. (patchset #1 id:1 of https://codereview.chromium.org/856813003/) Reason for revert: Revert didn't help. Original issue's description: > Revert of [turbofan] Improve memory layout of Node class. (patchset #1 id:1 of https://codereview.chromium.org/828803004/) > > Reason for revert: > Speculative revert to see if win XP recovers. Will reland if not. > > Original issue's description: > > [turbofan] Improve memory layout of Node class. > > > > Also gets rid of the reinterpret_cast on this. > > > > R=svenpanne@chromium.org > > > > Committed: https://crrev.com/f91e8046fad99365c74ccbb39d42de58398dab57 > > Cr-Commit-Position: refs/heads/master@{#26103} > > TBR=svenpanne@chromium.org,dcarney@chromium.org,bmeurer@chromium.org > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > > Committed: https://crrev.com/dc6f34d83650dce8faa3e82e87c050f12732d4b3 > Cr-Commit-Position: refs/heads/master@{#26106} TBR=svenpanne@chromium.org,dcarney@chromium.org,bmeurer@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/855723002 Cr-Commit-Position: refs/heads/master@{#26107}
-
machenbach authored
Revert of [turbofan] Improve memory layout of Node class. (patchset #1 id:1 of https://codereview.chromium.org/828803004/) Reason for revert: Speculative revert to see if win XP recovers. Will reland if not. Original issue's description: > [turbofan] Improve memory layout of Node class. > > Also gets rid of the reinterpret_cast on this. > > R=svenpanne@chromium.org > > Committed: https://crrev.com/f91e8046fad99365c74ccbb39d42de58398dab57 > Cr-Commit-Position: refs/heads/master@{#26103} TBR=svenpanne@chromium.org,dcarney@chromium.org,bmeurer@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/856813003 Cr-Commit-Position: refs/heads/master@{#26106}
-
bmeurer authored
Also gets rid of the reinterpret_cast on this. R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/828803004 Cr-Commit-Position: refs/heads/master@{#26103}
-
bmeurer authored
- Make Node::Inputs and Node::Uses mostly STL compliant. - Get rid of some pre-C++11 crappiness. - Start moving unit tests from cctest to unittests. - TrimInputCount() now tries to reserve inputs slots for later appending. - Fix numerous style guide violations. TEST=cctest,unittests R=dcarney@chromium.org Review URL: https://codereview.chromium.org/851263002 Cr-Commit-Position: refs/heads/master@{#26098}
-
- 07 Jan, 2015 1 commit
-
-
bmeurer authored
- Move NodeMarker to its own file, and introduce a non templatized base class. - Cleanup the include hell. - Sanitize the Node construction methods now that we got rid of that GenericNode/GenericGraph stuff. - Protect against NodeId overflow in Graph. - Various minor cleanups. TEST=cctest,mjsunit,unittests Review URL: https://codereview.chromium.org/838783002 Cr-Commit-Position: refs/heads/master@{#25977}
-
- 03 Dec, 2014 1 commit
-
-
bmeurer authored
- Remove use_count_, only used for debug builds and tests. - Use BitField instead of the unreliable C++ bit fields. - Improve memory layout. R=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/780503002 Cr-Commit-Position: refs/heads/master@{#25632}
-
- 02 Dec, 2014 2 commits
-
-
mstarzinger authored
R=titzer@chromium.org Review URL: https://codereview.chromium.org/760493003 Cr-Commit-Position: refs/heads/master@{#25620}
-
danno authored
- Create a first-class Edge type. - Separate node and edge iterators - Make iterators only responsible for iteration - Make it possible to modify the use edge iterator while iterating. - Add the ability to update inputs to Edges directly. Review URL: https://codereview.chromium.org/765983002 Cr-Commit-Position: refs/heads/master@{#25616}
-
- 28 Nov, 2014 4 commits
-
-
mstarzinger authored
R=titzer@chromium.org Review URL: https://codereview.chromium.org/768853002 Cr-Commit-Position: refs/heads/master@{#25575}
-
mstarzinger authored
R=titzer@chromium.org Review URL: https://codereview.chromium.org/756073004 Cr-Commit-Position: refs/heads/master@{#25572}
-
mstarzinger authored
R=titzer@chromium.org Review URL: https://codereview.chromium.org/767733002 Cr-Commit-Position: refs/heads/master@{#25569}
-
Ben L. Titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/768763002 Cr-Commit-Position: refs/heads/master@{#25567}
-
- 10 Nov, 2014 3 commits
-
-
bmeurer@chromium.org authored
Select lowering must not merge Select nodes that depend on each other, because the resulting graph is not schedulable. TEST=unittests R=jarin@chromium.org Review URL: https://codereview.chromium.org/709423003 Cr-Commit-Position: refs/heads/master@{#25245} git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@25245 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
bmeurer@chromium.org authored
This reverts commit cf08e8d7 for breaking Embenchen bullet. TBR=jarin@chromium.org Review URL: https://codereview.chromium.org/684413003 Cr-Commit-Position: refs/heads/master@{#25238} git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@25238 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
bmeurer@chromium.org authored
Select lowering must not merge Select nodes that depend on each other, because the resulting graph is not schedulable. TEST=unittests R=jarin@chromium.org Review URL: https://codereview.chromium.org/717473002 Cr-Commit-Position: refs/heads/master@{#25236} git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@25236 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 27 Oct, 2014 3 commits
-
-
jarin@chromium.org authored
This reverts commit a238443c (r24915) for tanking benchmarks. TBR=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/684693002 Cr-Commit-Position: refs/heads/master@{#24916} git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@24916 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jarin@chromium.org authored
BUG= R=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/676353002 Cr-Commit-Position: refs/heads/master@{#24915} git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@24915 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
Allow reservation of additional input capacity when creating nodes to prevent switching to deque representation when adding well-known additional inputs. Also ensure that only a single temporary buffer is used to create temporary input arrays before allocating nodes. R=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/644083003 Cr-Commit-Position: refs/heads/master@{#24896} git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@24896 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 15 Oct, 2014 1 commit
-
-
rossberg@chromium.org authored
- Extend verifier to check types of JS and Simplified nodes. - Untyped nodes now contain NULL as types, enforcing hard failure. - Typer immediately installs itself as a decorator; remove explicit decorator installation. - Decorator eagerly types all nodes that have typed inputs (subsumes typing of constant cache, removing its typing side-channel and various spurious dependencies on the typer). - Cut down typer interface to prevent inconsistently typed graphs. - Remove verification from start, since it caused too much trouble with semi-wellformed nodes. - Fix a couple of bugs on the way that got uncovered. To do: verifying machine operators. Also, various conditions in the verifier are currently commented out, because they don't yet hold. BUG= R=jarin@chromium.org,titzer@chromium.org Review URL: https://codereview.chromium.org/658543002 git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@24626 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 30 Sep, 2014 1 commit
-
-
bmeurer@chromium.org authored
Review URL: https://codereview.chromium.org/618643002 git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@24319 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 18 Sep, 2014 1 commit
-
-
sigurds@chromium.org authored
Original: https://codereview.chromium.org/573703002/ Reland Fixes: - Add deopt framestate to CollectStackTrace runtime call R=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/544953006 git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@24023 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 17 Sep, 2014 2 commits
-
-
sigurds@chromium.org authored
This reverts commit r24008. TBR=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/581673002 git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@24010 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
sigurds@chromium.org authored
- Lazy deopt from inlined calls - Lazy deopt from inlined calls with parameter mismatch R=jarin@chromium.org, mstarzinger@chromium.org, mstarzinger@chromium Review URL: https://codereview.chromium.org/573703002 git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@24008 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-