Commit f10b118
committed
Force SvPV_helper to be always inline
(when the compiler cooperates, that is)
I compiled toke.s using -O0 and looked at the output. It had a full
copy of SvPV_helper in it. But this function is called with various
constants that make much of the code dead when constant folded. That
was not being done here.
This commit changes to force inlining. Looking at a revised toke.s, I
saw that SvPV_helper was inserted multiple times in the code, and that
each occurrence had significantly fewer instructions than the original.
I infer that the compiler did the constant folding and pruned out the
dead code.1 parent 1359503 commit f10b118
2 files changed
+4
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3358 | 3358 | | |
3359 | 3359 | | |
3360 | 3360 | | |
3361 | | - | |
| 3361 | + | |
3362 | 3362 | | |
3363 | 3363 | | |
3364 | 3364 | | |
| |||
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
0 commit comments