[llvm] [BasicAA] Treat returns_twice functions as clobbering unescaped objects (PR #117902)

[Antonio Frighetto via llvm-commits]

antoniofrighetto wrote:

> Possibly. Note that callCapturesBefore() is mostly obsolete and only used by one place in MemCpyOpt at this point, so it likely doesn't make a difference in practice.

I'm only left with a doubt: being mostly obsolete, is there any particular reason why MemCpyOpt is not leveraging BatchAA with EarliestEscapeAnalysis? Or should it be possible (desirable) to move MemCpyOpt to employ the latter as well?

https://github.com/llvm/llvm-project/pull/117902