[PATCH 1/3] lto-plugin: support LDPT_GET_SYMBOLS_V3

2022-06-15 Thread Martin Liška
That supports skipping of an object file (LDPS_NO_SYMS). lto-plugin/ChangeLog: * lto-plugin.c (struct plugin_file_info): Add skip_file flag. (write_resolution): Write resolution only if get_symbols != LDPS_NO_SYMS. (all_symbols_read_handler): Ignore file if skip_file is t

[PATCH][pushed] clang: fix -Wunused-parameter warning

2022-06-15 Thread Martin Liška
Fixes: gcc/cp/decl2.cc:158:54: warning: unused parameter 'entry' [-Wunused-parameter] --- gcc/cp/decl2.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/cp/decl2.cc b/gcc/cp/decl2.cc index a8f3e6e5fba..0c4492f7354 100644 --- a/gcc/cp/decl2.cc +++ b/gcc/cp/decl2.cc @@ -155,

[PATCH, rs6000] Use CC for BCD operations [PR100736]

2022-06-15 Thread HAO CHEN GUI via Gcc-patches
Hi, This patch uses CC instead of CCFP for all BCD operations. Thus, infinite math flag has no impact on BCD operations. To support BCD overflow and invalid coding, ordered and unordered are added into CC mode. With CC, "ge" and "le" are converted to reverse comparison. So the invalid coding need

Re: [PATCH] gengtype: do not skip char after escape sequnce

2022-06-15 Thread Martin Liška
On 6/1/22 17:14, Jeff Law via Gcc-patches wrote: > > > On 5/4/2022 1:14 PM, Martin Liška wrote: >> Right now, when a \$x escape sequence occures, the >> next character after $x is skipped, which is bogus. >> >> The code has very low coverage right now. >> >> Patch can bootstrap on x86_64-linux-gn

Re: [PATCH] Add operators / and * for profile_{count,probability}.

2022-06-15 Thread Martin Liška
PING^2 On 5/24/22 13:35, Martin Liška wrote: > PING^1 > > On 5/5/22 20:15, Martin Liška wrote: >> On 5/5/22 15:49, Jan Hubicka wrote: >>> Hi, The patch simplifies usage of the profile_{count,probability} types. Patch can bootstrap on x86_64-linux-gnu and survives regression tests.

Re: [PATCH] ipa-icf: skip variables with body_removed

2022-06-15 Thread Martin Liška
@Honza: PING On 5/20/22 09:46, Martin Liška wrote: > On 5/19/22 17:02, Jan Hubicka wrote: >>> Similarly to cgraph_nodes, it may happen that body_removed is set >>> during merging of symbols. >>> >>> PR ipa/105600 >>> >>> Patch can bootstrap on x86_64-linux-gnu and survives regression tests. >>

[PATCH][wwwdocs] gcc-13: add arm star-mc1 cpu

2022-06-15 Thread Chung-Ju Wu via Gcc-patches
Hi, Recently we added arm star-mc1 cpu support to upstream: https://gcc.gnu.org/pipermail/gcc-patches/2022-June/596379.html It would be great if we can describe it on gcc-13 changes.html as well. Attached is the patch for gcc-wwwdocs repository. Is it OK? Regards, jasonwucjFrom 379e1b67b92beca

Re: [PATCH v2 2/2] riscv-cores.def: Add T-Head XuanTie C906

2022-06-15 Thread Kito Cheng via Gcc-patches
LGTM, thanks! On Wed, Jun 15, 2022 at 7:48 PM Christoph Muellner wrote: > > From: Christoph Müllner > > This adds T-Head's XuanTie C906 to the list of known cores as "thead-c906". > The C906 is shipped for quite some time (it is the core of the Allwinner D1). > Note, that the tuning struct for t

Re: [PATCH v2 1/2] riscv-cores.def: Fix description of RISCV_CORE() macro

