[builtins][Android] Define HAS_80_BIT_LONG_DOUBLE to 0

[rprichard]

Summary

Android 32-bit x86 uses a 64-bit long double.

Android 64-bit x86 uses a 128-bit quad-precision long double.

[compnerd]

compiler-rt/lib/builtins/int_types.h

149

Personal taste thing:

#if (defined(__i386__) || defined(__x86_64__)) && !(defined(_MSC_VER) || defined(__ANDROID__))

seems a bit easier to parse for me, especially since it sets HAS_80_BIT_LONG_DOUBLE to 1 which makes this then read as:

"x86 or x86_64 and NOT(MSVC OR ANDROID) has fp80"

[rprichard]

Refactor #if condition.

compiler-rt/lib/builtins/int_types.h

#else
   udwords high;
   udwords low;
 #endif // _YUGA_LITTLE_ENDIAN
 } uqwords;
 
 // Check if the target supports 80 bit extended precision long doubles.
 // Notably, on x86 Windows, MSVC only provides a 64-bit long double, but GCC
 // still makes it 80 bits. Clang will match whatever compiler it is trying to
-// be compatible with.
-#if ((defined(__i386__) || defined(__x86_64__)) && !defined(_MSC_VER)) ||      \
-    defined(__m68k__) || defined(__ia64__)
+// be compatible with. On 32-bit x86 Android, long double is 64 bits, while on
+// x86_64 Android, long double is 128 bits.
+#if (defined(__i386__) || defined(__x86_64__)) &&                              \
+    !(defined(_MSC_VER) || defined(__ANDROID__))
+#define HAS_80_BIT_LONG_DOUBLE 1
+#elif defined(__m68k__) || defined(__ia64__)
 #define HAS_80_BIT_LONG_DOUBLE 1
 #else
 #define HAS_80_BIT_LONG_DOUBLE 0
 #endif
 
 typedef union {
   uqwords u;
   long double f;