On 2015-03-23 12:35 PM, Ajit Kumar Agarwal wrote: > Hello All: > > Did you get a chance to look at the below patch. > > Thanks & Regards > Ajit > > -----Original Message----- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On > Behalf Of Ajit Kumar Agarwal > Sent: Wednesday, March 04, 2015 3:57 PM > To: vmaka...@redhat.com; GCC Patches > Cc: Vinod Kathail; Shail Aditya Gupta; Vidhumouli Hunsigida; Nagaraju Mekala > Subject: [Patch] OPT: Update heuristics for loop-invariant for address > arithmetic. > > 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. > > Sorry, I am not a maintainer/reviewer of loop invariant motion pass. I had no power to approve this, especially when you change mostly code for ! flag_ira_loop_pressure.
I only should say that microblaze is not a primary target. You should demonstrate improvement on a primary target, x86/x86-64 the best. SPEC2000 or SPEC2006 would be a good benchmark for this.
