Szelethus added a comment.

In D75163#1902816 <https://reviews.llvm.org/D75163#1902816>, @xazax.hun wrote:

> The name of the patch implies refactoring but some functional changes were 
> also done. Is it possible to separate the functional changes into a separate 
> patch? If it is not a big effort I would prefer that way.


It would indeed be nicer, but given that the patch is relatively small both in 
LOC and in scope, I think its okay. In the future, I would also prefer to 
separate functional and non-functional changes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75163



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

Reply via email to