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

Upgrade to LLVM 3.9.1 #1498

Merged
merged 3 commits into from
Jan 18, 2017
Merged

Upgrade to LLVM 3.9.1 #1498

merged 3 commits into from
Jan 18, 2017

Conversation

SeanTAllen
Copy link
Member

No description provided.

@SeanTAllen SeanTAllen changed the title Move to 3.9.1 as supported LLVM 3.9.x version Upgrade to LLVM 3.9.1 Jan 5, 2017
@SeanTAllen SeanTAllen added the changelog - added Automatically add "Added" CHANGELOG entry on merge label Jan 5, 2017
@SeanTAllen
Copy link
Member Author

Apparently there is no Ubuntu 3.9.1 prebuilt

http://releases.llvm.org/3.9.1/

@Praetonus
Copy link
Member

@SeanTAllen
Copy link
Member Author

@Praetonus i think so, i'm going to make a change and find out.

@SeanTAllen
Copy link
Member Author

Pushed some changes @Praetonus. Let's see how that works out.

@SeanTAllen
Copy link
Member Author

@ponylang/committer hmmm the windows version of this isn't going to pass until we update https://github.com/ponylang/ponyc-windows-llvm/

Anyone with vagrant et al installed want to do that?

@SeanTAllen
Copy link
Member Author

@Praetonus i switched it but how we have it set right now, that then breaks the other versions because in life we aren't allowed to have nice things. I'll fix this up another time. First I'm going to make sure that 3.9.1 works even if the others don't.

@SeanTAllen SeanTAllen added changelog - changed Automatically add "Changed" CHANGELOG entry on merge and removed changelog - added Automatically add "Added" CHANGELOG entry on merge labels Jan 7, 2017
@SeanTAllen
Copy link
Member Author

In case I don't make the sync on the 11th. Can we try and get someone to update our Windows LLVM version to 3.9.1 so we can get this merged?

@chalcolith
Copy link
Member

I'm working on Windows build stuff; I'll make sure 3.9.1 gets included.

@SeanTAllen
Copy link
Member Author

@kulibali Awesome. Ping me here when I should update and rerun this.

@chalcolith
Copy link
Member

#1515 builds with 3.9.1 for Windows. If you rebase this on master (without the appveyor.yml changes) things should work.

@SeanTAllen
Copy link
Member Author

without the appveyor changes @kulibali ?

I'm a little confused. What am I bringing over?

@chalcolith
Copy link
Member

The master branch is now building 3.9.1 on Windows, so you don't need to change the appveyor.yml in this PR.

@SeanTAllen
Copy link
Member Author

This is ready to merge

@Theodus Theodus merged commit 4c07d17 into master Jan 18, 2017
ponylang-main added a commit that referenced this pull request Jan 18, 2017
@SeanTAllen SeanTAllen deleted the llvm-3.9.1 branch June 7, 2018 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog - changed Automatically add "Changed" CHANGELOG entry on merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants