Expose internal python frames outside tracebacks

Author: msimacek

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/SysModuleBuiltins.java

@@ -864,7 +864,7 @@ static PFrame counted(VirtualFrame frame, int depth,
                         @Bind Node inliningTarget,
                         @Cached ReadFrameNode readFrameNode,
                         @Cached PRaiseNode raiseNode) {
-            PFrame requested = readFrameNode.getFrameForReference(frame, PArguments.getCurrentFrameInfo(frame), ReadFrameNode.VisiblePythonFramesSelector.INSTANCE, depth, false);
+            PFrame requested = readFrameNode.getFrameForReference(frame, PArguments.getCurrentFrameInfo(frame), ReadFrameNode.AllPythonFramesSelector.INSTANCE, depth, false);
             if (requested == null) {
                 throw raiseNode.raise(inliningTarget, ValueError, ErrorMessages.CALL_STACK_NOT_DEEP_ENOUGH);
             }

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/bytecode/PBytecodeRootNode.java

@@ -2932,7 +2932,7 @@ private int bytecodePrintExpr(VirtualFrame virtualFrame, boolean useCachedNodes,
 
     @BytecodeInterpreterSwitch
     private byte checkTracingAndProfilingEnabled(Assumption noTraceOrProfile, MutableLoopData mutableData) {
-        if (!noTraceOrProfile.isValid() && frameIsVisibleToPython()) {
+        if (!noTraceOrProfile.isValid()) {
             PythonContext.PythonThreadState ts = mutableData.getThreadState(this);
             Object profileFun = ts.getProfileFun();
             if (ts.getTraceFun() != null) {

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/bytecode_dsl/PBytecodeDSLRootNode.java

@@ -516,8 +516,8 @@ void clearPastLine() {
 
     @NonIdempotent
     public final boolean needsTraceAndProfileInstrumentation() {
-        // We need instrumentation only if the assumption is invalid and the root node is visible.
-        return !getLanguage().noTracingOrProfilingAssumption.isValid() && !isInternal();
+        // We need instrumentation only if the assumption is invalid
+        return !getLanguage().noTracingOrProfilingAssumption.isValid();
     }
 
     @NonIdempotent
@@ -529,7 +529,6 @@ public final PythonThreadState getThreadState() {
      * Reparses with instrumentations for settrace and setprofile enabled.
      */
     public final void ensureTraceAndProfileEnabled() {
-        assert !isInternal();
         getRootNodes().update(TRACE_AND_PROFILE_CONFIG);
     }
 

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/frame/ReadFrameNode.java

@@ -140,7 +140,7 @@ public final PFrame getCurrentPythonFrame(VirtualFrame frame, boolean needsLocal
     }
 
     public final PFrame getFrameForReference(Frame frame, PFrame.Reference startFrameInfo, int level, boolean needsLocals) {
-        return getFrameForReference(frame, startFrameInfo, VisiblePythonFramesSelector.INSTANCE, level, needsLocals);
+        return getFrameForReference(frame, startFrameInfo, AllPythonFramesSelector.INSTANCE, level, needsLocals);
     }
 
     public final PFrame getFrameForReference(Frame frame, PFrame.Reference startFrameInfo, FrameSelector selector, int level, boolean needsLocals) {