xgupta accepted this revision.
xgupta added a comment.
This revision is now accepted and ready to land.

In D104971#2842293 <https://reviews.llvm.org/D104971#2842293>, @SaurabhJha 
wrote:

> This is a light patch that probably does not require a review and I created a 
> patch anyway.
>
> Would it be appropriate to get this into main using `arc land` if this is 
> approved? I saw this https://reviews.llvm.org/D78867 which advises against 
> `arc land` but could not find anything related to that in the main docs.

I follow 
https://www.llvm.org/docs/Phabricator.html#committing-someone-s-change-from-phabricator
which says `git push https://github.com/llvm/llvm-project.git HEAD:main`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104971

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to