[llvm] [BOLT][AArch64] Enabling Inlining for Memcpy for AArch64 in BOLT (PR #154929)
Sjoerd Meijer via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 4 12:02:56 PDT 2025
sjoerdmeijer wrote:
>
> Beyond the lit test, I ran some brief smoketests on a few real-world binaries to validate the pattern detection of the immediate movs. I have shared a few of them below:
>
> Application Inlined Calls Total Calls Hit Rate
> Redis 22 558 4%
> PostgreSQL 78 956 8%
> Zstd 31 177 18%
> ## Correctness Testing
> * **Redis**: `redis-benchmark`./runtest --single unit/memefficiency`(memory-specific test) +`redis-benchmark` SET/GET` operations) - both passed.
> * **Zstd**: `make check` (complete compression/decompression test suite) - all test cases passed.
> * **PostgreSQL**: - Basic functionality tests (server startup, SQL operations) - passed.
These are some great numbers.
And great that the smoke tests pass. Did you check whether the code was actually executed? I want to avoid that we have missed some corner cases.
https://github.com/llvm/llvm-project/pull/154929
More information about the llvm-commits
mailing list