[PATCH] D22254: [ARM, AArch64] Adds builtins to get current SP and PC

[Renato Golin via cfe-commits]

rengolin added a comment.

In http://reviews.llvm.org/D22254#481972, @mgrang wrote:

> Hi Tim/Renato,
>
> On a similar note, I have a patch which adds the breakpoint intrinsic (for ARM and Thumb):
>  http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.dui0491c/CJAHBCFH.html


Hi Mandeep,

I don't think we should be implementing old ARMCC intrinsics either. GCC doesn't, especially since most of them are easy to do with one line inline assembly snippets.

Those intrinsics are not covered by the ACLE (mentioned, but not defined), which reinforces the notion that we shouldn't be doing anyway.

cheers,
--renato


http://reviews.llvm.org/D22254