[PATCH] Cleanup register pressure calculation in MachineLICM

Quentin Colombet qcolombet at apple.com
Fri Apr 3 09:58:49 PDT 2015


> Regression testing has revealed that there is actually one more distinction to be made to avoid functional changes in this refactoring: Whether to count unseen registers as defs. Now, there are no regressions in the LLVM test suite.


Could you add the related test to make check?
I like to improve our code coverage in make check when we get the chance :).

Other than that LGTM.

Thanks,
-Quentin


http://reviews.llvm.org/D8604

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






More information about the llvm-commits mailing list