This is an archive of the discontinued LLVM Phabricator instance.

[LLDB][Docs] Convert links.md & lldb-for-gdb-users.txt file to respective .rst files
ClosedPublic

Authored by xgupta on Aug 27 2021, 12:47 AM.

Details

Summary

Update links.md & lldb-for-gdb-users.txt file to respective .rst fles for consistency as most of the documentation is written in reStructuredText format.

Signed-off-by: Shivam Gupta <shivam98.tkg@gmail.com>

Diff Detail

Event Timeline

xgupta created this revision.Aug 27 2021, 12:47 AM
xgupta updated this revision to Diff 369063.Aug 27 2021, 3:43 AM

Convert lldb-for-gdb-users.txt

xgupta updated this revision to Diff 369099.Aug 27 2021, 7:41 AM

move lldb-for-gdb-users.rst to use/ folder

xgupta published this revision for review.Aug 27 2021, 7:46 AM
xgupta retitled this revision from [LLDB][Docs] Convert some .txt files to .rst to [LLDB][Docs] Convert links.md & lldb-for-gdb-users.txt file to respective .rst files.
xgupta edited the summary of this revision. (Show Details)
Herald added a project: Restricted Project. · View Herald TranscriptAug 27 2021, 7:46 AM
mgorny added inline comments.Aug 28 2021, 3:38 AM
lldb/docs/use/lldb-for-gdb-users.rst
42

This style looks a bit weird. Any reason not to use indentation instead of >?

xgupta updated this revision to Diff 369251.Aug 28 2021, 4:17 AM

remove extra ">"

xgupta added inline comments.Aug 28 2021, 4:20 AM
lldb/docs/use/lldb-for-gdb-users.rst
42

Yes, I mistakenly followed https://lldb.llvm.org/resources/build.html style, corrected it now.

mgorny accepted this revision.Aug 28 2021, 4:23 AM

Presuming you've verified that it renders correctly, LGTM.

This revision is now accepted and ready to land.Aug 28 2021, 4:23 AM
xgupta updated this revision to Diff 369253.Aug 28 2021, 4:43 AM

add lldb-for-gdb-users entry to index.rst

I can confirm that it renders correctly, please commit it at your convenience. Thanks.

This revision was automatically updated to reflect the committed changes.