Age | Commit message (Collapse) | Author |
|
As of VNDK deprecation, any libraries that defines vndk is no longer
valid anymore. This change removes all VNDK definition(s) from modules
which was VNDK.
Bug: 328994089
Test: AOSP CF build succeeded
Change-Id: I40d3faf1b395cdecdcc07e4c6e8f7356cc25e517
|
|
Original change: https://android-review.googlesource.com/c/platform/external/lz4/+/2968247
Change-Id: I7b5e272bd3e034761f8a53d0fa7f27a5a09f1fd1
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
When the GPU feature is turned on in crosvm, these modules are added as
dependencies. Since crosvm is included in the virt APEX, add the APEX to
the apex_available properties of the modules to make them available in
the APEX.
Bug: 325929966
Test: m
Change-Id: If1435570fd5697af3b9989f0477014452456acbc
|
|
Original change: https://android-review.googlesource.com/c/platform/external/lz4/+/2242764
Change-Id: If757d57a188bc9e3d3f427ab528c96407c500c71
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://android-review.googlesource.com/c/platform/external/lz4/+/2242764
Change-Id: I07345ef13c88ab0adebfea94b55a67454b085c27
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://android-review.googlesource.com/c/platform/external/lz4/+/2242764
Change-Id: I10ea4937eaf0cb6721ea50057ee861deaa157760
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Test: Treehugger
Change-Id: I4f5700930d903679f51966165867a92d24ef30ec
|
|
stage v1.9.4
|
|
|
|
Support MSVC 2022 (version 17.3)
|
|
|
|
https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/strtoui64-wcstoui64-strtoui64-l-wcstoui64-l
|
|
|
|
This patch fixes the following error from "make staticAnalyze"
datagencli.c:106:21: warning: Value stored to 'size' is never read
size=0;
^ ~
|
|
Suppress the following false positive warnings from MSVC:
- Disable all arithmetic overflow (C26451)
- Suppress C6385: Reading invalid data from 'compressedBuffer'.
- Add ULL suffix to unsigned 64-bits constants.
|
|
GH-Actions: Add ubuntu-22.04 and the latest C/C++ compilers for Linux
|
|
MSVC 2022 reports the follwing false positve warnings:
lz4\tests\datagencli.c(110): warning C26451: Arithmetic overflow: Using operator '-' on a 4 byte value and then casting the result to a 8 byte value. Cast the value to the wider type before calling operator '-' to avoid overflow (io.2).
lz4\tests\datagencli.c(134): warning C26451: Arithmetic overflow: Using operator '-' on a 4 byte value and then casting the result to a 8 byte value. Cast the value to the wider type before calling operator '-' to avoid overflow (io.2).
lz4\tests\datagencli.c(146): warning C26451: Arithmetic overflow: Using operator '-' on a 4 byte value and then casting the result to a 8 byte value. Cast the value to the wider type before calling operator '-' to avoid overflow (io.2).
Although they're absolutely compiler's and static analyzer's bug,
it'd always be nice to use the standard library.
|
|
Since rc.exe (the resource compiler) is legacy compiler, it truncates preprocessor symbol name length to 32 chars.
And it reports the following warning
lz4\build\VS2022\..\..\lib\lz4.h(314): warning RC4011: identifier truncated to 'LZ4_STATIC_LINKING_ONLY_DISABLE'
lz4\build\VS2022\..\..\lib\lz4.h(401): warning RC4011: identifier truncated to 'LZ4_STATIC_LINKING_ONLY_DISABLE'
This patch detects rc.exe and just skips long symbol.
|
|
MSVC (17.3 or earlier) reports the following warning
lz4\lib\lz4.c(527): warning C6385: Reading invalid data from 'v'.
Line 527 is : LZ4_memcpy(&v[4], v, 4);
But, obviously v[0..3] is always filled with meaningful value.
Therefore, this warning report is wrong.
We must revisit this issue with future version of MSVC.
|
|
|
|
|
|
faster CLI decompression speed for frames compressed with -BD4 setting
|
|
|
|
lz4frame favors the faster prefix mode
when decompressing a frame with linked blocks.
This significantly improved CLI decompression on files compressed with -BD4 setting.
On my laptop, decompressing `enwik9` went from 0.89s to 0.52s.
This improvement is only for linked blocks.
It's more visible for small block sizes.
|
|
cmake: move `cmake_minimum_required()` before `project()`
|
|
Update documentation in preparation for release v1.9.4
|
|
cmake_minimum_required() must always be the first instruction of a CMakeLists.
project() should come after cmake_minimum_required() as soon as possible. Therefore option() are moved after project().
|
|
|
|
|
|
Doc freestanding
|
|
|
|
|
|
Add document for LZ4_STATIC_LINKING_ONLY_DISABLE_MEMORY_ALLOCATION
|
|
Add LZ4_FREESTANDING
|
|
|
|
The 'freestanding' flag indicates a capability of compiling and executing freestanding code.
Currently it requires Linux, x86_64 and (relatively newer) gcc/g++.
|
|
|
|
Since test-freestanding is able to be compiled and executed
in specific environment, we should not run it in our standard test.
|
|
|
|
|
|
|
|
|
|
Also added tests/Makefile entry "test-freestanding".
|
|
LZ4_STATIC_LINKING_ONLY_DISABLE_MEMORY_ALLOCATION is enabled
|
|
|
|
|
|
|
|
Introduce LZ4_STATIC_LINKING_ONLY_DISABLE_MEMORY_ALLOCATION
|
|
Separate support of LZ4_decompress_fast*()
|
|
single sumtype return value
|