2022-06-15 Thread Kito Cheng via Gcc-patches
LGTM, thanks for correcting the comments ! On Wed, Jun 15, 2022 at 7:47 PM Christoph Muellner wrote: > > From: Christoph Müllner > > The current description of RISCV_CORE() does not match the > implementation. This commit provides a fix for that. > > gcc/ChangeLog: > > * config/riscv/ris

Re: [PATCH] RISC-V/testsuite: Fix pr105666.c under rv32

2022-06-15 Thread Kito Cheng via Gcc-patches
Committed with a minor commit log fix, thanks! > > In rv32 regression test, this cases will report an error: > > "cc1: error: ABI requires '-march=rv32'" > > Add '-mabi' option will fix this. > > gcc/testsuite/ChangeLog: > > * gcc.target/riscv/pr105666.c: New options. ^^^ here should be a

Re: [PATCH 1/5] xtensa: Document new -mextra-l32r-costs= Xtensa-specific option

2022-06-15 Thread Max Filippov via Gcc-patches
On Mon, Jun 13, 2022 at 8:54 PM Takayuki 'January June' Suwa wrote: > > > gcc/ChangeLog: > * doc/invoke.texi: Document -mextra-l32r-costs= option. > --- > gcc/doc/invoke.texi | 11 ++- > 1 file changed, 10 insertions(+), 1 deletion(-) Committed to master. -- Thanks. -- Max

Re: [PATCH 5/5] xtensa: Eliminate [DS]Cmode hard register clobber that is immediately followed by whole overwrite the register

2022-06-15 Thread Max Filippov via Gcc-patches
On Mon, Jun 13, 2022 at 8:54 PM Takayuki 'January June' Suwa wrote: > > RTL expansion of substitution to [DS]Cmode hard register includes obstructive > register clobber. > > A simplest example: > > double _Complex test(double _Complex c) { > return c; > } > > will be converted to: >

Re: [PATCH 4/5] xtensa: Eliminate unwanted reg-reg moves during DFmode input reloads

2022-06-15 Thread Max Filippov via Gcc-patches
On Mon, Jun 13, 2022 at 8:54 PM Takayuki 'January June' Suwa wrote: > > When spilled DFmode registers are reloaded in, once loaded into a pair of > SImode regs and then copied from that regs. Such unwanted reg-reg moves > seems not to be eliminated at the "cprop_hardreg" stage, despite no problem

Re: [PATCH 3/5] xtensa: Add some dedicated patterns that correspond to GIMPLE canonicalizations

2022-06-15 Thread Max Filippov via Gcc-patches
On Mon, Jun 13, 2022 at 8:54 PM Takayuki 'January June' Suwa wrote: > > This patch offers better RTL representations against straightforward > derivations from some tree optimizers' canonicalized forms. > > - rounding up to even, such as '(x + (x & 1))', is canonicalized to > '((x + 1) & -2)', b

Re: [PATCH v2 2/5] xtensa: Add support for sibling call optimization

2022-06-15 Thread Max Filippov via Gcc-patches
On Wed, Jun 15, 2022 at 5:23 AM Takayuki 'January June' Suwa wrote: > > On 2022/06/15 5:17, Max Filippov wrote: > > Hi Suwa-san, > hi! > > > This change results in a bunch of new regression test failures: > > The code generated for e.g. gcc.c-torture/execute/921208-2.c looks like > > this: > oh,

Re: [PATCH] Add -fextra-libc-function=memcmpeq for __memcmpeq

2022-06-15 Thread Fangrui Song via Gcc-patches
On Wed, Jun 15, 2022 at 2:44 PM H.J. Lu via Gcc-patches wrote: > > On Mon, Jun 13, 2022 at 9:01 AM Richard Biener > wrote: > > > > > > > > > Am 13.06.2022 um 16:36 schrieb H.J. Lu : > > > > > > On Mon, Jun 13, 2022 at 3:11 AM Richard Biener > > > wrote: > > >> > > >>> On Tue, Jun 7, 2022 at 9:0

