[PATCH] D76190: CET for Exception Handle

Pengfei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 29 18:12:06 PDT 2020


pengfei added a comment.

In D76190#1948762 <https://reviews.llvm.org/D76190#1948762>, @MaskRay wrote:

> @pengfei You can strip unneeded Phabricator tags with:
>
>   % which arcfilter
>   arcfilter () {
>           git log -1 --pretty=%B | awk '/Reviewers:|Subscribers:/{p=1} /Reviewed By:|Differential Revision:/{p=0} !p && !/^Summary:$/ {sub(/^Summary: /,
>   "");print}' | git commit --amend -F -
>   }
>
>
> `Subscribers:` and `Reviewers:` are just annoying. It is also not appropriate to use `Patch By: Xiang Zhang (xiangzhangllvm)`. See https://llvm.org/docs/DeveloperPolicy.html#commit-messages " git commit --amend --author="John Doe <jdoe at llvm.org> to correct the author property"


Thanks for your information. It's very helpful.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76190





More information about the llvm-commits mailing list