diff options
| author | Peter Collingbourne <peter@pcc.me.uk> | 2018-04-23 19:09:34 +0000 |
|---|---|---|
| committer | Peter Collingbourne <peter@pcc.me.uk> | 2018-04-23 19:09:34 +0000 |
| commit | 5ab4a4793efbe8805c3fedc8f6625ebad987ce7b (patch) | |
| tree | f160a8e343181015aee8ce2b4d54dc4e1dee4794 /llvm/lib/Transforms/Utils/InlineFunction.cpp | |
| parent | cc45e923c5c1be906ad9b56b2665b954f78c4a49 (diff) | |
| download | bcm5719-llvm-5ab4a4793efbe8805c3fedc8f6625ebad987ce7b.tar.gz bcm5719-llvm-5ab4a4793efbe8805c3fedc8f6625ebad987ce7b.zip | |
Reland r329956, "AArch64: Introduce a DAG combine for folding offsets into addresses.", with a fix for the bot failure.
This reland includes a check to prevent the DAG combiner from folding an
offset that is smaller than the existing one. This can cause oscillations
between two possible DAGs, which was the cause of the hang and later assertion
failure observed on the lnt-ctmark-aarch64-O3-flto bot.
http://green.lab.llvm.org/green/job/lnt-ctmark-aarch64-O3-flto/2024/
Original commit message:
> This is a code size win in code that takes offseted addresses
> frequently, such as C++ constructors that typically need to compute
> an offseted address of a vtable. This reduces the size of Chromium
> for Android's .text section by 108KB.
Differential Revision: https://reviews.llvm.org/D45199
llvm-svn: 330630
Diffstat (limited to 'llvm/lib/Transforms/Utils/InlineFunction.cpp')
0 files changed, 0 insertions, 0 deletions
