Add GCC CI - Githubissues

rapidfuzz / rapidfuzz-cpp

Rapid fuzzy string matching in C++ using the Levenshtein Distance

https://rapidfuzz.github.io/rapidfuzz-cpp

MIT License

238 stars 37 forks source link

Closed yaoyuan0553 closed 2 years ago

yaoyuan0553 commented 2 years ago

Note: GCC does not support -fsanitizer=fuzzer, so the fuzzing tests are disabled for the GCC CI.

maxbachmann commented 2 years ago

Thinking about this again we would need to restrict this to:

#if defined(__GNUC__) && !defined(__llvm__) && !defined(__INTEL_COMPILER) && __GNUC__ < 10

since other compilers define __GNUC__ to denote compatibility with a specific version of gcc.

yaoyuan0553 commented 2 years ago

Yes, I saw the same usages in the Catch2 library as well. The changes are updated.

maxbachmann commented 2 years ago

Thanks