diff options
| author |  Derek Mauro <dmauro@google.com> | 2022-11-30 10:58:38 -0800 |
|---|---|---|
| committer |  Copybara-Service <copybara-worker@google.com> | 2022-11-30 10:59:21 -0800 |
| commit | 66665d8d2e3fedff340b83f9841ca427145a7b26 (patch) | |
| tree | 747602a57a578dc3220d8e61298184c43bbf8deb /ci | |
| parent | 94e9ee3f75a617403049a29e6c932b8b3bb13177 (diff) | |
Fixes many compilation issues that come from having no external CI
coverage of the accelerated CRC implementation and some differences
bewteen the internal and external implementation.
This change adds CI coverage to the
linux_clang-latest_libstdcxx_bazel.sh script assuming this script
always runs on machines of at least the Intel Haswell generation.
Fixes include:
* Remove the use of the deprecated xor operator on crc32c_t
* Remove #pragma unroll_completely, which isn't known by GCC or Clang:
https://godbolt.org/z/97j4vbacs
* Fixes for -Wsign-compare, -Wsign-conversion and -Wshorten-64-to-32
PiperOrigin-RevId: 491965029
Change-Id: Ic5e1f3a20f69fcd35fe81ebef63443ad26bf7931
Diffstat (limited to 'ci')
| -rwxr-xr-x | ci/linux_clang-latest_libstdcxx_bazel.sh | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/ci/linux_clang-latest_libstdcxx_bazel.sh b/ci/linux_clang-latest_libstdcxx_bazel.sh index 720e776d..13d56fc9 100755 --- a/ci/linux_clang-latest_libstdcxx_bazel.sh +++ b/ci/linux_clang-latest_libstdcxx_bazel.sh @@ -77,6 +77,7 @@ for std in ${STD}; do --copt="--gcc-toolchain=/usr/local" \ --copt="-DGTEST_REMOVE_LEGACY_TEST_CASEAPI_=1" \ --copt="${exceptions_mode}" \ + --copt="-march=haswell" \ --copt=-Werror \ --define="absl=1" \ --distdir="/bazel-distdir" \ |