This is an archive of the discontinued LLVM Phabricator instance.

[libc] Add a README to the sub-directories under the utils directory.
ClosedPublic

Authored by sivachandra on Feb 12 2020, 1:14 PM.

Details

Summary

Also, the source layout document has been updated to reflect the current
layout of the utils directory.

Event Timeline

sivachandra created this revision.Feb 12 2020, 1:14 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 12 2020, 1:14 PM
PaulkaToast added inline comments.Feb 21 2020, 6:37 PM
libc/utils/CPP/README.md
1–6

"This directory contains re-implementations of some C++ standard library utilities" or "This directory contains a re-implementation of some of the C++ standard library"

libc/utils/UnitTest/README.md
23

"extend" instead of "extended"

Address comments.

sivachandra marked 2 inline comments as done.Feb 21 2020, 10:56 PM
PaulkaToast added inline comments.Feb 22 2020, 2:16 PM
libc/utils/CPP/README.md
1–6

I think you missed this one?

Add a missed word.

sivachandra marked 2 inline comments as done.Feb 22 2020, 10:58 PM
sivachandra added inline comments.
libc/utils/CPP/README.md
1–6

Ah, you wanted an additional "utilities".

PaulkaToast accepted this revision.Feb 23 2020, 12:28 AM
This revision is now accepted and ready to land.Feb 23 2020, 12:28 AM
MaskRay added inline comments.Feb 23 2020, 9:08 AM
libc/utils/HdrGen/README.md
7

Delete the trailing empty line.

sivachandra marked 2 inline comments as done.Feb 23 2020, 10:12 PM
This revision was automatically updated to reflect the committed changes.