This is an archive of the discontinued LLVM Phabricator instance.

[analyzer] Fixes to the checker developer manual.
ClosedPublic

Authored by NoQ on Jul 27 2016, 10:05 AM.

Details

Summary
  1. Fix the explanation of how to run tests after migration from autotools to cmake.
  1. Expand the "debugging" section with more interesting stuff, as accidentally discussed in D22622 and D22856.

Any other things to fix there?

Also, does anybody think it's a good idea to mention my unofficial guide (https://github.com/haoNoQ/clang-analyzer-guide/releases) in the "additional sources" section?

Diff Detail

Repository
rL LLVM

Event Timeline

NoQ updated this revision to Diff 65762.Jul 27 2016, 10:05 AM
NoQ retitled this revision from to [analyzer] Fixes to the checker developer manual..
NoQ updated this object.
NoQ added a reviewer: zaks.anna.
zaks.anna accepted this revision.Jul 27 2016, 3:19 PM
zaks.anna edited edge metadata.

A couple of nits. Otherwise, LGTM. Thanks!

www/analyzer/checker_dev_manual.html
534 ↗(On Diff #65762)

I suggest to just tell about -### and drop the explanation about attaching to the child process. Sounds good?

645 ↗(On Diff #65762)

I'd move this up after the description of "-analyze-function=foo".

This revision is now accepted and ready to land.Jul 27 2016, 3:19 PM
This revision was automatically updated to reflect the committed changes.