Mercurial > hg > graal-compiler
diff graal/com.oracle.truffle.sl/src/com/oracle/truffle/sl/nodes/instrument/SLStatementWrapper.java @ 16961:a1427e40deaf
Truffle/Instrumentation: some Javadoc revistions; minor code cleanups; remove one redundant operation; add tracing to the LineLocation maps.
author | Michael Van De Vanter <michael.van.de.vanter@oracle.com> |
---|---|
date | Tue, 26 Aug 2014 13:54:53 -0700 |
parents | 7661cc464239 |
children | 5d16da2ca0c8 |
line wrap: on
line diff
--- a/graal/com.oracle.truffle.sl/src/com/oracle/truffle/sl/nodes/instrument/SLStatementWrapper.java Tue Aug 26 09:35:08 2014 -0700 +++ b/graal/com.oracle.truffle.sl/src/com/oracle/truffle/sl/nodes/instrument/SLStatementWrapper.java Tue Aug 26 13:54:53 2014 -0700 @@ -30,14 +30,10 @@ import com.oracle.truffle.sl.runtime.*; /** - * SLStatmentWrapper is a Truffle AST node that gets inserted as the parent to the node that it is - * wrapping. Any debugging instruments are attached to this wrapper through {@link Probe}s (which - * themselves contain the instruments). It is through this mechanism that tools can interact - * directly with the AST. <br/> - * SLStatmentWrapper specifically wraps {@link SLStatementWrapper}s and overrides the executeVoid - * function of {@link SLStatementNode} to operate on the child of the wrapper instead of the wrapper - * itself. - * + * A Truffle node that can be inserted into a Simple AST (assumed not to have executed yet) to + * enable "instrumentation" of a {@link SLStatementNode}. Tools wishing to interact with AST + * execution may attach {@link Instrument}s to the {@link Probe} uniquely associated with the + * wrapper, and to which this wrapper routes {@link ExecutionEvents}. */ public final class SLStatementWrapper extends SLStatementNode implements Wrapper { @@ -50,8 +46,6 @@ assert !(child instanceof SLStatementWrapper); this.probe = context.createProbe(child.getSourceSection()); this.child = child; - // The child should only be inserted after a replace, so we defer inserting the child to the - // creator of the wrapper. } @Override @@ -98,10 +92,4 @@ } } - /** - * Sets the parent pointer of this wrapper's child. - */ - public void insertChild() { - insert(this.child); - } }