[PATCH] D109621: [clang][Driver] Default to loading clang.cfg if config file not specified

Arfrever Frehtes Taifersar Arahesis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 10 13:17:19 PDT 2021


Arfrever added inline comments.


================
Comment at: clang/docs/UsersManual.rst:877
 For example, if the Clang executable is named `armv7l-clang` (it may be a
 symbolic link to `clang`), then Clang will search for file `armv7l.cfg` in the
+directories mentioned above. If the executable is named `clang`, the Clang will
----------------
This says `armv7l.cfg`...


================
Comment at: clang/lib/Driver/Driver.cpp:906
   // from executable name. For instance, an executable 'armv7l-clang' will
   // search for config file 'armv7l-clang.cfg'.
   if (CfgFileName.empty() && !ClangNameParts.TargetPrefix.empty())
----------------
But this says `armv7l-clang.cfg`.
Could you also fix this inconsistency?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109621



More information about the cfe-commits mailing list