cor3ntin marked an inline comment as done.
cor3ntin added a comment.

In D119136#3418585 <https://reviews.llvm.org/D119136#3418585>, @ChuanqiXu wrote:

> I've just skimmed over the patch and it is a little bit hard to understand 
> for people who don't have a strong background. Is it possible to split this 
> one into several parts?

Unfortunately, I don't thing that would be very practical. The patch is not 
small but what it changes is rather small, and if we tried to split it the 
intermediate patches would be inconsistent and untested


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119136

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

Reply via email to