Merge pull request #91 from saulius/bugfix/ci_fixes

LLVM does not seem to be installed by default anymore

Author: saulius

.github/workflows/rust.yml

@@ -45,12 +45,6 @@ jobs:
       run: echo "LIBCLANG_PATH=C:\Program Files\LLVM\bin" >> $GITHUB_ENV
       shell: bash
 
-    # This confuses Rust's bindgen
-    # See https://github.com/rust-lang/rust-bindgen/issues/1506
-    - name: Uninstall LLVM from Homebrew
-      if: matrix.os == 'macos-latest'
-      run: brew uninstall llvm
-
     - name: Build
       run: cargo +${{ matrix.rust }} build
 

croaring/benches/performance_comparison.rs

@@ -209,7 +209,7 @@ fn perf_comp_and_inplace_rust_roaring(b: &mut Bencher) {
     let bitmap2: RoaringBitmap = (100..200).collect();
 
     b.iter(|| {
-        bitmap1.intersect_with(black_box(&bitmap2));
+        bitmap1 &= black_box(&bitmap2);
     });
 }
 
@@ -247,7 +247,7 @@ fn perf_comp_or_inplace_rust_roaring(b: &mut Bencher) {
     let bitmap2: RoaringBitmap = (100..200).collect();
 
     b.iter(|| {
-        bitmap1.union_with(black_box(&bitmap2));
+        bitmap1 |= &bitmap2;
     });
 }
 
@@ -285,7 +285,7 @@ fn perf_comp_xor_inplace_rust_roaring(b: &mut Bencher) {
     let bitmap2: RoaringBitmap = (100..200).collect();
 
     b.iter(|| {
-        bitmap1.symmetric_difference_with(black_box(&bitmap2));
+        bitmap1 ^= black_box(&bitmap2);
     });
 }