[PATCH] D28985: [Builtin][ARM] Add Thumb1 support for aeabi_c{f, d}cmp.S and dcmp.S

[Renato Golin via Phabricator via llvm-commits]

rengolin accepted this revision.
rengolin added a comment.
This revision is now accepted and ready to land.

Hi Weiming, this looks reasonable to me. Just use the same encoding for LSLS and feel free to commit.

Thanks!
--renato



================
Comment at: lib/builtins/arm/aeabi_cdcmp.S:80
+        movs r0, #1
+        lsls r0, #1
+        pop {r0-r3, pc}
----------------
Use LSLS one-reg or two-reg encoding (A8.8.94 T1), but not both.


https://reviews.llvm.org/D28985