Re: [PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-05-03 Thread Alex Coplan
On 22/04/2024 13:01, Ajit Agarwal wrote: > Hello Alex: > > On 14/04/24 10:29 pm, Ajit Agarwal wrote: > > Hello Alex: > > > > On 12/04/24 11:02 pm, Ajit Agarwal wrote: > >> Hello Alex: > >> > >> On 12/04/24 8:15 pm, Alex Coplan wrote: > >>> On 12/04/2024 20:02, Ajit Agarwal wrote: > Hello

Re: [PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-04-22 Thread Ajit Agarwal
Hello Alex: On 14/04/24 10:29 pm, Ajit Agarwal wrote: > Hello Alex: > > On 12/04/24 11:02 pm, Ajit Agarwal wrote: >> Hello Alex: >> >> On 12/04/24 8:15 pm, Alex Coplan wrote: >>> On 12/04/2024 20:02, Ajit Agarwal wrote: Hello Alex: On 11/04/24 7:55 pm, Alex Coplan wrote: > On

Re: [PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-04-14 Thread Ajit Agarwal
Hello Alex: On 12/04/24 11:02 pm, Ajit Agarwal wrote: > Hello Alex: > > On 12/04/24 8:15 pm, Alex Coplan wrote: >> On 12/04/2024 20:02, Ajit Agarwal wrote: >>> Hello Alex: >>> >>> On 11/04/24 7:55 pm, Alex Coplan wrote: On 10/04/2024 23:48, Ajit Agarwal wrote: > Hello Alex: > >

Re: [PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-04-12 Thread Ajit Agarwal
Hello Alex: On 12/04/24 8:15 pm, Alex Coplan wrote: > On 12/04/2024 20:02, Ajit Agarwal wrote: >> Hello Alex: >> >> On 11/04/24 7:55 pm, Alex Coplan wrote: >>> On 10/04/2024 23:48, Ajit Agarwal wrote: Hello Alex: On 10/04/24 7:52 pm, Alex Coplan wrote: > Hi Ajit, > > On

Re: [PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-04-12 Thread Alex Coplan
On 12/04/2024 20:02, Ajit Agarwal wrote: > Hello Alex: > > On 11/04/24 7:55 pm, Alex Coplan wrote: > > On 10/04/2024 23:48, Ajit Agarwal wrote: > >> Hello Alex: > >> > >> On 10/04/24 7:52 pm, Alex Coplan wrote: > >>> Hi Ajit, > >>> > >>> On 10/04/2024 15:31, Ajit Agarwal wrote: > Hello Alex:

Re: [PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-04-12 Thread Ajit Agarwal
Hello Alex: On 11/04/24 7:55 pm, Alex Coplan wrote: > On 10/04/2024 23:48, Ajit Agarwal wrote: >> Hello Alex: >> >> On 10/04/24 7:52 pm, Alex Coplan wrote: >>> Hi Ajit, >>> >>> On 10/04/2024 15:31, Ajit Agarwal wrote: Hello Alex: On 10/04/24 1:42 pm, Alex Coplan wrote: > Hi

Re: [PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-04-11 Thread Alex Coplan
On 10/04/2024 23:48, Ajit Agarwal wrote: > Hello Alex: > > On 10/04/24 7:52 pm, Alex Coplan wrote: > > Hi Ajit, > > > > On 10/04/2024 15:31, Ajit Agarwal wrote: > >> Hello Alex: > >> > >> On 10/04/24 1:42 pm, Alex Coplan wrote: > >>> Hi Ajit, > >>> > >>> On 09/04/2024 20:59, Ajit Agarwal wrote:

Re: [PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-04-10 Thread Ajit Agarwal
Hello Alex: On 10/04/24 7:52 pm, Alex Coplan wrote: > Hi Ajit, > > On 10/04/2024 15:31, Ajit Agarwal wrote: >> Hello Alex: >> >> On 10/04/24 1:42 pm, Alex Coplan wrote: >>> Hi Ajit, >>> >>> On 09/04/2024 20:59, Ajit Agarwal wrote: Hello Alex: On 09/04/24 8:39 pm, Alex Coplan

Re: [PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-04-10 Thread Alex Coplan
Hi Ajit, On 10/04/2024 15:31, Ajit Agarwal wrote: > Hello Alex: > > On 10/04/24 1:42 pm, Alex Coplan wrote: > > Hi Ajit, > > > > On 09/04/2024 20:59, Ajit Agarwal wrote: > >> Hello Alex: > >> > >> On 09/04/24 8:39 pm, Alex Coplan wrote: > >>> On 09/04/2024 20:01, Ajit Agarwal wrote: >

Re: [PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-04-10 Thread Ajit Agarwal
Hello Alex: On 10/04/24 1:42 pm, Alex Coplan wrote: > Hi Ajit, > > On 09/04/2024 20:59, Ajit Agarwal wrote: >> Hello Alex: >> >> On 09/04/24 8:39 pm, Alex Coplan wrote: >>> On 09/04/2024 20:01, Ajit Agarwal wrote: Hello Alex: On 09/04/24 7:29 pm, Alex Coplan wrote: > On

Re: [PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-04-10 Thread Alex Coplan
Hi Ajit, On 09/04/2024 20:59, Ajit Agarwal wrote: > Hello Alex: > > On 09/04/24 8:39 pm, Alex Coplan wrote: > > On 09/04/2024 20:01, Ajit Agarwal wrote: > >> Hello Alex: > >> > >> On 09/04/24 7:29 pm, Alex Coplan wrote: > >>> On 09/04/2024 17:30, Ajit Agarwal wrote: > > > On

Re: [PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-04-09 Thread Ajit Agarwal
Hello Alex: On 09/04/24 8:39 pm, Alex Coplan wrote: > On 09/04/2024 20:01, Ajit Agarwal wrote: >> Hello Alex: >> >> On 09/04/24 7:29 pm, Alex Coplan wrote: >>> On 09/04/2024 17:30, Ajit Agarwal wrote: On 05/04/24 10:03 pm, Alex Coplan wrote: > On 05/04/2024 13:53, Ajit Agarwal

Re: [PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-04-09 Thread Alex Coplan
On 09/04/2024 20:01, Ajit Agarwal wrote: > Hello Alex: > > On 09/04/24 7:29 pm, Alex Coplan wrote: > > On 09/04/2024 17:30, Ajit Agarwal wrote: > >> > >> > >> On 05/04/24 10:03 pm, Alex Coplan wrote: > >>> On 05/04/2024 13:53, Ajit Agarwal wrote: > Hello Alex/Richard: > > All

Re: [PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-04-09 Thread Ajit Agarwal
Hello Alex: On 09/04/24 7:29 pm, Alex Coplan wrote: > On 09/04/2024 17:30, Ajit Agarwal wrote: >> >> >> On 05/04/24 10:03 pm, Alex Coplan wrote: >>> On 05/04/2024 13:53, Ajit Agarwal wrote: Hello Alex/Richard: All review comments are incorporated. >>> >>> Thanks, I was kind-of

Re: [PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-04-09 Thread Alex Coplan
On 09/04/2024 17:30, Ajit Agarwal wrote: > > > On 05/04/24 10:03 pm, Alex Coplan wrote: > > On 05/04/2024 13:53, Ajit Agarwal wrote: > >> Hello Alex/Richard: > >> > >> All review comments are incorporated. > > > > Thanks, I was kind-of expecting you to also send the renaming patch as a > >

Re: [PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-04-09 Thread Ajit Agarwal
On 05/04/24 10:03 pm, Alex Coplan wrote: > On 05/04/2024 13:53, Ajit Agarwal wrote: >> Hello Alex/Richard: >> >> All review comments are incorporated. > > Thanks, I was kind-of expecting you to also send the renaming patch as a > preparatory patch as we discussed. > > Sorry for another meta

Re: [PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-04-05 Thread Alex Coplan
On 05/04/2024 13:53, Ajit Agarwal wrote: > Hello Alex/Richard: > > All review comments are incorporated. Thanks, I was kind-of expecting you to also send the renaming patch as a preparatory patch as we discussed. Sorry for another meta comment, but: I think the reason that the Linaro CI isn't

[PATCH V4 1/3] aarch64: Place target independent and dependent changed code in one file

2024-04-05 Thread Ajit Agarwal
Hello Alex/Richard: All review comments are incorporated. Common infrastructure of load store pair fusion is divided into target independent and target dependent changed code. Target independent code is the Generic code with pure virtual function to interface betwwen target independent and