Changeset 227725 in webkit for trunk/Source/JavaScriptCore/runtime/SamplingProfiler.h

Timestamp:

Jan 29, 2018, 2:43:13 AM (7 years ago)

Author:

Yusuke Suzuki

Message:

JSC Sampling Profiler: Detect tester and testee when sampling in RegExp JIT
​https://bugs.webkit.org/show_bug.cgi?id=152729

Reviewed by Saam Barati.

JSTests:

• stress/sampling-profiler-regexp.js: Added.

(platformSupportsSamplingProfiler.test):
(platformSupportsSamplingProfiler.baz):
(platformSupportsSamplingProfiler):

Source/JavaScriptCore:

This patch extends SamplingProfiler to recognize JIT RegExp execution. We record
executing RegExp in VM so that SamplingProfiler can detect it. This is better
than the previous VM::isExecutingInRegExpJIT flag approach since

1. isExecutingInRegExpJIT is set after starting executing JIT RegExp code. Thus,

if we suspend the thread just before executing this flag, or just after clearing
this flag, SamplingProfiler gets invalid frame, and frame validation fails. We
should set such a flag before and after executing JIT RegExp code.

2. This removes VM dependency from YarrJIT which is not essential one.

We add ExecutionContext enum to RegExp::matchInline not to mark execution if it
is done in non JS thread.

• bytecode/BytecodeDumper.cpp:

(JSC::regexpName):
(JSC::BytecodeDumper<Block>::dumpRegExps):
(JSC::regexpToSourceString): Deleted.

• heap/Heap.cpp:

(JSC::Heap::addCoreConstraints):

• runtime/RegExp.cpp:

(JSC::RegExp::compile):
(JSC::RegExp::match):
(JSC::RegExp::matchConcurrently):
(JSC::RegExp::compileMatchOnly):
(JSC::RegExp::toSourceString const):

• runtime/RegExp.h:
• runtime/RegExpInlines.h:

(JSC::RegExp::matchInline):

• runtime/RegExpMatchesArray.h:

(JSC::createRegExpMatchesArray):

• runtime/SamplingProfiler.cpp:

(JSC::SamplingProfiler::SamplingProfiler):
(JSC::SamplingProfiler::timerLoop):
(JSC::SamplingProfiler::takeSample):
(JSC::SamplingProfiler::processUnverifiedStackTraces):
(JSC::SamplingProfiler::StackFrame::nameFromCallee):
(JSC::SamplingProfiler::StackFrame::displayName):
(JSC::SamplingProfiler::StackFrame::displayNameForJSONTests):
(JSC::SamplingProfiler::StackFrame::functionStartLine):
(JSC::SamplingProfiler::StackFrame::functionStartColumn):
(JSC::SamplingProfiler::StackFrame::sourceID):
(JSC::SamplingProfiler::StackFrame::url):
(WTF::printInternal):
(JSC::SamplingProfiler::~SamplingProfiler): Deleted.

• runtime/SamplingProfiler.h:
• runtime/VM.h:
• yarr/YarrJIT.cpp:

(JSC::Yarr::YarrGenerator::generateEnter):
(JSC::Yarr::YarrGenerator::generateReturn):
(JSC::Yarr::YarrGenerator::YarrGenerator):
(JSC::Yarr::jitCompile):

• yarr/YarrJIT.h:

File:

• trunk/Source/JavaScriptCore/runtime/SamplingProfiler.h (modified) (7 diffs)

trunk/Source/JavaScriptCore/runtime/SamplingProfiler.h

@@ -40 +40 @@
 namespace JSC {
 
+class RegExp;
 class VM;
 class ExecutableBase;
@@ -69 +70 @@
         Executable,
         Host,
+        RegExp,
         C,
         Unknown
@@ -86 +88 @@
         ExecutableBase* executable { nullptr };
         JSObject* callee { nullptr };
+        RegExp* regExp { nullptr };
 
         struct CodeLocation {
@@ -142 +145 @@
         bool topFrameIsLLInt;
         void* llintPC;
+        RegExp* regExp;
         Vector<UnprocessedStackFrame> frames;
     };
@@ -157 +161 @@
 
     SamplingProfiler(VM&, RefPtr<Stopwatch>&&);
-    ~SamplingProfiler();
+    ~SamplingProfiler() = default;
     void noticeJSLockAcquisition();
     void noticeVMEntry();
@@ -186 +190 @@
     void createThreadIfNecessary(const AbstractLocker&);
     void timerLoop();
-    void takeSample(const AbstractLocker&, Seconds& stackTraceProcessingTime);
+    Seconds takeSample(const AbstractLocker&);
 
     VM& m_vm;
@@ -198 +202 @@
     RefPtr<Thread> m_thread;
     RefPtr<Thread> m_jscExecutionThread;
-    bool m_isPaused;
-    bool m_isShutDown;
+    bool m_isPaused { false };
+    bool m_isShutDown { false };
     bool m_needsReportAtExit { false };
     HashSet<JSCell*> m_liveCellPointers;