Re: [PATCH] RISC-V/testsuite: Fix pr105666.c under rv32

2022-06-15 Thread Palmer Dabbelt
On Wed, 15 Jun 2022 10:35:36 PDT (-0700), Vineet Gupta wrote: Hi Wei, On 6/8/22 02:35, jiawei wrote: From: Jia-wei Chen In rv32 regression test, this cases will report an error: "cc1: error: ABI requires '-march=rv32'" Add '-mabi' option will fix this. gcc/testsuite/ChangeLog: *

Re: [PATCH 0/9] Add debug_annotate attributes

2022-06-15 Thread Yonghong Song via Gcc-patches
On 6/15/22 1:57 PM, David Faust wrote: On 6/14/22 22:53, Yonghong Song wrote: On 6/7/22 2:43 PM, David Faust wrote: Hello, This patch series adds support for: - Two new C-language-level attributes that allow to associate (to "annotate" or to "tag") particular declarations and types

[committed] analyzer: fix up paths for inlining (PR analyzer/105962)

2022-06-15 Thread David Malcolm via Gcc-patches
-fanalyzer runs late compared to other code analysis tools, in that in runs on the partially-optimized gimple-ssa representation. I chose this point to run in the hope of easy integration with LTO. As PR analyzer/105962 notes, this means that function inlining can occur before the -fanalyzer "see

Re: PING Re: [PATCH 07/10] value-relation.h: add 'final' and 'override' to relation_oracle vfunc impls

2022-06-15 Thread David Malcolm via Gcc-patches
On Wed, 2022-06-15 at 09:33 -0400, Andrew MacLeod wrote: > On 6/13/22 21:24, David Malcolm wrote: > > On Mon, 2022-06-13 at 20:45 -0400, Aldy Hernandez wrote: > > > Final implies we can't further derive from the derived class, > > > right? > > "final" on a vfunc implies that nothing overrides that

[committed] analyzer: show saved diagnostics as nodes in .eg.dot dumps

2022-06-15 Thread David Malcolm via Gcc-patches
I've been using this tweak to the output of -fdump-analyzer-exploded-graph in my working copies for a while; the extra red nodes make it *much* easier to find the places where diagnostics are being emitted (or rejected by the diagnostic_manager). Successfully bootstrapped & regrtested on x86_64-pc

[committed] analyzer: add more uninit test coverage

2022-06-15 Thread David Malcolm via Gcc-patches
Tested on x86_64-pc-linux-gnu. Pushed to trunk as r13-1116-g44681d45473883. gcc/testsuite/ChangeLog: * gcc.dg/analyzer/uninit-1.c: Add test coverage of attempts to jump through an uninitialized function pointer, and of attempts to pass an uninitialized value to a function

Re: [PATCH] Add -fextra-libc-function=memcmpeq for __memcmpeq

2022-06-15 Thread H.J. Lu via Gcc-patches
On Mon, Jun 13, 2022 at 9:01 AM Richard Biener wrote: > > > > > Am 13.06.2022 um 16:36 schrieb H.J. Lu : > > > > On Mon, Jun 13, 2022 at 3:11 AM Richard Biener > > wrote: > >> > >>> On Tue, Jun 7, 2022 at 9:02 PM H.J. Lu via Gcc-patches > >>> wrote: > >>> > >>> Add -fextra-libc-function=memcmpe

nvptx: Allow '--with-arch' to override the default '-misa' (was: nvptx multilib setup)

2022-06-15 Thread Thomas Schwinge
Hi Tom! On 2022-05-13T16:20:14+0200, I wrote: > On 2022-02-04T13:09:29+0100, Tom de Vries via Gcc wrote: >> On 2/4/22 08:21, Thomas Schwinge wrote: >>> On 2022-02-03T13:35:55+, "vries at gcc dot gnu.org via Gcc-bugs" >>> wrote: I've tested this using (recommended) driver 470.94 on boar

