This patch adds a second kind of refactoring action rule that produces symbol occurrences. It will be used by the updated clang-refactor patch at https://reviews.llvm.org/D36574.
|29 ↗||(On Diff #112881)|
I might miss some context here. As per your comment in https://reviews.llvm.org/D36075#inline-323769, you'll try to remove this tag, so I think we will hold off this patch until that is done?
I'm not a fan of passing a const smart_pointer& as function parameter. The function being called doesn't care about the lifetime management, so I'd use a raw pointer here.