This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU/SI: Enable MemorySSA for EarlyCSE
Needs ReviewPublic

Authored by tstellarAMD on Nov 24 2016, 9:06 AM.

Details

Reviewers
arsenm
tstellar
Summary

With MemorySSA this pass does a much better job of CSE'ing intrinsic
with pointer arguments.

Event Timeline

tstellarAMD retitled this revision from to AMDGPU/SI: Enable MemorySSA for EarlyCSE.
tstellarAMD updated this object.
tstellarAMD added a reviewer: arsenm.
tstellarAMD added a subscriber: llvm-commits.
arsenm edited edge metadata.Nov 29 2016, 11:20 AM

Can you add a testcase where this changes something? I assume this works with the GVN path?

t-tye added a subscriber: t-tye.Mar 22 2017, 6:38 PM
tony-tye removed a subscriber: tony-tye.Mar 22 2017, 6:50 PM