[committed] d: Add `@no_sanitize' attribute to compiler and library.

2022-06-15 Thread Iain Buclaw via Gcc-patches
Hi, This patch adds the `@no_sanitize' attribute to the D front-end. The `@no_sanitize` attribute disables a particular sanitizer for this function, analogous to `__attribute__((no_sanitize))'. The library also defines `@noSanitize' to be compatible with the LLVM D compiler's `ldc.attributes'.

Re: [committed] openmp: Conforming device numbers and omp_{initial,invalid}_device

2022-06-15 Thread Thomas Schwinge
Hi Jakub! On 2022-06-15T10:46:30+0200, Jakub Jelinek wrote: > On Tue, Jun 14, 2022 at 06:41:37PM +0200, Thomas Schwinge wrote: >> On 2022-06-13T14:06:39+0200, Jakub Jelinek via Gcc-patches >> wrote: >> > OpenMP 5.2 changed once more what device numbers are allowed. >> >> > libgomp/ >> >> >

Re: [PATCH 0/9] Add debug_annotate attributes

2022-06-15 Thread David Faust via Gcc-patches
On 6/14/22 22:53, Yonghong Song wrote: > > > On 6/7/22 2:43 PM, David Faust wrote: >> Hello, >> >> This patch series adds support for: >> >> - Two new C-language-level attributes that allow to associate (to "annotate" >> or >>to "tag") particular declarations and types with arbitrary stri

Re: [PATCH RFA] ubsan: default to trap on unreachable at -O0 and -Og [PR104642]

2022-06-15 Thread Jason Merrill via Gcc-patches
On 6/14/22 07:44, Jakub Jelinek wrote: On Mon, Jun 13, 2022 at 03:53:13PM -0400, Jason Merrill via Gcc-patches wrote: When not optimizing, we can't do anything useful with unreachability in terms of code performance, so we might as well improve debugging by turning __builtin_unreachable into a t

[PATCH] Fortran: check POS and LEN arguments simplifying bit intrinsics [PR105986]

2022-06-15 Thread Harald Anlauf via Gcc-patches
Dear all, we need to check the POS (and LEN) arguments of bit intrinsics when simplifying, e.g. when used in array constructors. Otherwise we ICE. Found by Gerhard. The fix is straightforward, see attached. Regtested on x86_64-pc-linux-gnu. OK for mainline? Thanks, Harald From 32c95012378ada

Re: [PATCH] libcpp: Handle extended characters in user-defined literal suffix [PR103902]

2022-06-15 Thread Lewis Hyatt via Gcc-patches
On Tue, Jun 14, 2022 at 05:26:49PM -0400, Lewis Hyatt wrote: > Hello- > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103902 > > The attached patch resolves PR preprocessor/103902 as described in the patch > message inline below. bootstrap + regtest all languages was successful on > x86-64 Linux

[committed] d: Add `@visibility' and `@hidden' attributes.

2022-06-15 Thread Iain Buclaw via Gcc-patches
Hi, This patch adds the visibility attribute to the D compiler, and library helpers `@visibility' and `@hidden' to the run-time library. The `@visibility' attribute is functionality the same as `__attribute__((visibility))', and `@hidden' is a convenience alias to `@visibility("hidden")' defined

Re: [PATCH V2]rs6000: Store complicated constant into pool

2022-06-15 Thread Segher Boessenkool
Hi! On Wed, Jun 15, 2022 at 04:19:41PM +0800, Jiufu Guo wrote: > Segher Boessenkool writes: > > Have you tried with different limits? > I drafted cases(C code, and updated asm code) to test runtime costs for > different code sequences: building constants with 5 insns; with 3 > insns and 2 insns;

Re: [PATCH] RISC-V/testsuite: Fix pr105666.c under rv32

