Page MenuHomePhabricator

[CI] Disable crypto assembly when building with ASAN
ClosedPublic

Authored by Fabien on Jan 7 2020, 13:54.

Details

Summary

This is known to cause issue when the SHA256 SSE4 variant is selected.
This seems to be a long time open issue:
https://github.com/google/sanitizers/issues/192

This is expected to fix this kind of failures:
https://build.bitcoinabc.org/viewLog.html?tab=buildLog&logTab=tree&filter=debug&expand=all&buildId=25123&guest=1#footer

Test Plan

On a machine that does not support SHA-NI:

ABC_BUILD_NAME=build-asan ../contrib/teamcity/build-configurations.sh

Diff Detail

Repository
rABC Bitcoin ABC
Branch
ci_asan_disable_asm
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 8762
Build 15506: Default Diff Build & Tests
Build 15505: arc lint + arc unit

Event Timeline

deadalnix requested changes to this revision.Jan 7 2020, 14:25

Add a comment in the code as to why this is disabled.

This revision now requires changes to proceed.Jan 7 2020, 14:25

Add a comment explaining the issue.

This revision is now accepted and ready to land.Jan 7 2020, 15:43