Skip to content
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.

Sign up for GitHub

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

Jump to bottom

llvm: Apply workaround for Failed to evaluate function length in SEH unwind info in aarch64 #18690

Merged
merged 1 commit into from
Oct 13, 2023
Merged

llvm: Apply workaround for Failed to evaluate function length in SEH unwind info in aarch64 #18690

merged 1 commit into from
Oct 13, 2023

Conversation

hmartinez82
Copy link
Contributor

No description provided.

@hmartinez82 hmartinez82 changed the title clang: Apply workaround for Failed to evaluate function length in SEH unwind info in aarch64 llvm: Apply workaround for Failed to evaluate function length in SEH unwind info in aarch64 Oct 3, 2023
@lazka
Copy link
Member

lazka commented Oct 3, 2023

needs a pkgrel bump

@hmartinez82 hmartinez82 marked this pull request as ready for review October 3, 2023 07:23
@hmartinez82
Copy link
Contributor Author

Version bumped. I wonder if there's a limit of 6hrs per build and it's reaching that point

@mmuetzel
Copy link
Collaborator

Did this patch make it into the 17.0.2 release? Or do we still need to backport it in MSYS2 now that Clang 17.0.2 is being built for staging?

@hmartinez82
Copy link
Contributor Author

Humm. I don't know. llvm/llvm-project-release-prs#722 says merged for release 17.x, but I don't know if made into 17.0.2

@hmartinez82
Copy link
Contributor Author

Apparently, it did not https://github.com/llvm/llvm-project/commits/llvmorg-17.0.2

@mmuetzel
Copy link
Collaborator

Hmm... Looking at the commit history, this might have landed a couple of days after 17.0.2 was tagged...

@hmartinez82
Copy link
Contributor Author

hmartinez82 commented Oct 12, 2023 •
edited
Loading

Hmm... Looking at the commit history, this might have landed a couple of days after 17.0.2 was tagged...

Yes, they stated as much: llvm/llvm-project-release-prs#722 (comment) . I just found that.

@rimrul
Copy link
Contributor

rimrul commented Oct 12, 2023 •
edited
Loading

I wonder if there's a limit of 6hrs per build and it's reaching that point

There is a limit of 6hours per job.

https://docs.github.com/en/actions/learn-github-actions/usage-limits-billing-and-administration#usage-limits

Edit: There is some related discussion at #16101

@hmartinez82
Copy link
Contributor Author

I'll rebase this and let's try again

@rimrul rimrul mentioned this pull request Oct 12, 2023
Merged
@mmuetzel mmuetzel merged commit f172172 into msys2:master Oct 13, 2023
7 checks passed
@hmartinez82 hmartinez82 deleted the clang_aarch64_seh branch October 13, 2023 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@hmartinez82 @lazka @mmuetzel @rimrul