[PATCH] D28016: [Builtins] [ARM] Adding Thumb1 support for fcmp

Weiming Zhao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 7 01:11:47 PST 2017


weimingz added a comment.

I can fix the comments



================
Comment at: lib/builtins/arm/comparesf2.S:196
+    bne     LOCAL_LABEL(CHECK_NAN_2)
+    negs    r0, r0 // both are positive and a < b, return -1.
+    b       LOCAL_LABEL(CHECK_NAN_2)
----------------
do you mean comments like this and line 214 too?


https://reviews.llvm.org/D28016





More information about the llvm-commits mailing list