[all-commits] [llvm/llvm-project] c94667: [clang-format] [PR52595] clang-format does not rec...

MyDeveloperDay via All-commits all-commits at lists.llvm.org
Thu Nov 25 03:10:33 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c94667a810e44c44fd355f9f014ffe161d75d761
      https://github.com/llvm/llvm-project/commit/c94667a810e44c44fd355f9f014ffe161d75d761
  Author: mydeveloperday <mydeveloperday at gmail.com>
  Date:   2021-11-25 (Thu, 25 Nov 2021)

  Changed paths:
    M clang/lib/Format/TokenAnnotator.cpp
    M clang/unittests/Format/FormatTest.cpp

  Log Message:
  -----------
  [clang-format] [PR52595] clang-format does not recognize rvalue references to array

https://bugs.llvm.org/show_bug.cgi?id=52595

missing space between `T(&&)` but not between `T (&` due to && being incorrectly thought of as `UnaryOperator`  rather than `PointerOrReference`

```
int operator()(T (&)[N]) { return 0; }
int operator()(T(&&)[N]) { return 1; }
```

Existing Unit tests are changed because actually I think they are originally incorrect, and are inconsistent with the (&) cases that are 4 or 5 lines above them.

Reviewed By: curdeius

Differential Revision: https://reviews.llvm.org/D114519




More information about the All-commits mailing list