[PATCH] D108416: [llvm-libgcc] initial commit

[Christopher Di Bella via Phabricator via llvm-commits]

cjdb added inline comments.


================
Comment at: llvm/tools/llvm-libgcc/lib/gcc_s.ver:4
+    defined(__ARM_ARCH_7A__) && defined(__ARM_EABI__) && \
+    defined(__ARM_FP) && __ARM_FP == 0xC
+  #define LLVM_LIBGCC_ARM_GNUEABIHF
----------------
danielkiss wrote:
> that should work as 
> __ARM_FP means HW floating point is available as
> 	• Bit 1 - half precision (16-bit).
> 	• Bit 2 - single precision (32-bit).
> 	• Bit 3 - double precision (64-bit).
> or
> 	• 0x04 for single-support.
> 	• 0x0C for single- and double-support.
> 	• 0x0E for half-, single-, and double-support.
Should it be `__ARM_FP == 0xC` or `__ARM_FP >= 0x04`?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108416/new/

https://reviews.llvm.org/D108416