diff options
| author | Sanjay Patel <spatel@rotateright.com> | 2017-09-18 20:54:26 +0000 |
|---|---|---|
| committer | Sanjay Patel <spatel@rotateright.com> | 2017-09-18 20:54:26 +0000 |
| commit | 7765c93be2f3c0c8938ca17e80cd91b9a0c6daf1 (patch) | |
| tree | 96943fa6a79bc4f44fe60f929a8dbcb2e503cd4a /llvm/tools/llvm-cov/SourceCoverageViewText.cpp | |
| parent | 39cdb84560177ceb583f07f8e29d3883639fa86c (diff) | |
| download | llvm-7765c93be2f3c0c8938ca17e80cd91b9a0c6daf1.zip llvm-7765c93be2f3c0c8938ca17e80cd91b9a0c6daf1.tar.gz llvm-7765c93be2f3c0c8938ca17e80cd91b9a0c6daf1.tar.bz2 | |
[DAG, x86] allow store merging before and after legalization (PR34217)
rL310710 allowed store merging to occur after legalization to catch stores that are created late,
but this exposes a logic hole seen in PR34217:
https://bugs.llvm.org/show_bug.cgi?id=34217
We will miss merging stores if the target lowers vector extracts into target-specific operations.
This patch allows store merging to occur both before and after legalization if the target chooses
to get maximum merging.
I don't think the potential regressions in the other tests are relevant. The tests are for
correctness of weird IR constructs rather than perf tests, and I think those are still correct.
Differential Revision: https://reviews.llvm.org/D37987
llvm-svn: 313564
Diffstat (limited to 'llvm/tools/llvm-cov/SourceCoverageViewText.cpp')
0 files changed, 0 insertions, 0 deletions