[PATCH] D54911: [compiler-rt][builtins][PowerPC] Add ___fixunstfti builtin compiler-rt method support for PowerPC

Robert Enenkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 19 21:12:46 PST 2018


renenkel requested changes to this revision.
renenkel added a comment.
This revision now requires changes to proceed.

In floattitf_test.c,

  typedef union {
    long double ld;
    double doublePairs[2]; /* [0] is the high double, [1] is the low double. */
    unsigned long long doubleBitPatterns[2];
  } ldStructure;

is confusing. Clearer would be:

  typedef union {
    long double ld;
    double d[2]; /* [0] is the high double, [1] is the low double. */
    unsigned long long ull[2]; /* high and low doubles as 64-bit ints */
  } ldStructure;


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

https://reviews.llvm.org/D54911





More information about the llvm-commits mailing list