[PATCH] D68414: [SROA] Enhance AggLoadStoreRewriter to rewrite integer load/store if it covers multi fields in original aggregate

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 1 00:35:15 PDT 2020


lebedev.ri added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/SROA.cpp:4142-4143
+///
+///     S1   ------
+///     S2      ------
+///
----------------
Ignorant question: is it guaranteed that there isn't a situation like this:
```
///     S1   ------
///     S2      ------
///     S3     ------
```
I.e., there is no other slice S3 overlapping S2, with different overlap characteristic? 


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

https://reviews.llvm.org/D68414





More information about the llvm-commits mailing list