This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Implement getMemoryOpCost
AbandonedPublic

Authored by arsenm on Dec 18 2015, 12:20 PM.

Details

Reviewers
tstellarAMD

Diff Detail

Event Timeline

arsenm updated this revision to Diff 43256.Dec 18 2015, 12:20 PM
arsenm retitled this revision from to AMDGPU: Implement getMemoryOpCost.
arsenm updated this object.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.

Out of curiosity, what is the meaning underlying these cost numbers? Something like cycles?

Out of curiosity, what is the meaning underlying these cost numbers? Something like cycles?

See the comment on TargetCostConstants in TargetTransformInfo.h

tstellarAMD accepted this revision.Dec 23 2015, 8:28 PM
tstellarAMD edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Dec 23 2015, 8:28 PM

Looks like patch was not committed.

arsenm abandoned this revision.Apr 4 2020, 7:26 PM