This is an archive of the discontinued LLVM Phabricator instance.

Updated -help message in clang-check.
ClosedPublic

Authored by alexfh on Jul 11 2012, 10:41 AM.

Details

Summary

Provide more information on usage in -help

Diff Detail

Event Timeline

klimek added inline comments.Jul 11 2012, 11:24 AM
clang-check/ClangCheck.cpp
25

I think especially with the upcoming patch to allow other build system integration, I'd rather not call this cmake-output-dir. I think build-path is better, and having an explanation that mentions CMake as an example.

alexfh updated this revision to Unknown Object (????).Jul 12 2012, 4:11 AM

Addressed comments.

alexfh added inline comments.Jul 12 2012, 4:13 AM
clang-check/ClangCheck.cpp
25

makes sense.
what about "If the path is\n"

"relative, the current working directory needs to be in the CMake source\n"	
"tree and the file must be in a subdirectory of the current working\n"	
"directory." part? can we make this eat any form of paths?
alexfh updated this revision to Unknown Object (????).Jul 12 2012, 7:11 AM

Minor wording change

klimek accepted this revision.Jul 12 2012, 7:15 AM
alexfh updated this revision to Unknown Object (????).Jul 12 2012, 7:34 AM

Updated, resolved conflicts

alexfh closed this revision.Jul 12 2012, 7:34 AM