Re: [PATCH] Fix -fself-test ICE in non-english locale (PR bootstrap/77569)

2017-01-03 Thread Jakub Jelinek
On Tue, Jan 03, 2017 at 08:09:03PM -0500, David Malcolm wrote: > > LC_ALL=en_US.UTF-8 and LC_ALL=de_DE.UTF-8). Ok for trunk? > > Thanks for looking at this; OK for trunk. Thanks. > I wonder if it makes sense to add a target to gcc/Makefile.in to run > the selftests in a non-English locale, to c

Re: C/C++ PATCH to implement -Wpointer-compare warning (PR c++/64767)

2017-01-03 Thread Eric Gallager
On 10/2/16, Jason Merrill wrote: > OK, thanks. > > On Sat, Oct 1, 2016 at 10:16 AM, Marek Polacek wrote: >> On Fri, Sep 30, 2016 at 05:48:03PM -0400, Jason Merrill wrote: >>> On Fri, Sep 30, 2016 at 12:43 PM, Marek Polacek >>> wrote: >>> > On Fri, Sep 23, 2016 at 10:31:33AM -0400, Jason Merrill

Re: Implement -Wduplicated-branches (PR c/64279) (v3)

2017-01-03 Thread Eric Gallager
On 11/3/16, Jakub Jelinek wrote: > On Thu, Nov 03, 2016 at 09:27:55AM -0400, Jason Merrill wrote: >> On Thu, Nov 3, 2016 at 7:24 AM, Marek Polacek wrote: >> > On Tue, Nov 01, 2016 at 02:53:58PM +0100, Jakub Jelinek wrote: >> >> On Tue, Nov 01, 2016 at 09:41:20AM -0400, Jason Merrill wrote: >> >>

Contents of PO file 'cpplib-7.1-b20170101.ru.po'

2017-01-03 Thread Translation Project Robot
cpplib-7.1-b20170101.ru.po.gz Description: Binary data The Translation Project robot, in the name of your translation coordinator.

New Russian PO file for 'cpplib' (version 7.1-b20170101)

2017-01-03 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'cpplib' has been submitted by the Russian team of translators. The file is available at: http://translationproject.org/latest/cpplib/ru.po (This file, 'cpplib-7.1-b20170101

Re: [PATCH] c++/78771 ICE with inheriting ctor

2017-01-03 Thread Jason Merrill
On 12/19/2016 08:09 AM, Nathan Sidwell wrote: this patch fixes 78771, were an assert fires due to recursive instantiation of an inheriting ctor. Normally when a recursive instantiation is needed, we've already constructed and registered the declaration, so simply return it. For ctors though we

Patch Pings for fixes to bz33562 and bz61912.

2017-01-03 Thread Jeff Law
Pinging patches #1 and #2 from the 4 part series to improve DSE. ISTM that #3 and #4 should wait for gcc-7. Patches #1 and #2 included for reference. --- Begin Message --- This is the first of the 4 part patchkit to address deficiencies in our DSE implementation. This patch addresses the

Re: [PATCH] c++/78765

2017-01-03 Thread Jason Merrill
On 12/16/2016 07:23 AM, Nathan Sidwell wrote: when cxx_eval_constant_expression finds a nonconstant expression it returns a TREE without TREE_CONSTANT set. else if (non_constant_p && TREE_CONSTANT (r)) { /* This isn't actually constant, so unset TREE_CONSTANT. */ ... else //

Re: [PATCH] PR c++/66735 lambda capture by reference

2017-01-03 Thread Jason Merrill
On 01/03/2017 08:57 AM, Nathan Sidwell wrote: else if (!is_this && explicit_init_p) { - type = make_auto (); - type = do_auto_deduction (type, expr, type); + tree auto_node = make_auto (); + + type = auto_node; + if (by_reference_p) + { + /* Add the

[PATCH][PR tree-optimization/78856] Invalidate cached iteration information when threading across multiple loop headers

2017-01-03 Thread Jeff Law
So as noted in the BZ comments the jump threading code has code that detects when a jump threading path wants to cross multiple loop headers and truncates the jump threading path in that case. What we should have done instead is invalidate the cached loop information. Additionally, this BZ s

[doc,libstdc++] sourceforge.net defaults to https, adjust reference

2017-01-03 Thread Gerald Pfeifer
Applied. Gerald 2017-01-03 Gerald Pfeifer * doc/xml/manual/documentation_hacking.xml: sourceforge.net now defaults to https; adjust reference. Index: doc/xml/manual/documentation_hacking.xml === --- doc/xml/manua

[doc,rfc] clean up preprocessor option documentation

2017-01-03 Thread Sandra Loosemore
This patch is another installment in my series to clean up the preprocessor documentation. Most of the changes here are just copy-editing for style and markup and fixing/removing obvious bit-rot, but there's also one chunk that I've rewritten completely. The rewrite involves combining the dis

New Brazilian Portuguese PO file for 'cpplib' (version 7.1-b20170101)

2017-01-03 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'cpplib' has been submitted by the Brazilian Portuguese team of translators. The file is available at: http://translationproject.org/latest/cpplib/pt_BR.po (This file, 'cppl

Contents of PO file 'cpplib-7.1-b20170101.pt_BR.po'

2017-01-03 Thread Translation Project Robot
cpplib-7.1-b20170101.pt_BR.po.gz Description: Binary data The Translation Project robot, in the name of your translation coordinator.

Re: [PATCH] Fix -fself-test ICE in non-english locale (PR bootstrap/77569)

2017-01-03 Thread David Malcolm
On Wed, 2017-01-04 at 00:01 +0100, Jakub Jelinek wrote: > Hi! > > The cb.error hook is called in the case we are looking for with > _("conversion from %s to %s not supported by iconv") > where _(msgid) is dgettext ("cpplib", msgid), so if performing -fself > -test > on iconv that doesn't support e

GCC patch committed: Fix -fdump-go-spec for enums

2017-01-03 Thread Ian Lance Taylor
This patch to GCC fixes the -fdump-go-spec output for enums. Previously GCC always printed "int", which is often incorrect as the Go type "int" is 64 bits on a 64-bit hosts, while enums are typically 32 bits. This was mostly harmless but caused the representation of the ffi_cif type from libffi to

[PATCH] C++: fix fix-it hints for misspellings within explicit namespaces

2017-01-03 Thread David Malcolm
PR c++/77829 and PR c++/78656 identify an issue within the C++ frontend where it issues nonsensical fix-it hints for misspelled name lookups within an explicitly given namespace: it finds the closest name within all namespaces, and uses the location of the namespace for the replacement, rather than

Contents of PO file 'cpplib-7.1-b20170101.fi.po'

2017-01-03 Thread Translation Project Robot
cpplib-7.1-b20170101.fi.po.gz Description: Binary data The Translation Project robot, in the name of your translation coordinator.

New Finnish PO file for 'cpplib' (version 7.1-b20170101)

2017-01-03 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'cpplib' has been submitted by the Finnish team of translators. The file is available at: http://translationproject.org/latest/cpplib/fi.po (This file, 'cpplib-7.1-b20170101

[PATCH] move snprintf truncation warnings under own option ()

2017-01-03 Thread Martin Sebor
The -Wformat-length option warns about both overflow and truncation. I had initially debated introducing two options, one for each of the two kinds of problems, but decided to go with just one and consider breaking it up based on feedback. I feel that there has now been sufficient feedback (e.g.,

[PATCH] Change DWARF5 .debug_loclists location description sizes from 2-byte length to uleb128 lengths

2017-01-03 Thread Jakub Jelinek
Hi! http://dwarfstd.org/ShowIssue.php?issue=161102.1 got accepted today, so DWARF5 is going to use uleb128 sizes instead of 2-byte sizes in .debug_loclists section. On a randomly chosen *.i file I had around, this results in shrinking of .debug_loclists section size from 0xef7df to 0xddd65, so aro

[PATCH] Remove padding from DWARF5 headers

2017-01-03 Thread Jakub Jelinek
Hi! http://dwarfstd.org/ShowIssue.php?issue=161031.2 got approved today, so DWARF5 is changing and the various DW_UT_* kinds will no longer have the same size of the headers. So, DW_UT_compile/DW_UT_partial shrinks by 12/16 bytes (padding 1 and padding 2 is removed; 16 bytes for 64-bit DWARF), DW

[PATCH] Fix -fself-test ICE in non-english locale (PR bootstrap/77569)

2017-01-03 Thread Jakub Jelinek
Hi! The cb.error hook is called in the case we are looking for with _("conversion from %s to %s not supported by iconv") where _(msgid) is dgettext ("cpplib", msgid), so if performing -fself-test on iconv that doesn't support ebcdic in a locale that has translations for this string, gcc ICEs. The

libgo patch committed: Eliminate __go_alloc and __go_free

2017-01-03 Thread Ian Lance Taylor
This patch to libgo eliminates the __go_alloc and __go_free functions, in favor of fully typed memory allocation (except for one remaining call in parfor.c, which will go away later). Implementing this was simplified by moving the allgs slice from C to Go, which involved moving a few functions. B

[C++ PATCH] Avoid UB in cp_lexer_previous_token (PR c++/71182)

2017-01-03 Thread Jakub Jelinek
Hi! cp_lexer_new_from_tokens creates lexer that has NULL lexer->buffer, calling lexer->buffer->address () therefore is UB (diagnosed by --with-boot-config=bootstrap-ubsan). The following patch fixes this, or Markus offered gcc_assert (!lexer->buffer || tp != lexer->buffer->address ()); inst

[C++ PATCH] Reject invalid auto foo (), a = 5;

2017-01-03 Thread Jakub Jelinek
Hi! C++14 and above says that for auto specifier in [dcl.spec.auto]/7: "If the init-declarator-list contains more than one init-declarator, they shall all form declarations of variables." The following patch attempts to reject this. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for t

Re: [PATCH], PR tqrget/78953, Fix power9 insn does not meet its constraints

2017-01-03 Thread David Edelsohn
On Tue, Jan 3, 2017 at 4:43 PM, Michael Meissner wrote: > In builting Spec 2006 with -mcpu=power9 with -O3, two of the benchmarks > (gamess > and calculix) did not build due to an "insn does not match its constraints" > error. > > (insn 2674 2673 2675 37 (parallel [ > (set (reg:SI 0 0

[PATCH], PR tqrget/78953, Fix power9 insn does not meet its constraints

2017-01-03 Thread Michael Meissner
In builting Spec 2006 with -mcpu=power9 with -O3, two of the benchmarks (gamess and calculix) did not build due to an "insn does not match its constraints" error. (insn 2674 2673 2675 37 (parallel [ (set (reg:SI 0 0 [985]) (vec_select:SI (reg:V4SI 32 0 [orig:378 vect__5

[SPARC] Enable LRA by default

2017-01-03 Thread Eric Botcazou
I changed my mind and decided to give it a try for GCC 7, after bootstrapping and testing the 32-bit and 64-bit compilers, both in development and release modes, over the last weeks. The few encountered issues were minor: missed optimization in a specific case (PR rtl-optimization/78664) and co

Re: [PATCH/AARCH64] Handle ILP32 multi-arch

2017-01-03 Thread Andrew Pinski
Ping? On Sat, Dec 10, 2016 at 1:24 PM, Andrew Pinski wrote: > On Thu, Nov 10, 2016 at 6:58 PM, Andrew Pinski wrote: >> On Tue, Oct 25, 2016 at 3:25 PM, Matthias Klose wrote: >>> On 07.10.2016 23:08, Andrew Pinski wrote: Hi, This patch adds ilp32 multi-arch support. This is needed t

Re: [PATCH, libgo] Avoid compiling runtime/aeshash.c with older assemblers [PR go/78789]

2017-01-03 Thread Ian Lance Taylor
On Tue, Jan 3, 2017 at 10:37 AM, Uros Bizjak wrote: > > Attached patch detect support for AES instructions and avoids > compiling runtime/aeshash.c with older assemblers (on e.g. CentOS > 5.11). The result of configure is also communicated into go runtime, > so the library doesn't try to call non-

Re: [-fcompare-debug] find jump before debug insns in expand

2017-01-03 Thread Alexandre Oliva
On Jan 3, 2017, Richard Sandiford wrote: >>&& (last = get_last_insn ()) >> - && JUMP_P (last)) >> + && (JUMP_P (last) >> + || (DEBUG_INSN_P (last) >> + && JUMP_P (prev_nondebug_insn (last) > Would it be worth adding a get_last_nondebug_insn in case other patt

Re: [PATCH] Use the middle-end boolean_type_node

2017-01-03 Thread Jakub Jelinek
On Tue, Jan 03, 2017 at 09:40:02PM +0200, Janne Blomqvist wrote: > I do think this is fixable without waiting for potential improved > hardware. The vast majority of uses of boolean_type_node in the > Fortran frontend is code like > > tmp = fold_build2_loc (input_location, GT_EXPR, >

[gomp4] backport an acc directive matching cleanup for fortran

2017-01-03 Thread Cesar Philippidis
This patch contains a backport of the fortran OpenACC directive matching changes I made to trunk here . It's not a clean backport because gomp4 has some support for the device_type clause which is missing in trunk. Next I plan to backport J

Re: [PATCH] Use the middle-end boolean_type_node

2017-01-03 Thread Janne Blomqvist
On Tue, Jan 3, 2017 at 6:50 PM, Andreas Krebbel wrote: > The regression with 8 bit boolean types surfaced with the z10 machine. The > ABI is much older. Since > we cannot change it anymore we should rather talk to the hardware guys to add > the instruction we > need. So for now we probably have

Re: [PATCH] PR 78534 Change character length from int to size_t

2017-01-03 Thread Janne Blomqvist
On Tue, Jan 3, 2017 at 9:21 PM, FX wrote: >> r244027 reverts r244011. Sorry for the breakage. It seems to affect >> all i686 as well in addition to power, maybe all 32-bit hosts. > > The breakage is surprising, as the rejects-valid does not involve character > length at all. > Jane, any chance yo

Re: [bootstrap-O3] add a default initializer to avoid a warning at -O3

2017-01-03 Thread Jason Merrill
Are there bugzillas for these false positive warnings? On Tue, Jan 3, 2017 at 12:45 PM, Jeff Law wrote: > On 01/02/2017 10:28 PM, Alexandre Oliva wrote: >> >> Building with the bootstrap-O3 configuration option fails to compile >> input.c due to an AFAICT false-positive warning about an uninitial

Re: [PATCH] PR 78534 Change character length from int to size_t

2017-01-03 Thread FX
> r244027 reverts r244011. Sorry for the breakage. It seems to affect > all i686 as well in addition to power, maybe all 32-bit hosts. The breakage is surprising, as the rejects-valid does not involve character length at all. Jane, any chance you might have accidentally committed some unrelated c

Re: [PATCH] Use the middle-end boolean_type_node

2017-01-03 Thread FX
> The regression with 8 bit boolean types surfaced with the z10 machine. The > ABI is much older. Since > we cannot change it anymore we should rather talk to the hardware guys to add > the instruction we > need. So for now we probably have to live with the regression in the Fortran > cases sinc

Re: [PATCH] Use the middle-end boolean_type_node

2017-01-03 Thread FX
> The gfc_init_types change is an ABI change, at least if the fortran FE > bool type is ever stored in memory and accessed by multiple TUs, or > passed as argument etc. And the difference between the C/C++ _Bool/bool > and fortran FE bool has caused lots of issues in the past, so if it can be > th

Re: [PATCH], PR target/78900, Fix PowerPC __float128 signbit

2017-01-03 Thread David Edelsohn
On Fri, Dec 30, 2016 at 3:54 PM, Michael Meissner wrote: > The signbit-3.c test explicitly tests for the value coming from memory, a > vector register, or a GPR. Unfortunately, the code did not handle splitting > up > the registers when the value was in a GPR. > > These patches add teh GPR suppo

Re: [PATCH] [PR rtl-optimization/65618] Fix MIPS ADA bootstrap failure

2017-01-03 Thread Jeff Law
On 01/03/2017 04:04 AM, James Cowgill wrote: On 01/01/17 22:27, Jeff Law wrote: On 12/20/2016 07:38 AM, James Cowgill wrote: Hi, On 19/12/16 21:43, Jeff Law wrote: On 12/19/2016 08:44 AM, James Cowgill wrote: 2016-12-16 James Cowgill PR rtl-optimization/65618 * emit-rtl.c (try_sp

[PATCH, libgo] Avoid compiling runtime/aeshash.c with older assemblers [PR go/78789]

2017-01-03 Thread Uros Bizjak
Hello! Attached patch detect support for AES instructions and avoids compiling runtime/aeshash.c with older assemblers (on e.g. CentOS 5.11). The result of configure is also communicated into go runtime, so the library doesn't try to call non-existent aeshashbody routine. Patch was tested on x86_

Re: [bootstrap-O3,fortran] silence warning in simplify_transformation_to_array

2017-01-03 Thread Jeff Law
On 01/02/2017 10:29 PM, Alexandre Oliva wrote: simplify_transformation_to_array had the nested loop unrolled 7 times, which is reasonable given that it iterates over arrays of size GFC_MAX_DIMENSIONS == 7. The problem is that the last iteration increments the index, tests that it's less than res

Contents of PO file 'cpplib-7.1-b20170101.sv.po'

2017-01-03 Thread Translation Project Robot
cpplib-7.1-b20170101.sv.po.gz Description: Binary data The Translation Project robot, in the name of your translation coordinator.

New Swedish PO file for 'cpplib' (version 7.1-b20170101)

2017-01-03 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'cpplib' has been submitted by the Swedish team of translators. The file is available at: http://translationproject.org/latest/cpplib/sv.po (This file, 'cpplib-7.1-b20170101

Re: [bootstrap-O3] use unsigned type for regno in df-scan

2017-01-03 Thread Jeff Law
On 01/03/2017 11:11 AM, Jakub Jelinek wrote: On Tue, Jan 03, 2017 at 11:08:05AM -0700, Jeff Law wrote: What if REGNO is 2147483648 (assume 32 bit host). That will get us into the else block in df_ref_record as it's >= FIRST_PSEUDO_REGISTER. In df_ref_create_structure, we use the same expressio

Re: [bootstrap-O3] use unsigned type for regno in df-scan

2017-01-03 Thread Jakub Jelinek
On Tue, Jan 03, 2017 at 11:08:05AM -0700, Jeff Law wrote: > What if REGNO is 2147483648 (assume 32 bit host). That will get us into the > else block in df_ref_record as it's >= FIRST_PSEUDO_REGISTER. > > In df_ref_create_structure, we use the same expression to compute REGNO, but > this time it's

Re: [bootstrap-O3] use unsigned type for regno in df-scan

2017-01-03 Thread Jeff Law
On 01/02/2017 10:29 PM, Alexandre Oliva wrote: This patch fixes a false-positive warning in df-scan, at bootstrap-O3 failed, and enables GCC to optimize out the code that leads to the warning. df_ref_create_structure was inlined into the else part of df_ref_record. Due to the condition of the c

Re: [PATCH] PR 78534 Change character length from int to size_t

2017-01-03 Thread Janne Blomqvist
On Tue, Jan 3, 2017 at 4:07 PM, David Edelsohn wrote: > This patch broke bootstrap. I now am seeing numerous errors when > building libgomp. > > Please fix or revert immediately. r244027 reverts r244011. Sorry for the breakage. It seems to affect all i686 as well in addition to power, maybe all

Re: [PATCH 1/2] [ADA] Fix MIPS big-endian build

2017-01-03 Thread Eric Botcazou
> Thanks, can you commit it for me? I screwed up the patch in the email I > sent a few minutes ago but the patch below should apply. I installed both patches on the mainline. -- Eric Botcazou

Re: [bootstrap-O3] add a default initializer to avoid a warning at -O3

2017-01-03 Thread Jeff Law
On 01/02/2017 10:28 PM, Alexandre Oliva wrote: Building with the bootstrap-O3 configuration option fails to compile input.c due to an AFAICT false-positive warning about an uninitialized use of a variable. This patch adds a default initializer to silence it. Regstrapped on x86_64-linux-gnu and

Re: [bootstrap-O3,fortran] add a NULL initializer to avoid a warning at -O3

2017-01-03 Thread Jeff Law
On 01/02/2017 10:28 PM, Alexandre Oliva wrote: Building with the bootstrap-O3 configuration option fails to compile fortran/module.c due to an AFAICT false-positive warning about an uninitialized use of a variable. This patch adds a dummy initializer to silence it. Regstrapped on x86_64-linux-g

Re: [PR tree-optimization/71691] Fix unswitching in presence of maybe-undef SSA_NAMEs (take 2)

2017-01-03 Thread Aldy Hernandez
On 12/20/2016 09:16 AM, Richard Biener wrote: On Fri, Dec 16, 2016 at 3:41 PM, Aldy Hernandez wrote: Hi folks. This is a follow-up on Jeff and Richi's interaction on the aforementioned PR here: https://gcc.gnu.org/ml/gcc-patches/2016-08/msg01397.html I decided to explore the idea of

Re: [bootstrap-O1] add initializers to avoid warnings at -O1

2017-01-03 Thread Jeff Law
On 01/02/2017 10:28 PM, Alexandre Oliva wrote: Building with the bootstrap-O1 configuration option fails to compile a number of files due to AFAICT false-positive warnings about uses of uninitialized variables. This patch adds dummy initializers to silence them all. Regstrapped on x86_64-linux-

Re: [bootstrap-O1] enlarge sprintf output buffer to avoid warning

2017-01-03 Thread Jeff Law
On 01/02/2017 10:28 PM, Alexandre Oliva wrote: In stage2 of bootstrap-O1, the code that warns if sprintf might overflow its output buffer cannot tell that an unsigned value narrowed to 16 bits will fit in 4 bytes with %4x. I couldn't find a better way to avoid the warning at -O1 than growing the

New French PO file for 'gcc' (version 7.1-b20170101)

2017-01-03 Thread Translation Project Robot
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 French team of translators. The file is available at: http://translationproject.org/latest/gcc/fr.po (This file, 'gcc-7.1-b20170101.fr.po', h

Re: [PATCH, GCC/ARM 2/2, ping4] Allow combination of aprofile and rmprofile multilibs

2017-01-03 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 06/12/16 11:35, Thomas Preudhomme wrote: Ping? *** gcc/ChangeLog *** 2016-10-03 Thomas Preud'homme * config.gcc: Allow combinations of aprofile and rmprofile values for --with-multilib-list. * config/arm/t-multilib: New file.

Re: [PATCH, GCC/testsuite/ARM, ping] Fix empty_fiq_handler target selector

2017-01-03 Thread Thomas Preudhomme
Happy new year! Ping? Best regards, Thomas On 09/12/16 15:28, Thomas Preudhomme wrote: Hi, The current target selector for empty_fiq_handler.c testcase skips the test when targeting Thumb mode on a device with ARM execution state. Because it checks Thumb mode by looking for an -mthumb option

Re: [PATCH, GCC/testsuite/ARM, ping] Skip optional_mthumb tests if GCC has a default mode

2017-01-03 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 12/12/16 17:52, Thomas Preudhomme wrote: Hi, The logic to make -mthumb optional for Thumb-only devices is only executed when no -marm or -mthumb is given on the command-line. This includes configuring GCC with --with-mode= because this makes the option to be pass

Re: [PATCH] Use the middle-end boolean_type_node

2017-01-03 Thread Andreas Krebbel
On 01/03/2017 03:31 PM, Janne Blomqvist wrote: > On Tue, Jan 3, 2017 at 4:20 PM, Jakub Jelinek wrote: >> On Tue, Jan 03, 2017 at 03:14:46PM +0100, Dominik Vogt wrote: >>> This patch costs several thousand additional instructions in >>> Spec2006 on s390x ("lines" = instructions): >>> >>> 410.bwav

PING Re: [PATCH] Add x86_64-specific selftests for RTL function reader (v2)

2017-01-03 Thread David Malcolm
Ping: https://gcc.gnu.org/ml/gcc-patches/2016-12/msg01616.html (the patch has been successfully bootstrap®rtested on x86_64-pc-linux-gnu, and also tested on i686-pc-linux-gnu). On Mon, 2016-12-19 at 12:12 -0500, David Malcolm wrote: > Note to i386 maintainters: this patch is part of the RTL fro

Re: [patch,avr] PR78883: Implement CANNOT_CHANGE_MODE_CLASS.

2017-01-03 Thread Segher Boessenkool
On Tue, Jan 03, 2017 at 01:43:01PM +, Richard Sandiford wrote: > An alternative would be to add a new macro to control this block in > general_operand: > > #ifdef INSN_SCHEDULING > /* On machines that have insn scheduling, we want all memory >reference to be explicit, so outlaw p

Re: [Patch][ARM,AArch64] more poly64 intrinsics and tests

2017-01-03 Thread Christophe Lyon
Ping? On 14 December 2016 at 23:09, Christophe Lyon wrote: > On 14 December 2016 at 17:55, James Greenhalgh > wrote: >> On Mon, Dec 12, 2016 at 05:03:31PM +0100, Christophe Lyon wrote: >>> Hi, >>> >>> After the recent update from Tamar, I noticed a few discrepancies >>> between ARM and AArch64

Re: [ARM] PR 78253 do not resolve weak ref locally

2017-01-03 Thread Christophe Lyon
Ping? The patch is at https://gcc.gnu.org/ml/gcc-patches/2016-12/msg00078.html On 14 December 2016 at 16:29, Christophe Lyon wrote: > Ping^2 ? > > As a reminder, this patch mimics what aarch64 does wrt to references to weak > symbols such that they are not resolved by the assembler, in case a s

Re: [PATCH] libstdc++: Allow using without lock free atomic int

2017-01-03 Thread Jonathan Wakely
On 19/12/16 17:52 +, Jonathan Wakely wrote: On 16/12/16 17:52 +, Jonathan Wakely wrote: On 09/11/16 23:26 +0200, Pauli wrote: Compiling programs using std::future for old arm processors fails. The problem is caused by preprocessor check for atomic lock free int. Future can be changed t

Re: [PATCH v2] aarch64: Add split-stack initial support

2017-01-03 Thread Wilco Dijkstra
Adhemerval Zanella wrote:   Sorry for the late reply - but I think it's getting there. A few more comments: + /* If function uses stacked arguments save the old stack value so morestack + can return it. */ + reg11 = gen_rtx_REG (Pmode, R11_REGNUM); + if (cfun->machine->frame.saved_regs_si

Re: [PATCH 2/2] [ADA] Fix constants in s-linux-mips.ads

2017-01-03 Thread Arnaud Charlet
> This patch corrects various constants in s-linux-mips.ads. A large > proportion (especially the signals) were simply wrong on MIPS. It also > fixes the struct sigaction offsets which are incorrect on 64-bit > systems because sa_flags is an int (always 32-bits), and not a pointer. > > Thanks, > J

[PATCH 2/2] [ADA] Fix constants in s-linux-mips.ads

2017-01-03 Thread James Cowgill
Hi, This patch corrects various constants in s-linux-mips.ads. A large proportion (especially the signals) were simply wrong on MIPS. It also fixes the struct sigaction offsets which are incorrect on 64-bit systems because sa_flags is an int (always 32-bits), and not a pointer. Thanks, James gcc

Re: [PATCH 1/2] [ADA] Fix MIPS big-endian build

2017-01-03 Thread James Cowgill
Hi, On 03/01/17 14:47, Arnaud Charlet wrote: >> This patch merges the mips and mipsel sections in >> gcc-interface/Makefile.in favoring the existing variables in mipsel. >> Over time, the mipsel target was tested much more than the mips target >> and a number of fixes were applied which should hav

Re: [PATCH 1/2] [ADA] Fix MIPS big-endian build

2017-01-03 Thread Arnaud Charlet
> This patch merges the mips and mipsel sections in > gcc-interface/Makefile.in favoring the existing variables in mipsel. > Over time, the mipsel target was tested much more than the mips target > and a number of fixes were applied which should have been applied to > both. Since the only real diff

[PATCH 1/2] [ADA] Fix MIPS big-endian build

2017-01-03 Thread James Cowgill
Hi, This patch merges the mips and mipsel sections in gcc-interface/Makefile.in favoring the existing variables in mipsel. Over time, the mipsel target was tested much more than the mips target and a number of fixes were applied which should have been applied to both. Since the only real differenc

Re: [PATCH, wwwdocs] Add gcc-7/porting_to.html, Fortran changes

2017-01-03 Thread Gerald Pfeifer
On Tue, 3 Jan 2017, Janne Blomqvist wrote: > the attached patch mentions some changes in the Fortran frontend for > the GCC 7 cycle, and also adds the so-far missing gcc-7/porting_to.html > file. Thank you, Janne. > Ok to commit? Yes, modulo Joseph's comment and a missing towards the end of p

Re: [PATCH] Use the middle-end boolean_type_node

2017-01-03 Thread Janne Blomqvist
On Tue, Jan 3, 2017 at 4:20 PM, Jakub Jelinek wrote: > On Tue, Jan 03, 2017 at 03:14:46PM +0100, Dominik Vogt wrote: >> This patch costs several thousand additional instructions in >> Spec2006 on s390x ("lines" = instructions): >> >> 410.bwaves: +28 lines (2 funcs bigger) >> 437.leslie3d:

[wwwdocs] Trim GCJ references from "Current Development" in onlinedocs/

2017-01-03 Thread Gerald Pfeifer
Applied. Gerald Index: onlinedocs/index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/onlinedocs/index.html,v retrieving revision 1.161 diff -u -r1.161 index.html --- onlinedocs/index.html 21 Dec 2016 10:57:36 - 1.161 +++

Re: [PATCH] Use the middle-end boolean_type_node

2017-01-03 Thread Jakub Jelinek
On Tue, Jan 03, 2017 at 03:14:46PM +0100, Dominik Vogt wrote: > This patch costs several thousand additional instructions in > Spec2006 on s390x ("lines" = instructions): > > 410.bwaves: +28 lines (2 funcs bigger) > 437.leslie3d: +43 lines (5 funcs bigger) > 434.zeusmp: +2650 lines (

Re: [PATCH] Use the middle-end boolean_type_node

2017-01-03 Thread Dominik Vogt
On Tue, Dec 13, 2016 at 10:59:09PM +0200, Janne Blomqvist wrote: > Use the boolean_type_node setup by the middle-end instead of > redefining it. boolean_type_node is not used in GFortran for any > ABI-visible stuff, only internally as the type of boolean > expressions. There appears to be one excep

Re: [-fcompare-debug] find jump before debug insns in expand

2017-01-03 Thread Richard Sandiford
Alexandre Oliva writes: > A debug insn after the final jump of a basic block may cause the > expander to emit a dummy move where the non-debug compile won't > because it finds the jump insn at the end of the insn stream. > > Fix the condition so that, instead of requiring the jump as the last > in

Re: [PATCH] PR 78534 Change character length from int to size_t

2017-01-03 Thread David Edelsohn
This patch broke bootstrap. I now am seeing numerous errors when building libgomp. Please fix or revert immediately. Thanks, David omp_lib.f90:184:40: logical (4) :: omp_test_lock 1 Error: Symbol 'omp_test_lock' at (1) has already been host

[PATCH] PR c++/66735 lambda capture by reference

2017-01-03 Thread Nathan Sidwell
This patch fixes 66735, where we lose cv qualifiers on an explicit lambda capture by reference: [&r = ] () {...} The problem is the partitioning between lambda_capture_field_type and add_capture, where the latter is responsible for adding the referenceness. That leaves auto deduction cluel

[doc] md.texi -- "trap" is no longer used by the Java front end

2017-01-03 Thread Gerald Pfeifer
Applied. Gerald 2017-01-03 Gerald Pfeifer * doc/md.texi (Standard Names): Remove reference to Java frontend. Index: doc/md.texi === --- doc/md.texi (revision 244022) +++ doc/md.texi (working copy) @@ -6534,8 +6534,7 @@

Re: [patch,avr] PR78883: Implement CANNOT_CHANGE_MODE_CLASS.

2017-01-03 Thread Richard Sandiford
Georg-Johann Lay writes: > On 02.01.2017 15:54, Dominik Vogt wrote: >> On Mon, Jan 02, 2017 at 03:47:43PM +0100, Georg-Johann Lay wrote: >>> This fixes PR78883 which is a problem in reload revealed by a >>> change to combine.c. The fix is as proposed by Segher: implement >>> CANNOT_CHANGE_MODE_CL

C++ PATCH for c++/77545 and c++/77284 (ICE with CLEANUP_STMT)

2017-01-03 Thread Marek Polacek
The problem here is that we've gotten to potential_constant_expression_1 with a CLEANUP_STMT, but it doesn't know how to handle that so we ICE. I thought it'd be possible to look into CLEANUP_{BODY,EXPR} to determine whether the CLEANUP_STMT can be potentially const, but cxx_eval_constant_expressi

[PATCH] Add deleted std::thread(const thread&&) constructor

2017-01-03 Thread Jonathan Wakely
LWG 2097 says that the templated std::thread(Callable&&, Args&&...) constructor should not participate in overload resolution when decay_t is std::thread. Rather than removing it from the overload set we just ensure that there are better matches, but we fail to do so for const rvalues. This fixes

Re: [PATCH, wwwdocs] Add gcc-7/porting_to.html, Fortran changes

2017-01-03 Thread Janne Blomqvist
On Tue, Jan 3, 2017 at 2:04 PM, Joseph Myers wrote: > On Tue, 3 Jan 2017, Janne Blomqvist wrote: > >> (I added and tags to the >> porting_to.html file, which is apparently what is recommended these >> days; I didn't add this to any existing file) > > DOCTYPE etc. are added automatically via sty

New template for 'gcc' made available

2017-01-03 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. (If you have any questions, send them to .) A new POT file for textual domain 'gcc' has been made available to the language teams for translation. It is archived as: http://translationproject.org/POT-files/gcc-

Re: [PATCH, wwwdocs] Add gcc-7/porting_to.html, Fortran changes

2017-01-03 Thread Joseph Myers
On Tue, 3 Jan 2017, Janne Blomqvist wrote: > (I added and tags to the > porting_to.html file, which is apparently what is recommended these > days; I didn't add this to any existing file) DOCTYPE etc. are added automatically via style.mhtml. They do not belong in any individual checked-in .h

New Ukrainian PO file for 'cpplib' (version 7.1-b20170101)

2017-01-03 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'cpplib' has been submitted by the Ukrainian team of translators. The file is available at: http://translationproject.org/latest/cpplib/uk.po (This file, 'cpplib-7.1-b201701

Contents of PO file 'cpplib-7.1-b20170101.uk.po'

2017-01-03 Thread Translation Project Robot
cpplib-7.1-b20170101.uk.po.gz Description: Binary data The Translation Project robot, in the name of your translation coordinator.

New template for 'cpplib' made available

2017-01-03 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. (If you have any questions, send them to .) A new POT file for textual domain 'cpplib' has been made available to the language teams for translation. It is archived as: http://translationproject.org/POT-files/c

[PATCH] Fix typos in libstdc++ docs and update copyright years

2017-01-03 Thread Jonathan Wakely
* doc/xml/manual/spine.xml: Update copyright years. * doc/xml/manual/build_hacking.xml: Fix spelling of libbuilddir. * doc/xml/manual/test.xml: Likewise. * doc/html/*: Regenerate. Comitted to trunk. commit 6c912caa212670131a9d3ed61c89d2182c79b1c0 Author: Jonathan

[PATCH] [msp430] Sync msp430_mcu_data with devices.csv

2017-01-03 Thread Joe Seymour
This patch syncs the generated msp430_mcu_data structure: - With the latest version of devices.csv released by TI. - Between msp430.c and driver-msp430.c. The former was updated more recently than the latter. - With the copy of the same data structure in binutils, for which a similar patch has

Re: [PATCH] [PR rtl-optimization/65618] Fix MIPS ADA bootstrap failure

2017-01-03 Thread James Cowgill
On 01/01/17 22:27, Jeff Law wrote: > On 12/20/2016 07:38 AM, James Cowgill wrote: >> Hi, >> >> On 19/12/16 21:43, Jeff Law wrote: >>> On 12/19/2016 08:44 AM, James Cowgill wrote: 2016-12-16 James Cowgill PR rtl-optimization/65618 * emit-rtl.c (try_split): Update "afte

Re: [C++ PATCH] Implement LWG2296 helper intrinsic

2017-01-03 Thread Jonathan Wakely
On 01/01/17 15:53 +0100, Jakub Jelinek wrote: On Sun, Jan 01, 2017 at 10:27:24AM -0400, Gerald Pfeifer wrote: On Fri, 7 Oct 2016, Jakub Jelinek wrote: > The following patch adds __builtin_addressof with the semantics it has in > clang, i.e. it is a constexpr & operator alternative that never use

Re: [PATCH][GCC][PATCHv3] Improve fpclassify w.r.t IEEE like numbers in GIMPLE.

2017-01-03 Thread Tamar Christina
Hi Jeff, I wasn't sure if you saw the updated patch attached to the previous email or if you just hadn't had the time to look at it yet. Cheers, Tamar From: Jeff Law Sent: Monday, December 19, 2016 8:27:33 PM To: Tamar Christina; Joseph Myers Cc: GCC Pa

[PATCH 2/4] S/390: Unroll mvc/xc loop for memset with small constant lengths.

2017-01-03 Thread Andreas Krebbel
When expanding a memset we emit a loop of MVCs/XCs instructions dealing with 256 byte blocks. This loop used to get unrolled with older GCCs when using constant length operands. GCC lost this ability probably when more of the loop unrolling stuff has been moved to tree level. With this patch the

[PATCH 4/4] S/390: Additional memset/memcpy runtime tests.

2017-01-03 Thread Andreas Krebbel
These were provided by Dominik to check more of the corner case in our memset/memcpy inline code. gcc/testsuite/ChangeLog: 2017-01-03 Dominik Vogt * gcc.target/s390/memcpy-2.c: New test. * gcc.target/s390/memset-2.c: New test. --- gcc/testsuite/gcc.target/s390/memcpy-2.c | 94

[PATCH 3/4] S/390: Unroll mvc loop for memcpy with small constant lengths.

2017-01-03 Thread Andreas Krebbel
See the memset unrolling patch. The very same applies to memcpys with constant lengths. 2017-01-03 Andreas Krebbel * config/s390/s390.c (s390_expand_movmem): Unroll MVC loop for small constant length operands. gcc/testsuite/ChangeLog: 2017-01-03 Andreas Krebbel *

[PATCH 1/4] S/390: memset: Avoid overlapping MVC operands between iterations.

2017-01-03 Thread Andreas Krebbel
A memset with a value != 0 is currently implemented using the mvc instruction propagating the first byte through 256 byte blocks. While for the first mvc the byte is written with a separate instruction subsequent MVCs used the last byte of the previous 256 byte block. Starting with z13 this cause

[PATCH 0/4] S/390: memset/memcpy inline code improvements

2017-01-03 Thread Andreas Krebbel
Please see the individual patches for descriptions. I'll commit these after leaving a few days for comments. Andreas Krebbel (4): S/390: memset: Avoid overlapping MVC operands between iterations. S/390: Unroll mvc/xc loop for memset with small constant lengths. S/390: Unroll mvc loop for me

  1   2   >