Fix incorrect use of GetProcessMemoryInfo's return value. (#1315)

Summary:
According to the API documentation [GetProcessMemoryInfo function (psapi.h)
](https://learn.microsoft.com/en-us/windows/win32/api/psapi/nf-psapi-getprocessmemoryinfo)

> If the function succeeds, the return value is nonzero.
> If the function fails, the return value is zero. To get extended error information, call [GetLastError](https://learn.microsoft.com/en-us/windows/desktop/api/errhandlingapi/nf-errhandlingapi-getlasterror).

Pull Request resolved: #1315

Test Plan:
- Turn on ShouldRecordStats ( GCConfig.h )
- Be able to get the actual value instead of 0

Reviewed By: neildhar

Differential Revision: D53901267

Pulled By: tmikov

fbshipit-source-id: 9053e0e94eddee30a9bcf6b50e32bd01a0172eb5

lib/Support/OSCompatWindows.cpp

@@ -355,8 +355,8 @@ int pages_in_ram(const void *p, size_t sz, llvh::SmallVectorImpl<int> *runs) {
 
 uint64_t peak_rss() {
   PROCESS_MEMORY_COUNTERS pmc;
-  auto ret = GetProcessMemoryInfo(GetCurrentProcess(), &pmc, sizeof(pmc));
-  if (ret != 0) {
+  BOOL ret = GetProcessMemoryInfo(GetCurrentProcess(), &pmc, sizeof(pmc));
+  if (!ret) {
     // failed
     return 0;
   }
@@ -365,8 +365,8 @@ uint64_t peak_rss() {
 
 uint64_t current_rss() {
   PROCESS_MEMORY_COUNTERS pmc;
-  auto ret = GetProcessMemoryInfo(GetCurrentProcess(), &pmc, sizeof(pmc));
-  if (ret != 0) {
+  BOOL ret = GetProcessMemoryInfo(GetCurrentProcess(), &pmc, sizeof(pmc));
+  if (!ret) {
     // failed
     return 0;
   }