[PATCH] D62345: [MachineScheduler] checkResourceLimit boundary condition update

[Jinsong Ji via Phabricator via llvm-commits]

jsji created this revision.
jsji added reviewers: atrick, hfinkel, MatzeB, fhahn, jonpa, stefanp, javed.absar, niravd.
Herald added a subscriber: hiraditya.
Herald added a project: LLVM.

When we call checkResourceLimit in bumpCycle or bumpNode, and we
know the resource count has just reached the limit (the equations
 are equal). We should return true to mark that we are resource
limited for next schedule, or else we might continue to schedule
in favor of latency for 1 more schedule and create a schedule that
 actually overbook the resource.

When we call checkResourceLimit to estimate the resource limit before
scheduling, we don't need to return true even if the equations are
equal, as it shouldn't limit the schedule for it .


Repository:
  rL LLVM

https://reviews.llvm.org/D62345

Files:
  llvm/lib/CodeGen/MachineScheduler.cpp
  llvm/test/CodeGen/PowerPC/csr-save-restore-order.ll
  llvm/test/CodeGen/PowerPC/f128-passByValue.ll
  llvm/test/CodeGen/PowerPC/vec_conv_fp_to_i_8byte_elts.ll
  llvm/test/CodeGen/PowerPC/vec_conv_i_to_fp_8byte_elts.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D62345.201063.patch
Type: text/x-patch
Size: 9674 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190523/51a95f97/attachment.bin>