Page MenuHomePhabricator

docs: Emphasize ArrayRef over SmallVectorImpl
AcceptedPublic

Authored by dexonsmith on Jul 26 2018, 3:29 PM.

Details

Reviewers
bogner
Summary

The section on SmallVector has a note about preferring SmallVectorImpl for APIs but doesn't mention ArrayRef. Although ArrayRef is discussed elsewhere, let's re-emphasize here.

Diff Detail

Event Timeline

dexonsmith created this revision.Jul 26 2018, 3:29 PM

Drive-by-review here.

llvm/docs/ProgrammersManual.rst
1476

s/basically just//

1493–1494

I suggest "discouraged" instead of "bad".

1504–1505

s/is so widely/is widely/
s/really isn't/is no longer/
s/"private to the implementation" anymore/an implementation detail/

bogner accepted this revision.Sep 25 2018, 11:12 AM

LGTM once dberris's suggestions are taken into account

This revision is now accepted and ready to land.Sep 25 2018, 11:12 AM