[snappy] Support for non-BMI2 build

[ludekvodicka]

We migrated the snappy library from our build to vcpkg. Unfortunately on some of our Windows virtual machines running on ESXI we're getting illegal instruction crash:

Based on this post (https://stackoverflow.com/questions/61053436/how-to-build-snappy-on-windows-10-in-vs2015) it's necessary to disable SNAPPY_HAVE_BMI2 option.

Would be great to have a port option to build SNAPPY without BMI2 support

[ludekvodicka]

This seems to be an issue in the Snappy: conda-forge/snappy-feedstock#24

This piece of code wrongly detects BMI2 also on non-supported platforms:

check_cxx_source_compiles("
#include <immintrin.h>
int main() {
  return _bzhi_u32(0, 1);
}" SNAPPY_HAVE_BMI2) 

[JackBoosY]

Hmm... that seems like a upstream bug, can you report this to the upstream?

Thanks.

[ludekvodicka]

By upstream bug, you mean a bug directly in the snappy library?

I tried to report it but there is no "issue" section in the snappy repo and I didn't find any other way how to let them know.

https://github.com/google/snappy

[JackBoosY]

@ludekvodicka Here https://groups.google.com/forum/#!forum/snappy-compression.

[FrankXie05]

@ludekvodicka Is still work for this issue? Does the problem still exist?

[JackBoosY]

Close this PR until upstream fixes this.