diff options
author | Argyrios Kyrtzidis <akyrtzi@gmail.com> | 2011-08-23 21:02:41 +0000 |
---|---|---|
committer | Argyrios Kyrtzidis <akyrtzi@gmail.com> | 2011-08-23 21:02:41 +0000 |
commit | 2797df6a24e9065b37022f922da813a133cfaac2 (patch) | |
tree | 73b27677989416e40aa65651831409f78d27668b /lldb/source/Expression/ClangExpressionParser.cpp | |
parent | e7f7516148e5e295af7bfb3aa64da4aaf6137687 (diff) | |
download | bcm5719-llvm-2797df6a24e9065b37022f922da813a133cfaac2.tar.gz bcm5719-llvm-2797df6a24e9065b37022f922da813a133cfaac2.zip |
Amend r138129 (reduction of SLocEntries) which introduced performance regression due
to increased calls to SourceManager::getFileID. (rdar://9992664)
Use a slightly different approach that is more efficient both in terms of speed
(no extra getFileID calls) and in SLocEntries reduction.
Comparing pre-r138129 and this patch we get:
For compiling SemaExpr.cpp reduction of SLocEntries by 26%.
For the boost enum library:
-SLocEntries -34% (note that this was -5% for r138129)
-Memory consumption -50%
-PCH size -31%
Reduced SLocEntries also benefit the hot function SourceManager::getFileID,
evident by the reduced "FileID scans".
llvm-svn: 138380
Diffstat (limited to 'lldb/source/Expression/ClangExpressionParser.cpp')
0 files changed, 0 insertions, 0 deletions