This is an archive of the discontinued LLVM Phabricator instance.

[LangRef] Mention vector support for bitreverse/bswap intrinsics (PR38012)
ClosedPublic

Authored by RKSimon on Jan 27 2019, 1:40 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

RKSimon created this revision.Jan 27 2019, 1:40 PM
spatel accepted this revision.Jan 28 2019, 8:08 AM

LGTM - see inline for some wording/grammar nits.

docs/LangRef.rst
12249 ↗(On Diff #183773)

integers values -> integer values

12257 ↗(On Diff #183773)

perform -> operate?

12258 ↗(On Diff #183773)

Run-on sentence: change comma after 'basis' to period or semi-colon.

12279–12280 ↗(On Diff #183773)

...family of intrinsics is used to byte swap an integer value or vector of integer values...

12282 ↗(On Diff #183773)

...operations on data that are not...

Or just remove that sentence? Doesn't add much value IMO.

12295 ↗(On Diff #183773)

Run-on sentence: change comma after 'basis' to period or semi-colon.

This revision is now accepted and ready to land.Jan 28 2019, 8:08 AM
This revision was automatically updated to reflect the committed changes.