[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-9427-gc2be6fc
abarth at webkit.org
abarth at webkit.org
Wed Dec 22 11:26:23 UTC 2010
The following commit has been merged in the debian/experimental branch:
commit cd493c4845fe269900b8397e1001198323efaf8a
Author: abarth at webkit.org <abarth at webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Fri Jul 23 14:19:18 2010 +0000
2010-07-23 Adam Barth <abarth at webkit.org>
Reviewed by Eric Seidel.
Inline HTMLTokenizer::InputStreamPreprocessor::peek
https://bugs.webkit.org/show_bug.cgi?id=42880
According to sampling HTMLTokenizer::InputStreamPreprocessor::peek is
the hotest function in the tokenizer. Inlining this function is about
a 2.2% speedup on the parser benchmark. Turns out the tokenizer is
much hotter than the tree builder. :)
* html/HTMLTokenizer.h:
(WebCore::HTMLTokenizer::InputStreamPreprocessor::peek):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@63970 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 3bd6213..f69666c 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,18 @@
+2010-07-23 Adam Barth <abarth at webkit.org>
+
+ Reviewed by Eric Seidel.
+
+ Inline HTMLTokenizer::InputStreamPreprocessor::peek
+ https://bugs.webkit.org/show_bug.cgi?id=42880
+
+ According to sampling HTMLTokenizer::InputStreamPreprocessor::peek is
+ the hotest function in the tokenizer. Inlining this function is about
+ a 2.2% speedup on the parser benchmark. Turns out the tokenizer is
+ much hotter than the tree builder. :)
+
+ * html/HTMLTokenizer.h:
+ (WebCore::HTMLTokenizer::InputStreamPreprocessor::peek):
+
2010-07-23 Andras Becsi <abecsi at webkit.org>
Unreviewed build fix.
diff --git a/WebCore/html/HTMLTokenizer.h b/WebCore/html/HTMLTokenizer.h
index 96c2d7c..6778af4 100644
--- a/WebCore/html/HTMLTokenizer.h
+++ b/WebCore/html/HTMLTokenizer.h
@@ -149,7 +149,7 @@ private:
// Returns whether we succeeded in peeking at the next character.
// The only way we can fail to peek is if there are no more
// characters in |source| (after collapsing \r\n, etc).
- bool peek(SegmentedString& source, int& lineNumber)
+ ALWAYS_INLINE bool peek(SegmentedString& source, int& lineNumber)
{
m_nextInputCharacter = *source;
if (m_nextInputCharacter == '\n' && m_skipNextNewLine) {
--
WebKit Debian packaging
More information about the Pkg-webkit-commits
mailing list