This is an archive of the discontinued LLVM Phabricator instance.

[docs] Added llvm/cmake section
ClosedPublic

Authored by pooja2299 on May 5 2021, 10:48 AM.

Details

Summary

Added information about the cmake inside llvm.

Diff Detail

Event Timeline

pooja2299 created this revision.May 5 2021, 10:48 AM
pooja2299 requested review of this revision.May 5 2021, 10:48 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 5 2021, 10:48 AM
xgupta accepted this revision.May 5 2021, 8:23 PM

After addressing these changes it seems good to me.

llvm/docs/GettingStarted.rst
824

I think it is in more detail than needed on the getting started page. You may remove OCaml language, Sphinx, TensorFlow compiler, atomic builtins, version control information.

This revision is now accepted and ready to land.May 5 2021, 8:23 PM
pooja2299 updated this revision to Diff 343289.May 5 2021, 10:56 PM

Removed redundant lines.

If @mstorsjo and @jroelofs have no issue I will commit it tomorrow.

jroelofs accepted this revision.May 12 2021, 9:50 AM

@pooja2299 I think you have made a few good patches, If you want to continue contributing, you may request commit access by following the steps mention here: https://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access.

This revision was automatically updated to reflect the committed changes.