[PATCH] D59480: [NFC] Add SchedState to allow forwarding the Scheduling state between MBB

Qing Shan Zhang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 18 19:09:04 PDT 2019


steven.zhang added a comment.

In D59480#1432943 <https://reviews.llvm.org/D59480#1432943>, @fhahn wrote:

> On a first look, the patch looks useful! Any chance to provide the follow-up patch that uses it as well? that would be helpful to see the bigger picture. Also, are you seeing any benefits by preserving the state?


I believe SystemZ has already seen some benefit basing on this. And as I replied jonpa below, I tried to add some NFC patches to complete the work. However, for PowerPC, I haven't seen the benefit from the spec2017 yet, and I am still investigating,  In theory, we should have some benefit as the resource is model more precise.


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

https://reviews.llvm.org/D59480





More information about the llvm-commits mailing list