[PATCH] D134270: [clang] [Driver] Support multiple configuration files
Serge Pavlov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Sep 20 22:37:03 PDT 2022
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
More information about the cfe-commits
mailing list