[PATCH] D83645: Bump the default target CPU for i386-freebsd to i686

2020-07-22 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. llvm/utils/git/pre-push.py is recommended for your future commits. I guess `arc land` is a bad choice because llvm/llvm-project uses the github community version which can't set a pre-receive hook. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION http

[PATCH] D83645: Bump the default target CPU for i386-freebsd to i686

2020-07-19 Thread Dimitry Andric via Phabricator via cfe-commits
dim added a comment. In D83645#2160761 , @MaskRay wrote: > @dim > > Hi, your git commit contains extra Phabricator tags. You can drop > `Reviewers:` `Subscribers:` `Tags:` and the text `Summary:` from the git > commit with the following script: > > arc

[PATCH] D83645: Bump the default target CPU for i386-freebsd to i686

2020-07-18 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. @dim Hi, your git commit contains extra Phabricator tags. 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:|Su

[PATCH] D83645: Bump the default target CPU for i386-freebsd to i686

2020-07-12 Thread Dimitry Andric via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG02cfa7530d9e: Bump the default target CPU for i386-freebsd to i686 (authored by dim). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83645/new/ https://revie

[PATCH] D83645: Bump the default target CPU for i386-freebsd to i686

2020-07-12 Thread Ed Maste via Phabricator via cfe-commits
emaste accepted this revision. emaste added a comment. This revision is now accepted and ready to land. Fine with me Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83645/new/ https://reviews.llvm.org/D83645 ___

[PATCH] D83645: Bump the default target CPU for i386-freebsd to i686

2020-07-12 Thread Dimitry Andric via Phabricator via cfe-commits
dim created this revision. dim added reviewers: emaste, brooks, rsmith. Herald added subscribers: jfb, krytarowski, arichardson. Herald added a project: clang. Similar to what we have done downstream, some time ago: https://svnweb.freebsd.org/changeset/base/353936 This followed some discussions o