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.

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

[llvm] add features enable-{eh,ffi,bindings,terminfo,threads} #16877

Merged
merged 1 commit into from
Mar 29, 2021

Conversation

abique
Copy link
Contributor

@abique abique commented Mar 25, 2021

Adds missing features to LLVM

@abique abique force-pushed the llvm-no-curses branch 3 times, most recently from 81d9f25 to 6773251 Compare March 25, 2021 14:42
@yurybura
Copy link
Contributor

Related to #16232

@abique
Copy link
Contributor Author

abique commented Mar 25, 2021

They do not overlap through.

@abique
Copy link
Contributor Author

abique commented Mar 25, 2021

Yay, all the tests passed, this looks good to me.

@abique abique changed the title [llvm] add more features [llvm] add features enable-{eh,ffi,bindings,terminfo,threads} Mar 25, 2021
@Cheney-W Cheney-W self-assigned this Mar 26, 2021
@Cheney-W Cheney-W added category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist info:reviewed Pull Request changes follow basic guidelines labels Mar 26, 2021
Copy link
Contributor

@yurybura yurybura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check my comments below.

ports/llvm/CONTROL Show resolved Hide resolved
ports/llvm/CONTROL Show resolved Hide resolved
ports/llvm/CONTROL Show resolved Hide resolved
ports/llvm/CONTROL Show resolved Hide resolved
ports/llvm/CONTROL Show resolved Hide resolved
@Cheney-W Cheney-W added requires:author-response and removed info:reviewed Pull Request changes follow basic guidelines labels Mar 26, 2021
@abique
Copy link
Contributor Author

abique commented Mar 26, 2021

@yurybura Thank you very much for the review 👍

@abique
Copy link
Contributor Author

abique commented Mar 26, 2021

It seems that the osgearth failure is unrelated right?

@Cheney-W
Copy link
Contributor

Cheney-W commented Mar 26, 2021

Yes, this is a CI issue, it will be fixed by PR: #16904.

@JackBoosY JackBoosY added the depends:different-pr This PR or Issue depends on a PR which has been filed label Mar 26, 2021
@Cheney-W Cheney-W added info:reviewed Pull Request changes follow basic guidelines and removed depends:different-pr This PR or Issue depends on a PR which has been filed labels Mar 29, 2021
@vicroms vicroms merged commit 9b90e8a into microsoft:master Mar 29, 2021
@abique abique deleted the llvm-no-curses branch March 30, 2021 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants