[all-commits] [llvm/llvm-project] cbea67: [clang][driver] Restore the original help text for...
Andrzej WarzyĆski via All-commits
all-commits at lists.llvm.org
Wed Jan 13 01:26:14 PST 2021
Branch: refs/heads/master
Home: https://github.com/llvm/llvm-project
Commit: cbea6737d5130724c7c8cf8ee4ccf1c3dd099450
https://github.com/llvm/llvm-project/commit/cbea6737d5130724c7c8cf8ee4ccf1c3dd099450
Author: Andrzej Warzynski <andrzej.warzynski at arm.com>
Date: 2021-01-13 (Wed, 13 Jan 2021)
Changed paths:
M clang/docs/ClangCommandLineReference.rst
M clang/include/clang/Driver/Options.td
Log Message:
-----------
[clang][driver] Restore the original help text for `-I`
The help text for `-I` was recently expanded in [1]. The expanded
version focuses on explaining the semantics of `-I` in Clang. We are now
in the process of adding support for `-I` in Flang and this new
description is incompatible with the semantics of `-I` in Flang. This
was brought up in this review:
* https://reviews.llvm.org/D93453
This patch reverts the original change in Options.td. This way the help
text for `-I` remains generic enough so that it applies to both Clang
and Flang.
The expanded description of `-I` from [1] is moved to the
`DocBrief` field for `-I`. This field is prioritised over the help text
when generating ClangCommandLineReference.rst, so the user facing
documentation for Clang retains the expanded description:
* https://clang.llvm.org/docs/ClangCommandLineReference.html
`DocBrief` fields are currently not used in Flang.
As requested in the reviews, the help text and the expanded description
are slightly refined.
[1] Commit: 8dd4e3ceb804a58bcf25e6856fc6fde5e1995a66
Differential Revision: https://reviews.llvm.org/D94169
More information about the All-commits
mailing list