Re: [PATCH][AArch64] Improve SHA1 scheduling

2017-01-17 Thread James Greenhalgh
hours to object before pushing it. Thanks, James > From: Wilco Dijkstra > Sent: 25 October 2016 18:08 > To: GCC Patches > Cc: nd > Subject: [PATCH][AArch64] Improve SHA1 scheduling >     > SHA1H instructions may be scheduled after a SHA1C instruction > that uses the sa

Re: [PATCH][AArch64] Improve SHA1 scheduling

2017-01-17 Thread Wilco Dijkstra
Wilco Dijkstra wrote: > James Greenhalgh wrote: > > > I haven't seen a follow-up to Andrew's point regarding other > > read-modify-write operations. > > > > Did youi investigate the cost of these? > > I looked at whether there are other similar cases, but it appears SHA1 > is unique due to the odd

Re: [PATCH][AArch64] Improve SHA1 scheduling

2016-12-07 Thread Wilco Dijkstra
James Greenhalgh wrote: > I haven't seen a follow-up to Andrew's point regarding other > read-modify-write operations. > > Did youi investigate the cost of these? I looked at whether there are other similar cases, but it appears SHA1 is unique due to the odd dataflow, the mismatch in latencies a

Re: [PATCH][AArch64] Improve SHA1 scheduling

2016-12-06 Thread James Greenhalgh
er 2016 18:08 > To: GCC Patches > Cc: nd > Subject: [PATCH][AArch64] Improve SHA1 scheduling >     > SHA1H instructions may be scheduled after a SHA1C instruction > that uses the same input register.  However SHA1C updates its input, > so if SHA1H is scheduled after it, it requ

Re: [PATCH][AArch64] Improve SHA1 scheduling

2016-12-06 Thread Wilco Dijkstra
  ping From: Wilco Dijkstra Sent: 25 October 2016 18:08 To: GCC Patches Cc: nd Subject: [PATCH][AArch64] Improve SHA1 scheduling     SHA1H instructions may be scheduled after a SHA1C instruction that uses the same input register.  However SHA1C updates its input, so if SHA1H is scheduled

Re: [PATCH][AArch64] Improve SHA1 scheduling

2016-11-14 Thread Wilco Dijkstra
ping From: Wilco Dijkstra Sent: 25 October 2016 18:08 To: GCC Patches Cc: nd Subject: [PATCH][AArch64] Improve SHA1 scheduling     SHA1H instructions may be scheduled after a SHA1C instruction that uses the same input register.  However SHA1C updates its input, so if SHA1H is scheduled after

Re: [PATCH][AArch64] Improve SHA1 scheduling

2016-11-03 Thread Wilco Dijkstra
Andrew Pinski wrote: > On Tue, Oct 25, 2016 at 10:08 AM, Wilco Dijkstra > wrote: > > SHA1H instructions may be scheduled after a SHA1C instruction > > that uses the same input register.  However SHA1C updates its input, > > so if SHA1H is scheduled after it, it requires an extra move. > > Increas

Re: [PATCH][AArch64] Improve SHA1 scheduling

2016-11-02 Thread Andrew Pinski
On Tue, Oct 25, 2016 at 10:08 AM, Wilco Dijkstra wrote: > SHA1H instructions may be scheduled after a SHA1C instruction > that uses the same input register. However SHA1C updates its input, > so if SHA1H is scheduled after it, it requires an extra move. > Increase the priority of SHA1H to ensure

Re: [PATCH][AArch64] Improve SHA1 scheduling

2016-11-02 Thread Wilco Dijkstra
ping From: Wilco Dijkstra Sent: 25 October 2016 18:08 To: GCC Patches Cc: nd Subject: [PATCH][AArch64] Improve SHA1 scheduling   SHA1H instructions may be scheduled after a SHA1C instruction that uses the same input register.  However SHA1C updates its input, so if SHA1H is scheduled after

[PATCH][AArch64] Improve SHA1 scheduling

2016-10-25 Thread Wilco Dijkstra
SHA1H instructions may be scheduled after a SHA1C instruction that uses the same input register. However SHA1C updates its input, so if SHA1H is scheduled after it, it requires an extra move. Increase the priority of SHA1H to ensure it gets scheduled earlier, avoiding the move. Is this something