Xinlong-Wu wrote:

> > > > This patch is moved from https://reviews.llvm.org/D134600
> > > 
> > > 
> > > Why? That loses all the context in the Phabricator review. Keep the 
> > > review there.
> > 
> > 
> > yes, I will keep it there.
> > But the [Phabricator shutdown 
> > timeline](https://discourse.llvm.org/t/update-on-github-pull-requests/71540)
> >  said that
> > > Phabricator becomes read-only after October 1
> > 
> > 
> > So I thought it may means we can't add comments or update it? That's why I 
> > moved it from Phabricator.
> > If you mean we still can review&update it at Phabricator, I will close this 
> > pr. )
> 
> Scroll down the thread and you will find 
> https://discourse.llvm.org/t/update-on-github-pull-requests/71540/125. 
> There's consensus that Phabricator will remain usable for existing revisions 
> for a while longer for this specific kind of situation, and that migrating 
> revisions off of it loses valuable context and should be avoided.

That make sense. I will close this pr

https://github.com/llvm/llvm-project/pull/68551
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to