forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit28d750c
committed
Reorder FmgrBuiltin members, saving 25% in size.
That's worth it, as fmgr_builtins is frequently accessed, and asfmgr_builtins is one of the biggest constant variables in a backend.On most 64bit systems this will change the size of the struct from32byte to 24bytes. While that could make indexing into the arraymarginally more expensive, the higher cache hit ratio is worth more,especially because these days fmgr_builtins isn't searched with abinary search anymore (c.f.212e6f3).Discussion:https://postgr.es/m/20181016201145.aa2dfeq54rhqzron@alap3.anarazel.de1 parent68fc227 commit28d750c
2 files changed
+2
-2
lines changedLines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
230 | 230 |
| |
231 | 231 |
| |
232 | 232 |
| |
233 |
| - | |
| 233 | + | |
234 | 234 |
| |
235 | 235 |
| |
236 | 236 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
25 | 25 |
| |
26 | 26 |
| |
27 | 27 |
| |
28 |
| - | |
29 | 28 |
| |
30 | 29 |
| |
31 | 30 |
| |
| 31 | + | |
32 | 32 |
| |
33 | 33 |
| |
34 | 34 |
| |
|
0 commit comments
Comments
(0)