gamesh411 wrote:

> Usually its hard to see how the implementation changes if anyways all the 
> surrounding code moves. For such changes, if the change is large (like in 
> this case, its a 2k+ diff), it feels better to review the individual commits 
> instead of everything all at once. If we do that right, the bulk of the 
> change will be attributed to a handful of NFC mechanical changes, and I could 
> focus on whats left.

We can do that. I will just create separate PRs for these commits, emphasizing 
in each one whether they are NFCs or the key points where they deviate from 
being NFCs.

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

Reply via email to