[PATCH] D133325: [Driver] Allow search of included response files as configuration files

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 7 11:23:11 PDT 2022


aaron.ballman added inline comments.


================
Comment at: clang/docs/UsersManual.rst:947-949
+Option `--search-config-dirs` changes the search algorith if the included
+file is specified without a directory separator. In this case the file is looked
+for in the same directories as configuration files.
----------------



================
Comment at: llvm/lib/Support/CommandLine.cpp:1188-1194
+  if (InConfigFile) {
+    if (std::find_if(NewArgv.begin(), NewArgv.end(),
+                     [](const char *Arg) -> bool {
+                       return StringRef("--search-config-dirs") == Arg;
+                     }) != NewArgv.end())
+      setSearchAsConfig(true);
+  }
----------------



================
Comment at: llvm/lib/Support/CommandLine.cpp:1384-1387
+    if (llvm::sys::path::is_relative(FileName)) {
+      if (FS->makeAbsolute(CfgFilePath))
+        return false;
+    }
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133325



More information about the cfe-commits mailing list