On Tue, Jan 13, 2015 at 6:13 AM, Marcus Shawcroft <marcus.shawcr...@gmail.com> wrote: > On 11 January 2015 at 02:37, Andrew Pinski <pins...@gmail.com> wrote: >> On Tue, Nov 11, 2014 at 6:47 AM, Marcus Shawcroft >> <marcus.shawcr...@gmail.com> wrote: >>> On 30 October 2014 08:54, Gopalasubramanian, Ganesh >>> <ganesh.gopalasubraman...@amd.com> wrote: >>> >>>> 2014-10-30 Ganesh Gopalasubramanian <ganesh.gopalasubraman...@amd.com> >>> >>> Check the whitespace in your ChangeLog line. >>> >>>> * config/arm/types.md (define_attr "type"): Add prefetch. >>> >>> The existing schedulers use 'load1'. We can of course split that into >>> two introducing "prefetch" and update all of the existing schedulers >>> to reflect the change. However I suggest we do that as a separate >>> activity when someone actually needs the distinction, note this change >>> will require updating the schedulers for both ARM and AArch64 backends >>> not just those relevant to AArch64. For this prefetch patch I suggest >>> we go with the existing "load1". >> >> I will need this change for ThunderX schedule. The Pref instruction >> is single issued while load1 can be dual issued. > > Hi > > https://gcc.gnu.org/ml/gcc-patches/2015-01/msg00802.html > > Philipp when you deal with Ramana's request above to split > load1->load1/prefetch in the existing schedulers I suggest you also > split it in aarch64/thunderx.md in order to retain existing behaviour. > Andrew can then follow up add the "right" behaviour when he is ready. > Andrew OK ?
Yes that sounds ok to me. I was going to submit an update to thunderx.md file this week anyways. Thanks, Andrew > > Cheers > /Marcus
