nikic wrote:

> This PR only adds support for the following architectures: x86 - x86_64. 
> Support for other architectures can be added in future patches.

This makes for pretty bad UX. To the degree it is feasible, this should be 
supported on all arches from day one, at least if a Clang builtin is exported. 
Don't we already have the necessary logic for this for llvm.stacksave? There's 
a ISD::STACKSAVE opcode and targets call setStackPointerRegisterToSaveRestore() 
to specify the stack register.

I'm not sure if there is any expected difference in behavior between STACKSAVE 
and STACKADDRESS, but we should at least be able to reuse something here.

https://github.com/llvm/llvm-project/pull/148281
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to