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

[Daniel Kiss via Phabricator via llvm-commits]

danielkiss 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
----------------
cjdb wrote:
> 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`?
`__ARM_FP >= 0x04`  is better because on some targets the 64bit operations are available.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108416