Re: [PATCH] improve note location and refactor warn_uninit

2021-08-26 Thread Richard Biener via Gcc-patches
On Wed, Aug 25, 2021 at 10:03 PM Martin Sebor wrote: > > Richard, some time ago you mentioned you'd had trouble getting > -Wuninitialized to print the note pointing to the uninitialized > variable. I said I'd look into it, and so I did. The attached > patch simplifies the warn_uninit() function

Re: [PATCH] Make sure we're playing with integral modes before call extract_integral_bit_field.

2021-08-26 Thread Richard Biener via Gcc-patches
On Thu, Aug 26, 2021 at 1:16 AM Jeff Law wrote: > > > > On 8/24/2021 3:44 AM, Hongtao Liu via Gcc-patches wrote: > > On Tue, Aug 24, 2021 at 5:40 PM Hongtao Liu wrote: > > On Tue, Aug 17, 2021 at 9:52 AM Hongtao Liu wrote: > > On Mon, Aug 9, 2021 at 4:34 PM Hongtao Liu wrote: > > On Fri, Aug 6,

Re: [PATCH] Inline IBM long double __gcc_qsub

2021-08-26 Thread Andreas Schwab
On Aug 25 2021, David Edelsohn via Gcc-patches wrote: > rs6000: inline ldouble __gcc_qsub > > While performing some tests of IEEE 128 float for PPC64LE, Michael > Meissner noticed that __gcc_qsub is substantially slower than > __gcc_qadd. __gcc_qsub valls __gcc_add with the second

Re: [PATCH] Inline IBM long double __gcc_qsub

2021-08-26 Thread Andreas Schwab
On Aug 25 2021, David Edelsohn via Gcc-patches wrote: > rs6000: inline ldouble __gcc_qsub > > While performing some tests of IEEE 128 float for PPC64LE, Michael > Meissner noticed that __gcc_qsub is substantially slower than > __gcc_qadd. __gcc_qsub valls __gcc_add with the second

[PATCH] mklog: fix renaming with version 0.7.0.

2021-08-26 Thread Martin Liška
There's a significant change that breaks tests: v0.7.0: Renamed files return target filename as PatchedFile.path. Fixed and pushed. Martin contrib/ChangeLog: * mklog.py: Use file.{source,target}_file for proper rename handling. --- contrib/mklog.py | 23 +--

Re: Merge stores/loads in modref summaries

2021-08-26 Thread Christophe Lyon via Gcc-patches
Hi, On Wed, Aug 25, 2021 at 7:58 PM Jan Hubicka wrote: > Hi, > this patch adds logic needed to merge neighbouring accesses in ipa-modref > summaries. This helps analyzing array initializers and similar code. It > is > bit of work, since it breaks the fact that modref tree makes a good > latti

Re: [PATCH] Make sure we're playing with integral modes before call extract_integral_bit_field.

2021-08-26 Thread Richard Sandiford via Gcc-patches
Richard Biener via Gcc-patches writes: > One thought I had is whether we can "fix" validate_subreg to have less > "weird" allowed float-int > special cases. As said upthread I think that we either should allow > all of those, implying that > subregs work semantically as if there's subregs to same

Re: Merge stores/loads in modref summaries

2021-08-26 Thread Martin Liška
On 8/26/21 10:33, Christophe Lyon via Gcc-patches wrote: Can you have a look? Please create a PR for it. Thanks, Martin

Re: Merge stores/loads in modref summaries

2021-08-26 Thread Jan Hubicka
> > This patch is causing ICEs on arm: > FAIL: g++.dg/torture/pr89303.C -O1 (internal compiler error) > FAIL: g++.dg/torture/pr89303.C -O1 (test for excess errors) It happens on 32bit arches only it seems. For some reason we end up merging access: Parm 0 param offset:12 offset:0 size:96

Re: Merge stores/loads in modref summaries

2021-08-26 Thread Jan Hubicka
> On 8/26/21 10:33, Christophe Lyon via Gcc-patches wrote: > > Can you have a look? > > Please create a PR for it. I have fix, so perhaps there is no need for PR :) I am testing the following - the problem was that try_merge_with missed some merges because how unoredered_remove handles the vector.

Re: [PATCH 7/7] doc: BPF CO-RE documentation

2021-08-26 Thread Richard Biener via Gcc-patches
On Wed, Aug 4, 2021 at 8:01 PM David Faust via Gcc-patches wrote: > > Document the new command line options (-mcore and -mno-core), the new > BPF target builtin (__builtin_preserve_access_index), and the new BPF > target attribute (preserve_access_index) introduced with BPF CO-RE. OK. > gcc/Chan

Re: [PATCH 4/7] btf: expose get_btf_id

2021-08-26 Thread Richard Biener via Gcc-patches
On Wed, Aug 4, 2021 at 7:57 PM David Faust via Gcc-patches wrote: > > Expose the function get_btf_id, so that it may be used by the BPF > backend. This enables the BPF CO-RE machinery in the BPF backend to > lookup BTF type IDs, in order to create CO-RE relocation records. > > A prototype is added

Re: [PATCH, V2 2/3] targhooks: New target hook for CTF/BTF debug info emission

2021-08-26 Thread Richard Biener via Gcc-patches
On Tue, Aug 24, 2021 at 7:07 PM Indu Bhagat wrote: > > On 8/18/21 12:00 AM, Richard Biener wrote: > > On Tue, Aug 17, 2021 at 7:26 PM Indu Bhagat wrote: > >> > >> On 8/17/21 1:04 AM, Richard Biener wrote: > >>> On Mon, Aug 16, 2021 at 7:39 PM Indu Bhagat > >>> wrote: > > On 8/10/21 4:

Re: [PATCH,V2 1/3] bpf: Add new -mcore option for BPF CO-RE

2021-08-26 Thread Richard Biener via Gcc-patches
On Tue, Aug 10, 2021 at 5:45 PM Jose E. Marchesi wrote: > > > > On Thu, Aug 5, 2021 at 2:54 AM Indu Bhagat via Gcc-patches > > wrote: > >> > >> -mcore in the BPF backend enables code generation for the CO-RE usecase. > >> LTO is > >> disabled for CO-RE compilations. > > > > -mcore reads like "co

Re: [PATCH] Make sure we're playing with integral modes before call extract_integral_bit_field.

2021-08-26 Thread Richard Biener via Gcc-patches
On Thu, Aug 26, 2021 at 11:06 AM Richard Sandiford wrote: > > Richard Biener via Gcc-patches writes: > > One thought I had is whether we can "fix" validate_subreg to have less > > "weird" allowed float-int > > special cases. As said upthread I think that we either should allow > > all of those,

[PATCH] configure: Adjust several assembler checks to remove an unused parm.

2021-08-26 Thread Iain Sandoe
Hi, noticed while trying to apply a new change to gcc/configure.ac --- In r12-3048-ge0b6d0b39c6, the GAS version parameter was removed from the gcc_GAS_CHECK_FEATURE macro. It seems that overlapping comit/test cycles resulted in several AMDGCN and one Darwin commit with the now extra parame

Re: [PATCH] Make sure we're playing with integral modes before call extract_integral_bit_field.

2021-08-26 Thread Richard Sandiford via Gcc-patches
Richard Biener via Gcc-patches writes: > On Thu, Aug 26, 2021 at 11:06 AM Richard Sandiford > wrote: >> >> Richard Biener via Gcc-patches writes: >> > One thought I had is whether we can "fix" validate_subreg to have less >> > "weird" allowed float-int >> > special cases. As said upthread I thi

Re: [PATCH] Optimize macro: make it more predictable

2021-08-26 Thread Richard Biener via Gcc-patches
On Tue, Aug 24, 2021 at 3:04 PM Martin Liška wrote: > > On 8/24/21 14:13, Richard Biener wrote: > > On Thu, Jul 1, 2021 at 3:13 PM Martin Liška wrote: > >> > >> On 10/23/20 1:47 PM, Martin Liška wrote: > >>> Hey. > >> > >> Hello. > >> > >> I deferred the patch for GCC 12. Since the time, I messed

Re: [PATCH] configure: Adjust several assembler checks to remove an unused parm.

2021-08-26 Thread Richard Biener via Gcc-patches
On Thu, Aug 26, 2021 at 12:44 PM Iain Sandoe wrote: > > Hi, > > noticed while trying to apply a new change to gcc/configure.ac > > --- > > In r12-3048-ge0b6d0b39c6, the GAS version parameter was removed from > the gcc_GAS_CHECK_FEATURE macro. It seems that overlapping comit/test > cycles resu

Re: [PATCH] Make sure we're playing with integral modes before call extract_integral_bit_field.

2021-08-26 Thread Richard Biener via Gcc-patches
On Thu, Aug 26, 2021 at 12:50 PM Richard Sandiford wrote: > > Richard Biener via Gcc-patches writes: > > On Thu, Aug 26, 2021 at 11:06 AM Richard Sandiford > > wrote: > >> > >> Richard Biener via Gcc-patches writes: > >> > One thought I had is whether we can "fix" validate_subreg to have less >

Re: [RFC] Don't move cold code out of loop by checking bb count

