Commit 15cb74a7 authored by jamesr@chromium.org's avatar jamesr@chromium.org

Whitelist QuadF from ChromeClassTester checks

There's a measurable (2.3-3%) perf improvement from inlining QuadF's dtor and
ctor, even though FindBadConstructs thinks it's too large.

BUG=
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/11280279

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170868 0039d316-1c4b-4281-b951-d872f2087c98
parent 4785ac5f
...@@ -200,6 +200,10 @@ void ChromeClassTester::BuildBannedLists() { ...@@ -200,6 +200,10 @@ void ChromeClassTester::BuildBannedLists() {
// Part of our public interface that nacl and friends use. (Arguably, this // Part of our public interface that nacl and friends use. (Arguably, this
// should mean that this is a higher priority but fixing this looks hard.) // should mean that this is a higher priority but fixing this looks hard.)
ignored_record_names_.insert("PluginVersionInfo"); ignored_record_names_.insert("PluginVersionInfo");
// Measured performance improvement on cc_perftests. See
// https://codereview.chromium.org/11299290/
ignored_record_names_.insert("QuadF");
} }
std::string ChromeClassTester::GetNamespaceImpl(const DeclContext* context, std::string ChromeClassTester::GetNamespaceImpl(const DeclContext* context,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment