Merge pull request #9134 from fjeremic/jitdump-improvements-9133

Consolidate jitdump functionality into JitDump.cpp

runtime/compiler/build/files/common.mk

@@ -275,6 +275,7 @@ JIT_PRODUCT_SOURCE_FILES+=\
     compiler/control/DLLMain.cpp \
     compiler/control/HookedByTheJit.cpp \
     compiler/control/J9Options.cpp \
+    compiler/control/JitDump.cpp \
     compiler/control/MethodToBeCompiled.cpp \
     compiler/control/rossa.cpp \
     compiler/env/ClassLoaderTable.cpp \

runtime/compiler/control/CMakeLists.txt

@@ -27,6 +27,7 @@ j9jit_files(
 	control/HookedByTheJit.cpp
 	control/J9Options.cpp
 	control/J9Recompilation.cpp
+	control/JitDump.cpp
 	control/MethodToBeCompiled.cpp
 	control/rossa.cpp
 )

runtime/compiler/control/CompilationThread.cpp

@@ -49,6 +49,7 @@
 #include "codegen/PrivateLinkage.hpp"
 #include "compile/CompilationTypes.hpp"
 #include "compile/ResolvedMethod.hpp"
+#include "control/JitDump.hpp"
 #include "control/Recompilation.hpp"
 #include "control/RecompilationInfo.hpp"
 #include "control/MethodToBeCompiled.hpp"
@@ -194,19 +195,6 @@ TR::CompilationInfoPerThreadBase::setCompilation(TR::Compilation *compiler)
    _compiler = compiler;
    }
 
-#ifdef J9VM_RAS_DUMP_AGENTS
-static UDATA
-blankDumpSignalHandler(struct J9PortLibrary *portLibrary, U_32 gpType, void *gpInfo, void *arg)
-   {
-   J9VMThread *vmThread = (J9VMThread *) arg;
-   TR_VerboseLog::writeLineLocked(TR_Vlog_JITDUMP, "vmThread=%p Recursive crash occurred. Aborting JIT dump.", vmThread);
-
-   // Returning J9PORT_SIG_EXCEPTION_RETURN will make us come back to the same crashing instruction over and over
-   //
-   return J9PORT_SIG_EXCEPTION_RETURN; // FIXME: is this the right return type? - This appears to be the right return type
-   }
-#endif
-
 #if defined(J9VM_OPT_JITSERVER)
 thread_local TR::CompilationInfoPerThread *TR::compInfoPT;
 #endif /* defined(J9VM_OPT_JITSERVER) */