2021-08-26 Thread Richard Biener via Gcc-patches
On Tue, Aug 10, 2021 at 4:03 AM Xionghu Luo wrote: > > Hi, > > On 2021/8/6 20:15, Richard Biener wrote: > > On Mon, Aug 2, 2021 at 7:05 AM Xiong Hu Luo wrote: > >> > >> There was a patch trying to avoid move cold block out of loop: > >> > >> https://gcc.gnu.org/pipermail/gcc/2014-November/215551.

[committed] libstdc++: Make Networking TS headers more portable [PR100285]

2021-08-26 Thread Jonathan Wakely via Gcc-patches
Add more preprocessor conditions to check for constants being defined before using them, so that the Networking TS headers can be compiled on a wider range of platforms. Signed-off-by: Jonathan Wakely libstdc++-v3/ChangeLog: PR libstdc++/100285 * configure.ac: Check for O_NONBLO

Re: [PATCH] Inline IBM long double __gcc_qsub

2021-08-26 Thread Segher Boessenkool
Hi! On Wed, Aug 25, 2021 at 08:23:32PM -0400, David Edelsohn wrote: > rs6000: inline ldouble __gcc_qsub > > While performing some tests of IEEE 128 float for PPC64LE, Michael > Meissner noticed that __gcc_qsub is substantially slower than > __gcc_qadd. __gcc_qsub valls __gcc_add

Re: [gcc r12-3142] Merge load/stores in ipa-modref summaries

2021-08-26 Thread Andreas Schwab
New tests that FAIL (2 tests): compiler driver --help=param option(s): "^ +-.*[^:.]$" absent from output: " --param=modref-max-adjustments= Maximum number of times a given range is adjusted during the dataflow" compiler driver --help=params option(s): "[^.]$" absent from output: "w" Andreas.

Re: [PATCH] Optimize macro: make it more predictable

2021-08-26 Thread Martin Liška
On 8/26/21 13:04, Richard Biener wrote: On Tue, Aug 24, 2021 at 3:04 PM Martin Liška wrote: On 8/24/21 14:13, Richard Biener wrote: On Thu, Jul 1, 2021 at 3:13 PM Martin Liška wrote: On 10/23/20 1:47 PM, Martin Liška wrote: Hey. Hello. I deferred the patch for GCC 12. Since the time, I

Re: [PATCH] avoid printing range table header alone

2021-08-26 Thread Andrew MacLeod via Gcc-patches
On 8/25/21 7:49 PM, Martin Sebor wrote: On 8/25/21 1:34 PM, Andrew MacLeod wrote: FWIW, I see duplicate output in the EVRP1 dump that looks just like it could be the output under Non-varying global ranges. I wonder if it's the result of the block in ranger_cache::fill_block_cache guarded by

Re: [PATCH 14/34] rs6000: Add remaining overloads

2021-08-26 Thread Bill Schmidt via Gcc-patches
Hi Segher, On 8/25/21 6:27 PM, Segher Boessenkool wrote: On Thu, Jul 29, 2021 at 08:31:01AM -0500, Bill Schmidt wrote: * config/rs6000/rs6000-overload.def: Add remaining overloads. +; TODO: Note that the entry for VEC_ADDE currently gets ignored in +; altivec_resolve_overloaded_builtin.

Re: [PATCH] Optimize macro: make it more predictable

2021-08-26 Thread Richard Biener via Gcc-patches
On Thu, Aug 26, 2021 at 2:39 PM Martin Liška wrote: > > On 8/26/21 13:04, Richard Biener wrote: > > On Tue, Aug 24, 2021 at 3:04 PM Martin Liška wrote: > >> > >> On 8/24/21 14:13, Richard Biener wrote: > >>> On Thu, Jul 1, 2021 at 3:13 PM Martin Liška wrote: > > On 10/23/20 1:47 PM, Ma

Re: [PATCH,V2 1/3] bpf: Add new -mcore option for BPF CO-RE

2021-08-26 Thread Jose E. Marchesi via Gcc-patches
Hi Richard. > On Tue, Aug 10, 2021 at 5:45 PM Jose E. Marchesi > wrote: >> >> >> > On Thu, Aug 5, 2021 at 2:54 AM Indu Bhagat via Gcc-patches >> > wrote: >> >> >> >> -mcore in the BPF backend enables code generation for the CO-RE usecase. >> >> LTO is >> >> disabled for CO-RE compilations. >>

Re: [PATCH 14/34] rs6000: Add remaining overloads

2021-08-26 Thread Segher Boessenkool
Hi! On Thu, Aug 26, 2021 at 07:59:04AM -0500, Bill Schmidt wrote: > On 8/25/21 6:27 PM, Segher Boessenkool wrote: > >On Thu, Jul 29, 2021 at 08:31:01AM -0500, Bill Schmidt wrote: > >>* config/rs6000/rs6000-overload.def: Add remaining overloads. > >>+; TODO: Note that the entry for VEC_ADDE cur

[PATCH v2] libgcc: Add a backchain fallback to _Unwind_Backtrace() on PowerPC

2021-08-26 Thread Raphael Moreira Zinsly via Gcc-patches
Without dwarf2 unwind tables available _Unwind_Backtrace() is not able to return the full backtrace. This patch adds a fallback function on powerpc to get the backtrace by doing a backchain, this code was originally at glibc. libgcc/ChangeLog: * config/rs6000/linux-unwind.h (struct rt_sig

Improve handling of modref --params

2021-08-26 Thread Jan Hubicka
Hi, this patch makes insertion to modref access tree smarter when --param modref-max-bases and moredref-max-refs are hit. Instead of giving up we either give up on base alias set (make it equal to ref) or turn the alias set to 0. This lets us to track useful info on quite large functions, such as

Re: Merge stores/loads in modref summaries

2021-08-26 Thread H.J. Lu via Gcc-patches
On Thu, Aug 26, 2021 at 2:49 AM Jan Hubicka wrote: > > > On 8/26/21 10:33, Christophe Lyon via Gcc-patches wrote: > > > Can you have a look? > > > > Please create a PR for it. > I have fix, so perhaps there is no need for PR :) > I am testing the following - the problem was that try_merge_with mis

[PATCH] [i386] Set all_regs to true in the call to replace_rtx [PR102057]

2021-08-26 Thread Uros Bizjak via Gcc-patches
We want to replace all REGs equal to FROM. 2021-08-26 Uroš Bizjak gcc/ PR target/102057 * config/i386/i386.md (cmove reg-reg move elimination peephole2s): Set all_regs to true in the call to replace_rtx. I was not able to create a testcase without warnings. Bootstrapped and regre

Re: Merge stores/loads in modref summaries

2021-08-26 Thread Jeff Law via Gcc-patches
On 8/26/2021 9:27 AM, H.J. Lu via Gcc-patches wrote: On Thu, Aug 26, 2021 at 2:49 AM Jan Hubicka wrote: On 8/26/21 10:33, Christophe Lyon via Gcc-patches wrote: Can you have a look? Please create a PR for it. I have fix, so perhaps there is no need for PR :) I am testing the following - t

Re: [PATCH v3] Fix for powerpc64 long double complex divide failure

2021-08-26 Thread Patrick McGehearty via Gcc-patches
Apologies in advance for the length of this reply. Details seemed necessary to be sure we are on the same page of understanding. Aug 26, 2021 by Patrick McGehearty The only IBM machines I have current access to are in the gcc fsf farm. All studies were done on gcc135.fsffrance.org Linux gcc135.

[PATCH] testsuite: Make sure double-precision is supported in g++.dg/eh/arm-vfp-unwind.C

2021-08-26 Thread Christophe Lyon via Gcc-patches
g++.dg/eh/arm-vfp-unwind.C uses an asm statement relying on double-precision FPU support, but does not make sure it is actually supported by the target. Check (__ARM_FP & 8) to ensure this. 2021-08-26 Christophe Lyon gcc/testsuite/ * g++.dg/eh/arm-vfp-unwind.C: Check __ARM_FP.

Re: [PATCH v3] Fix for powerpc64 long double complex divide failure

2021-08-26 Thread Joseph Myers
On Thu, 26 Aug 2021, Patrick McGehearty via Gcc-patches wrote: > The only complex divide routines in $HOME/usr/lib64/libgcc_s.so.1 are: > __divdc3, __divsc3, __divtc3 Because no symbol versions are assigned to the KFmode symbols in the .ver files, so they are only exported from libgcc.a. I thin

Re: [PATCH] improve note location and refactor warn_uninit

2021-08-26 Thread Martin Sebor via Gcc-patches
On 8/26/21 1:00 AM, Richard Biener wrote: On Wed, Aug 25, 2021 at 10:03 PM Martin Sebor wrote: Richard, some time ago you mentioned you'd had trouble getting -Wuninitialized to print the note pointing to the uninitialized variable. I said I'd look into it, and so I did. The attached patch si

Ping: [PATCH, rs6000 V2] Add store fusion support for Power10

2021-08-26 Thread Pat Haugen via Gcc-patches
Ping. On 8/11/21 11:02 AM, Pat Haugen via Gcc-patches wrote: > Enable store fusion on Power10. > > Use the SCHED_REORDER hook to implement Power10 specific ready list > reordering. > As of now this is just store fusion. > > Things changed in this version of the patch > - Separate patch for addi

Re: [PATCH v3] Fix for powerpc64 long double complex divide failure

2021-08-26 Thread Segher Boessenkool
On Thu, Aug 26, 2021 at 10:51:51AM -0500, Patrick McGehearty wrote: > I'm concerned that the old environment on gcc135.fsffrance.org > is not showing me a typical build environment for IBM linux users > with more modern environments. It runs CentOS 7. This is still supported. It is getting long

Re: [PATCH v3] Fix for powerpc64 long double complex divide failure

2021-08-26 Thread Segher Boessenkool
Hi! On Fri, Aug 13, 2021 at 05:12:47PM +, Joseph Myers wrote: > That should be the function __divtc3. (A single libgcc binary supports > multiple long double formats, so libgcc function names referring to > floating-point modes need to be understood as actually referring to a > particular

Re: Merge stores/loads in modref summaries

2021-08-26 Thread Jan Hubicka
> > commit f075b8c5adcf9cb6336563c472c8d624c54184db > Author: Jan Hubicka > Date: Thu Aug 26 15:33:56 2021 +0200 > > Fix off-by-one error in try_merge_with > > gcc/ChangeLog: > > * ipa-modref-tree.h (modref_ref_node::verify): New member > functoin. >

Re: Merge stores/loads in modref summaries

2021-08-26 Thread Jeff Law via Gcc-patches
On 8/26/2021 11:10 AM, Jan Hubicka wrote: commit f075b8c5adcf9cb6336563c472c8d624c54184db Author: Jan Hubicka Date: Thu Aug 26 15:33:56 2021 +0200 Fix off-by-one error in try_merge_with gcc/ChangeLog: * ipa-modref-tree.h (modref_ref_node::verify): New member

Re: [PATCH, rs6000 V2] Add store fusion support for Power10

2021-08-26 Thread Segher Boessenkool
Hi! On Wed, Aug 11, 2021 at 11:02:25AM -0500, Pat Haugen wrote: > * config/rs6000/rs6000-cpus.def (ISA_3_1_MASKS_SERVER): Add new flag. > (POWERPC_MASKS): Likewise. "Add OPTION_MASK_P10_FUSION_2STORE", instead. > +static bool > +is_fusable_store (rtx_insn *insn, rtx *str_mem) > +{ >

[r12-3159 Regression] FAIL: gfortran.dg/pr68251.f90 -O (test for excess errors) on Linux/x86_64

2021-08-26 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, f075b8c5adcf9cb6336563c472c8d624c54184db is the first bad commit commit f075b8c5adcf9cb6336563c472c8d624c54184db Author: Jan Hubicka Date: Thu Aug 26 15:33:56 2021 +0200 Fix off-by-one error in try_merge_with caused FAIL: gcc.c-torture/compile/960514-1.c -O3 -fomit-fra

[Committed] Tidy up !POINTER_TYPE_P test in match.pd LSHIFT_EXPR folding

2021-08-26 Thread Roger Sayle
As suggested by Richard Biener in the comments of PR middle-end/102029, the new test "INTEGRAL_TYPE_P (type) && !POINTER_TYPE_P (type) ..." is redundant, and just "INTEGRAL_TYPE_P (type)" is the preferred form. Committed as obvious/pre-approved, after a "make bootstrap" and "make -k check" with n

Re: Merge stores/loads in modref summaries

2021-08-26 Thread Jeff Law via Gcc-patches
On 8/26/2021 11:10 AM, Jan Hubicka wrote: commit f075b8c5adcf9cb6336563c472c8d624c54184db Author: Jan Hubicka Date: Thu Aug 26 15:33:56 2021 +0200 Fix off-by-one error in try_merge_with gcc/ChangeLog: * ipa-modref-tree.h (modref_ref_node::verify): New member

Re: [PATCH, V2 2/3] targhooks: New target hook for CTF/BTF debug info emission

2021-08-26 Thread Indu Bhagat via Gcc-patches
On 8/26/21 3:03 AM, Richard Biener wrote: On Tue, Aug 24, 2021 at 7:07 PM Indu Bhagat wrote: On 8/18/21 12:00 AM, Richard Biener wrote: On Tue, Aug 17, 2021 at 7:26 PM Indu Bhagat wrote: On 8/17/21 1:04 AM, Richard Biener wrote: On Mon, Aug 16, 2021 at 7:39 PM Indu Bhagat wrote: On 8/1

[PATCH v2] Inline IBM long double __gcc_qsub

2021-08-26 Thread David Edelsohn via Gcc-patches
While performing some tests of IEEE 128 float for PPC64LE, Michael Meissner noticed that __gcc_qsub is substantially slower than __gcc_qadd. __gcc_qsub calls __gcc_add with the second operand negated. Because the functions normally are invoked through libgcc shared object, the

Re: [PATCH] testsuite, Darwin : Do not claim 'GAS' for cctools assembler.

2021-08-26 Thread Mike Stump via Gcc-patches
On Aug 19, 2021, at 1:02 PM, Iain Sandoe wrote: > > Although the cctools assembler is based of GNU GAS, it is from a > very old version (1.38) which does not support many of the features > that the target supports test is expecting***. > > tested on i686 and x86_64 darwin versions using the ccto

Re: [PATCH] Generate XXSPLTIDP on power10.

2021-08-26 Thread will schmidt via Gcc-patches
On Wed, 2021-08-25 at 15:46 -0400, Michael Meissner wrote: > Generate XXSPLTIDP on power10. > > This patch implements XXSPLTIDP support for SF and DF scalar constants and > V2DF > vector constants. The XXSPLTIDP instruction is given a 32-bit immediate that > is converted to a vector of two DFmod

Re: [PATCH] improve note location and refactor warn_uninit

2021-08-26 Thread Martin Sebor via Gcc-patches
On 8/26/21 10:38 AM, Martin Sebor wrote: On 8/26/21 1:00 AM, Richard Biener wrote: On Wed, Aug 25, 2021 at 10:03 PM Martin Sebor wrote: Richard, some time ago you mentioned you'd had trouble getting -Wuninitialized to print the note pointing to the uninitialized variable.  I said I'd look int

[PATCH] [i386] Call force_reg unconditionally.

2021-08-26 Thread Uros Bizjak via Gcc-patches
There is no point to check RTXes before calling force_reg, force_reg checks for REG RTX by itself. 2021-08-26 Uroš Bizjak gcc/ * config/i386/i386.md (*btr_1): Call force_reg unconditionally. (conditional moves with memory inputs splitters): Ditto. * config/i386/sse.md (one_cmpl2):

Re: [PATCH] Generate XXSPLTIDP on power10.

2021-08-26 Thread Michael Meissner via Gcc-patches
On Thu, Aug 26, 2021 at 02:17:57PM -0500, will schmidt wrote: > On Wed, 2021-08-25 at 15:46 -0400, Michael Meissner wrote: > > Generate XXSPLTIDP on power10. > > > > I have added a temporary switch (-mxxspltidp) to control whether or not the > > XXSPLTIDP instruction is generated. > > How tempora

[PATCH v4] Fix for powerpc64 long double complex divide failure

2021-08-26 Thread Patrick McGehearty via Gcc-patches
The v4 revision adds a test in libgcc/libgcc2.c for when "__LIBGCC_TF_MANT_DIG__ == 106" to use __LIBGCC_DF_EPSILON__ instead of __LIBGCC_TF_EPSILON__. That is specific to IBM 128-bit format long doubles where EPSILON is very, very small and 1/EPSILON oveflows to infinity. This change avoids the ov

Re: [PATCH v4] Fix for powerpc64 long double complex divide failure

2021-08-26 Thread Joseph Myers
On Thu, 26 Aug 2021, Patrick McGehearty via Gcc-patches wrote: > The revision retains the use of __LIBGCC_DF_* in .../rs6000/_divkc3.c > instead of __LIBGCC_KF_* since some older but still supported environments > do not recognize the KF floating point mode properly. This change has a That part o

Re: [PATCH v2] Inline IBM long double __gcc_qsub

2021-08-26 Thread Segher Boessenkool
Hi! On Thu, Aug 26, 2021 at 02:57:35PM -0400, David Edelsohn wrote: > * config/rs6000/ibm-ldouble.c (ldouble_qadd_internal): Rename from > __gcc_qadd. > (__gcc_qadd): Call ldouble_qadd_internal. > (__gcc_qsub): Call ldouble_qadd_internal with second

Re: [PATCH v2] Inline IBM long double __gcc_qsub

2021-08-26 Thread David Edelsohn via Gcc-patches
On Thu, Aug 26, 2021 at 6:53 PM Segher Boessenkool wrote: > > Hi! > > On Thu, Aug 26, 2021 at 02:57:35PM -0400, David Edelsohn wrote: > > * config/rs6000/ibm-ldouble.c (ldouble_qadd_internal): Rename > > from > > __gcc_qadd. > > (__gcc_qadd): Call ldouble_qadd_

[committed] libstdc++: Add static assertions for std::function requirements

2021-08-26 Thread Jonathan Wakely via Gcc-patches
Add static assertions to std::function, so that more user-friendly diagnostics are given when trying to store a non-copyable target object. These preconditions were added as "Mandates:" by LWG 2447, but I'm committing them separately from implementing that, to allow just this change to be backport

[committed] libstdc:: Use markdown for Doxygen comments in std::function

2021-08-26 Thread Jonathan Wakely via Gcc-patches
Signed-off-by: Jonathan Wakely libstdc++-v3/ChangeLog: * include/bits/std_function.h (function): Adjust doxygen comments. * include/bits/unique_ptr.h (make_unique_for_overwrite): Change parameter name to match doxygen comment. Tested powerpc64le-linux. Committed

[committed] libstdc++: Avoid a move in std::function construction (LWG 2447)

2021-08-26 Thread Jonathan Wakely via Gcc-patches
This makes the std::function constructor use perfect forwarding, to avoid an unnecessary move-construction of the target. This means we need to rewrite the _Function_base::_Base_manager::_M_init_functor function to use a forwarding reference, and so can reuse it for the clone operation. Also simpl

Re: [PATCH 15/34] rs6000: Execute the automatic built-in initialization code

2021-08-26 Thread Segher Boessenkool
Hi! On Thu, Jul 29, 2021 at 08:31:02AM -0500, Bill Schmidt wrote: > * config/rs6000/rs6000-call.c (rs6000-builtins.h): New #include. > (rs6000_init_builtins): Call rs6000_autoinit_builtins; skip the old > initialization logic when new builtins are enabled. s/; s/. S/ > + /* E

Re: [PATCH v4] Fix for powerpc64 long double complex divide failure

2021-08-26 Thread Patrick McGehearty via Gcc-patches
I will prepare a patch without the changes to .../rs6000/_divkc3.c but I have no way of testing it to confirm it fixes the original complaint. On 8/26/2021 5:09 PM, Joseph Myers wrote: On Thu, 26 Aug 2021, Patrick McGehearty via Gcc-patches wrote: The revision retains the use of __LIBGCC_DF_*

[PATCH] AIX packed structure alignment [PR102068]

2021-08-26 Thread David Edelsohn via Gcc-patches
Further fixes to structure alignment when the structure is packed and contains double. This patch checks for packed attribute at the top level. Bootstrapped on powerpc-ibm-aix7.2.3.0 gcc/ChangeLog: PR target/102068 * config/rs6000/rs6000.c (rs6000_adjust_

[PATCH] i386: Fix wrong optimization for consecutive masked scatters [PR 101472]

2021-08-26 Thread Kong, Lingling via Gcc-patches
Hi, For avx512f_scattersi, mask operand only affect set src, we need to refine the pattern to let gcc know mask register also affect the dest. So we put mask operand into UNSPEC_VSIBADDR. Bootstrapped and regression tested on x86_64-linux-gnu{-m32,-m64}. Ok for master? gcc/ChangeLog: P

Re: [PATCH v3] Fix for powerpc64 long double complex divide failure

2021-08-26 Thread Michael Meissner via Gcc-patches
On Fri, Aug 13, 2021 at 05:22:47PM +, Joseph Myers wrote: > On Fri, 13 Aug 2021, Andreas Schwab wrote: > > > On Aug 12 2021, Patrick McGehearty via Gcc-patches wrote: > > How can it happen that __LONG_DOUBLE_IEEE128__ is not defined? This > > file is always compiled with -mfloat128 and this l

Re: [PATCH] Generate XXSPLTIDP on power10.

2021-08-26 Thread Michael Meissner via Gcc-patches
On Thu, Aug 26, 2021 at 05:28:42PM -0400, Michael Meissner wrote: > I have patches for adding XXSPLTI32DX, but so far, I'm not sure whether it is > a > win or not. This has the xxsplti32dx_operand predicate and the > xxsplti32dx_constant_p internal function and a separate constraint ("eD") for >

Re: [PATCH 1/2] RISC-V: Add arch flags for T-HEAD.

2021-08-26 Thread Jojo R via Gcc-patches
Hi, - Add documents of XuanTie series [1]. - Add QEMU binary [2] by now, you can execute application with argument ‘-cpu c910’,    and opening source is on the way. - Add my colleague [3] to commit patches of binutils [1] https://github.com/rjiejie/XuanTie-doc [2]

[PATCH] MIPS: use N64 ABI by default if the triple end with -gnuabi64

2021-08-26 Thread YunQiang Su
gcc/ChangeLog: PR target/102089 * config.gcc: MIPS: use N64 ABI by default if the triple end with -gnuabi64, which is used by Debian since 2013. --- gcc/config.gcc | 14 ++ 1 file changed, 14 insertions(+) diff --git a/gcc/config.gcc b/gcc/config.gcc index 0

Re: [PATCH] MIPS: use N64 ABI by default if the triple end with -gnuabi64

2021-08-26 Thread Xi Ruoyao via Gcc-patches
On Thu, 2021-08-26 at 23:56 -0400, YunQiang Su wrote: > gcc/ChangeLog: > > PR target/102089 > * config.gcc: MIPS: use N64 ABI by default if the triple end >   with -gnuabi64, which is used by Debian since 2013. > --- >  gcc/config.gcc | 14 ++ >  1 file changed,

Re: [PATCH] Make sure we're playing with integral modes before call extract_integral_bit_field.

2021-08-26 Thread Hongtao Liu via Gcc-patches
On Thu, Aug 26, 2021 at 7:09 PM Richard Biener via Gcc-patches wrote: > > On Thu, Aug 26, 2021 at 12:50 PM Richard Sandiford > wrote: > > > > Richard Biener via Gcc-patches writes: > > > On Thu, Aug 26, 2021 at 11:06 AM Richard Sandiford > > > wrote: > > >> > > >> Richard Biener via Gcc-patches

[PATCH] libffi: Fix MIPS r6 support

2021-08-26 Thread YunQiang Su
for some instructions, MIPS r6 uses different encoding other than the previous releases. 1. mips/n32.S disable .set mips4: since it casuses old insn encoding is used. https://github.com/libffi/libffi/pull/396 2. mips/ffi.c: the encoding for JR is hardcoded: we need to use different value

Re: [PATCH] i386: Fix wrong optimization for consecutive masked scatters [PR 101472]

2021-08-26 Thread Hongtao Liu via Gcc-patches
On Fri, Aug 27, 2021 at 10:03 AM Kong, Lingling via Gcc-patches wrote: > > Hi, > > For avx512f_scattersi, mask operand only affect set src, we need > to refine the pattern to let gcc know mask register also affect the dest. > So we put mask operand into UNSPEC_VSIBADDR. > > Bootstrapped and regre

Re: [PATCH v2] [MIPS]: add .module mipsREV to all output asm file

2021-08-26 Thread YunQiang Su
在 2021/6/18 11:29, YunQiang Su 写道: Currently, the asm output file for MIPS has no rev info. It can make some trouble, for example: assembler is mips1 by default, gcc is fpxx by default. To assemble the output of gcc -S, we have to pass -mips2 to assembler. gcc/ChangeLog: * gcc/config

Re: [PATCH v2] [MIPS]: add .module mipsREV to all output asm file

2021-08-26 Thread Xi Ruoyao via Gcc-patches
On Fri, 2021-08-27 at 13:11 +0800, YunQiang Su wrote: > 在 2021/6/18 11:29, YunQiang Su 写道: > > Currently, the asm output file for MIPS has no rev info. > > It can make some trouble, for example: > >    assembler is mips1 by default, > >    gcc is fpxx by default. > > To assemble the output of gcc -

Re: [PATCH, V2 2/3] targhooks: New target hook for CTF/BTF debug info emission

2021-08-26 Thread Richard Biener via Gcc-patches
On Thu, Aug 26, 2021 at 8:55 PM Indu Bhagat wrote: > > On 8/26/21 3:03 AM, Richard Biener wrote: > > On Tue, Aug 24, 2021 at 7:07 PM Indu Bhagat wrote: > >> > >> On 8/18/21 12:00 AM, Richard Biener wrote: > >>> On Tue, Aug 17, 2021 at 7:26 PM Indu Bhagat > >>> wrote: > > On 8/17/21 1:

Re: [PATCH] improve note location and refactor warn_uninit

2021-08-26 Thread Richard Biener via Gcc-patches
On Thu, Aug 26, 2021 at 9:30 PM Martin Sebor wrote: > > On 8/26/21 10:38 AM, Martin Sebor wrote: > > On 8/26/21 1:00 AM, Richard Biener wrote: > >> On Wed, Aug 25, 2021 at 10:03 PM Martin Sebor wrote: > >>> > >>> Richard, some time ago you mentioned you'd had trouble getting > >>> -Wuninitialized

[PATCH] Check the type of mask while generating cond_op in gimple simplication.

2021-08-26 Thread liuhongt via Gcc-patches
When gimple simplifcation try to combine op and vec_cond_expr to cond_op, it doesn't check if mask type matches. It causes an ICE when expand cond_op with mismatched mode. This patch add a function named cond_vectorized_internal_fn_supported_p to additionally check mask type than vectorized_in