[PATCH] D29485: [Builtin][ARM] Implement addsf3/__aeabi_fadd for Thumb1

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

> all constants are materialized instead of loading from constant pool
>  no stack spills (C version uses 136 bytes stack space)
>  clz() is called only when necessary. (C version always calls it)

If we have to hand-write assembly to efficiently materialize constants, something has gone very wrong.  It's better to fix the compiler if we can; please file bugs for these issues.


https://reviews.llvm.org/D29485