-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Build fails on ppc64le due to MUST_TAIL_CALL when tail call is not possible #33906
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
Conversation
EWS run on previous version of this PR (hash 7c79dd4) |
For avoidance of doubt: the changes to Skia are contributed only under Skia's standard BSD-style license. |
I would rather submit this to Skia, like we have been doing so far with all the patches we have needed until now, instead of permanently carrying a patch that needs to be applied on our end every time we want to update Skia. |
Unfortunately I'm not authorized to contribute to Skia. |
@mcatanzaro Would it be okay with you if I submit the patch to upstream Skia myself, mentioning you are the author in the commit log? |
Yes, except it's mostly authored by Dan, not be me. We'd love to see this included in Skia. (However, it seems likely that if you state you're not the author, the change will be rejected. Good luck....) |
Submitted to Skia, let's see how that goes: https://skia-review.googlesource.com/c/skia/+/910216 |
7c79dd4
to
0d29de4
Compare
EWS run on previous version of this PR (hash 0d29de4)
|
0d29de4
to
6b424d0
Compare
EWS run on current version of this PR (hash 6b424d0) |
…ssible https://bugs.webkit.org/show_bug.cgi?id=279985 Reviewed by Adrian Perez de Castro. Our use of clang::musttail attribute is not supported by Clang on ppc64le and needs to be removed. See llvm/llvm-project#93267 and llvm/llvm-project#98859 and llvm/llvm-project#108014 for more info. This change is authored by Dan Horák (thank you!). * Source/WTF/wtf/Compiler.h: Canonical link: https://commits.webkit.org/285261@main
6b424d0
to
1f02186
Compare
Committed 285261@main (1f02186): https://commits.webkit.org/285261@main Reviewed commits have been landed. Closing PR #33906 and removing active labels. |
1f02186
6b424d0