[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