[PATCH] D113319: [clang-format] Improve require and concept handling

Marek Kurdej via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 22 06:49:21 PST 2022


curdeius added a comment.

In D113319#3337485 <https://reviews.llvm.org/D113319#3337485>, @krasimir wrote:

> It appears that after this patch clang-format started breaking up `requires` in javascript, e.g.:
>
>   // before
>   function f() {
>     var requires = {};
>   }
>   // after
>   function f() {
>     var
>       requires
>     = {};
>   }
>
> Could we restrict the requires logic only to (obj)C++? (normally I'd go and come up with a patch for cases like this, but this patch is quite large and it's not immediately clear where's the best place for this restriction).

Addressed in https://reviews.llvm.org/D120324.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D113319/new/

https://reviews.llvm.org/D113319



More information about the cfe-commits mailing list