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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to