Fix warnings in the jitter code

This change fixes some of the warnings in the jitter code on Linux. With this change
combined with an upcoming change in the rest of the codebase, 16 warning disabling
options can be removed.

The issues in the jitter were:
1) Incorrect typedefs for const_reference and const_pointer (the const was ignored)
2) Member initiazalization order in class constructor didn't correspond to the member order in some classes
3) Objects with vtables were copied via memcpy. While this is intentional, cast of the pointers to void*
   is required to silence clang warning
4) Comparing values of different enums - there are cases of two enums containing the same values.
5) Casting int to pointer - the cast was legal (forming a handle), adding cast to size_t in between used to fix the warning
6) "static struct" - removed the static keyword
7) Missing return value in methods with _ASSERTE
8) Class name classifier on methods in the .h
9) Specialized template member functions need to be defined out of the class

1 files changed, 2 insertions, 1 deletions

diff --git a/src/jit/codegeninterface.h b/src/jit/codegeninterface.h
index 75d210a80e..672c51706d 100644
--- a/src/jit/codegeninterface.h
+++ b/src/jit/codegeninterface.h
@@ -88,6 +88,8 @@ public:
 
     void                genCalcFrameSize ();
 
+    GCInfo              gcInfo;
+
     RegSet              regSet;
     RegState            intRegState;
     RegState            floatRegState;
@@ -319,7 +321,6 @@ protected:
 #endif // LATE_DISASM
 
 public:
-    GCInfo              gcInfo;
 
 #ifdef DEBUG
     void                setVerbose(bool value)      { verbose = value; }