- Address warnings about 64-bit to 32-bit conversions.

runtime/platform/globals.h

Index: runtime/platform/globals.h
===================================================================
--- runtime/platform/globals.h	(revision 31816)
+++ runtime/platform/globals.h	(working copy)
@@ -255,11 +255,6 @@
 typedef intptr_t word;
 typedef uintptr_t uword;
 
-#if defined(TARGET_OS_WINDOWS) || defined(TARGET_OS_MACOS)
-// off64_t is not defined on Windows or Mac OS.
-typedef int64_t off64_t;
-#endif
-
 // Byte sizes.
 const int kWordSize = sizeof(word);
 const int kDoubleSize = sizeof(double);  // NOLINT
@@ -469,12 +464,12 @@
 #if !defined(TEMP_FAILURE_RETRY)
 // TEMP_FAILURE_RETRY is defined in unistd.h on some platforms. The
 // definition below is copied from Linux and adapted to avoid lint
-// errors (type long int changed to int64_t and do/while split on
+// errors (type long int changed to intptr_t and do/while split on
 // separate lines with body in {}s).
 #define TEMP_FAILURE_RETRY(expression)                                         \
-    ({ int64_t __result;                                                       \
+    ({ intptr_t __result;                                                      \
        do {                                                                    \
-         __result = static_cast<int64_t>(expression);                          \
+         __result = static_cast<intptr_t>(expression);                         \

[siva, 2014/01/16 00:15:59]

We probably should not have a static cast here to avoid inadvertant usage of
expressions that return non intptr_t values. Without the case we will get a
compilation error.

[Ivan Posva, 2014/01/16 05:05:41]

You are absolutely right. I did review the current calls to be correct, but by
not casting it we will trigger if we ever go outside of the bounds.

        } while (__result == -1L && errno == EINTR);                            \
        __result; })
 #endif  // !defined(TEMP_FAILURE_RETRY)