[PATCH] D39700: [Builtins] Do not use tailcall for Thumb1

[Peter Smith via Phabricator via llvm-commits]

peter.smith added a comment.

I agree that we must avoid the 16-bit branch on Thumb1 and that the code sequence is fine for that purpose.

I do wonder if compiler-rt should define the __aeabi_... forwarding functions in it? I would have expected the C-library to define them. For example the v6-m libgcc.a does not define the __aeabi memcpy family, the forwarding is done in newlib. Similarly Arm's proprietary compiler's C library defines __aeabi_memcpy that just falls through directly into memcpy. I'm guessing that there is some C-library that we support that does not define the __aeabi_ function?


https://reviews.llvm.org/D39700