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 ? Cheers /Marcus
