-
Notifications
You must be signed in to change notification settings - Fork 13k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CBE should support llvm.stacksave/stackrestore #1400
Comments
Possible patch here: http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20070115/042825.html |
Gordon, The patch came through completely garbled. Could you please attach it to this PR Thanks, Reid. |
Sure. patch include/llvm/Intrinsics.td < llvm/llvm-bugzilla-archive#1028 .patch The original was a .tar.gz, which might account for the corruption. |
Hi, Gordon. I'm just testing your patches now. If it all works out, I'll commit shortly. FYI, the script in utils/mkpatch might help with future patch generation. It Thanks, Reid. |
Nice patch, Gordon. This fixes 176.gcc/CBE. Its been a long outstanding issue. Well done. Reid. |
*** Bug llvm/llvm-bugzilla-archive#1273 has been marked as a duplicate of this bug. *** |
Extended Description
The CBE should translate llvm.stacksave/stackrestore into calls to GCC's __builtin_stack_save/
__builtin_stack_restore functions. The only tricky part of this is figuring out which versions of GCC have
these builtins (if we care).
This would get 176.gcc working with the CBE, which would be very useful when bugpointing.
-Chris
The text was updated successfully, but these errors were encountered: