[MERGE #4731 @leirocks] fixes for dynamic profile caching

Merge pull request #4731 from leirocks:dpcmaster

Previously there's leak because dynamic profile info is pinned and causes two roots(another is global function body) on a ring.
The fix is to use a heap allocated structure to save the information to dump to dynamic profile cache.
The fix missed a scenario the while loading from cache, it didn't save to the heap allocated dynamicProfileInfoMap,
and later some function is inlined, cause the info is missed while saving to the cache file, causes the cache file mismatch with the script code

lib/Runtime/Language/DynamicProfileInfo.cpp

@@ -1971,11 +1971,13 @@ namespace Js
 
             if (!reader->Read(functionId))
             {
+                AssertOrFailFast(false);
                 return nullptr;
             }
 
             if (!reader->Read(&paramInfoCount))
             {
+                AssertOrFailFast(false);
                 return nullptr;
             }
 
@@ -2206,6 +2208,7 @@ namespace Js
         }
 
     Error:
+        AssertOrFailFast(false);
         return nullptr;
     }
 

lib/Runtime/Language/DynamicProfileInfo.h

@@ -889,6 +889,7 @@ namespace Js
         {
             if (lengthLeft < sizeof(T))
             {
+                AssertOrFailFast(false);
                 return false;
             }
             *data = *(T *)current;
@@ -914,6 +915,7 @@ namespace Js
             size_t size = sizeof(T) * len;
             if (lengthLeft < size)
             {
+                AssertOrFailFast(false);
                 return false;
             }
             memcpy_s(data, size, current, size);