alexfh added a comment.

In D72527#1817518 <https://reviews.llvm.org/D72527#1817518>, @Eugene.Zelenko 
wrote:

> Both scripts works fine. However rename script should also sort entries 
> alphabetically, but probably this should be separate patch.


Yep, sorting of the list on rename is a separate issue.
The patch LGTM.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72527



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

Reply via email to