Skip to content

Pull recent code from origin#1

Merged
sergey-dryabzhinsky merged 20 commits intosergey-dryabzhinsky:devfrom
facebook:dev
Nov 5, 2025
Merged

Pull recent code from origin#1
sergey-dryabzhinsky merged 20 commits intosergey-dryabzhinsky:devfrom
facebook:dev

Conversation

@sergey-dryabzhinsky
Copy link
Owner

No description provided.

Cyan4973 and others added 20 commits September 24, 2025 22:46
allow the existence of a `CMakeLists.txt` file at root,
for easier integration with other projects expecting this file at root.

Existing integration point, within `build/cmake/`, still works as expected.
for CMakeLists.txt at root
Revert a branch optimization that was based on an incorrect
assumption in the AArch64 part of ZSTD_decodeSequence. In extreme
cases the existing implementation could lead to data corruption.

Insert an UNLIKELY hint to guide the compilers toward generating more
efficient machine code.
Remove the 32-bit code paths from the AArch64 only sections of
ZSTD_decodeSequence.
AArch64: Revert previous branch optimization
minor: fix mingw warnings: calloc argument order
Fix variable name typos in automated_benchmarking.py

% `ruff check`
```
Error: tests/automated_benchmarking.py:237:21: F821 Undefined name `baseline_label`
Error: tests/automated_benchmarking.py:250:21: F821 Undefined name `baseline_label`
Error: tests/automated_benchmarking.py:318:55: F821 Undefined name `frequenc`
Error: Process completed with exit code 1.
```
% [`ruff rule F821`](https://docs.astral.sh/ruff/rules/undefined-name)
Fix undefined names in automated_benchmarking.py
it's not helpful for performance, and can in some times be detrimental.
the synchronous mode is also much easier to follow and debug.
this does not provide speed benefits,
since most of the leverage happens internally within the library,
and can even become detrimental in certain scenario, due to complex and wasteful memory management.
At a minimum, it makes the logic simpler, easier to debug, at essentially the same performance.
Remove asyncio from the compression path
[cmake] propose a root wrapper
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4 to 5.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v4...v5)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '5'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
…ons/upload-artifact-5

Bump actions/upload-artifact from 4 to 5
Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.30.1 to 4.31.2.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@f1f6e5f...0499de3)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-version: 4.31.2
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
…ub/codeql-action-4.31.2

Bump github/codeql-action from 3.30.1 to 4.31.2
@sergey-dryabzhinsky sergey-dryabzhinsky self-assigned this Nov 5, 2025
@sergey-dryabzhinsky sergey-dryabzhinsky added the Update Pull code from origin label Nov 5, 2025
@sergey-dryabzhinsky sergey-dryabzhinsky merged commit f43df45 into sergey-dryabzhinsky:dev Nov 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Update Pull code from origin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants