Re: [PATCH] [PowerPC/RTEMS] Add 64-bit support using ELFv2 ABI

On 26/07/17 19:56, Segher Boessenkool wrote: On Tue, Jul 25, 2017 at 02:25:21PM +0200, Sebastian Huber wrote: Add 64-bit support for RTEMS using the ELFv2 ABI with 64-bit long double. diff --git a/gcc/config.gcc b/gcc/config.gcc index 2ae0218b5fc..aab7f65c1df 100644 --- a/gcc/config.gcc +++

Re: [PATCH] Switch vec_init and vec_extract optabs to 2 mode optab to allow extraction of vector from vector or initialization of vector from smaller vectors (PR target/80846)

On Tue, 25 Jul 2017, Jakub Jelinek wrote: > Hi! > > The following patch adjusts the vec_init and vec_extract optabs, so that > they don't have in the expander names just the vector mode, but also another > mode, for vec_extract the mode of the result and for vec_init the mode of > the elts of

[PATCH,AIX] Enable Go for AIX

Description: * This patch enables Go on AIX. Tests: * Fedora25/x86_64 + GCC trunk : Configure/Build: SUCCESS - build made by means of gmake. ChangeLog: * configure.ac, configure: Enable Go for AIX * contrib/config-list.mk: Enable Go for AIX Cordialement, Tony Reix Bull - ATOS IBM Coop

Re: [PATCH] Switch vec_init and vec_extract optabs to 2 mode optab to allow extraction of vector from vector or initialization of vector from smaller vectors (PR target/80846)

On Wed, Jul 26, 2017 at 09:09:04AM +0200, Jakub Jelinek wrote: > On Tue, Jul 25, 2017 at 03:52:56PM -0500, Segher Boessenkool wrote: > > On Tue, Jul 25, 2017 at 11:14:32AM +0200, Jakub Jelinek wrote: > > > This patch only adds new vector from vector extract and init patterns to > > > the i386

[PATCH 2/2, i386]: Introduce ASM_FPRINTF_EXTENSIONS and simplify ASM_OUTPUT_REG_{PUSH|POP}

Hello! Attached patch introduces a couple of asm_fprintf extensions to simplify handling of word-mode operands. 2017-07-26 Uros Bizjak * config/i386/i386.h (ASM_PRINTF_EXTENSIONS): New macro. (ASM_OUTPUT_REG_PUSH): Rewrite with new operand modifiers.

Re: [PATCH] Switch vec_init and vec_extract optabs to 2 mode optab to allow extraction of vector from vector or initialization of vector from smaller vectors (PR target/80846)

On Wed, 26 Jul 2017, Jakub Jelinek wrote: > On Tue, Jul 25, 2017 at 03:52:56PM -0500, Segher Boessenkool wrote: > > On Tue, Jul 25, 2017 at 11:14:32AM +0200, Jakub Jelinek wrote: > > > This patch only adds new vector from vector extract and init patterns to > > > the i386 backend, but I had to

[PATCH,AIX] Manage .go_export section for AIX

Description: * This patch manages the .go_export section as an EXCLUDE section on AIX. Tests: * Fedora25/x86_64 + GCC trunk : Configure/Build: SUCCESS - build made by means of gmake. ChangeLog: * go-backend.c (go_write_export_data): Use EXCLUDE section for AIX. Cordialement, Tony Reix

Re: [PATCH] Switch vec_init and vec_extract optabs to 2 mode optab to allow extraction of vector from vector or initialization of vector from smaller vectors (PR target/80846)

On Tue, Jul 25, 2017 at 11:14 AM, Jakub Jelinek wrote: > Hi! > > The following patch adjusts the vec_init and vec_extract optabs, so that > they don't have in the expander names just the vector mode, but also another > mode, for vec_extract the mode of the result and for

Re: [PATCH GCC][1/2]Feed bound computation to folder in loop split

On Wed, Jul 26, 2017 at 9:48 AM, Richard Biener wrote: > On Tue, Jul 25, 2017 at 7:45 PM, Marc Glisse wrote: >> On Tue, 25 Jul 2017, Richard Biener wrote: >> I think we need Richard to say what the intent is for the valueization

Re: [COMMITED][AArch64] Fix PR79041

Andreas Schwab wrote: > That fails in ILP32 mode. Well -mabi-ilp32 and -mcmodel=large make no sense at all, that should really give an error... I've committed a patch to trunk and GCC7 to disable this test with ILP32. Wilco

Re: [PATCH] Fix infinite recursion with div-by-zero (PR middle-end/70992)

On Wed, Jul 26, 2017 at 1:35 PM, Marek Polacek wrote: > On Tue, Jul 25, 2017 at 03:47:31PM +0200, Richard Biener wrote: >> On Tue, Jul 25, 2017 at 3:30 PM, Eric Botcazou wrote: >> >> Eric, any comments? >> > >> > No objection for the build2_stat hunk, I

[PATCH][AArch64] Remove '*' from movsi/di/ti patterns

Remove the remaining uses of '*' from the movsi/di/ti patterns. Using '*' in alternatives is typically incorrect at it tells the register allocator to ignore those alternatives. So remove these from all the integer move patterns. This removes unnecessary int to float moves, for example

Re: Patch ping

On Tue, 25 Jul 2017, Jakub Jelinek wrote: > Hi! > > I'd like to ping 2 patches: > > - UBSAN -fsanitize=pointer-overflow support > - http://gcc.gnu.org/ml/gcc-patches/2017-06/msg01365.html The probablility stuff might need updating? Can you put the TYPE_PRECISION (sizetype) != POINTER_SIZE

Re: [PATCH GCC][1/2]Feed bound computation to folder in loop split

On Wed, Jul 26, 2017 at 11:57 AM, Marc Glisse wrote: > On Wed, 26 Jul 2017, Richard Sandiford wrote: > >> Marc Glisse writes: >>> >>> On Wed, 26 Jul 2017, Richard Sandiford wrote: Richard Biener writes: >

Re: [PATCH v2] [SPARC] Add -mfsmuld option

> Add the -mfsmuld option to control the generation of the FsMULd > instruction. In general, this instruction is available in architecture > version V8 and V9 CPUs with FPU. Some CPUs of this category do not > support this instruction properly, e.g. AT697E, AT697F and UT699. Some > CPUs of this

[PATCH 1/2, c-family]: Add 'z' to asm_fprintf_char_table

Hello! This patch is the prerequisite for my next patch. It enables %z extension for asm_fprintf that will emit 'q' or 'l' suffixes for instructions with word-sized operands. 2017-07-26 Uros Bizjak * c-format.c (asm_fprintf_char_table): Add 'z' to format_chars.

Re: [PATCH v2] [SPARC] Add -mfsmuld option

On 26/07/17 14:13, Eric Botcazou wrote: Add the -mfsmuld option to control the generation of the FsMULd instruction. In general, this instruction is available in architecture version V8 and V9 CPUs with FPU. Some CPUs of this category do not support this instruction properly, e.g. AT697E,

Re: PING^2: Fwd: SSA range class and removal of VR_ANTI_RANGEs

On Tue, Jul 25, 2017 at 4:50 PM, Andrew MacLeod wrote: > On 07/25/2017 03:12 AM, Richard Biener wrote: >> >> On Fri, Jul 21, 2017 at 9:30 PM, Aldy Hernandez wrote: >>> >>> On Mon, Jul 17, 2017 at 6:23 AM, Richard Biener >>>

Re: [PATCH v2] [SPARC] Add -mfsmuld option

On 26/07/17 14:44, Sebastian Huber wrote: On 26/07/17 14:13, Eric Botcazou wrote: Add the -mfsmuld option to control the generation of the FsMULd instruction. In general, this instruction is available in architecture version V8 and V9 CPUs with FPU. Some CPUs of this category do not support

Re: [PATCH] Fix infinite recursion with div-by-zero (PR middle-end/70992)

On Tue, Jul 25, 2017 at 03:47:31PM +0200, Richard Biener wrote: > On Tue, Jul 25, 2017 at 3:30 PM, Eric Botcazou wrote: > >> Eric, any comments? > > > > No objection for the build2_stat hunk, I think it's in keeping with the Ada > > semantics. But the tree_could_trap_p

Re: Patch ping

On Wed, Jul 26, 2017 at 12:34:10PM +0200, Richard Biener wrote: > On Tue, 25 Jul 2017, Jakub Jelinek wrote: > > > Hi! > > > > I'd like to ping 2 patches: > > > > - UBSAN -fsanitize=pointer-overflow support > > - http://gcc.gnu.org/ml/gcc-patches/2017-06/msg01365.html > > The probablility

New Spanish PO file for 'gcc' (version 7.1.0)

Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Spanish team of translators. The file is available at: http://translationproject.org/latest/gcc/es.po (This file, 'gcc-7.1.0.es.po', has

[PATCH v2] [SPARC] Add -mfsmuld option

Add the -mfsmuld option to control the generation of the FsMULd instruction. In general, this instruction is available in architecture version V8 and V9 CPUs with FPU. Some CPUs of this category do not support this instruction properly, e.g. AT697E, AT697F and UT699. Some CPUs of this category

[patch,v7,committed] Backport PR81487: asprintf -> xasprintf

Applied as https://gcc.gnu.org/r250562 Johann lto-plugin/ Backport from 2017-07-21 trunk r250428. PR lto/81487 * lto-plugin.c (claim_file_handler): Use xasprintf instead of asprintf. [hi!=0]: Swap hi and lo arguments supplied to xasprintf. gcc/

Re: Fix thinko in gimple_assign_set_rhs_with_ops

On Tue, Jul 25, 2017 at 3:59 PM, Eric Botcazou wrote: > The attached fix to the Ada front-end introduces a regression in the ACATS > testsuite for cb4009a. The backtrace is: > > #0 operation_could_trap_helper_p(tree_code, bool, bool, bool, bool, > tree_node*, bool*) () at

Re: [PATCH 1/2] [SPARC] Drop superfluous MASK_FPU enable

> All TARGET_DEFAULT defines set MASK_FPU. There is no need to set it in > some CPU target flags enable. > > gcc/ > config/sparc/sparc.c (sparc_option_override): Remove MASK_FPU > from all CPU target flags enable members. OK for mainline and 7 branch, thanks. -- Eric Botcazou

Re: [PATCH GCC][1/2]Feed bound computation to folder in loop split

On Wed, 26 Jul 2017, Richard Sandiford wrote: Marc Glisse writes: On Wed, 26 Jul 2017, Richard Sandiford wrote: Richard Biener writes: On Tue, Jul 25, 2017 at 7:45 PM, Marc Glisse wrote: On Tue, 25 Jul 2017, Richard

RE: [PATCH] Switch vec_init and vec_extract optabs to 2 mode optab to allow extraction of vector from vector or initialization of vector from smaller vectors (PR target/80846)

On Tue, 25 Jul 2017, Matthew Fortune wrote: > Jakub Jelinek writes: > > Bootstrapped/regtested on x86_64-linux and i686-linux, where it improves > > e.g. the code generation for slp-43.c and slp-45.c testcases. > > make cc1 tested in cross-compilers to the remaining targets. >

[PATCH,AIX] Fully enable XCOFF in libbacktrace on AIX

Description: * This patch fully enables XCOFF in libbacktrace on AIX. Tests: * Fedora25/x86_64 + GCC v7.1.0 : Configure/Build: SUCCESS - build made by means of gmake. ChangeLog: * configure.ac, filetype.awk: Separate AIX XCOFF32 and XCOFF64. * xcoff.c: Add support for AIX XCOFF32 and

Re: [patch] Ad PR81487: More asprintf -> xasprintf replacements

On Tue, Jul 25, 2017 at 10:01 PM, Georg-Johann Lay wrote: > Richard Biener schrieb: >> >> On Mon, Jul 24, 2017 at 10:19 AM, Georg-Johann Lay wrote: >>> >>> Hi, as proposed in >>> >>> https://gcc.gnu.org/ml/gcc-patches/2017-07/msg01294.html >>> >>> this patch does

Re: [PATCH 2/2] [SPARC] Add -mfsmuld option

> Add the -mfsmuld option to control the generation of the FsMULd > instruction. In general, this instruction is available in architecture > version V8 and V9 CPUs with FPU. Some CPUs of this category do not > support this instruction properly, e.g. AT697E, AT697F and UT699. Some > CPUs of this

Re: [PATCH 2/2] [SPARC] Add -mfsmuld option

On 26/07/17 10:13, Sebastian Huber wrote: On 26/07/17 10:09, Eric Botcazou wrote: Add the -mfsmuld option to control the generation of the FsMULd instruction. In general, this instruction is available in architecture version V8 and V9 CPUs with FPU. Some CPUs of this category do not support

Re: [PATCH 2/2] [SPARC] Add -mfsmuld option

On 26/07/17 10:09, Eric Botcazou wrote: What about automatically setting MASK_FSMULD if MASK_FPU is set and disabling it by default for v7, cypress and leon (i.e MASK_ISA|MASK_FSMULD for them)? What about the SPARCLITE and SPARCLET processors? I guess they previously didn't use the FSMULD

Re: [PATCH GCC][1/2]Feed bound computation to folder in loop split

Marc Glisse writes: > On Wed, 26 Jul 2017, Richard Sandiford wrote: >> Richard Biener writes: >>> On Tue, Jul 25, 2017 at 7:45 PM, Marc Glisse wrote: On Tue, 25 Jul 2017, Richard Biener wrote: >> I think we

[PATCH 1/2] [SPARC] Drop superfluous MASK_FPU enable

All TARGET_DEFAULT defines set MASK_FPU. There is no need to set it in some CPU target flags enable. gcc/ config/sparc/sparc.c (sparc_option_override): Remove MASK_FPU from all CPU target flags enable members. --- gcc/config/sparc/sparc.c | 10 +- 1 file changed, 5

[PATCH 2/2] [SPARC] Add -mfsmuld option

Add the -mfsmuld option to control the generation of the FsMULd instruction. In general, this instruction is available in architecture version V8 and V9 CPUs with FPU. Some CPUs of this category do not support this instruction properly, e.g. AT697E, AT697F and UT699. Some CPUs of this category

Re: [PATCH] Switch vec_init and vec_extract optabs to 2 mode optab to allow extraction of vector from vector or initialization of vector from smaller vectors (PR target/80846)

> Bootstrapped/regtested on x86_64-linux and i686-linux, where it improves > e.g. the code generation for slp-43.c and slp-45.c testcases. > make cc1 tested in cross-compilers to the remaining targets. The SPARC bits are OK by me. -- Eric Botcazou

Re: [RFC] If conversion min/max search, costs and problems

> Do you have an example where wrong code is generated through the > noce_convert_multiple_sets_p path (with or without bodged costs)? > > Both AArch64 and x86-64 reject your testcase along this codepath because > of the constant set of 1. If we work around that by setting bla = n rather > than

Re: [PATCH GCC][1/2]Feed bound computation to folder in loop split

On Wed, 26 Jul 2017, Richard Sandiford wrote: Richard Biener writes: On Tue, Jul 25, 2017 at 7:45 PM, Marc Glisse wrote: On Tue, 25 Jul 2017, Richard Biener wrote: I think we need Richard to say what the intent is for the valueization

Re: [PATCH][AArch64] Add addr_type attribute

Hi James, Thanks for the review and comments on the patch. >> What am I missing - you add a new function which is never called? >> Should this have been in series with a scheduling model change? Sorry. You are right. This patch is one in series for scheduling and addition of attributes to

Re: [PATCH 2/2] [SPARC] Add -mfsmuld option

On 26/07/17 10:09, Eric Botcazou wrote: Add the -mfsmuld option to control the generation of the FsMULd instruction. In general, this instruction is available in architecture version V8 and V9 CPUs with FPU. Some CPUs of this category do not support this instruction properly, e.g. AT697E,

Re: [PATCH GCC][1/2]Feed bound computation to folder in loop split

Richard Biener writes: > On Tue, Jul 25, 2017 at 7:45 PM, Marc Glisse wrote: >> On Tue, 25 Jul 2017, Richard Biener wrote: >> I think we need Richard to say what the intent is for the valueization function. It is used both to stop

Re: [PATCH] Switch vec_init and vec_extract optabs to 2 mode optab to allow extraction of vector from vector or initialization of vector from smaller vectors (PR target/80846)

On Tue, Jul 25, 2017 at 03:52:56PM -0500, Segher Boessenkool wrote: > On Tue, Jul 25, 2017 at 11:14:32AM +0200, Jakub Jelinek wrote: > > This patch only adds new vector from vector extract and init patterns to > > the i386 backend, but I had to change many other targets too, because > > it needs

Re: [PATCH GCC][1/2]Feed bound computation to folder in loop split

On Tue, Jul 25, 2017 at 7:45 PM, Marc Glisse wrote: > On Tue, 25 Jul 2017, Richard Biener wrote: > >>> I think we need Richard to say what the intent is for the valueization >>> function. It is used both to stop looking at defining stmt if the return >>> is >>> NULL, and to

Re: [PATCH] Add macro DISABLE_COPY_AND_ASSIGN

On 17-07-19 10:30:45, Yao Qi wrote: > We have many classes that copy cotr and assignment operator are deleted > in different projects, gcc, gdb and gold. So this patch adds a macro > to do this, and replace these existing mechanical code with macro > DISABLE_COPY_AND_ASSIGN. > > The patch was

[PATCH 2/2] Introduce Python testcases to check DWARF output

For now, this supports only platforms that have an objdump available for the corresponding target. There are several things that would be nico to have in the future: * add support for more DWARF dumping tools, such as otool on Darwin; * have a DWARF location expression decoder, to be able to

[PATCH 1/2] Introduce testsuite support to run Python tests

gcc/testsuite/ * lib/gcc-python.exp: New test library. * python/testutils.py: New Python helper. --- gcc/testsuite/lib/gcc-python.exp | 95 +++ gcc/testsuite/python/testutils.py | 45 +++ 2 files changed, 140 insertions(+)

[PATCH 0/2] Python testcases to check DWARF output

Hello, At the last GNU Cauldron, Richard Biener and I talked about DWARF output testing. Except for guality tests, which are disabled on several targets, the only way tests check the DWARF is scanning the annotated assembly (-dA), making it hard to write reliable tests. For instance, checking

[PATCH] Backport to GCC7

Hi. I'm going to install following 3 revision in order to fix GCC.x branch: 0001-Backport-r249728.patch 0002-Backport-r249833.patch 0003-Backport-r250561.patch Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. Martin >From ce5f2421462ebf6bc02eec9729502671c3483883 Mon

Re: [PATCH 12/17] Add server.h and server.c

On Mon, 2017-07-24 at 16:05 -0400, David Malcolm wrote: >  > + > +You should have received a copy of the GNU General Public License > +along with GCC; see the file COPYING3.  If not see > +.  */ > + > +#ifndef GCC_SERVER_H > +#define GCC_SERVER_H > + > +/* Wrapper

[PATCH,AIX] Enable libffi for AIX

Description: * This patch enables libffi on AIX. Tests: * Fedora25/x86_64 + GCC trunk : Configure/Build: SUCCESS - build made by means of gmake. ChangeLog: * configure.ac, configure: Enable libffi for AIX Cordialement, Tony Reix Bull - ATOS IBM Coop Architect & Technical Leader Office :

Re: [PATCH] PR libstdc++/53984 handle exceptions in basic_istream::sentry

Hi, On 26/07/2017 16:21, Andreas Schwab wrote: ERROR: 27_io/basic_fstream/53984.cc: unknown dg option: dg-require-file-io 18 {} for " dg-require-file-io 18 "" " Should be already fixed, a trivial typo. Thanks, Paolo.

Re: [PATCH 12/17] Add server.h and server.c

On Wed, 2017-07-26 at 23:35 +0900, Oleg Endo wrote: > On Mon, 2017-07-24 at 16:05 -0400, David Malcolm wrote: > > > > + > > +You should have received a copy of the GNU General Public License > > +along with GCC; see the file COPYING3. If not see > > +.  */ > > + >

Re: [PATCH v12] add -fpatchable-function-entry=N,M option

On Wed, Jul 26, 2017 at 04:16:25PM +0200, Andreas Schwab wrote: > On Jul 07 2017, Torsten Duwe wrote: > > > diff --git a/gcc/testsuite/c-c++-common/patchable_function_entry-decl.c > > b/gcc/testsuite/c-c++-common/patchable_function_entry-decl.c > > new file mode 100644 > > index

Re: [PATCH v2] [SPARC] Add -mfsmuld option

> I would like to add this to the web site: > > Index: htdocs/gcc-7/changes.html > === > RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-7/changes.html,v > retrieving revision 1.88 > diff -r1.88 changes.html > 1248a1249,1251 > > > Use of

Re: Patch ping

On Wed, 26 Jul 2017, Jakub Jelinek wrote: > On Wed, Jul 26, 2017 at 12:34:10PM +0200, Richard Biener wrote: > > On Tue, 25 Jul 2017, Jakub Jelinek wrote: > > > > > Hi! > > > > > > I'd like to ping 2 patches: > > > > > > - UBSAN -fsanitize=pointer-overflow support > > > -

Re: [PATCH] PR libstdc++/53984 handle exceptions in basic_istream::sentry

On Jul 25 2017, Jonathan Wakely wrote: > diff --git a/libstdc++-v3/testsuite/27_io/basic_fstream/53984.cc > b/libstdc++-v3/testsuite/27_io/basic_fstream/53984.cc > new file mode 100644 > index 000..e84072e > --- /dev/null > +++

RE: [PATCH][GCC][AArch64] optimize float immediate moves (1 /4) - infrastructure.

Hi James, I have updated the patch and have responded to your question blow. Ok for trunk? Thanks, Tamar > > static bool > > @@ -5857,12 +5955,6 @@ aarch64_preferred_reload_class (rtx x, > reg_class_t regclass) > >return NO_REGS; > > } > > > > - /* If it's an integer immediate

Re: [PATCH v12] add -fpatchable-function-entry=N,M option

On Jul 07 2017, Torsten Duwe wrote: > diff --git a/gcc/testsuite/c-c++-common/patchable_function_entry-decl.c > b/gcc/testsuite/c-c++-common/patchable_function_entry-decl.c > new file mode 100644 > index 000..8514b10e820 > --- /dev/null > +++

Re: [PATCH v12] add -fpatchable-function-entry=N,M option

On Jul 26 2017, Torsten Duwe wrote: > On Wed, Jul 26, 2017 at 04:16:25PM +0200, Andreas Schwab wrote: >> On Jul 07 2017, Torsten Duwe wrote: >> >> > diff --git a/gcc/testsuite/c-c++-common/patchable_function_entry-decl.c >> >

Re: [PATCH 1/2] Introduce testsuite support to run Python tests

On Wed, 2017-07-26 at 18:35 +0200, Pierre-Marie de Rodat wrote: > On 07/26/2017 06:25 PM, David Malcolm wrote: > > str.format was introduced in Python 2.6, so presumably the minimum > > python 2 version here is at least 2.6+; for Python 3 I believe it > > was > > present in Python 3.0 onwards. >

Re: ping [PATCH] [MSP430] Fix PR78849: ICE on initialization of global struct containing __int20 array

On 05/19/2017 07:35 AM, Jozef Lawrynowicz wrote: > Original post: https://gcc.gnu.org/ml/gcc-patches/2017-04/msg01030.html > > The attached patch fixes an issue for the msp430 target where the TYPE_SIZE of > the __int20 type was set using the precision (20 bits) instead of the > in-memory > size

Re: [PATCH] Improve extraction of changed file in contrib/mklog

On 07/09/2017 01:03 PM, Yuri Gribov wrote: > Hi, > > Currently mklog will fail to analyze lines like this in patches: > diff -rupN gcc/gcc/testsuite/lib/profopt.exp > gcc-compare-checks/gcc/testsuite/lib/profopt.exp > (it fails with "Error: failed to parse diff for ... and ..."). > > This patch

Re: [PATCH v2] New C++ warning option '-Waccess-specifiers'

On 7/24/17, Franz Sirl wrote: > Am 2017-07-24 um 00:19 schrieb Volker Reichelt: >> On 23 Jul, Eric Gallager wrote: >>> On 7/23/17, Volker Reichelt wrote: Hi again, here is an updated patch for a new warning about

Re: [PATCH] toplev: avoid recursive emergency_dump_function

On Sat, 22 Jul 2017, Segher Boessenkool wrote: > On Thu, Jul 20, 2017 at 05:40:28PM +0300, Alexander Monakov wrote: > > Segher pointed out on IRC that ICE reporting with dumps enabled got worse: > > if emergency_dump_function itself leads to an ICE (e.g. by segfaulting), > > nested ICE reporting

[Patch AArch64 Obvious] Unify branch costs to generic_branch_cost

Hi, All the cores in AArch64 use the pair {1, 3} for their branch costs. As that is covered by generic_branch_cost, we can just use that directly and save the tiny amount of redundant code. If in future any core wants to modify this, they can always add a special-case branch-cost back. Comitted

Re: [PATCH] Fix when -lssp is added by driver (PR middle-end/81400).

On 07/12/2017 07:38 AM, Martin Liška wrote: > Hi. > > Following patch adds -lspp when one uses -mstack-protector-guard=global. > > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. > > Ready to be installed? > > Martin > > gcc/ChangeLog: > > 2017-07-12 Martin Liska

Re: [C++ Patch] PR 71570 ("[6/7/8 regression] ICE on invalid variable capture in cxx_incomplete_type_diagnostic...")

OK; this is trivial enough for 6/7/8. On Wed, Jul 26, 2017 at 2:49 PM, Paolo Carlini wrote: > Hi, > > avoiding the error recovery issues noticed in the bug seems just matter of > early returning error_mark_node from add_capture, like we already do a few > lines below in

Re: [PATCH] Improve alloca alignment

On 07/26/2017 11:39 AM, Wilco Dijkstra wrote: > This patch improves alloca alignment. Currently alloca reserves > too much space as it aligns twice, and generates unnecessary stack > alignment code. For example alloca (16) generates: > > sub sp, sp, #32 ??? > mov x1, sp >

Re: [PATCH] x86: Properly check register CFA offset

On Wed, Jul 26, 2017 at 6:14 PM, H.J. Lu wrote: > > X86 epilogue saves register at CFA offset. Since its location on stack > is computed as CFA - its CFA_OFFSET, CFA_OFFSET points the end of the > saved register location on stack. This patch updates sp_valid_at and >

Re: [PING] [PATCH v4 0/12] [i386] Improve 64-bit Microsoft to System V ABI pro/epilogues

On Sun, May 14, 2017 at 3:23 AM, Uros Bizjak wrote: > On Sun, May 14, 2017 at 12:34 AM, Daniel Santos > wrote: >> On 05/13/2017 11:52 AM, Uros Bizjak wrote: >>> >>> On Sat, May 13, 2017 at 1:01 AM, Daniel Santos >>> wrote:

[C++ Patch] PR 71570 ("[6/7/8 regression] ICE on invalid variable capture in cxx_incomplete_type_diagnostic...")

Hi, avoiding the error recovery issues noticed in the bug seems just matter of early returning error_mark_node from add_capture, like we already do a few lines below in a similar case of ill-formed capture. Tested x86_64-linux. If we are going to fix this in a simply way, maybe we could also

[PATCH v5] aarch64: Add split-stack initial support

This is an update patch based on my previous submission [1]. The changes from previous version are: - Update aarch64_supports_split_stack to return true an let the loader to actually emit an error if it does not provide the required TCB field. The TCB field support is planed for GLIBC

Re: [PATCH][AArch64] Simplify frame layout for stack probing

On 07/25/2017 07:58 AM, Wilco Dijkstra wrote: > This patch makes some changes to the frame layout in order to simplify > stack probing. We want to use the save of LR as a probe in any non-leaf > function. With shrinkwrapping we may only save LR before a call, so it > is useful to define a fixed

Re: [PATCH] x86: Properly check register CFA offset

On Wed, Jul 26, 2017 at 12:04 PM, Uros Bizjak wrote: > On Wed, Jul 26, 2017 at 6:14 PM, H.J. Lu wrote: >> >> X86 epilogue saves register at CFA offset. Since its location on stack >> is computed as CFA - its CFA_OFFSET, CFA_OFFSET points the end of the

Re: [PATCH] enhance -Wrestrict for sprintf %s arguments

On 07/19/2017 10:10 AM, Martin Sebor wrote: > On 07/19/2017 12:42 AM, Jeff Law wrote: >> On 07/02/2017 02:00 PM, Martin Sebor wrote: >>> The attached patch enhances the -Wrestrict warning to detect more >>> than just trivial instances of overlapping copying by sprintf and >>> related functions.

[PATCH] Fix PR middle-end/81564: ICE in group_case_labels_stmt()

The test case for PR81564 exposes an issue where the case labels for a switch statement point to blocks that have already been removed by an earlier call to cleanup_tree_cfg(). In that case, the code in group_case_labels_stmt() that does: base_bb = label_to_block (CASE_LABEL (base_case));

Re: [PATCH 1/2] x86,s390: add compiler memory barriers when expanding atomic_thread_fence (PR 80640)

On Wed, 26 Jul 2017, Alexander Monakov wrote: > On Wed, 26 Jul 2017, Jeff Law wrote: > > I'm not sure what you mean by extraneous compiler barriers -- isn't the > > worst case scenario here that the target emits them as well? So there > > would be an extraneous one in that case, but that ought

Re: [PATCH #4, cleanup] Remove PowerPC -mvsx-small-integer

Hi Mike, On Wed, Jul 26, 2017 at 12:24:17AM -0400, Michael Meissner wrote: > * config/rs6000/rs6000-cpus.def (ISA_2_7_MASKS_SERVER): Delete > -mvsx-small-integer option. > (ISA_3_0_MASKS_IEEE): Likewise. > (POWERPC_MASKS): Likewise. (OTHER_VSX_VECTOR_MASKS): Likewise. >

[PATCH,AIX] Changes for linking gotools on AIX.

Description: * This patch adds linker options for gotools for AIX. Tests: * Fedora25/x86_64 + GCC trunk : Configure/Build: SUCCESS - build remade by means of gmake. - some test redone in libgo (gmake check) * AIX + GCC 7.1.0 : - build remade by means of gmake. - some test redone in

Re: [PATCH 1/2] x86,s390: add compiler memory barriers when expanding atomic_thread_fence (PR 80640)

On 05/26/2017 04:58 AM, Alexander Monakov wrote: > On Wed, 17 May 2017, Alexander Monakov wrote: > >> Ping. > > Ping^2? > >> (to be clear, patch 2/2 is my previous followup in this thread, I forgot to >> adjust the subject line; it should have said: >> "[PATCH 2/2] x86: add compiler memory

Re: [PATCH] Fix indirect call optimization done by autoFDO.

On 07/11/2017 04:37 AM, Martin Liška wrote: > Hello. > > Following is a typo fix which nobody has noticed during testing of > e.g. gcc/testsuite/gcc.dg/tree-prof/indir-call-prof.c. > > > Patch can bootstrap and survives regression tests. > > Ready for trunk? > Thanks, > Martin > > >

C++ PATCH to implement P0702R1, list-deduction of vector

At the Toronto meeting we adjusted the semantics of class template argument deduction for a class such as std::vector that has an initializer-list constructor: previously, we would get vector v1 { 42 }; // vector vector v2 { v1 }; // vector but now v2 is also deduced to vector. Tested

[Patch AArch64 obvious] Unify address costs to generic_addrcost_table

Hi, The special case address cost tables for Cortex-A57 and qdf24xx are no different from the generic address cost table. We should just use the generic address cost table directly. If this changes in future, a core is welcome to add new address cost tables. Comitted as obvious as revision

Re: [PATCH] Initialize counters in autoFDO to zero, not to uninitialized.

On 07/11/2017 04:35 AM, Martin Liška wrote: > Hello. > > This fixes majority of autoFDO test-cases. > > Patch can boostrap and survives regression tests. > > Ready for trunk? > Thanks, > Martin > > gcc/ChangeLog: > > 2017-07-11 Martin Liska > > * auto-profile.c

Re: [PATCH 1/2] Introduce testsuite support to run Python tests

On 07/26/2017 06:25 PM, David Malcolm wrote: str.format was introduced in Python 2.6, so presumably the minimum python 2 version here is at least 2.6+; for Python 3 I believe it was present in Python 3.0 onwards. Hm… Python 2.6 is fairly old: last binary release was ages ago, last source

Re: Patch ping

On Wed, Jul 26, 2017 at 04:13:30PM +0200, Richard Biener wrote: > > > You don't seem to use 'size' anywhere. > > > > size I thought about but then decided not to do anything with it. > > There are two cases, one is where there is no ADDR_EXPR and it actually > > a memory reference. > > In that

Re: [PATCH] PR libstdc++/53984 handle exceptions in basic_istream::sentry

Hi again, On 26/07/2017 16:27, Paolo Carlini wrote: Hi, On 26/07/2017 16:21, Andreas Schwab wrote: ERROR: 27_io/basic_fstream/53984.cc: unknown dg option: dg-require-file-io 18 {} for " dg-require-file-io 18 "" " Should be already fixed, a trivial typo. ... but now the new test simply fails

[PATCH] x86: Properly check register CFA offset

X86 epilogue saves register at CFA offset. Since its location on stack is computed as CFA - its CFA_OFFSET, CFA_OFFSET points the end of the saved register location on stack. This patch updates sp_valid_at and fp_valid_at to properly check register CFA offset. Tested on x86-64. OK for trunk

Re: [PATCH 0/2] Python testcases to check DWARF output

On 07/26/2017 06:15 PM, David Malcolm wrote: (FWIW I'm a big fan of Python, so am happy to see this proposal) Me too. :-) Which version of Python did you test against? As far as I can see you've coded this using the common subset of Python 2 and Python 3; it's worth spelling out what the

[PATCH] PR c++/67054 Allow inheriting constructor with non-default-constructible members

This patch addresses PR 67054 (duplicates 62310, 80851). An implicitly-defined inheriting constructor was wrongly considered deleted when it would initialize a non-default-constructible member, even when a brace-or-equal-initializer was present. The bug only affects deduction of the

Re: [PATCH] PR c++/67054 Allow inheriting constructor with non-default-constructible members

OK, thanks. On Wed, Jul 26, 2017 at 12:45 PM, Leonid Koppel wrote: > This patch addresses PR 67054 (duplicates 62310, 80851). An implicitly-defined > inheriting constructor was wrongly considered deleted when it would > initialize a non-default-constructible member, even

Re: [PATCH,AIX] Changes for linking gotools on AIX.

On Wed, Jul 26, 2017 at 12:41 PM, REIX, Tony wrote: > Description: > * This patch adds linker options for gotools for AIX. > > Tests: > * Fedora25/x86_64 + GCC trunk : Configure/Build: SUCCESS >- build remade by means of gmake. >- some test redone in libgo (gmake

Re: [PATCH 2/2] Introduce Python testcases to check DWARF output

On Wed, 2017-07-26 at 18:00 +0200, Pierre-Marie de Rodat wrote: [...] > diff --git a/gcc/testsuite/python/dwarfutils/__init__.py > b/gcc/testsuite/python/dwarfutils/__init__.py > new file mode 100644 > index 000..246fbbd15be > --- /dev/null > +++

Re: [PATCH 00/17] RFC: New source-location representation; Language Server Protocol

On 07/24/2017 01:04 PM, David Malcolm wrote: * The LSP implementation is a just a proof-of-concept, to further motivate capturing the extra data. Turning it into a "proper" LSP server implementation would be a *lot* more work, and I'm unlikely to actually do that (but maybe someone on

Re: [PATCH 1/2] Introduce testsuite support to run Python tests

On Wed, 2017-07-26 at 18:00 +0200, Pierre-Marie de Rodat wrote: [...snip...] > diff --git a/gcc/testsuite/python/testutils.py > b/gcc/testsuite/python/testutils.py > new file mode 100644 > index 000..503105ad9d0 > --- /dev/null > +++ b/gcc/testsuite/python/testutils.py > @@ -0,0 +1,45 @@

Re: [rs6000] Avoid rotates of floating-point modes

Hi! On Tue, Jul 25, 2017 at 04:10:19PM +0100, Richard Sandiford wrote: > Segher Boessenkool writes: > --- gcc/config/rs6000/rs6000.c2017-07-13 09:25:13.909213921 +0100 > +++ gcc/config/rs6000/rs6000.c2017-07-25 11:14:27.692739547 +0100 > @@ -10503,17

Re: [PATCH 1/2] x86,s390: add compiler memory barriers when expanding atomic_thread_fence (PR 80640)

On Wed, 26 Jul 2017, Jeff Law wrote: > I'm not sure what you mean by extraneous compiler barriers -- isn't the > worst case scenario here that the target emits them as well? So there > would be an extraneous one in that case, but that ought to be a "don't > care". Yes, exactly this. > In the

Re: [PATCH 0/2] Python testcases to check DWARF output

On Wed, 2017-07-26 at 18:00 +0200, Pierre-Marie de Rodat wrote: > Hello, > > At the last GNU Cauldron, Richard Biener and I talked about DWARF > output > testing. Except for guality tests, which are disabled on several > targets, the only way tests check the DWARF is scanning the annotated >

Re: [PATCH 1/2] x86,s390: add compiler memory barriers when expanding atomic_thread_fence (PR 80640)

On Wed, 26 Jul 2017, Jeff Law wrote: > So I think this is up to the target maintainers. I have no concerns > with enabling use of expand_asm_memory_barrier to be used outside of > optabs. So if the s390/x86 maintainers want to go forward, the optabs > changes are pre-approved. Please see the

  1   2   >