Very delayed answer, sorry... Ajit Kumar Agarwal <ajit.kumar.agar...@xilinx.com> writes: > Hello All: > > The changes are made in the patch to update the heuristics for loop > invariant for address arithemetic at RTL Level. The heuristics are > updated with the consideration of single def and use for register > pressure calculation instead Of ignoring it and also to update the > estimated register pressure cost along with the check of actual uses > with Address uses. > > With the above change, gains are seen in the Geomean for Mibench/EEMBC > benchmarks for microblaze target. No Regression is seen in deja GNU > regressions tests for microblaze.
Since thispatch is basically removing code, were you able to analyse why that code was having a detrimental effect? I assume it benefited some target originally. Thanks, Richard
