diff options
| author | Roman Lebedev <lebedev.ri@gmail.com> | 2019-05-28 17:53:43 +0000 |
|---|---|---|
| committer | Roman Lebedev <lebedev.ri@gmail.com> | 2019-05-28 17:53:43 +0000 |
| commit | 19f51ec04ac1edd0b278e23b8b5cb3ff49c2dd08 (patch) | |
| tree | 59e9eae5d753f713b063f1e3bea7f92eec169c20 /llvm/lib/CodeGen/SafeStackColoring.h | |
| parent | f612b18720aa6da2843c562a266d0850039418dd (diff) | |
| download | bcm5719-llvm-19f51ec04ac1edd0b278e23b8b5cb3ff49c2dd08.tar.gz bcm5719-llvm-19f51ec04ac1edd0b278e23b8b5cb3ff49c2dd08.zip | |
[DAGCombiner][X86][AArch64][AMDGPU] (x + C) - y -> (x - y) + C fold
Summary:
The main motivation is shown by all these `neg` instructions that are now created.
In particular, the `@reg32_lshr_by_negated_unfolded_sub_b` test.
AArch64 test changes all look good (`neg` created), or neutral.
X86 changes look neutral (vectors), or good (`neg` / `xor eax, eax` created).
I'm not sure about `X86/ragreedy-hoist-spill.ll`, it looks like the spill
is now hoisted into preheader (which should still be good?),
2 4-byte reloads become 1 8-byte reload, and are elsewhere,
but i'm not sure how that affects that loop.
I'm unable to interpret AMDGPU change, looks neutral-ish?
This is hopefully a step towards solving [[ https://bugs.llvm.org/show_bug.cgi?id=41952 | PR41952 ]].
https://rise4fun.com/Alive/pkdq (we are missing more patterns, i'll submit them later)
Reviewers: craig.topper, RKSimon, spatel, arsenm
Reviewed By: RKSimon
Subscribers: bjope, qcolombet, kzhuravl, jvesely, wdng, nhaehnle, yaxunl, javed.absar, dstuttard, tpr, t-tye, kristof.beyls, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D62223
llvm-svn: 361852
Diffstat (limited to 'llvm/lib/CodeGen/SafeStackColoring.h')
0 files changed, 0 insertions, 0 deletions

