[PATCH] D108221: [LoopIdiom] Keep TBAA when creating memcpy/memmove

[Jeroen Dobbelaere via Phabricator via llvm-commits]

jeroen.dobbelaere added a comment.

- A `memmove` test seems to be missing ?
- When the tbaa format is in 'new struct path tbaa' (-Xclang -new-struct-path-tbaa)  format, the resulting shuffling should not be valid ? aka, the new struct path tbaa contains offset and size. Offset is likely to be 0 and size should be the size of a single element.
  - Either that should be filtered out,
  - or the size should be adapted,  (But what with about an unknown size ?)
  - or maybe it should be converted into 'struct path tbaa' ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108221