diff options
author | Peter Collingbourne <peter@pcc.me.uk> | 2018-07-20 21:43:20 +0000 |
---|---|---|
committer | Peter Collingbourne <peter@pcc.me.uk> | 2018-07-20 21:43:20 +0000 |
commit | 9e45fcd250e8f1acddba63f2d2bd90a353ad7d1c (patch) | |
tree | ade4f189a842ac3ebe17b74b58eef5c90894a2d7 | |
parent | c3554027940f5d19133a3a335e5ef667d3a251e2 (diff) |
Change the cap on the amount of padding for each vtable to 32-byte (previously it was 128-byte)
We tested different cap values with a recent commit of Chromium. Our results show that the 32-byte cap yields the smallest binary and all the caps yield similar performance.
Based on the results, we propose to change the cap value to 32-byte.
Patch by Zhaomo Yang!
Differential Revision: https://reviews.llvm.org/D49405
git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@337622 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | test/cfi/create-derivers.test | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/test/cfi/create-derivers.test b/test/cfi/create-derivers.test index 8b569d001..b651d9be6 100644 --- a/test/cfi/create-derivers.test +++ b/test/cfi/create-derivers.test @@ -7,15 +7,15 @@ B0: {{1B|B@@}}: {{.*}} size 1 RUN: %clangxx_cfi -DB32 -flto -c -o %t2.o %S/simple-fail.cpp RUN: opt -lowertypetests -debug-only=lowertypetests -o /dev/null %t2.o 2>&1 | FileCheck --check-prefix=B32 %s -B32: {{1B|B@@}}: {{.*}} size 24 +B32: {{1B|B@@}}: {{.*}} size 2{{3|4}} B32-NOT: all-ones RUN: %clangxx_cfi -DB64 -flto -c -o %t3.o %S/simple-fail.cpp RUN: opt -lowertypetests -debug-only=lowertypetests -o /dev/null %t3.o 2>&1 | FileCheck --check-prefix=B64 %s -B64: {{1B|B@@}}: {{.*}} size 54 +B64: {{1B|B@@}}: {{.*}} size 5{{3|4}} B64-NOT: all-ones RUN: %clangxx_cfi -DBM -flto -c -o %t4.o %S/simple-fail.cpp RUN: opt -lowertypetests -debug-only=lowertypetests -o /dev/null %t4.o 2>&1 | FileCheck --check-prefix=BM %s -BM: {{1B|B@@}}: {{.*}} size 84 +BM: {{1B|B@@}}: {{.*}} size 8{{3|4}} BM-NOT: all-ones |