[PATCH] D27898: [compiler-rt] [builtins] Implement __floattitf() & __floatuntitf()
Weiming Zhao via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Dec 20 22:35:35 PST 2016
weimingz added a comment.
The code looks OK except for the formatting. Since the logic is almost the same as floattixf / floatuntixf except that some constants are different, I'm wondering if it makes sense to reuse the existing code? For example, make existing code as floattixf_impl.inc
================
Comment at: lib/builtins/floattitf.c:32
+COMPILER_RT_ABI fp_t
+__floattitf(ti_int a)
+{
----------------
Please put "{" on the same line like __floattitf(ti_int a) {
Same for other occurrences below
https://reviews.llvm.org/D27898
More information about the cfe-commits
mailing list