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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to