On 15/04/15 21:59, Maxim Kuvyrkov wrote: >> On Apr 15, 2015, at 12:41 AM, Kugan <kugan.vivekanandara...@linaro.org> >> wrote: >> >> This patch uses clobber with match_scratch instead of earlyclobber for >> aarch64_lshr_sisd_or_int_<mode>3 so that RA can have more freedom in >> selecting suitable register, as discussed in >> http://thread.gmane.org/gmane.comp.gcc.patches/336162 and reported in >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65139. >> >> This is based on Maxim's patch. I have bootstrapped and regression >> tested on aarch64-linux-gnu with no new regressions. Is this OK for trunk? >> >> Thanks, >> Kugan >> >> gcc/ChangeLog: >> >> 2015-04-15 Kugan Vivekanandarajah <kug...@linaro.org> >> Maxim Kuvyrkov <maxim.kuvyr...@linaro.org> >> >> PR target/65139 >> * config/aarch64/aarch64.md (<optab><mode>3): Expand lshr with >> gen_aarch64_lshr_sisd_or_int_<mode>3. >> (*aarch64_lshr_sisd_or_int_<mode>3): Rename to >> aarch64_lshr_sisd_or_int_<mode>3 and use clobber with >> match_scratch instead of early clobber. >> <p.txt> > > For the benefit of other reviewers -- I have reviewed this patch internally > and it looks OK. > > Kugan, did you audit other patterns in aarch64.md to see if any other > early-clobbers can be optimized in this way?
Not yet. I will after this patch is gone through the review. Thanks, Kugan
