This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU/GlobalISel: Add documentation for RegisterBankInfo
ClosedPublic

Authored by arsenm on Jan 13 2020, 8:00 AM.

Details

Summary

Document some high level strategies that should be used for register
bank selection. The constant bus restriction section hasn't actually
been implemented yet.

Diff Detail

Event Timeline

arsenm created this revision.Jan 13 2020, 8:00 AM

A nice summary!

llvm/lib/Target/AMDGPU/AMDGPURegisterBankInfo.cpp
22

operand -> operands

34

Fix: "disambiguate determine"

arsenm updated this revision to Diff 238918.Jan 17 2020, 4:00 PM
arsenm marked 2 inline comments as done.

Fix typos

kerbowa accepted this revision.Jan 20 2020, 7:50 AM

LGTM

This revision is now accepted and ready to land.Jan 20 2020, 7:50 AM