[PATCH] D151047: [clang-format] Fix indentation for selective formatting.
Sergei Barannikov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu May 25 04:14:03 PDT 2023
barannikov88 added a comment.
In D151047#4369742 <https://reviews.llvm.org/D151047#4369742>, @Sedeniono wrote:
> To create a new fix, do I understand the guide <https://llvm.org/docs/Phabricator.html#using-patch-summaries> correctly that I basically execute `arc diff --verbatim` and mention `Depends on D151047` somewhere in the message? Will it then appear here automatically and reopen the review?
If you want to create a new review, use `arc diff --create`. It will always create a new review and assign it a different number.
If you want to reopen review because the patch was reverted:
- cherry-pick the reverted commit;
- do any necessary adjustments for the tests to pass, amend them;
- If you're using arc, use `arc diff --update D151047` as usual. It will update the current review, preserving its number.
(The --update option is not necessary if the commit message contains URL to this review, but it prevents you from accidentally creating a new review.)
I don't remember if the above steps will change the review status to Reopened.
If that doesn't happen, you can reopen the review manually from the menu below (Add Action... -> Reopen Revision).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D151047/new/
https://reviews.llvm.org/D151047
More information about the cfe-commits
mailing list