Skip to content

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

mcatanzaro
Copy link
Contributor

@mcatanzaro mcatanzaro commented Sep 19, 2024

@mcatanzaro mcatanzaro self-assigned this Sep 19, 2024
@mcatanzaro mcatanzaro added the WebKitGTK Bugs related to the Gtk API layer. label Sep 19, 2024
@mcatanzaro
Copy link
Contributor Author

For avoidance of doubt: the changes to Skia are contributed only under Skia's standard BSD-style license.

@aperezdc
Copy link
Contributor

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.

@mcatanzaro
Copy link
Contributor Author

Unfortunately I'm not authorized to contribute to Skia.

@aperezdc
Copy link
Contributor

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?

@mcatanzaro
Copy link
Contributor Author

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....)

@aperezdc
Copy link
Contributor

Submitted to Skia, let's see how that goes: https://skia-review.googlesource.com/c/skia/+/910216

@mcatanzaro mcatanzaro force-pushed the eng/Skia-Build-fails-on-ppc64le-due-to-MUST_TAIL_CALL-when-tail-call-is-not-possible branch from 7c79dd4 to 0d29de4 Compare October 15, 2024 21:59
@mcatanzaro mcatanzaro changed the title [Skia] Build fails on ppc64le due to MUST_TAIL_CALL when tail call is not possible Build fails on ppc64le due to MUST_TAIL_CALL when tail call is not possible Oct 15, 2024
@mcatanzaro mcatanzaro force-pushed the eng/Skia-Build-fails-on-ppc64le-due-to-MUST_TAIL_CALL-when-tail-call-is-not-possible branch from 0d29de4 to 6b424d0 Compare October 15, 2024 22:00
@mcatanzaro mcatanzaro requested a review from aperezdc October 15, 2024 22:01
@mcatanzaro mcatanzaro added the merge-queue Applied to send a pull request to merge-queue label Oct 16, 2024
…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
@webkit-commit-queue webkit-commit-queue force-pushed the eng/Skia-Build-fails-on-ppc64le-due-to-MUST_TAIL_CALL-when-tail-call-is-not-possible branch from 6b424d0 to 1f02186 Compare October 16, 2024 14:13
@webkit-commit-queue
Copy link
Collaborator

Committed 285261@main (1f02186): https://commits.webkit.org/285261@main

Reviewed commits have been landed. Closing PR #33906 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit 1f02186 into WebKit:main Oct 16, 2024
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WebKitGTK Bugs related to the Gtk API layer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants