Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Phabricator shutdown timeline

Differential D58467

[DAGCombine] Generalize Dead Store to overlapping stores.
ClosedPublic
Actions

Authored by niravd on Feb 20 2019, 11:36 AM.

Details

Reviewers
courbet
rnk
Commits
rG48cf37b55cc9: [DAGCombine] Generalize Dead Store to overlapping stores.
rL354518: [DAGCombine] Generalize Dead Store to overlapping stores.
Summary

Remove stores that are immediately overwritten by larger
stores.

Diff Detail

Event Timeline

niravd created this revision.Feb 20 2019, 11:36 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 20 2019, 11:36 AM
Herald added subscribers: hiraditya, javed.absar. · View Herald Transcript
Harbormaster completed remote builds in B28338: Diff 187636.Feb 20 2019, 11:36 AM
niravd added a child revision: D58468: [DAGCombine] Fold overlapping constant stores.Feb 20 2019, 11:43 AM
rnk accepted this revision.Feb 20 2019, 11:59 AM

lgtm, nice

This revision is now accepted and ready to land.Feb 20 2019, 11:59 AM
Closed by commit rL354518: [DAGCombine] Generalize Dead Store to overlapping stores. (authored by niravd). · Explain WhyFeb 20 2019, 1:07 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
llvm/
lib/
CodeGen/
SelectionDAG/
31 lines
test/
CodeGen/
AArch64/
6 lines
X86/
1 line

Diff 187636

View Options

llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp

Loading...
View Options

llvm/test/CodeGen/AArch64/ldst-paired-aliasing.ll

Loading...
View Options

llvm/test/CodeGen/X86/pr40631_deadstore_elision.ll

Loading...