diff options
| author | Volodymyr Sapsai <vsapsai@apple.com> | 2022-11-29 17:49:40 -0800 |
|---|---|---|
| committer | Volodymyr Sapsai <vsapsai@apple.com> | 2022-12-02 11:59:22 -0800 |
| commit | b4b54697b7aacda1fabef36d4e74d5ee45641618 (patch) | |
| tree | f19943f54ee9c42ff51d1043357e0a8eae20df30 /llvm/lib/ProfileData/Coverage/CoverageMapping.cpp | |
| parent | b77533306876fc807e58e355d95d848a0077131f (diff) | |
| download | llvm-b4b54697b7aacda1fabef36d4e74d5ee45641618.zip llvm-b4b54697b7aacda1fabef36d4e74d5ee45641618.tar.gz llvm-b4b54697b7aacda1fabef36d4e74d5ee45641618.tar.bz2 | |
[clang][Driver] Don't overwrite `DiagnosticsEngine::IgnoreAllWarnings`, rely on `DiagnosticOptions::IgnoreWarnings` value.
Driver overwrites `DiagnosticsEngine::IgnoreAllWarnings` based on `-w` flag
without taking into account `DiagnosticOptions::IgnoreWarnings` that is
propagated to `DiagnosticsEngine` in `ProcessWarningOptions` (called from
`CompilerInstance::createDiagnostics`). It makes it hard to manipulate
`DiagnosticOptions` directly and pushes towards string-based API.
Most of in-tree tools use `DiagnosticOptions` already, so migrate
`clang_parseTranslationUnit_Impl` to use it too. Don't parse `-w`
directly but rely on
```
def w : Flag<["-"], "w">, HelpText<"Suppress all warnings">, Flags<[CC1Option]>,
MarshallingInfoFlag<DiagnosticOpts<"IgnoreWarnings">>;
```
Allows to reland D138252.
Differential Revision: https://reviews.llvm.org/D138970
Diffstat (limited to 'llvm/lib/ProfileData/Coverage/CoverageMapping.cpp')
0 files changed, 0 insertions, 0 deletions