This is an archive of the discontinued LLVM Phabricator instance.

clang-rename: let -force handle multiple renames
ClosedPublic

Authored by vmiklos on Sep 8 2017, 11:40 AM.

Details

Summary

The use case is that renaming multiple symbols in a large enough codebase is
much faster if all of these can be done with a single invocation, but
there will be multiple translation units where one or more symbols are
not found.

Old behavior was to exit with an error (default) or exit without
reporting an error (-force). New behavior is that -force results in a
best-effort rename: rename symbols which are found and just ignore the
rest.

The existing help for -force sort of already implies this behavior.

Diff Detail

Repository
rL LLVM

Event Timeline

vmiklos created this revision.Sep 8 2017, 11:40 AM
This revision is now accepted and ready to land.Sep 11 2017, 1:20 AM
This revision was automatically updated to reflect the committed changes.