Fix compilation warnings on ARM64 with old GCC versions. #52
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@cmumford PTAL? |
cmumford
reviewed
Jun 22, 2021
Tested on Chrome OS M93 Linux (Debian 10.10) on kukui.
|
@cmumford Can you please take a look at the latest revision? Is this clearer? |
|
(Also, thank you very much for the review and feedback!) |
|
Thanks Victor. |
|
Thank you very much for the quick review! |
swtrse
added a commit
to muecoin/crc32c-subtree
that referenced
this pull request
Sep 24, 2021
commit b9d6e82 Author: Victor Costan <costan@google.com> Date: Wed Sep 1 09:19:47 2021 -0700 Fix Windows CI build. (google#54) The Windows SDK versions present on our CI options (GitHub Actions hosted runners, AppVeyor) have headers that cause compilation warnings when included with MSVC operating in modern C modes (C11+). This PR disables the compilation warning caused by the Windows SDK headers, so we can get Windows feedback from CI. The warnings can be re-enabled when the Windows SDK used by our CI is upgraded to a version that doesn't trigger warnings. commit bbbb93a Author: Victor Costan <costan@google.com> Date: Wed Sep 1 09:17:07 2021 -0700 Switch CI to GitHub Actions (google#55) * Fix Windows CI build. The Windows SDK versions present on our CI options (GitHub Actions hosted runners, AppVeyor) have headers that cause compilation warnings when included with MSVC operating in modern C modes (C11+). This PR disables the compilation warning caused by the Windows SDK headers, so we can get Windows feedback from CI. The warnings can be re-enabled when the Windows SDK used by our CI is upgraded to a version that doesn't trigger warnings. * Switch CI to GitHub Actions. commit d46cd17 Author: Victor Costan <pwnall@chromium.org> Date: Mon Aug 30 19:11:12 2021 -0700 Add clangd cache directory to .gitignore. commit fa5ade4 Author: Victor Costan <costan@google.com> Date: Tue Jun 22 14:30:59 2021 -0700 Fix compilation warnings on ARM64 with old GCC versions. (google#52) Tested on Chrome OS M93 Linux (Debian 10.10) on kukui. commit db08d22 Author: Chris Mumford <cmumford@cmumford.com> Date: Sun Jun 20 08:18:01 2021 -0700 Updated Travis-CI configuration. (google#51) clang/llvm: 10 → 12 gcc: 10 → 11 commit e31619a Author: Francois Marier <francois@fmarier.org> Date: Wed Jun 2 12:18:58 2021 -0700 Fix GitHub links. (google#50) These repos have moved to different GitHub user accounts or orgs. commit 7fa4c26 Author: Victor Costan <costan@google.com> Date: Sun Dec 13 22:55:19 2020 -0800 Update Travis CI config. (google#49) commit a3d9e6d Author: Victor Costan <costan@google.com> Date: Sun Dec 13 19:23:42 2020 -0800 Updated third_party/ and Travis CI config. (google#48)
maflcko
pushed a commit
to bitcoin-core/crc32c-subtree
that referenced
this pull request
Sep 29, 2021
fa5ade4 Fix compilation warnings on ARM64 with old GCC versions. (google#52) (Victor Costan) db08d22 Updated Travis-CI configuration. (google#51) (Chris Mumford) e31619a Fix GitHub links. (google#50) (Francois Marier) 7fa4c26 Update Travis CI config. (google#49) (Victor Costan) a3d9e6d Updated third_party/ and Travis CI config. (google#48) (Victor Costan) Pull request description: Only change is a warning fix for arm ACKs for top commit: sipa: Code review ACK cac7ca8 Tree-SHA512: a4e0b1033a4cbdfdcdf8288d92be34a73e4a3d4f9660bfef9c6f12388a1510a3af49924e0d73f4c12c8b785e420c7db74b0159431a3509dee636b4b1434f8ff9
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Tested on Chrome OS M93 Linux (Debian 10.10) on kukui.