GC OS interface refactoring

This change replaces all calls of OS specific functions in the GC by a call to
a platform agnostic interface. Critical sections were abstracted too.
The logging file access was changed to use CRT functions instead of Windows specific APIs.
A "size" member was added to the card_table_info so that we can pass the right
size to the VirtualRelease method when destroying the card table.
I have also fixed a bug in the gc_heap::make_card_table error path where when VirtualCommit
failed, it called VirtualRelease with size that was not the reserved size, but
the committed size.
Other related changes
- All interlocked operations moved to Interlocked class as static methods
- Removed unused function prototypes
- Shuffled stuff in the root CMakeLists.txt to enable building the GC sample using the
  settings inherited from the root CMakeLists.txt and to clean up some things that have
  rotted over time, like the FEATURE_xxx macros not being in one alphabetically ordered
  block
- Fixed the VOLATILE_MEMORY_BARRIER macro in the gcenv.base.h
- Replaced uint32_t thread id by EEThreadId
- Removed thread handles storage (g_gc_thread) from the GC. The thread handle is closed right after the thread is launched. That allowed me to get rid of the GCThreadHandle
- Renamed the methods of the EEThreadId to be easier to understand
- Moved the gcenv.windows.cpp and gcenv.unix.cpp to the sample folder

1 files changed, 3 insertions, 1 deletions

diff --git a/src/vm/common.h b/src/vm/common.h
index e323333ef5..2d23f8c471 100644
--- a/src/vm/common.h
+++ b/src/vm/common.h
@@ -288,7 +288,6 @@ namespace Loader
     } LoadFlag;
 }
 
-
 // src/inc
 #include "utilcode.h"
 #include "log.h"
@@ -297,6 +296,9 @@ namespace Loader
 #include "lazycow.h"
 
 // src/vm
+#include "gcenv.interlocked.h"
+#include "gcenv.interlocked.inl"
+
 #include "util.hpp"
 #include "ibclogger.h"
 #include "eepolicy.h"