[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 11/34] xen/arm: Introduce __builtin_stack_pointer

On 26/03/14 10:31, Ian Campbell wrote:
On Tue, 2014-03-25 at 18:01 +0000, Julien Grall wrote:
Hi Ian,

On 03/25/2014 05:18 PM, Ian Campbell wrote:
On Tue, 2014-03-25 at 16:55 +0000, Julien Grall wrote:
Clang doesn't support named register. Introduce __builtin_stack_pointer
to be able to use named register with gcc.

I think gcc considers the __builtin namespace to be its own, not sure
about clang.

I'm able to compile on GCC without any issue.

That's not the point. The point of a namespacing rule is that gcc can
add such a function at any time it likes, and if that breaks your
application then that is tough luck.

Thanks, I didn't find anything useful on internet about namespace. It seems the common rule is __foo is reserved by the compiler.

I think get_stack_pointer() would be a fine name for this macro. It
seems like the clang version should work for both gcc and clang.

Google seems to suggest that __builtin_stack_pointer might become a real
compiler builtin at some point.

I took the idea to the llvmlinux project:

But I can rename into get_stack_pointer for the next version.

Sounds good, thanks.

I will rename it and send the pathc.


Julien Grall

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.