[PATCH] D82070: [scudo][standalone] Allow Primary allocations to fail up multiple times.

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 18 21:16:08 PDT 2020


cryptoad added a comment.

I did the commit and followed the instructions @ https://llvm.org/docs/Phabricator.html#committing-someone-s-change-from-phabricator.
Let me know if they are no longer relevant.

In D82070#2102280 <https://reviews.llvm.org/D82070#2102280>, @MaskRay wrote:

> @cferris
>
> Hi, you can drop `Reviewers:` `Subscribers:` `Tags:` and the text `Summary:` from the git commit with the following script:
>
>   arcfilter () {
>           arc amend
>           git log -1 --pretty=%B | awk '/Reviewers:|Subscribers:/{p=1} /Reviewed By:|Differential Revision:/{p=0} !p && !/^Summary:$/ {sub(/^Summary: /,"");print}' | git commit --amend --date=now -F -
>   }
>   
>
> `Reviewed By: ` is considered important by some people. Please keep the tag. (I have updated my script to use `--date=now` (setting author date to committer date))
>
> `https://reviews.llvm.org/D80978` contains a git pre-push hook to automate this.





Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82070





More information about the llvm-commits mailing list