ci: Remove llvm 5, introduce llvm 16

LLVM 5 is ancient.

Author: emilio

.github/workflows/bindgen.yml

@@ -142,7 +142,7 @@ jobs:
  - debian: null
  cross: null
  rust: null
- llvm_version: ["5.0", "9.0"]
+ llvm_version: ["9.0", "16.0"]
  main_tests: [1]
  release_build: [0, 1]
  no_default_features: [0, 1]
@@ -155,7 +155,7 @@ jobs:
  # Test with extra asserts + docs just with latest llvm versions to
  # prevent explosion
  - os: ubuntu-latest
- llvm_version: "9.0"
+ llvm_version: "16.0"
  release_build: 0
  no_default_features: 0
  feature_extra_asserts: 1
@@ -169,17 +169,20 @@ jobs:
  # debian: arm64
  # cross: aarch64-linux-gnu
  # rust: aarch64-unknown-linux-gnu
- # llvm_version: "9.0"
+ # llvm_version: "16.0"
  # main_tests: 0
  # release_build: 0
  # feature_extra_asserts: 0
 
  # Ensure stuff works on macos too
- - os: macos-latest
- llvm_version: "9.0"
- release_build: 0
- no_default_features: 0
- feature_extra_asserts: 0
+ # Disabled for now because llvm doesn't provide releases for x86-64
+ # macOS which is what the runner uses.
+ #
+ # - os: macos-latest
+ # llvm_version: "16.0"
+ # release_build: 0
+ # no_default_features: 0
+ # feature_extra_asserts: 0
  steps:
  - uses: actions/checkout@v3
 

CONTRIBUTING.md

@@ -208,8 +208,8 @@ can add multiple test expectations, one for each supported `libclang`
 version. Instead of having a single `bindgen-tests/tests/expectations/tests/my_test.rs` file,
 add each of:
 
+* `bindgen-tests/tests/expectations/tests/libclang-16/my_test.rs`
 * `bindgen-tests/tests/expectations/tests/libclang-9/my_test.rs`
-* `bindgen-tests/tests/expectations/tests/libclang-5/my_test.rs`
 
 If you need to update the test expectations for a test file that generates
 different bindings for different `libclang` versions, you *don't* need to have
@@ -226,12 +226,6 @@ to check the bindings against with a cargo feature:
 $ cargo test --features __testing_only_libclang_$VERSION
 ```
 
-Where `$VERSION` is one of:
-
-* `4`
-* `3_9`
-* `3_8`
-
 depending on which version of `libclang` you have installed.
 
 ### Integration Tests

appveyor.yml

@@ -2,18 +2,18 @@ environment:
  RUST_BACKTRACE: 1
  RUST_CHANNEL: "%Configuration%"
  matrix:
- - TARGET: gnu
- LLVM_VERSION: 5.0.0-1
- BINDGEN_FEATURES: __testing_only_libclang_5
  - TARGET: gnu
  LLVM_VERSION: 9.0.0-1
  BINDGEN_FEATURES: __testing_only_libclang_9
- - TARGET: msvc
- LLVM_VERSION: 5.0.0
- BINDGEN_FEATURES: __testing_only_libclang_5
+ - TARGET: gnu
+ LLVM_VERSION: 16.0.4
+ BINDGEN_FEATURES: __testing_only_libclang_16
  - TARGET: msvc
  LLVM_VERSION: 9.0.0
  BINDGEN_FEATURES: __testing_only_libclang_9
+ - TARGET: msvc
+ LLVM_VERSION: 16.0.4
+ BINDGEN_FEATURES: __testing_only_libclang_16
 
 configuration:
 - stable

bindgen-cli/Cargo.toml

@@ -40,7 +40,7 @@ which-rustfmt = ["bindgen/which-rustfmt"]
 # Features used for CI testing 
 __testing_only_extra_assertions = ["bindgen/__testing_only_extra_assertions"]
 __testing_only_libclang_9 = ["bindgen/__testing_only_libclang_9"]
-__testing_only_libclang_5 = ["bindgen/__testing_only_libclang_5"]
+__testing_only_libclang_16 = ["bindgen/__testing_only_libclang_16"]
 
 [package.metadata.release]
 release = true

bindgen-cli/main.rs

@@ -13,10 +13,10 @@ use crate::options::builder_from_flags;
 #[cfg(feature = "logging")]
 fn clang_version_check() {
  let version = bindgen::clang_version();
- let expected_version = if cfg!(feature = "__testing_only_libclang_9") {
+ let expected_version = if cfg!(feature = "__testing_only_libclang_16") {
+ Some((16, 0))
+ } else if cfg!(feature = "__testing_only_libclang_9") {
  Some((9, 0))
- } else if cfg!(feature = "__testing_only_libclang_5") {
- Some((5, 0))
  } else {
  None
  };

bindgen-integration/Cargo.toml

@@ -16,4 +16,4 @@ runtime = ["bindgen/runtime"]
 
 __testing_only_extra_assertions = ["bindgen/__testing_only_extra_assertions"]
 __testing_only_libclang_9 = ["bindgen/__testing_only_libclang_9"]
-__testing_only_libclang_5 = ["bindgen/__testing_only_libclang_5"]
+__testing_only_libclang_16 = ["bindgen/__testing_only_libclang_16"]

bindgen-tests/Cargo.toml

@@ -23,4 +23,4 @@ which-rustfmt = ["bindgen/which-rustfmt"]
 
 __testing_only_extra_assertions = ["bindgen/__testing_only_extra_assertions"]
 __testing_only_libclang_9 = ["bindgen/__testing_only_libclang_9"]
-__testing_only_libclang_5 = ["bindgen/__testing_only_libclang_5"]
+__testing_only_libclang_16 = ["bindgen/__testing_only_libclang_16"]

bindgen-tests/tests/expectations/build.rs

@@ -8,7 +8,7 @@ use std::fs;
 use std::io::Write;
 use std::path::Path;
 
-const LIBCLANG_VERSION_DIRS: &[&str] = &["libclang-5", "libclang-9"];
+const LIBCLANG_VERSION_DIRS: &[&str] = &["libclang-9"];
 
 fn main() {
  println!("cargo:rerun-if-changed=build.rs");