build: pin -std=c11 for instrument-hooks core.c#84
Conversation
The Zig-generated dist/core.c is rejected under C23 (GCC 15's default), so pin C11 via cflags_c (C-only, so the C++ sources are unaffected).
Merging this PR will improve performance by ×2.6
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Memory | long body |
960 B | 6,296 B | -84.75% |
| ❌ | Memory | wait 1ms |
7 B | 16 B | -56.25% |
| ⚡ | Memory | short body 3 |
25,960 B | 11 B | ×2,400 |
| ⚡ | Memory | one |
45.9 KB | 14.3 KB | ×3.2 |
| ⚡ | Memory | wait 500ms |
64.1 KB | 23 KB | ×2.8 |
| ⚡ | WallTime | switch 2 |
336 ns | 300 ns | +12% |
| ⚡ | WallTime | short body |
2.7 µs | 2.4 µs | +11.62% |
| ⚡ | WallTime | short body |
1.6 µs | 1.5 µs | +10.66% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing cod-2953-use-stdc11-for-instrument-hooks-in-integrations (0b9689b) with main (bcd7e64)
Footnotes
-
1 benchmark was skipped, so the baseline result was used instead. If it was deleted from the codebase, click here and archive it to remove it from the performance reports. ↩
No description provided.