tree 56269c54bb5915d2e23fdd4b16516d3fd01b3a9a
parent 659dbcd57891738d8e9a43e6bc8659fdad61457d
author Fangrui Song <maskray@google.com> 1555660671 +0000
committer Fangrui Song <maskray@google.com> 1555660671 +0000

[MergeFunc] removeUsers: call remove() only on direct users

removeUsers uses a work list to collect indirect users and call remove()
on those functions. However it has a bug (`if (!Visited.insert(UU).second)`).

Actually, we don't have to collect indirect users.
After the merge of F and G, G's callers will be considered (added to
Deferred). If G's callers can be merged, G's callers' callers will be
considered.

Update the test unnamed-addr-reprocessing.ll to make it clear we can
still merge indirect callers.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@358741 91177308-0d34-0410-b5e6-96231b3b80d8