diff options
| author | Yann Collet <cyan@fb.com> | 2022-09-09 18:05:32 (GMT) |
|---|---|---|
| committer | Yann Collet <cyan@fb.com> | 2022-09-09 18:05:32 (GMT) |
| commit | 9bed6b56e9beb1cf820f01e47e70a13442d33544 (patch) | |
| tree | 6f0506e758c8869cf689c6d63510e84b221d44fa /.github/workflows | |
| parent | 46ecb595d86defab7fa869ebee45b80b189a3331 (diff) | |
| download | lz4-9bed6b56e9beb1cf820f01e47e70a13442d33544.zip lz4-9bed6b56e9beb1cf820f01e47e70a13442d33544.tar.gz lz4-9bed6b56e9beb1cf820f01e47e70a13442d33544.tar.bz2 | |
updated Github Actions tests documentation
notably regarding ubsan test status (now enabled)
Diffstat (limited to '.github/workflows')
| -rw-r--r-- | .github/workflows/README.md | 23 |
1 files changed, 12 insertions, 11 deletions
diff --git a/.github/workflows/README.md b/.github/workflows/README.md index 306d875..9ad2d99 100644 --- a/.github/workflows/README.md +++ b/.github/workflows/README.md @@ -4,12 +4,17 @@ This directory contains [GitHub Actions](https://github.com/features/actions) wo ## USAN, ASAN (`lz4-ubsan-x64`, `lz4-ubsan-x86`, `lz4-asan-x64`) -For now, `lz4-ubsan-*` ignores the exit code of `make usan` and `make usan32`. -Because there are several issues which may take relatively long time to resolve. - -We'll fully enable it when we ensure `make usan` is ready for all commits and PRs. - -See [#983](https://github.com/lz4/lz4/pull/983) for details. +For now, `lz4-ubsan-*` uses the `-fsanitize-recover=pointer-overflow` flag: +there are known cases of pointer overflow arithmetic within `lz4.c` fast compression. +These cases are not dangerous with current architecture, +but they are not guaranteed to work by the C standard, +which means that, in some future, some new architecture or some new compiler +may decide to do something funny that would break this behavior. +Hence it's proper to remove them. +This has been done in `lz4hc.c`. +However, the same attempt in `lz4.c` resulted in massive speed loss, +which is not acceptable to solve a "potential future" problem that does not exist anywhere today. +Therefore, a better work-around will have to be found. ## C Compilers (`lz4-c-compilers`) @@ -50,11 +55,7 @@ Also sometimes it reports false positives. # Difference with `.travis.yml` -The following tests are not included yet. - -- name: Compile OSS-Fuzz targets - -The following tests will not be included due to limitation of GH-Actions. +The following tests are not be included due to limitation of GH-Actions. - name: aarch64 real-hw tests - name: PPC64LE real-hw tests |