PiotrZSL added a comment.

Thank you for your feedback regarding the code review. I understand that the 
review includes the removal of 35 lines and the addition of 42 lines. However, 
I would like to point out that the areStatementsIdentical function and the 
other function in question are only around 24 lines long each. Therefore, I 
believe that the changes made to these functions are not overly complex and can 
be easily reviewed as they are currently presented.

Additionally, I would like to express my concern that pushing anything through 
the review process currently takes up to a month. Separating the changes into 
two patches would only prolong this process and potentially delay the 
completion of this task.

If you have any questions related to the scope of the changes, I would be more 
than happy to answer them.

Thank you again for your time and feedback. Please let me know if you have any 
further concerns, and I'll do my best to address them.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148995

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

Reply via email to