[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: texi2any 7.0 performance regression (non-XS)
From: |
Gavin Smith |
Subject: |
Re: texi2any 7.0 performance regression (non-XS) |
Date: |
Sun, 12 Feb 2023 12:04:38 +0000 |
On Sun, Feb 12, 2023 at 11:38:43AM +0000, Gavin Smith wrote:
> I'll try to look at this commit more to see if I can find anything else.
It's good news: we can keep the uninlined function. The slowdown almost
goes away completely if we eliminate the use of goto statements, e.g.:
diff --git a/tp/Texinfo/ParserNonXS.pm b/tp/Texinfo/ParserNonXS.pm
index 81ca96b3b6..b4765549ef 100644
--- a/tp/Texinfo/ParserNonXS.pm
+++ b/tp/Texinfo/ParserNonXS.pm
@@ -3921,7 +3921,7 @@ sub _process_remaining_on_line($$$$)
'extra' => {'arg_line' => $line }};
$current = $current->{'contents'}->[-1];
$retval = $GET_A_NEW_LINE;
- goto funexit;
+ return ($current, $line, $source_info, $retval);
} elsif ($line =~ /^(\s*?)\@end\s+([a-zA-Z][\w-]*)/
and ($2 eq $current->{'cmdname'})) {
if ($line =~ s/^(\s+)//) {
Making this change throughout _process_remaining_on_line reduces the
run time for the manual in question from 45.6 s to 26.8 s.
This is with perl 5.30.0.
> I suspect it may just be that Perl subroutine calls are very slow, so it
> is faster to inline everything into one long subroutine that is thousands
> of lines long.
This is true to some extent, but wasn't the main cause of the longer
runtimes on this occasion.
- Re: texi2any is too slow because of Unicode::Collate, (continued)
- Re: texi2any is too slow because of Unicode::Collate, Gavin Smith, 2023/02/11
- Re: texi2any is too slow because of Unicode::Collate, Eli Zaretskii, 2023/02/11
- Re: texi2any is too slow because of Unicode::Collate, Gavin Smith, 2023/02/11
- Re: texi2any is too slow because of Unicode::Collate, pertusus, 2023/02/11
- Re: texi2any is too slow because of Unicode::Collate, Eli Zaretskii, 2023/02/12
- Re: texi2any is too slow because of Unicode::Collate, Gavin Smith, 2023/02/12
- Re: texi2any is too slow because of Unicode::Collate, pertusus, 2023/02/12
- Re: texi2any is too slow because of Unicode::Collate, Gavin Smith, 2023/02/12
- texi2any 7.0 performance regression (non-XS), Gavin Smith, 2023/02/12
- Re: texi2any 7.0 performance regression (non-XS), pertusus, 2023/02/12
- Re: texi2any 7.0 performance regression (non-XS),
Gavin Smith <=
- Re: texi2any 7.0 performance regression (non-XS), pertusus, 2023/02/12
- Obsolete XS overrides are gone, Gavin Smith, 2023/02/27
- Re: texi2any is too slow because of Unicode::Collate, Eli Zaretskii, 2023/02/12
use getSortKey in Unicode::Collate, Gavin Smith, 2023/02/13