[PATCH] D82317: [Clang/Test]: Update tests where `noundef` attribute is necessary
Jessica Clarke via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 6 12:35:36 PDT 2020
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
More information about the cfe-commits
mailing list