jrtc27 added a comment.

If changes like this are required for all these tests this is going to be a 
complete pain for downstream forks like ours. At the very least, make whatever 
script you used to update these public, as I don't want to have to recreate it 
from scratch when merging this in. I had enough "fun" with the LLD 
mass-renaming (UpperCamel -> lowerCamel) and that was _with_ a 
supposedly-working script (it didn't quite do the right thing and I seem to 
recall there being two refactoring commits, only one of which had a script); I 
do not want a repeat of that experience.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82317

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

Reply via email to