Hello Alex:
On 16/05/24 10:21 pm, Alex Coplan wrote:
> Hi Ajit,
>
> Thanks a lot for working through the review feedback.
>
> The patch LGTM with the two minor suggested changes below. I can't
> approve the patch, though, so you'll need an OK from Richard S.
>
> Also, I'm not sure if it makes
Hello Richard:
On 17/05/24 11:07 pm, Richard Sandiford wrote:
> Ajit Agarwal writes:
>> Hello Alex/Richard:
>>
>> All review comments are addressed.
>>
>> Common infrastructure of load store pair fusion is divided into target
>> independent and target dependent changed code.
>>
>> Target
Ajit Agarwal writes:
> Hello Alex/Richard:
>
> All review comments are addressed.
>
> 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
Hello Alex:
On 17/05/24 6:22 pm, Alex Coplan wrote:
> Hi Ajit,
>
> On 17/05/2024 18:05, Ajit Agarwal wrote:
>> Hello Alex:
>>
>> On 16/05/24 10:21 pm, Alex Coplan wrote:
>>> Hi Ajit,
>>>
>>> Thanks a lot for working through the review feedback.
>>>
>>
>> Thanks a lot for reviewing the code and
Hi Ajit,
On 17/05/2024 18:05, Ajit Agarwal wrote:
> Hello Alex:
>
> On 16/05/24 10:21 pm, Alex Coplan wrote:
> > Hi Ajit,
> >
> > Thanks a lot for working through the review feedback.
> >
>
> Thanks a lot for reviewing the code and approving the patch.
To be clear, I didn't approve the patch
Hello Alex:
On 16/05/24 10:21 pm, Alex Coplan wrote:
> Hi Ajit,
>
> Thanks a lot for working through the review feedback.
>
Thanks a lot for reviewing the code and approving the patch.
> The patch LGTM with the two minor suggested changes below. I can't
> approve the patch, though, so you'll
Hi Ajit,
Thanks a lot for working through the review feedback.
The patch LGTM with the two minor suggested changes below. I can't
approve the patch, though, so you'll need an OK from Richard S.
Also, I'm not sure if it makes sense to apply the patch in isolation, it
might make more sense to
Hello Alex/Richard:
All review comments are addressed.
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 between target independent and