2022-06-15 Thread Vineet Gupta
Hi Wei, On 6/8/22 02:35, jiawei wrote: From: Jia-wei Chen In rv32 regression test, this cases will report an error: "cc1: error: ABI requires '-march=rv32'" Add '-mabi' option will fix this. gcc/testsuite/ChangeLog: * gcc.target/riscv/pr105666.c: New options. --- gcc/testsuite/

[pushed] Revert recent internal-fn changes [PR105975]

2022-06-15 Thread Richard Sandiford via Gcc-patches
The recent internal-fn “clean-ups” triggered problems on nvptx because some of the omp_simt_* patterns had modeless operands. I wondered about adapting expand_fn_using_insn to cope with that, but then the problem becomes: what should the mode of operand 0 be when there is no lhs? The answer depend

[committed] arm: big-endian issue in gen_cpymem_ldrd_strd [PR105981]

2022-06-15 Thread Richard Earnshaw via Gcc-patches
The code in gen_cpymem_ldrd_strd has been incorrect for big-endian since r230663. The problem is that we use gen_lowpart, etc. to split the 64-bit quantity, but fail to account for the fact that these routines are really dealing with 64-bit /values/ and in big-endian the ordering of the sub-regis

c++: Use better module partition naming

2022-06-15 Thread Nathan Sidwell via Gcc-patches
It turns out that 'implementation partition' is not a term used in the std, and is confusing to users. Let's use the better term 'internal partition'. While there, adjust header unit naming. nathan -- Nathan SidwellFrom 052d89537a4c09e1e1437042e2d1ea215656325f Mon Sep 17 00:00:00 2001 From:

Re: PING Re: [PATCH 07/10] value-relation.h: add 'final' and 'override' to relation_oracle vfunc impls

2022-06-15 Thread Andrew MacLeod via Gcc-patches
On 6/13/22 21:24, David Malcolm wrote: On Mon, 2022-06-13 at 20:45 -0400, Aldy Hernandez wrote: Final implies we can't further derive from the derived class, right? "final" on a vfunc implies that nothing overrides that vfunc, but you could still have further derived classes. You can think of

Re: [PATCH] Simplify (B * v + C) * D -> BD* v + CD when B, C, D are all INTEGER_CST.

