denzp / rust-ptx-linker

The missing puzzle piece for NVPTX experience with Rust
MIT License
52 stars 11 forks source link

Update dependencies (except for llvm-sys) #24

Closed gnzlbg closed 5 years ago

gnzlbg commented 5 years ago

Updating llvm-sys requires some more work than just upgrading to llvm 70 APIs.

denzp commented 5 years ago

Thanks a lot! v0.8.2 is out.

Would updating llvm-sys to 70 give us immediate benefits? I could only think of methods that got deprecated in LLVM 7.0, but the crate is still using them. Is there any other reason to update?

gnzlbg commented 5 years ago

It cuts compile-times a bit, and the different llvm-sys versions are incompatible, so we should upgrade to be able to use the last rustc-llvm-proxy crate here IIUC (a workspace solution for the fundamental crates would have forced us to do the llvm-sys upgrade for all crates at once, making sure everything works).