[PATCH] D57001: [libunwind] Don't define unw_fpreg_t to uint64_t for __ARM_DWARF_EH__

Saleem Abdulrasool via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 24 09:24:18 PST 2019


compnerd accepted this revision.
compnerd added a comment.
This revision is now accepted and ready to land.

`double` should be safe for ARM DWARF EH, though, technically, `long double` is more appropriate of a type definition (the FPU state that is saved should be the widest floating point type).  That would be `long double` (aka FP80 on x86, FP128 on AArch64/PPC, FP64 elsewhere).  This happens to work because ARM uses FP64 irrespective of DWARF or EHABI.


Repository:
  rUNW libunwind

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

https://reviews.llvm.org/D57001





More information about the cfe-commits mailing list