[PATCH] Fix instruction scheduling live register tracking

PaweĊ‚ Bylica chfast at gmail.com
Mon Jun 22 03:06:41 PDT 2015


I have a filling I'm going in circles.

Now it looks the best "fix" is to remove the case `I->getSUnit()->getHeight() < LiveRegGens[I->getReg()]->getHeight()` when updating LiveRegGen. Without this case LiveRegGen is set during unscheduling only if it was empty before.

I checked selfhosted regression tests and  test-suite. Now using the clang build with this change in my other projects.

Can someone give me an example when this removed case is needed?


http://reviews.llvm.org/D9993

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list