[PATCH] D100717: [InstCombine] Transform memcpy to ptr load/stores if TBAA says so

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

In D100717#2706571 <https://reviews.llvm.org/D100717#2706571>, @nikic wrote:

> Once the LangRef changes are there, I'd suggest cross-posting this to llvm-dev for visibility. This patch was discussed at the last alias analysis meeting,

(hm, was there no reminder mail, or did i miss it? really need to get cale)

> and I think most people were in favor of using metadata for this purpose, but weren't super fond of determining the type to use based on string matching. Maybe someone else has more detailed notes...

To be noted, isn't really inventing a new approach though, it's already done for vtable pointers.
I feel like that if this isn't ok, then the existing code also should be removed.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100717/new/

https://reviews.llvm.org/D100717