[PATCH] D95692: [Pipeliner] Fixed optimization remarks and debug dumps Initiation Interval value

[Brendon Cahoon via Phabricator via llvm-commits]

bcahoon added a comment.

Hi Marianne,

Thanks for the patch. Sorry that it took a while to get to this. I just have a couple of questions.

Thanks,
Brendon



================
Comment at: llvm/lib/CodeGen/MachinePipeliner.cpp:2126
   }
+  --II;
 
----------------
Does this need to be decremented only if scheduledFound is true?

Another possibility is to use Schedule.getInitiationInterval() instead of II in the subsequent debug and remark messages.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95692