[PATCH] D59036: Memory writes overlap in the pipelined loop

Denis Antrushin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 14 10:51:12 PDT 2019


dantrushin added inline comments.


================
Comment at: llvm/lib/CodeGen/MachinePipeliner.cpp:3170
   if (OffsetS >= OffsetD)
     return OffsetS + AccessSizeS > DeltaS;
   else
----------------
BTW, is this code correct?
I would expect something like

```
(OffsetS + DeltaS) <= (OffsetD + AccessSizeD) && (OffsetS + AccessSizeS + Delta) >= OffsetD
```

Am I missing something?


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

https://reviews.llvm.org/D59036





More information about the llvm-commits mailing list