-
-
Notifications
You must be signed in to change notification settings - Fork 419
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
Upgrade to LLVM 3.9.1 #1498
Conversation
Apparently there is no Ubuntu 3.9.1 prebuilt |
@SeanTAllen Couldn't we use the one for Debian? http://releases.llvm.org/3.9.1/clang+llvm-3.9.1-x86_64-linux-gnu-debian8.tar.xz |
@Praetonus i think so, i'm going to make a change and find out. |
Pushed some changes @Praetonus. Let's see how that works out. |
@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? |
@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. |
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? |
I'm working on Windows build stuff; I'll make sure 3.9.1 gets included. |
@kulibali Awesome. Ping me here when I should update and rerun this. |
#1515 builds with 3.9.1 for Windows. If you rebase this on master (without the appveyor.yml changes) things should work. |
without the appveyor changes @kulibali ? I'm a little confused. What am I bringing over? |
The master branch is now building 3.9.1 on Windows, so you don't need to change the appveyor.yml in this PR. |
This is ready to merge |
No description provided.