This is an archive of the discontinued LLVM Phabricator instance.

Document the -analyze opt flag in the LLVM testing guide (NFC)
ClosedPublic

Authored by mehdi_amini on Jul 14 2020, 2:44 PM.

Details

Summary

This came up in a recent review, @mtrofin was wondering were was
this all documented and I couldn't find a reference to provide.

Diff Detail

Event Timeline

mehdi_amini created this revision.Jul 14 2020, 2:44 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 14 2020, 2:44 PM

(minor rewording)

I'm trying to deprecate -analyze, given it doesn't really apply to NPM and there are better alternatives.
http://lists.llvm.org/pipermail/llvm-dev/2020-July/143372.html

One example of it making things harder is https://reviews.llvm.org/D83798.

Harbormaster failed remote builds in B64231: Diff 277982!

Refer to the new PM convention instead of -analyze

aeubanks accepted this revision.Jul 14 2020, 3:59 PM

lgtm with some minor wording nits

llvm/docs/TestingGuide.rst
50

reserved to test -> reserved to tests?

71

grammar nit: "An analysis is a pass that infers..." (or some other way)

This revision is now accepted and ready to land.Jul 14 2020, 3:59 PM

Adjust wordings as suggested

mehdi_amini marked 2 inline comments as done.

(minor wording change)

lattner accepted this revision.Jul 15 2020, 1:24 PM
This revision was automatically updated to reflect the committed changes.