[PATCH] D120806: [clang-format] Recognize "if consteval".

2022-03-02 Thread Marek Kurdej via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG13351fdf8cb4: [clang-format] Recognize "if consteval". (authored by curdeius). Changed prior to commit: https://reviews.llvm.org/D120806?vs=412366&id=412444#toc Repository: rG LLVM Github Monorepo C

[PATCH] D120806: [clang-format] Recognize "if consteval".

2022-03-02 Thread Marek Kurdej via Phabricator via cfe-commits
curdeius added a comment. Great, I'll add a few test cases when landing. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120806/new/ https://reviews.llvm.org/D120806 ___ cfe-commits mailing list cfe-commit

[PATCH] D120806: [clang-format] Recognize "if consteval".

2022-03-02 Thread Johel Ernesto Guerrero Peña via Phabricator via cfe-commits
JohelEGP accepted this revision. JohelEGP added a comment. My few cases format consistently with the condition case: diff diff --git a/tests/integration/test.waarudo.io.dynamic.cpp b/tests/integration/test.waarudo.io.dynamic.cpp index be5a626..e1265b9 100644 --- a/tests/integration/test.

[PATCH] D120806: [clang-format] Recognize "if consteval".

2022-03-02 Thread Marek Kurdej via Phabricator via cfe-commits
curdeius added a comment. I'll handle `AllowShortIfStatements` and update this review. Should be pretty straightforward. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120806/new/ https://reviews.llvm.org/D120806 __

[PATCH] D120806: [clang-format] Recognize "if consteval".

2022-03-02 Thread Johel Ernesto Guerrero Peña via Phabricator via cfe-commits
JohelEGP added a comment. In D120806#3354119 , @MyDeveloperDay wrote: > LFTM > > Do we also need to consider this in the context of "AllowShortIfStatments"? > (that would be in a different review in my view) Definitely. Since it didn't work at all, I

[PATCH] D120806: [clang-format] Recognize "if consteval".

2022-03-02 Thread MyDeveloperDay via Phabricator via cfe-commits
MyDeveloperDay accepted this revision. MyDeveloperDay added a comment. This revision is now accepted and ready to land. LFTM Do we also need to consider this in the context of "AllowShortIfStatments"? (that would be in a different review in my view) Repository: rG LLVM Github Monorepo CHAN

[PATCH] D120806: [clang-format] Recognize "if consteval".

2022-03-02 Thread Marek Kurdej via Phabricator via cfe-commits
curdeius created this revision. curdeius added reviewers: MyDeveloperDay, HazardyKnusperkeks, owenpan. Herald added a project: All. curdeius requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Fixes https://github.com/llvm/llvm-project/issues/5