diff options
| author | Marek Kurdej <marek.kurdej+llvm.org@gmail.com> | 2022-02-21 14:17:58 +0100 |
|---|---|---|
| committer | Marek Kurdej <marek.kurdej+llvm.org@gmail.com> | 2022-02-24 10:16:10 +0100 |
| commit | 46f6c834d9f95d99e0a85aa0c6dc07419aa6dee2 (patch) | |
| tree | 7044a67226feed8dba9f2b8c5aa3dd12c0bb0fb1 /llvm/tools/llvm-objdump/llvm-objdump.cpp | |
| parent | ff3f3a54e2d1b05c36943bf88ae0be7475d622ed (diff) | |
| download | llvm-46f6c834d9f95d99e0a85aa0c6dc07419aa6dee2.zip llvm-46f6c834d9f95d99e0a85aa0c6dc07419aa6dee2.tar.gz llvm-46f6c834d9f95d99e0a85aa0c6dc07419aa6dee2.tar.bz2 | |
[clang-format] Fix QualifierOrder breaking the code with requires clause.
Fixes https://github.com/llvm/llvm-project/issues/53962.
Given the config:
```
BasedOnStyle: LLVM
QualifierAlignment: Custom
QualifierOrder: ['constexpr', 'type']
```
The code:
```
template <typename F>
requires std::invocable<F>
constexpr constructor();
```
was incorrectly formatted to:
```
template <typename F>
requires
constexpr std::invocable<F> constructor();
```
because we considered `std::invocable<F> constexpr` as a type, not recognising the requires clause.
This patch avoids moving the qualifier across the boundary of the requires clause (checking `ClosesRequiresClause`).
Reviewed By: HazardyKnusperkeks, owenpan
Differential Revision: https://reviews.llvm.org/D120309
Diffstat (limited to 'llvm/tools/llvm-objdump/llvm-objdump.cpp')
0 files changed, 0 insertions, 0 deletions