This is an archive of the discontinued LLVM Phabricator instance.

[LLVM][Alignment] Update documentation
ClosedPublic

Authored by gchatelet on Aug 1 2019, 12:28 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

gchatelet created this revision.Aug 1 2019, 12:28 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 1 2019, 12:28 AM
aprantl accepted this revision.Aug 1 2019, 10:13 AM

LGTM with one fix inline.

llvm/include/llvm/Support/Alignment.h
328 ↗(On Diff #212744)

this one is too much :-)

This revision is now accepted and ready to land.Aug 1 2019, 10:13 AM
gchatelet marked an inline comment as done.Aug 2 2019, 12:12 AM

Thx for the review, I'll proceed with the push.

This revision was automatically updated to reflect the committed changes.