This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Split AMDGPUTTI into GCNTTI and R600TTI
ClosedPublic

Authored by tstellar on May 24 2018, 10:37 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

tstellar created this revision.May 24 2018, 10:37 PM
arsenm added inline comments.May 25 2018, 12:17 AM
lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
572–575 ↗(On Diff #148544)

Why not implement separately for GCNTargetMachine and R600TargetMachine?

tstellar updated this revision to Diff 148699.May 25 2018, 6:50 PM

Move getTargetTransforInfo to TargetMachine sub-classes.

tstellar marked an inline comment as done.May 25 2018, 6:51 PM
arsenm accepted this revision.May 25 2018, 10:23 PM

LGTM

This revision is now accepted and ready to land.May 25 2018, 10:23 PM
This revision was automatically updated to reflect the committed changes.