On 11/28/14 07:10, Wilco Dijkstra wrote:
Jeff Law wrote:
Do you have a testcase that shows the expected improvements from this
change? It's OK if it's specific to a target.
Have you bootstrapped and regression tested this change?
With a test for the testsuite and assuming it passes bootstrap and
regression testing, this will almost certainly be OK.
I've added a test, see below. It bootstraps OK on AArch64 and the only
difference in the regression tests after the patch is the one extra pass.
OK for commit?
OK with the appropropriate ChangeLog entires. THe original for
ira-costs.c was fine, so you just need the trivial one for the testcase.
jeff
