[PATCH] D45266: [PowerPC] Add a Memory Latency Mutation to the scheduler

Lei Huang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 26 13:14:07 PDT 2018


lei added inline comments.


================
Comment at: lib/Target/PowerPC/PPCSubtarget.cpp:53
+    // Looking for loads and stores.
+    if (SU.getInstr()->mayLoad() || SU.getInstr()->mayStore()) {
+      // If we have a memory dependency then set the latency to 4 cycles.
----------------
Maybe do an early exit here:
```
if (!SU.getInstr()->mayLoad() && ! SU.getInstr()->mayStore())
  continue;
```
Then we don't need to nest the two for loops below.


https://reviews.llvm.org/D45266





More information about the llvm-commits mailing list