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

Removing LLVM 3.7 support #2063

Closed
Praetonus opened this issue Jul 20, 2017 · 6 comments · Fixed by #2461
Closed

Removing LLVM 3.7 support #2063

Praetonus opened this issue Jul 20, 2017 · 6 comments · Fixed by #2461

Comments

@Praetonus
Copy link
Member

We're currently in the process of adding support for LLVM 4. When we added LLVM 3.9, we removed support of LLVM 3.6 soon after. The motivation for that was that the burden of maintaining as many LLVM versions was deemed to high.

By this logic, we should remove LLVM 3.7 once LLVM 4 is in.

@ponylang/committer Any objections?
@SeanTAllen
Copy link
Member

I believe we removed 3.6 when homebrew stopped supporting it. I don't believe we made the decision because of the number of LLVM versions that we were supporting. That said, I don't see any reason to continue supporting LLVM 3.7. I think it makes sense to email the user and developer lists and verify no one needs 3.7. If they do, I think it behooves us to find out why 3.7 is required before removing support.

@SeanTAllen SeanTAllen mentioned this issue Jul 20, 2017
Closed
@Praetonus
Copy link
Member Author

@SeanTAllen Yes that's right, I didn't remember the whole story correctly.

I'll send the email.

@SeanTAllen
Copy link
Member

@Praetonus I sent emails and a tweet. I'm in favor of removal but I think its best to make sure no one needs 3.7.

@Praetonus
Copy link
Member Author

Ah, I didn't see your email. Thanks.

@SeanTAllen
Copy link
Member

@Praetonus no comments on this so far. seems like there are no objections.

@SeanTAllen
Copy link
Member

We are in agreement. This can go forward.

SeanTAllen added a commit that referenced this issue Jan 3, 2018
@SeanTAllen
Deprecate LLVM 3.7 and 3.8
e867c39 
On the most recent sync call we decided to only officially support LLVM
3.9.1. Additionally we are maintaing CI for our experimentally supported
LLVM 4 and 5 work. The goal is to get LLVM 4 and 5 issues fixed so they
are no longer experimental.

Closes #2063
SeanTAllen added a commit that referenced this issue Jan 3, 2018
@SeanTAllen
Deprecate LLVM 3.7 and 3.8
1352c31 
On the most recent sync call we decided to only officially support LLVM
3.9.1. Additionally we are maintaing CI for our experimentally supported
LLVM 4 and 5 work. The goal is to get LLVM 4 and 5 issues fixed so they
are no longer experimental.

Closes #2063
@SeanTAllen SeanTAllen mentioned this issue Jan 3, 2018
Merged
SeanTAllen added a commit that referenced this issue Jan 4, 2018
@SeanTAllen
Deprecate LLVM 3.7 and 3.8
33bcf80 
On the most recent sync call we decided to only officially support LLVM
3.9.1. Additionally we are maintaing CI for our experimentally supported
LLVM 4 and 5 work. The goal is to get LLVM 4 and 5 issues fixed so they
are no longer experimental.

Closes #2063
SeanTAllen added a commit that referenced this issue Jan 4, 2018
@SeanTAllen
Deprecate LLVM 3.7 and 3.8 (#2461)
0447816 
On the most recent sync call we decided to only officially support LLVM
3.9.1. Additionally we are maintaing CI for our experimentally supported
LLVM 4 and 5 work. The goal is to get LLVM 4 and 5 issues fixed so they
are no longer experimental.

Closes #2063
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Deprecate LLVM 3.7.1 and 3.8.1 support ponylang/ponyc
2 participants
@SeanTAllen @Praetonus