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

[Jinsong Ji via Phabricator via llvm-commits]

jsji added a comment.

In D62345#1515781 <https://reviews.llvm.org/D62345#1515781>, @javed.absar wrote:

> Would this end up prioritising 'resourcelimited' too much? Looks like there can be performance differences between before and after this change - any  numbers / thoughts on that?


The SPEC2k17 performance test on P9 <https://reviews.llvm.org/P9> system shows GEOMEAN 0.3% improvement, 
with 1 biggest deg -0.50%, 2 biggest improvement +1.64%, +1.52%, others are within +-0.4%.

@javed.absar


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

https://reviews.llvm.org/D62345