Re: [PATCH] options: Make -Ofast switch off -fsemantic-interposition

2022-05-11 Thread Fāng-ruì Sòng via Gcc-patches
On Fri, Nov 19, 2021 at 9:55 AM Martin Jambor wrote: > > Hi, > > On Fri, Nov 19 2021, Jan Hubicka wrote: > >> > Hi, > >> > > >> > On Fri, Nov 12 2021, Martin Jambor wrote: > >> > > Hi, > >> > > > >> > > using -fno-semantic-interposition has been reported by various people > >> > > to bring about c

[committed] analyzer: fix memory leaks

2022-05-11 Thread David Malcolm via Gcc-patches
These leaks all relate to logging within -fdump-analyzer[-stderr] or are one-time leaks; seen with valgrind. Successfully bootstrapped & regrtested on x86_64-pc-linux-gnu. Tested with valgrind. Pushed to trunk as r13-334-g99988b0e8b57b3. gcc/analyzer/ChangeLog: * checker-path.cc (state_ch

RE: [EXTERNAL] Re: [PATCH] Guard against applying scale with 0 denominator

2022-05-11 Thread Eugene Rozenfeld via Gcc-patches
In my case this is not exactly what the FIXME in the comment above says. The count is 0 even before the initial scaling happens. I hit this case with some changes I'm working on to enable per-instruction discriminators for AutoFDO. I looked into saving/restoring the old counts but it would be aw

Re: [PATCH] Fold GCC_VERSION >= $old_version to TRUE, otherwise to FALSE.

2022-05-11 Thread Eric Gallager via Gcc-patches
On Thu, May 5, 2022 at 2:26 PM Martin Liška wrote: > > On 5/5/22 14:24, Richard Biener wrote: > > Hmm, but we support C++11 host compilers that are not GCC but > > may claim to be, with GCC_VERSION 4.2.x for example. Are we sure > > all those liars implement what we guard with the version checks?

[PATCH] Avoid visiting newly-created blocks in harden-conditionals

2022-05-11 Thread Alexandre Oliva via Gcc-patches
Reverse iteration over blocks, in gimple-harden-conditionals.cc, was supposed to avoid visiting blocks introduced by hardening and introducing further reversed conditionals and traps for them, but newly-created blocks may be inserted before the current block, as shown by the PR105455 testcase. N

[PATCH] [PR105455] Set edge probabilities when hardening conditionals

2022-05-11 Thread Alexandre Oliva via Gcc-patches
When turning unconditional edges into conditional, as in gimple-harden-conditionals.cc:insert_check_and_trap, the newly-created edge's probability comes out uninitialized, while the previously unconditional edge's probability is presumably profile_probability::always. Mixing initialized and unin

Re: [PATCH v2] Strip of a vector load which is only used partially.

2022-05-11 Thread Hongtao Liu via Gcc-patches
On Tue, May 10, 2022 at 2:54 PM Richard Biener via Gcc-patches wrote: > > On Mon, May 9, 2022 at 7:11 AM liuhongt via Gcc-patches > wrote: > > > > Here's adjused patch. > > Ok for trunk? > > > > Optimize > > > > _4 = VEC_PERM_EXPR <_1, _1, { 4, 5, 6, 7, 4, 5, 6, 7 }>; > > _5 = BIT_FIELD_REF <

[PATCH] [PR105455] predict: Check for no REG_BR_PROB in uninitialized case

2022-05-11 Thread Alexandre Oliva via Gcc-patches
There is an assumption in force_edge_cold that, if any edge out of the same src block has uninitialized probability, then a conditional branch out of src won't have REG_BR_PROB set. This assumption is supposed to hold, but buggy gimple passes may turn unconditional edges into conditional ones, a

RE: [PATCH] [i386]Add combine splitter to transform pxor/pcmpeqb/pmovmskb/cmp 0xffff to ptest.

2022-05-11 Thread Jiang, Haochen via Gcc-patches
Hi all, I just refined this patch with more explanation in commit message. No code change compare to last change, which removed ix86_match_ccmode. Ok for trunk? BRs, Haochen > -Original Message- > From: Jiang, Haochen > Sent: Saturday, May 7, 2022 9:55 AM > To: Uros Bizjak > Cc: gcc-p

[PATCH] Remove -mpower8-fusion options

2022-05-11 Thread Michael Meissner via Gcc-patches
Eliminate power8-fusion and power8-fusion-sign options. As part of PR target/102059, one of the things came up is that we should eliminate the power8 fusion options altogether. This patch eliminates the -mpower8-fusion option. It does enable power8 fusion if the code is being tuned for power8.

Re: [PATCH] testsuite: Update Wconversion testcase check type.

2022-05-11 Thread Palmer Dabbelt
On Wed, 11 May 2022 19:37:21 PDT (-0700), jia...@iscas.ac.cn wrote: Thank you, it hasn't commited into the trunk yet. Should be now. ISCAS å¨ Palmer Dabbelt ï¼2022å¹´5æ11æ¥ ä¸å5:59åéï¼ On Thu, 05 May 2022 11:45:50 PDT (-0700), [1]gcc-patches@gcc.gnu.org wrote: > On T

[committed] openmp: Add omp_all_memory support (C/C++ only so far)

2022-05-11 Thread Jakub Jelinek via Gcc-patches
Hi! The ugly part is that OpenMP 5.1 made omp_all_memory a reserved identifier which isn't allowed to be used anywhere but in the depend clause, this is against how everything else has been handled in OpenMP so far (where some identifiers could have special meaning in some OpenMP clauses or pragma

Re: [PATCH] Add -fcf-check-attribute=[yes|no|none] for Linux kernel

2022-05-11 Thread Richard Biener via Gcc-patches
On Wed, 11 May 2022, H.J. Lu wrote: > On Tue, May 10, 2022 at 11:39 PM Richard Biener wrote: > > > > On Tue, 10 May 2022, H.J. Lu wrote: > > > > > When compiling Linux kernel with -fcf-protection=branch to enable x86 > > > Indiret Branch Tracking (IBT), ENDBR is added to all global functions. > >

<    1   2