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.

Tags

Subscribers

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

Repository: rL LLVM

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

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

Path

Size

llvm/

trunk/

lib/

CodeGen/

SelectionDAG/

DAGCombiner.cpp

31 lines

test/

CodeGen/

AArch64/

ldst-paired-aliasing.ll

6 lines

X86/

pr40631_deadstore_elision.ll

1 line

Diff 187663

llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp

Loading...

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

Loading...

llvm/trunk/test/CodeGen/X86/pr40631_deadstore_elision.ll

Loading...