This is an archive of the discontinued LLVM Phabricator instance.

[docs] [clang] Small documentation change for compilation databases
ClosedPublic

Authored by croepha on Jan 9 2022, 12:56 AM.

Details

Summary

We have an page dedicated to compliation databases including
various ways to generate them, but we don't mention that clang
has a built in method to do this. This addresses that.

Diff Detail

Event Timeline

croepha requested review of this revision.Jan 9 2022, 12:56 AM
croepha created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptJan 9 2022, 12:56 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
joerg accepted this revision.Jan 9 2022, 1:30 PM
joerg added inline comments.
clang/docs/JSONCompilationDatabase.rst
33

I think you mean "concatenate", but otherwise LGTM?

This revision is now accepted and ready to land.Jan 9 2022, 1:30 PM
croepha updated this revision to Diff 398468.Jan 9 2022, 4:05 PM

Fixed spelling

croepha marked an inline comment as done.Jan 9 2022, 4:06 PM
croepha added inline comments.
clang/docs/JSONCompilationDatabase.rst
33

Yes, thank you

croepha marked an inline comment as done.Jan 11 2022, 9:32 AM

Hey, @joerg I don't have commit access, would you mind landing this for me?