[PATCH] D27898: [compiler-rt] [builtins] Implement __floattitf() & __floatuntitf()

Michał Górny via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 22 10:04:06 PST 2016


mgorny added inline comments.


================
Comment at: lib/builtins/floatuntitf.c:73
+    long_double_bits fb;
+    fb.u.high.all = (du_int)(e + 16383) << 48            /* exponent */
+                  | ((a >> 64) & 0x0000ffffffffffffLL);  /* mantissa */
----------------
rengolin wrote:
> nit: an hexadecimal pattern here would be clearer. same above.
Do you mean something like: `(foo << 48) & 0xffff....`?


https://reviews.llvm.org/D27898





More information about the cfe-commits mailing list