When @xonixx was adding code coverage, I asked if he could make the coverage AST output COVER["1"] instead of COVER[1] to give a slight optimization. However, it's better to do that as an optimization that all users can benefit from instead.
The speedup for the ArrayOperations benchmark is about 11%, so I think it's worth it:
When @xonixx was adding code coverage, I asked if he could make the coverage AST output COVER["1"] instead of COVER[1] to give a slight optimization. However, it's better to do that as an optimization that all users can benefit from instead.
The speedup for the ArrayOperations benchmark is about 11%, so I think it's worth it:
ArrayOperations-16 312ns ± 3% 276ns ± 3% -11.54% (p=0.008 n=5+5)
This commit adds the compiler optimization, but also makes the cover.go output __COVER[1] now that these expressions are optimized.
Fixes #157