Re: [PATCH]rs6000: Load high and low part of 64bit constant independently

2022-11-27 Thread Jiufu Guo via Gcc-patches
Hi Kewen/Segher, Thanks a lot for your review! I updated the patch accordingly as below for message/code/testcase: For a complicate 64bit constant, blow is one instruction-sequence to build: lis 9,0x800a ori 9,9,0xabcd sldi 9,9,32 oris 9,9,0xc167 ori

Re: [PATCH]rs6000: Load high and low part of 64bit constant independently

2022-11-27 Thread Kewen.Lin via Gcc-patches
Hi Segher, on 2022/11/25 23:46, Segher Boessenkool wrote: > Hi! > > On Fri, Nov 25, 2022 at 09:21:21PM +0800, Jiufu Guo wrote: >> "Kewen.Lin" writes: >>> on 2022/9/15 16:30, Jiufu Guo wrote: For a complicate 64bit constant, blow is one instruction-sequence to build: lis

Re: [PATCH]rs6000: Load high and low part of 64bit constant independently

2022-11-27 Thread Jiufu Guo via Gcc-patches
Hi Segher! Thanks for your helpful comments! Segher Boessenkool writes: > Hi! > > On Fri, Nov 25, 2022 at 09:21:21PM +0800, Jiufu Guo wrote: >> "Kewen.Lin" writes: >> > on 2022/9/15 16:30, Jiufu Guo wrote: >> >> For a complicate 64bit constant, blow is one instruction-sequence to >> >>

Re: [PATCH]rs6000: Load high and low part of 64bit constant independently

2022-11-25 Thread Segher Boessenkool
Hi! On Fri, Nov 25, 2022 at 09:21:21PM +0800, Jiufu Guo wrote: > "Kewen.Lin" writes: > > on 2022/9/15 16:30, Jiufu Guo wrote: > >> For a complicate 64bit constant, blow is one instruction-sequence to > >> build: > >>lis 9,0x800a > >>ori 9,9,0xabcd > >>sldi 9,9,32 > >>oris

Re: [PATCH]rs6000: Load high and low part of 64bit constant independently

2022-11-25 Thread Jiufu Guo via Gcc-patches
Jiufu Guo via Gcc-patches writes: > Hi Kewen, > > Thanks for your review on this patch! > > "Kewen.Lin" writes: > >> Hi Jeff, >> >> Sorry for the late review. >> >> on 2022/9/15 16:30, Jiufu Guo wrote: >>> Hi, >>> >>> For a complicate 64bit constant, blow is one instruction-sequence to >>>

Re: [PATCH]rs6000: Load high and low part of 64bit constant independently

2022-11-25 Thread Jiufu Guo via Gcc-patches
Hi Kewen, Thanks for your review on this patch! "Kewen.Lin" writes: > Hi Jeff, > > Sorry for the late review. > > on 2022/9/15 16:30, Jiufu Guo wrote: >> Hi, >> >> For a complicate 64bit constant, blow is one instruction-sequence to >> build: >> lis 9,0x800a >> ori 9,9,0xabcd >>

Re: [PATCH]rs6000: Load high and low part of 64bit constant independently

2022-11-25 Thread Kewen.Lin via Gcc-patches
Hi Jeff, Sorry for the late review. on 2022/9/15 16:30, Jiufu Guo wrote: > Hi, > > For a complicate 64bit constant, blow is one instruction-sequence to > build: > lis 9,0x800a > ori 9,9,0xabcd > sldi 9,9,32 > oris 9,9,0xc167 > ori 9,9,0xfa16 > > while we can also

[PATCH]rs6000: Load high and low part of 64bit constant independently

2022-09-15 Thread Jiufu Guo via Gcc-patches
Hi, For a complicate 64bit constant, blow is one instruction-sequence to build: lis 9,0x800a ori 9,9,0xabcd sldi 9,9,32 oris 9,9,0xc167 ori 9,9,0xfa16 while we can also use below sequence to build: lis 9,0xc167 lis 10,0x800a ori