[PATCH] D134337: [clang] [Driver] More flexible rules for loading default configs

Michał Górny via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 29 09:51:17 PDT 2022


mgorny marked 2 inline comments as done.
mgorny added inline comments.


================
Comment at: clang/lib/Driver/Driver.cpp:1065-1080
+  StringRef EffectiveDriverMode;
+  if (CLOptions)
+    EffectiveDriverMode = CLOptions->getLastArgValue(options::OPT_driver_mode);
+  if (EffectiveDriverMode.empty())
+    EffectiveDriverMode = ClangNameParts.DriverMode;
+
+  llvm::Triple EffectiveTriple =
----------------
MaskRay wrote:
> mgorny wrote:
> > This part is mostly proof-of-concept, I need to look into how to do this properly.
> The next common mentions "second part" but it isn't very clear what "first part" and "second part" mean.
> Perhaps repeat <triple>.cfg here and replace "second part" with "<mode>.cfg"
Oh, it's so weird because the comment was moved when I was swapping the logic but didn't notice it needs to be updated.


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

https://reviews.llvm.org/D134337



More information about the cfe-commits mailing list