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

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 7 15:20:12 PST 2017


rengolin added a comment.

In https://reviews.llvm.org/D39700#918644, @dalias wrote:

> This commit in musl has details on why calling memset, etc. doesn't work and violates the ABI:
>
> https://git.musl-libc.org/cgit/musl/commit?id=e6def544358afd5648a428d2e02c147a1f901048


oh, hum, that's a good point, actually.

Unfortunately, I think that "fix" is present in Android and Chromium, at least. Plus in RT for the past 5 years... :/

I don't think anyone uses it, though.


https://reviews.llvm.org/D39700





More information about the llvm-commits mailing list