Fix native validation - #9020
Conversation
Execution-Time Benchmarks Report ⏱️Execution-time results for samples comparing This PR (9020) and master.
|
|||||||||||||||||||||||||
| Metric | Master (Mean ± 95% CI) | Current (Mean ± 95% CI) | Change | Status |
|---|---|---|---|---|
| .NET Framework 4.8 - Baseline | ||||
| duration | 201.62 ± (201.56 - 202.46) ms | 214.33 ± (213.86 - 214.78) ms | +6.3% | ❌⬆️ |
| .NET Framework 4.8 - Bailout | ||||
| duration | 206.01 ± (205.80 - 206.57) ms | 217.49 ± (217.29 - 218.24) ms | +5.6% | ❌⬆️ |
BenchmarksBenchmark execution time: 2026-08-10 17:12:12 Comparing candidate commit 6fc3055 in PR branch Found 0 performance improvements and 1 performance regressions! Performance is the same for 71 metrics, 0 unstable metrics, 68 known flaky benchmarks, 58 flaky benchmarks without significant changes.
|
|
@codex review |
|
Codex Review: Didn't find any major issues. Keep them coming! Reviewed commit: ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback". |
Summary of changes
Reduce native-symbol validation log noise and update the Alpine x64 symbol snapshot.
Reason for change
nmoutput flooded CI logs, and current Azure and GitLab Alpine amd64 builds no longer import theceilsymbol.For example, here:
Implementation details
nmoutput for validation without streaming the full symbol table.ceilfrom the Alpine x64 verified snapshot.Test coverage
ceilremoval in amd64 builds.Other details