Bug 160357 - Refactor DFG::Node::hasLocal() to accessesStack()
Summary: Refactor DFG::Node::hasLocal() to accessesStack()
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Michael Saboff
URL:
Keywords:
Depends on:
Blocks: 160355
  Show dependency treegraph
 
Reported: 2016-07-29 14:24 PDT by Michael Saboff
Modified: 2016-07-29 15:05 PDT (History)
4 users (show)

See Also:


Attachments
Patch (8.39 KB, patch)
2016-07-29 14:41 PDT, Michael Saboff
fpizlo: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Saboff 2016-07-29 14:24:44 PDT
The Node class currently has two accessors, hasVariableAccessData() and hasLocal() that provide the same answer.  As part of the work for <https://bugs.webkit.org/show_bug.cgi?id=160355> - "JSVALUE64: Pass arguments in platform argument registers when making JavaScript calls", there is need for distinct accessors.  One of those accessors is needed to answer the question: "Does this node have a reference to a VariableAccessData object?".  A second accessor is needed to answer the question: "Does this node read/write the stack?".  That second question implies the need for such a node to have a reference to the appropriate VariableAccessData object, but within the DFG and FTL tier asserts a much stronger fact about the node.

This patch is for the work to refactor hasLocal() to accessesStack().
Comment 1 Michael Saboff 2016-07-29 14:41:43 PDT
Created attachment 284903 [details]
Patch
Comment 2 Filip Pizlo 2016-07-29 14:46:48 PDT
Comment on attachment 284903 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=284903&action=review

> Source/JavaScriptCore/ChangeLog:14
> +        The hasVariableAccessData() check now iplies no more than the node has a

*implies
Comment 3 Michael Saboff 2016-07-29 15:05:17 PDT
Committed r203923: <http://trac.webkit.org/changeset/203923>