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

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 18 02:01:56 PDT 2021


nikic added a comment.

I'm uncomfortable with using TBAA for this purpose. The problem is that not all languages use TBAA, simply because that does not match their aliasing model. They shouldn't be penalized because TBAA information gets reused for an unrelated purpose.

Might it make sense to simply always use a pointer type here (assuming size matches of course)?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100717



More information about the llvm-commits mailing list