order limits.h before stdlib.h to workaround for glibc _FORTIFY_SOURCE (#1184)

`limits.h` must be included before `stdlib.h` when building with glibc
and having `_FORTIFY_SOURCE` set to a non-zero value.

When building with `_FORTIFY_SOURCE`, `realpath()` is inlined, and its
definition depends on whether `limits.h` has been included or not
(clearly, this is a terrible idea in terms of interacting with Clang
modules and should probably be fixed upstream). If the definition
differs from the one in SwiftGlibc, then _TestingInternals will not
build.

Author: lhoward

Sources/_TestingInternals/include/Includes.h

@@ -29,6 +29,12 @@
 #include <ctype.h>
 #include <errno.h>
 #include <inttypes.h>
+/// limits.h must be included before stdlib.h with glibc, otherwise the
+/// fortified realpath() in this module will differ from the one in SwiftGlibc.
+/// glibc bug: https://sourceware.org/bugzilla/show_bug.cgi?id=30516
+#if __has_include(<limits.h>)
+#include <limits.h>
+#endif
 /// Guard against including `signal.h` on WASI. The `signal.h` header file
 /// itself is available in wasi-libc, but it's just a stub that doesn't actually
 /// do anything. And also including it requires a special macro definition
@@ -97,10 +103,6 @@
 #include <pwd.h>
 #endif
 
-#if __has_include(<limits.h>)
-#include <limits.h>
-#endif
-
 #if __has_include(<spawn.h>)
 #include <spawn.h>
 #endif