Re: [PATCH] loading float member of parameter stored via int registers

2022-12-29 Thread Segher Boessenkool
Hi! On Fri, Dec 30, 2022 at 10:22:31AM +0800, Jiufu Guo wrote: > Considering the limitations of CSE, I try to find other places > to handle this issue, and notice DSE can optimize below code: > "[sfp:DI]=x:DI ; y:SI=[sfp:DI]" to "y:SI=x:DI#0". > > So, I drafted a patch to update DSE to handle DI-

Re: [PATCH v4] LoongArch: Fixed a compilation failure with '%c' in inline assembly [PR107731].

2022-12-29 Thread Lulu Cheng
Hi, Richard:  Could you please help me look at this document? Is there any problem with my modification? Thanks! 在 2022/12/27 下午2:42, Lulu Cheng 写道: Co-authored-by: Yang Yujie gcc/ChangeLog: * config/loongarch/loongarch.cc (loongarch_classify_address): Add precessint for

Re: [PATCH] loading float member of parameter stored via int registers

2022-12-29 Thread Jiufu Guo via Gcc-patches
Hi, Jiufu Guo via Gcc-patches writes: > Hi, > > Jiufu Guo via Gcc-patches writes: > >> Hi, >> >> Segher Boessenkool writes: >> >>> On Fri, Dec 23, 2022 at 08:13:48PM +0100, Richard Biener wrote: > Am 23.12.2022 um 17:55 schrieb Segher Boessenkool > : > There are at least six ve

Re: [PATCH] i386: correct division modeling in lujiazui.md

2022-12-29 Thread Mayshao-oc
>Ping. If there are any questions or concerns about the patch, please let me >know: I'm interested in continuing this cleanup at least for older AMD models. > Hi Alexander: According to the speccpu2017 benchmark result, the patch looks good in lujiazui. BR Mayshao >I noticed I had an ext

Re: [PATCH] RISC-V: Optimize min/max with SImode sources on 64-bit

2022-12-29 Thread Jeff Law via Gcc-patches
On 12/29/22 07:50, Philipp Tomsich wrote: We have two issues around min/max here: 1. That it doesn't apply to the SImode abs case (which is due to expand_abs_nojump() blindly testing for the current mode in smax_optab). Mode testing is inherent in the optab query interface. 2. That we h

[COMMITTED] gcc: xtensa: use GP_RETURN_* instead of magic constant

2022-12-29 Thread Max Filippov via Gcc-patches
gcc/ * config/xtensa/xtensa.cc (xtensa_return_in_memory): Use GP_RETURN_* instead of magic constant. --- gcc/config/xtensa/xtensa.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/config/xtensa/xtensa.cc b/gcc/config/xtensa/xtensa.cc index e726a115029f..ae4

Re: [PATCH] xtensa: Check DF availability before use

2022-12-29 Thread Max Filippov via Gcc-patches
On Thu, Dec 29, 2022 at 4:33 AM Takayuki 'January June' Suwa wrote: > > Parhaps no problem, but for safety. > > gcc/ChangeLog: > > * config/xtensa/xtensa.cc (xtensa_expand_prologue): Fix to check > DF availability before use of DF_* macros. > --- > gcc/config/xtensa/xtensa.cc | 2

[PATCH] RISC-V: Fix inferior codegen for vse intrinsics.

2022-12-29 Thread juzhe . zhong
From: Ju-Zhe Zhong Currently we use pred_mov to to do the codegen for vse intrinsics. However, it generates inferior codegen when I am testing AVL model of VSETVL PASS using vse intrinsics. Consider this following code: void f2 (int * restrict in, int * restrict out, void * restrict mask_in, in

Re: [PATCH] RISC-V: Optimize min/max with SImode sources on 64-bit

2022-12-29 Thread Philipp Tomsich
On Wed, 28 Dec 2022 at 19:18, Raphael Moreira Zinsly < rzin...@ventanamicro.com> wrote: > The Zbb min/max pattern was not matching 32-bit sources when > compiling for 64-bit. > This patch separates the pattern into SImode and DImode, and > use a define_expand to handle SImode on 64-bit. > zbb-min-

Re: [PATCH] RISC-V: Optimize min/max with SImode sources on 64-bit

2022-12-29 Thread Jeff Law via Gcc-patches
On 12/29/22 05:23, Raphael Zinsly wrote: On Wed, Dec 28, 2022 at 10:36 PM Jeff Law wrote: On 12/28/22 11:18, Raphael Moreira Zinsly wrote: The Zbb min/max pattern was not matching 32-bit sources when compiling for 64-bit. This patch separates the pattern into SImode and DImode, and use a

[PATCH] xtensa: Check DF availability before use

2022-12-29 Thread Takayuki 'January June' Suwa via Gcc-patches
Parhaps no problem, but for safety. gcc/ChangeLog: * config/xtensa/xtensa.cc (xtensa_expand_prologue): Fix to check DF availability before use of DF_* macros. --- gcc/config/xtensa/xtensa.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/config/xtensa/xte

Re: [PATCH] RISC-V: Optimize min/max with SImode sources on 64-bit

2022-12-29 Thread Raphael Zinsly
On Wed, Dec 28, 2022 at 10:36 PM Jeff Law wrote: > > > > On 12/28/22 11:18, Raphael Moreira Zinsly wrote: > > The Zbb min/max pattern was not matching 32-bit sources when > > compiling for 64-bit. > > This patch separates the pattern into SImode and DImode, and > > use a define_expand to handle SI

Re: [PATCH modula2] PR-108142 Remove empty directories created in the build directory

2022-12-29 Thread Gaius Mulley via Gcc-patches
Iain Sandoe writes: > Hi Gaius, > > I’m finding it hard to figure out how the configuration is supposed to work > for cross compilers (and more so for Canadian/native crosses). > > Apologies if this is already answered in other review (but maybe some notes > would help in that case). > > questi