[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.

Thanks for the elucidation. I'll consider opening a patch. GVN MemorySSA main change would become a user of callCapturesBefore() as well. I imagine we may want to migrate these APIs to getModRefInfo()/getCapturesBefore() sometime soon.

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