diff options
| author | Kostya Kortchinsky <kostyak@google.com> | 2017-12-06 16:53:24 +0000 |
|---|---|---|
| committer | Kostya Kortchinsky <kostyak@google.com> | 2017-12-06 16:53:24 +0000 |
| commit | ddf4ef395973d79098df10b2dfe282476b84e60f (patch) | |
| tree | 293e7b8fa12a16782957fef13ae0195480d61213 /clang/unittests/Lex/LexerTest.cpp | |
| parent | 9e5e51aeed06226552f2d89624a2570bdc4e80ad (diff) | |
| download | llvm-ddf4ef395973d79098df10b2dfe282476b84e60f.zip llvm-ddf4ef395973d79098df10b2dfe282476b84e60f.tar.gz llvm-ddf4ef395973d79098df10b2dfe282476b84e60f.tar.bz2 | |
[scudo] Correct performance regression in Secondary
Summary:
This wasn't noticed: `RoundUpTo` doesn't produce a constant expression, so the
sizes were not constant either. Enforce them to be static const, replace
`RoundUpTo` by its expression. The compiler can now optimize the associated
computations accordingly.
Also looking at the produced assembly, `PageSize` was fetched multiple times
during `Allocate`, so keep a local value of it. As a result it's fetched once
and kept in a register.
Reviewers: alekseyshl, flowerhack
Reviewed By: alekseyshl
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D40862
llvm-svn: 319903
Diffstat (limited to 'clang/unittests/Lex/LexerTest.cpp')
0 files changed, 0 insertions, 0 deletions