2022-06-15 Thread Andrew MacLeod via Gcc-patches
On 6/15/22 04:24, Richard Biener wrote: diff --git a/gcc/match.pd b/gcc/match.pd index 44a385b912d..54f53a1f988 100644 --- a/gcc/match.pd +++ b/gcc/match.pd @@ -489,6 +489,88 @@ DEFINE_INT_AND_FLOAT_ROUND_FN (RINT) (if (!overflow || TYPE_OVERFLOW_WRAPS (type)) (mult @0 { wide_int_to_tr

[committed] arm: fix thinko in arm_bfi_1_p() [PR105974]

2022-06-15 Thread Richard Earnshaw via Gcc-patches
I clearly wasn't thinking straight when I wrote the arm_bfi_1_p function and used XUINT rather than UINTVAL when extracting CONST_INT values. It seemed to work in testing, but was incorrect and failed RTL checking. Fixed thusly: gcc/ChangeLog: PR target/105974 * config/arm/arm.

[PATCH v2 2/5] xtensa: Add support for sibling call optimization

2022-06-15 Thread Takayuki 'January June' Suwa via Gcc-patches
On 2022/06/15 5:17, Max Filippov wrote: > Hi Suwa-san, hi! > This change results in a bunch of new regression test failures: > The code generated for e.g. gcc.c-torture/execute/921208-2.c looks like this: oh, PICed... indirect (incl. via function pointer, virtual functions and of course PIC ones

[committed] d: Set TYPE_ARTIFICIAL on internal TypeInfo types

2022-06-15 Thread Iain Buclaw via Gcc-patches
Hi, This patch sets the TYPE_ARTIFICIAL flag on internally generated object.TypeInfo types, preventing them from triggering warnings when compiling with `-Wpadded'. Regression tested on x86_64-linux-gnu/-m32/-mx32, committed to mainline, and backported to the release/gcc-10, gcc-11, and gcc-12 br

[PATCH v2 2/2] riscv-cores.def: Add T-Head XuanTie C906

2022-06-15 Thread Christoph Muellner
From: Christoph Müllner This adds T-Head's XuanTie C906 to the list of known cores as "thead-c906". The C906 is shipped for quite some time (it is the core of the Allwinner D1). Note, that the tuning struct for the C906 is already part of GCC (it is also name "thead-c906"). gcc/ChangeLog:

[PATCH v2 1/2] riscv-cores.def: Fix description of RISCV_CORE() macro

2022-06-15 Thread Christoph Muellner
From: Christoph Müllner The current description of RISCV_CORE() does not match the implementation. This commit provides a fix for that. gcc/ChangeLog: * config/riscv/riscv-cores.def: Fix comment. Signed-off-by: Christoph Müllner --- gcc/config/riscv/riscv-cores.def | 12 +---

Re: [PATCH]middle-end Use subregs to expand COMPLEX_EXPR to set the lowpart.

2022-06-15 Thread Richard Sandiford via Gcc-patches
Jeff Law via Gcc-patches writes: > On 6/13/2022 5:54 AM, Richard Biener wrote: >> On Sun, Jun 12, 2022 at 7:27 PM Jeff Law via Gcc-patches >> wrote: >> [...] >>> On a related topic, any thoughts on keeping complex objects as complex >>> types/modes through gimple and into at least parts of the RT

[PATCH] tree-optimization/105969 - FPE with array diagnostics

2022-06-15 Thread Richard Biener via Gcc-patches
For a [0][0] array we have to be careful when dividing by the element size which is zero for the outermost dimension. Luckily the division is only for an overflow check which is pointless for array size zero. Bootstrapped and tested on x86_64-unknown-linux-gnu, pushed. 2022-06-15 Richard Biener

[PATCH] tree-optimization/105971 - less surprising refs_may_alias_p_2

2022-06-15 Thread Richard Biener via Gcc-patches
When DSE asks whether __real a is using __imag a it gets a surprising result when a is a FUNCTION_DECL. The following makes sure this case is less surprising to callers but keeping the bail-out for the non-decl case where it is true that PTA doesn't track aliases to code correctly. Bootstrapped a

[committed] d: Delay completing aggregate and enum types until after attributes have been applied.

2022-06-15 Thread Iain Buclaw via Gcc-patches
Hi, This patch rejigs the logic for completing D types with (eventually) recursive references to itself within its members. Because of these forward/recursive references, the TYPE_SIZE, TYPE_ALIGN, and TYPE_MODE of structs and enums were set before laying out its members. This adds a new macro T

[pushed] aarch64: Revert bogus fix for PR105254

2022-06-15 Thread Richard Sandiford via Gcc-patches
In f2ebf2d98efe0ac2314b58cf474f44cb8ebd5244 I'd forced the chosen unroll factor to be a factor of the VF, in order to work around an exact_div ICE in PR105254. This was completely bogus -- clearly I didn't look in enough detail at why we ended up with an unrolled VF that wasn't a multiple of the U

[pushed] gen: Allow unspec numbers in .md attributes

2022-06-15 Thread Richard Sandiford via Gcc-patches
Tamar pointed out that: (unspec:M ... ) didn't work when a value of attribute FOO was defined by define_constant, such as in: (define_int_attribute FOO [(UNSPEC_A "UNSPEC_B") ...]) This is because symbolic constants are substituted during lexing and only apply to bare symbol names, not stri

Re: [PATCH] libgompd: Fix sizes in OMPD support and add local ICVs finctions.

2022-06-15 Thread Mohamed Atef via Gcc-patches
Ping في الجمعة، ١٠ يونيو، ٢٠٢٢ ٥:٥٦ م Mohamed Atef كتب: > > libgomp/ChangeLog > > 2022-06-10 Mohamed Atef > > * ompd-helper.h (DEREFERENCE, ACCESS_VALUE): New macros. > * ompd-helper.c (gompd_get_nthread, gompd_get_thread_limit, > gomp_get_run_shed, gompd_get_run_sched_chunk_size, > gompd_get

Re: [PATCH] expand: Fix up IFN_ATOMIC_{BIT*,*CMP_0} expansion [PR105951]

2022-06-15 Thread Jakub Jelinek via Gcc-patches
On Wed, Jun 15, 2022 at 10:26:39AM +0200, Richard Biener wrote: > Ah, that might be a nice way to simplify the fallback indeed. > > > > The back-and-forth is a big ugly IMHO and it would suggest we can > > > always match to the IFN since we can fall back to the builtin > > > call as expansion...?

Re: [PATCH 2/2] riscv-cores.def: Add Allwinner D1 core

2022-06-15 Thread Philipp Tomsich
Please update the commit message to reflect this. On Wed, 15 Jun 2022 at 10:56, Christoph Müllner wrote: > > On Wed, Jun 15, 2022 at 10:39 AM Philipp Tomsich > wrote: > > > > On Wed, 15 Jun 2022 at 10:30, Christoph Müllner > > wrote: > > > > > > On Mon, Jun 13, 2022 at 3:20 PM Christoph Muellne

Re: [PATCH 2/2] riscv-cores.def: Add Allwinner D1 core

2022-06-15 Thread Christoph Müllner
On Wed, Jun 15, 2022 at 10:39 AM Philipp Tomsich wrote: > > On Wed, 15 Jun 2022 at 10:30, Christoph Müllner > wrote: > > > > On Mon, Jun 13, 2022 at 3:20 PM Christoph Muellner > > wrote: > > > > > > From: Christoph Müllner > > > > > > This adds Allwinner's D1 to the list of known cores. > > > T

Re: [committed] openmp: Conforming device numbers and omp_{initial,invalid}_device

2022-06-15 Thread Jakub Jelinek via Gcc-patches
On Tue, Jun 14, 2022 at 06:41:37PM +0200, Thomas Schwinge wrote: > Hi Jakub! > > On 2022-06-13T14:06:39+0200, Jakub Jelinek via Gcc-patches > wrote: > > OpenMP 5.2 changed once more what device numbers are allowed. > > > libgomp/ > > > * testsuite/libgomp.c-c++-common/target-is-accessibl

Re: [PATCH 2/2] riscv-cores.def: Add Allwinner D1 core

2022-06-15 Thread Philipp Tomsich
On Wed, 15 Jun 2022 at 10:30, Christoph Müllner wrote: > > On Mon, Jun 13, 2022 at 3:20 PM Christoph Muellner > wrote: > > > > From: Christoph Müllner > > > > This adds Allwinner's D1 to the list of known cores. > > The Allwinner includes a single-core XuanTie C906 and is available > > for quite

[PATCH] x86-64: Remove HAVE_LD_PIE_COPYRELOC

2022-06-15 Thread Fangrui Song via Gcc-patches
This was introduced in 2014-12 to use local binding for external symbols for -fPIE. It avoids a GOT indirection but the same optimizationis obtained with ld's R_X86_64_[REX_]GOTPCRELX optimization (albeit with slightly longer code). One design goal of -fPIE was to avoid copy relocations. HAVE_LD_

Re: [PATCH 2/2] riscv-cores.def: Add Allwinner D1 core

2022-06-15 Thread Christoph Müllner
On Mon, Jun 13, 2022 at 3:20 PM Christoph Muellner wrote: > > From: Christoph Müllner > > This adds Allwinner's D1 to the list of known cores. > The Allwinner includes a single-core XuanTie C906 and is available > for quite some time. Note, that the tuning struct for the C906 > is already part of

Re: [PATCH] expand: Fix up IFN_ATOMIC_{BIT*,*CMP_0} expansion [PR105951]

2022-06-15 Thread Richard Biener via Gcc-patches
On Wed, 15 Jun 2022, Jakub Jelinek wrote: > On Wed, Jun 15, 2022 at 09:45:25AM +0200, Richard Biener wrote: > > I wonder if that matching to IFN_ATOMIC_BIT_TEST_AND_* should then > > happen at the time we expand the original builtin call to RTL? Or > > did the matching expose secondary optimizati

Re: [PATCH] Simplify (B * v + C) * D -> BD* v + CD when B, C, D are all INTEGER_CST.

2022-06-15 Thread Richard Biener via Gcc-patches
On Tue, Jun 7, 2022 at 9:54 AM liuhongt wrote: > > >> + (mult:c (plus:c@4 (mult:c@5 @0 INTEGER_CST@1) INTEGER_CST@2) > >> INTEGER_CST@3) > > >since canonicalization puts INTEGER_CSTs last the :c should not be necessary. > > Changed. > > >> + (if (single_use (@4) > >> + && single_use (@5)) >

Re: [PATCH V2]rs6000: Store complicated constant into pool

2022-06-15 Thread Jiufu Guo via Gcc-patches
Segher Boessenkool writes: Hi Segher, Thanks so much for your review and sugguestions! > Hi! > > On Tue, Jun 14, 2022 at 09:23:55PM +0800, Jiufu Guo wrote: >> This patch reduces the threshold of instruction number for storing >> constant to pool and update cost for constant and mem accessing. >

Re: [PATCH] expand: Fix up IFN_ATOMIC_{BIT*,*CMP_0} expansion [PR105951]

2022-06-15 Thread Jakub Jelinek via Gcc-patches
On Wed, Jun 15, 2022 at 09:45:25AM +0200, Richard Biener wrote: > I wonder if that matching to IFN_ATOMIC_BIT_TEST_AND_* should then > happen at the time we expand the original builtin call to RTL? Or > did the matching expose secondary optimization opportunities? I don't see how we could do what

[PATCH] Revert "[PATCH] RISC-V: Use new linker emulations for glibc ABI."

2022-06-15 Thread Fangrui Song via Gcc-patches
This reverts commit 37d57ac9a636f2235f9060e84fb8dd7968abd1dc. The resolution to https://sourceware.org/bugzilla/show_bug.cgi?id=22962 let GCC pass -m emulation to ld and let the ld emulation configure default library paths. This scheme is problematic: * It's not ld's business to specify default

Re: [PATCH] expand: Fix up IFN_ATOMIC_{BIT*,*CMP_0} expansion [PR105951]

2022-06-15 Thread Richard Biener via Gcc-patches
On Wed, 15 Jun 2022, Jakub Jelinek wrote: > Hi! > > Both IFN_ATOMIC_BIT_TEST_AND_* and IFN_ATOMIC_*_FETCH_CMP_0 ifns > are matched if their corresponding optab is implemented for the particular > mode. The fact that those optabs are implemented doesn't guarantee > they will succeed though, they

[PATCH] expand: Fix up IFN_ATOMIC_{BIT*,*CMP_0} expansion [PR105951]

2022-06-15 Thread Jakub Jelinek via Gcc-patches
Hi! Both IFN_ATOMIC_BIT_TEST_AND_* and IFN_ATOMIC_*_FETCH_CMP_0 ifns are matched if their corresponding optab is implemented for the particular mode. The fact that those optabs are implemented doesn't guarantee they will succeed though, they can just FAIL in their expansion. The expansion in that

[committed][gdb/build] Fix build for gcc < 11

2022-06-15 Thread Tom de Vries via Gcc-patches
Hi, When building trunk on openSUSE Leap 15.3 with system gcc 7.5.0, I run into: ... In file included from ../bfd/bfd.h:46:0, from gdb/defs.h:37, from gdb/debuginfod-support.c:19: gdb/debuginfod-support.c: In function ‘bool debuginfod_is_enabled()’: gdb/../include