[PATCH] D24926: Added support of configuration files

Serge Pavlov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 1 19:02:00 PST 2017


sepavloff updated this revision to Diff 86760.
sepavloff added a comment.

Updated patch

Simplified the patch:

- removed suport of reading config file name from environment variable.

It is more convenient to read options from such variable rather than the
name of config file. Config file still can be specified by environment
variable using option `--config`.

- API function do not return error code anymore, they set string argument

to the error text.

Also the function `readConfigFile` now returns boolean value to indicate
if config file was successfully read.


https://reviews.llvm.org/D24926

Files:
  docs/CommandLine.rst
  include/llvm/Support/CommandLine.h
  lib/Support/CommandLine.cpp
  unittests/Support/CommandLineTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D24926.86760.patch
Type: text/x-patch
Size: 29605 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170202/58b812e9/attachment.bin>


More information about the llvm-commits mailing list