[PATCH] D134270: [clang] [Driver] Support multiple configuration files

[Serge Pavlov via Phabricator via cfe-commits]

sepavloff added a comment.

What about the case, when executable file has a target prefix (so implicit config file may be used) and the option `--config` is specified in command line? Now this case becomes possible and we should either reject it (because explicit config files have precedence) or support it (because configuration is built up of a few files). I would be inclined to the first variant but there could be use cases for the other. In any case it worth to be documented.



================
Comment at: clang/lib/Driver/Driver.cpp:990
 
 bool Driver::loadConfigFile() {
   // Process options that change search path for config files.
----------------
Maybe rename it to `loadConfigFiles`?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134270/new/

https://reviews.llvm.org/D134270