awarzynski updated this revision to Diff 315765.
awarzynski added a comment.
Implement SUGGESTION 1
I haven't updated the patch description yet - I'd rather keep it as is while
the actual direction for this patch is being discussed.
Also, I regenerated ClangCommandLineReference.rst and discovered that many
options are out-of-sync with Options.td. In order not to digress, I've only
updated the description for `-I`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D94169/new/
https://reviews.llvm.org/D94169
Files:
clang/docs/ClangCommandLineReference.rst
clang/include/clang/Driver/Options.td
Index: clang/include/clang/Driver/Options.td
===================================================================
--- clang/include/clang/Driver/Options.td
+++ clang/include/clang/Driver/Options.td
@@ -649,12 +649,12 @@
"remove current directory from include path">;
def I : JoinedOrSeparate<["-"], "I">, Group<I_Group>,
Flags<[CC1Option,CC1AsOption]>, MetaVarName<"<dir>">,
- HelpText<"Add directory to include search path. If there are multiple -I "
- "options, these directories are searched in the order they are "
- "given before the standard system directories are searched. "
- "If the same directory is in the SYSTEM include search paths, for
"
- "example if also specified with -isystem, the -I option will be "
- "ignored">;
+ HelpText<"Add directory to include search path. In Clang drivers, if "
+ "there are multiple -I options, these directories are searched "
+ "in the order they are given before the standard system directories "
+ "are searched. If the same directory is in the SYSTEM include search "
+ "paths, for example if also specified with -isystem, the -I option "
+ "will be ignored">;
def L : JoinedOrSeparate<["-"], "L">, Flags<[RenderJoined]>, Group<Link_Group>,
MetaVarName<"<dir>">, HelpText<"Add directory to library search path">;
def MD : Flag<["-"], "MD">, Group<M_Group>,
Index: clang/docs/ClangCommandLineReference.rst
===================================================================
--- clang/docs/ClangCommandLineReference.rst
+++ clang/docs/ClangCommandLineReference.rst
@@ -1016,7 +1016,7 @@
.. option:: -I<dir>, --include-directory <arg>, --include-directory=<arg>
-Add directory to include search path. If there are multiple -I options, these
directories are searched in the order they are given before the standard system
directories are searched. If the same directory is in the SYSTEM include search
paths, for example if also specified with -isystem, the -I option will be
ignored
+Add directory to include search path. In Clang drivers, if there are multiple
-I options, these directories are searched in the order they are given before
the standard system directories are searched. If the same directory is in the
SYSTEM include search paths, for example if also specified with -isystem, the
-I option will be ignored
.. option:: -I-, --include-barrier
Index: clang/include/clang/Driver/Options.td
===================================================================
--- clang/include/clang/Driver/Options.td
+++ clang/include/clang/Driver/Options.td
@@ -649,12 +649,12 @@
"remove current directory from include path">;
def I : JoinedOrSeparate<["-"], "I">, Group<I_Group>,
Flags<[CC1Option,CC1AsOption]>, MetaVarName<"<dir>">,
- HelpText<"Add directory to include search path. If there are multiple -I "
- "options, these directories are searched in the order they are "
- "given before the standard system directories are searched. "
- "If the same directory is in the SYSTEM include search paths, for "
- "example if also specified with -isystem, the -I option will be "
- "ignored">;
+ HelpText<"Add directory to include search path. In Clang drivers, if "
+ "there are multiple -I options, these directories are searched "
+ "in the order they are given before the standard system directories "
+ "are searched. If the same directory is in the SYSTEM include search "
+ "paths, for example if also specified with -isystem, the -I option "
+ "will be ignored">;
def L : JoinedOrSeparate<["-"], "L">, Flags<[RenderJoined]>, Group<Link_Group>,
MetaVarName<"<dir>">, HelpText<"Add directory to library search path">;
def MD : Flag<["-"], "MD">, Group<M_Group>,
Index: clang/docs/ClangCommandLineReference.rst
===================================================================
--- clang/docs/ClangCommandLineReference.rst
+++ clang/docs/ClangCommandLineReference.rst
@@ -1016,7 +1016,7 @@
.. option:: -I<dir>, --include-directory <arg>, --include-directory=<arg>
-Add directory to include search path. If there are multiple -I options, these directories are searched in the order they are given before the standard system directories are searched. If the same directory is in the SYSTEM include search paths, for example if also specified with -isystem, the -I option will be ignored
+Add directory to include search path. In Clang drivers, if there are multiple -I options, these directories are searched in the order they are given before the standard system directories are searched. If the same directory is in the SYSTEM include search paths, for example if also specified with -isystem, the -I option will be ignored
.. option:: -I-, --include-barrier
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
