Re: [ARC PATCH] Split SImode shifts pre-reload on !TARGET_BARREL_SHIFTER.

2023-10-04 Thread Claudiu Zissulescu Ianculescu
Hi Roger, The patch as it is passed the validation, and it is in general OK. Although it doesn't address the elephant in the room, namely output_shift function, it is a welcome cleanup. I would like you to split the patch in two. One which deals with improvements on shifts in absence of a barrel

Re: [ARC PATCH] Split SImode shifts pre-reload on !TARGET_BARREL_SHIFTER.

2023-10-03 Thread Claudiu Zissulescu Ianculescu
ger Sayle ; gcc-patches@gcc.gnu.org > > Subject: RE: [ARC PATCH] Split SImode shifts pre-reload on > > !TARGET_BARREL_SHIFTER. > > > > Hi Roger, > > > > It was nice to meet you too. > > > > Thank you in looking into the ARC's non-Barrel Shi

RE: [ARC PATCH] Split SImode shifts pre-reload on !TARGET_BARREL_SHIFTER.

2023-10-03 Thread Roger Sayle
-- > -Original Message- > From: Claudiu Zissulescu > Sent: 03 October 2023 15:26 > To: Roger Sayle ; gcc-patches@gcc.gnu.org > Subject: RE: [ARC PATCH] Split SImode shifts pre-reload on > !TARGET_BARREL_SHIFTER. > > Hi Roger, > > It was nice to meet you too. > >

RE: [ARC PATCH] Split SImode shifts pre-reload on !TARGET_BARREL_SHIFTER.

2023-10-03 Thread Claudiu Zissulescu
Hi Roger, It was nice to meet you too. Thank you in looking into the ARC's non-Barrel Shifter configurations. I will dive into your patch asap, but before starting here are a few of my comments: -Original Message- From: Roger Sayle Sent: Thursday, September 28, 2023 2:27 PM To: