Fix after review:

* Remove unneeded whitespace changes
* Extract code related with managed delegate to separate header
* Remove unneded files and defines from cmake files

Author: Evgeny Pavlov

src/coreclr/hosts/inc/coreclrhost.h

@@ -46,7 +46,5 @@ CORECLR_HOSTING_API(coreclr_execute_assembly,
             unsigned int* exitCode);
 
 #undef CORECLR_HOSTING_API
-
+                      
 #endif // __CORECLR_HOST_H__
-
-

src/coreclr/hosts/unixcoreruncommon/CMakeLists.txt

@@ -2,10 +2,6 @@ project(unixcoreruncommon)
 
 add_compile_options(-fPIC)
 
-if(FEATURE_GDBJIT)    
-   add_definitions(-DFEATURE_GDBJIT)
-endif(FEATURE_GDBJIT)
-
 _add_library(unixcoreruncommon
     STATIC
     coreruncommon.cpp

src/coreclr/hosts/unixcoreruncommon/coreruncommon.cpp

@@ -34,6 +34,7 @@
 // If set to 1, server GC is enabled on startup. If 0, server GC is
 // disabled. Server GC is off by default.
 static const char* serverGcVar = "CORECLR_SERVER_GC";
+
 #if defined(__linux__)
 #define symlinkEntrypointExecutable "/proc/self/exe"
 #elif !defined(__APPLE__)

src/dlls/mscoree/unixinterface.cpp

@@ -15,6 +15,9 @@
 #include <utilcode.h>
 #include <corhost.h>
 #include <configuration.h>
+#ifdef FEATURE_GDBJIT
+#include "../../vm/gdbjithelpers.h"
+#endif // FEATURE_GDBJIT
 
 typedef int (STDMETHODCALLTYPE *HostMain)(
     const int argc,
@@ -254,7 +257,9 @@ int coreclr_initialize(
 
         if (!SUCCEEDED(hr))
         {
-            fprintf(stderr, "coreclr_create_delegate failed - status: 0x%08x\n", hr);
+            fprintf(stderr,
+                    "Can't create delegate for 'System.Diagnostics.Debug.SymbolReader.SymbolReader.GetInfoForMethod' "
+                    "method - status: 0x%08x\n", hr);
         }
 #endif
     }

src/inc/utilcode.h

@@ -5750,21 +5750,4 @@ extern SpinConstants g_SpinConstants;
 
 // ======================================================================================
 
-#ifdef FEATURE_GDBJIT
-struct SequencePointInfo
-{
-    int lineNumber, ilOffset;
-    char16_t* fileName;
-};
-
-struct MethodDebugInfo
-{
-    SequencePointInfo* points;
-    int size;
-};
-
-typedef int (*GetInfoForMethodDelegate)(const char*, unsigned int, MethodDebugInfo& methodDebugInfo);
-extern GetInfoForMethodDelegate getInfoForMethodDelegate;
-#endif // FEATURE_GDBJIT
-
 #endif // __UtilCode_h__

src/vm/CMakeLists.txt

@@ -29,10 +29,6 @@ endif(CLR_CMAKE_PLATFORM_UNIX)
 
 if(FEATURE_GDBJIT)
     set(VM_SOURCES_GDBJIT
-    ../gc/gccommon.cpp
-    ../gc/gcscan.cpp
-    ../gc/gcsvr.cpp
-    ../gc/gcwks.cpp
     gdbjit.cpp
     )
    add_definitions(-DFEATURE_GDBJIT)

src/vm/gdbjit.cpp

@@ -11,8 +11,8 @@
 //*****************************************************************************
 
 #include "common.h"
-#include "../inc/utilcode.h"
 #include "gdbjit.h"
+#include "gdbjithelpers.h"
 
 struct DebuggerILToNativeMap
 {

src/vm/gdbjithelpers.h

@@ -0,0 +1,31 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+// See the LICENSE file in the project root for more information.
+//*****************************************************************************
+// File: gdbjithelpers.h
+//
+//
+// Helper file with managed delegate for GDB JIT interface implemenation.
+//
+//*****************************************************************************
+
+
+#ifndef __GDBJITHELPERS_H__
+#define __GDBJITHELPERS_H__
+
+struct SequencePointInfo
+{
+    int lineNumber, ilOffset;
+    char16_t* fileName;
+};
+
+struct MethodDebugInfo
+{
+    SequencePointInfo* points;
+    int size;
+};
+
+typedef int (*GetInfoForMethodDelegate)(const char*, unsigned int, MethodDebugInfo& methodDebugInfo);
+extern GetInfoForMethodDelegate getInfoForMethodDelegate;
+
+#endif // !__GDBJITHELPERS_H__