[PATCH] Const arguments
- Passing larger than 4-byte values by reference to avoid copy-on-move and temporary objects. ("PVS studio" can pinpoint/list all those automatically)
- const size_t* const is but a way to point for aliasing and therefore "restrict" optimisations and use. (themself leading to potential openmp "optimisations spot")
- "int getStuff(const size_t arg)" is more developper safeguard and code documentation than real perf. (preventing people using args as local vars and easing code reading, by pointing possible optimisation when code evolves)
- inlines are added and tested using profiling results (with very sleepy profiler).
Change History
(9)
Description: |
modified (diff)
|
Summary: |
Const arguments → [PATCH] Const arguments
|
Milestone: |
Backlog → Alpha 15
|
Milestone: |
Alpha 15 → Alpha 14
|
Keywords: |
patch review added; temporary copies memory removed
|
Priority: |
Should Have → Nice to Have
|
Owner: |
set to tuan kuranes
|
Milestone: |
Alpha 14 → Alpha 15
|
Keywords: |
review removed
|
Milestone: |
Alpha 15 → Backlog
|
Resolution: |
→ wontfix
|
Status: |
new → closed
|
patch