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