diff options
| author | Florian Hahn <flo@fhahn.com> | 2020-06-30 10:39:23 +0100 |
|---|---|---|
| committer | Florian Hahn <flo@fhahn.com> | 2020-06-30 20:34:04 +0100 |
| commit | 1ccc49924aeb7bb5fbd2eb956243d16db3a02b08 (patch) | |
| tree | 7ad7535727f1e7860d359826ac9d769bba25314c /clang/lib/Frontend/CompilerInvocation.cpp | |
| parent | 4ab43980450baf3c49bebbc526c6c96c3ed9f06e (diff) | |
| download | llvm-1ccc49924aeb7bb5fbd2eb956243d16db3a02b08.zip llvm-1ccc49924aeb7bb5fbd2eb956243d16db3a02b08.tar.gz llvm-1ccc49924aeb7bb5fbd2eb956243d16db3a02b08.tar.bz2 | |
[AArch64] Add getCFInstrCost, treat branches as free for throughput.
D79164/2596da31740f changed getCFInstrCost to return 1 per default.
AArch64 did not have its own implementation, hence the throughput cost
of CFI instructions is overestimated. On most cores, most branches should
be predicated and essentially free throughput wise.
This restores a 9% performance regression on a SPEC2006 benchmark on
AArch64 with -O3 LTO & PGO.
This patch effectively restores pre 2596da31740f behavior for AArch64
and undoes the AArch64 test changes of the patch.
Reviewers: samparker, dmgreen, anemet
Reviewed By: samparker
Differential Revision: https://reviews.llvm.org/D82755
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions