Re: [C PATCH] Fix up file-scope _Atomic expansion (PR c/65345)

2015-04-23 Thread Marek Polacek
On Thu, Mar 12, 2015 at 06:37:48PM +0100, Marek Polacek wrote: This is not a regression, so not sure if I shouldn't defer this patch to the next stage1 at this juncture... I've committed this patch now after another regtest/bootstrap (x86_64-linux). Marek

Re: [PATCH][ARM] Rewrite vccond NEON patterns to use RTL operations rather than UNSPECs

2015-04-23 Thread Ramana Radhakrishnan
On Wed, Feb 4, 2015 at 12:12 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi all, This patch improves the vccond patterns in neon.md to use proper RTL operations rather than UNSPECS. It is done in a similar way to the analogous aarch64 operations i.e. vceq is expressed as (neg (eq (...)

[Bug fortran/65548] [5/6 Regression] gfc_conv_procedure_call

2015-04-23 Thread juergen.reuter at desy dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65548 --- Comment #27 from Jürgen Reuter juergen.reuter at desy dot de --- And Example #2 is: module foo type :: t integer :: n character(32), dimension(:), allocatable :: md5 contains procedure :: init = t_init end type t

[Bug libstdc++/65861] libstdc++ is silently generating wrong code when its std::search is given an input iterator

2015-04-23 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65861 --- Comment #6 from Jonathan Wakely redi at gcc dot gnu.org --- And even if they don't read the libstdc++ documentation, std::search doesn't work with input iterators, that's always been true: https://www.sgi.com/tech/stl/search.html

Re: [PATCH] Quiet down -Wlogical-op a bit (PR c/61534)

2015-04-23 Thread Dodji Seketeli
Hi! Marek Polacek pola...@redhat.com writes: This patch stifles -Wlogical-op a bit: don't warn if either operand comes from a macro expansion. As the comment says, it doesn't fix the bug completely, but it's a simple improvement. I cannot approve this patch, but for what it's worth, I like

[Bug libstdc++/65861] New: libstdc++ is silently generating wrong code when its std::search is given an input iterator

2015-04-23 Thread howarth.at.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65861 Bug ID: 65861 Summary: libstdc++ is silently generating wrong code when its std::search is given an input iterator Product: gcc Version: 5.1.0 Status: UNCONFIRMED

[Bug inline-asm/65859] Optimizes out TLS variable called in a child function

2015-04-23 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65859 Andrew Pinski pinskia at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED

Re: [PATCH] Emit -Waddress warnings for comparing address of reference against NULL

2015-04-23 Thread Jason Merrill
On 04/20/2015 10:36 PM, Patrick Palka wrote: + if (decl_with_nonnull_addr_p (inner)) Using decl_with_nonnull_addr_p doesn't make sense for reference variables, since we're using their pointer value rather than their address. + warning_at (location, +

C++ PATCH for c++/50800 (ICE with may_alias and templates)

2015-04-23 Thread Jason Merrill
We try to strip attributes that aren't reflected in mangling from template arguments, but were failing to do that in this case. Fixed by making strip_typedefs strip such attributes as well. Tested x86_64-pc-linux-gnu, applying to trunk. commit 2068bd13c95465ece265d46daa0f9e3c3c4e2f8e Author:

FIx ipa-icf.c build failure

2015-04-23 Thread Jan Hubicka
Hi, while comitting previous patch, I frogot the attached change that exports attribute_value_equal. Comitted and my apologize for breakage. Honza * tree.h (attribute_value_equal): Declare. * tree.c (attribute_value_equal): Export. Index: tree.h

[Bug preprocessor/65860] New: Stringification of User Defined Literals

2015-04-23 Thread marcin.konarski at codestation dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65860 Bug ID: 65860 Summary: Stringification of User Defined Literals Product: gcc Version: 4.9.2 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

[Bug libstdc++/65861] libstdc++ is silently generating wrong code when its std::search is given an input iterator

2015-04-23 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65861 --- Comment #2 from Jonathan Wakely redi at gcc dot gnu.org --- This is user error, std::search() requires forward iterators and the library is not required to diagnose it. Defining _GLIBCXX_CONCEPT_CHECKS causes it to be rejected.

Re: [PATCH] Fix LTO option streaming

2015-04-23 Thread Jan Hubicka
It looks like when transitioning to using target and optimization option nodes for compile-time to link-time option streaming you didn't adjust lto-opts.c nor lto-wrapper.c. The following fixes Yep, I assumed that lto-wrapper's merging is now redundant for optimization options, while it is

[Bug libstdc++/65861] libstdc++ is silently generating wrong code when its std::search is given an input iterator

2015-04-23 Thread howarth.at.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65861 --- Comment #4 from Jack Howarth howarth.at.gcc at gmail dot com --- (In reply to Jonathan Wakely from comment #2) This is user error, std::search() requires forward iterators and the library is not required to diagnose it. Defining

[PATCH, rs6000, testsuite, committed] Clean up one more test for unaligned loads handling

2015-04-23 Thread Bill Schmidt
Hi, Between the time my unaligned-loads patch was approved and trunk reopened for business, another test showed up that needs to be cleaned up in the same way as the others. This patch does that. Verified on powerpc64le-unknown-linux-gnu, committed as obvious. Thanks, Bill 2015-04-23 Bill

Fix more of ICF's comparsion (attributes and references)

2015-04-23 Thread Jan Hubicka
Hi, this patch introduces sem_item::compare_attributes that is based on comp_type_attributes, just simpler. We can clearly be a lot smarter if we started annotating attributes with a safety WRT various transformations, but I am not sure we care. I think it may make more sense to actually lower

Re: Re: [PATCH] Emit -Waddress warnings for comparing address of reference against NULL

2015-04-23 Thread Manuel López-Ibáñez
On 04/23/2015 05:12 PM, Jason Merrill wrote: On 04/20/2015 10:36 PM, Patrick Palka wrote: Implementation is pretty straightforward. The only catch is that the middle-end doesn't actually assume that REFERENCE_TYPEs are non-NULL so code like int a = *(int *)0; if (a != 0) will warn

[Bug c++/50800] Internal compiler error in finish_member_declarations, possibly related to may_alias attribute

2015-04-23 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=50800 --- Comment #13 from Jason Merrill jason at gcc dot gnu.org --- Author: jason Date: Thu Apr 23 15:55:21 2015 New Revision: 222377 URL: https://gcc.gnu.org/viewcvs?rev=222377root=gccview=rev Log: PR c++/50800 * tree.c (strip_typedefs):

RFA (tree.c): PATCH for may_alias vs. TYPE_CANONICAL, related to c++/50800

2015-04-23 Thread Jason Merrill
In general, TYPE_CANONICAL of a type strips all attributes. An exception to this seems to be that TYPE_REF_CAN_ALIAS_ALL remains set on the TYPE_CANONICAL of a pointer/reference type even though its TREE_TYPE no longer has the may_alias attribute, and is inconsistent with

[Bug target/26702] .size is not emitted for BSS variables

2015-04-23 Thread ramana at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=26702 --- Comment #11 from Ramana Radhakrishnan ramana at gcc dot gnu.org --- Author: ramana Date: Thu Apr 23 14:49:45 2015 New Revision: 222371 URL: https://gcc.gnu.org/viewcvs?rev=222371root=gccview=rev Log: Fix PR target/26702 For Kwok Cheung

[Bug libstdc++/65861] libstdc++ is silently generating wrong code when its std::search is given an input iterator

2015-04-23 Thread howarth.at.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65861 --- Comment #3 from Jack Howarth howarth.at.gcc at gmail dot com --- (In reply to Jonathan Wakely from comment #2) This is user error, std::search() requires forward iterators and the library is not required to diagnose it. Defining

[Bug preprocessor/65860] Stringification of User Defined Literals

2015-04-23 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65860 Jonathan Wakely redi at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |NEW Last

[Bug fortran/65548] [5/6 Regression] gfc_conv_procedure_call

2015-04-23 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65548 --- Comment #28 from Dominique d'Humieres dominiq at lps dot ens.fr --- And Example #2 is: ... Confirmed too, but no ICE under debugger.

[Bug rtl-optimization/65862] [MIPS] IRA/LRA issue: integers spilled to floating-point registers

2015-04-23 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65862 --- Comment #1 from Andrew Pinski pinskia at gcc dot gnu.org --- The kernel should have been compiled with -msoft-float and I thought it was.

Re: [WIP] OpenMP 4 NVPTX support

2015-04-23 Thread Jakub Jelinek
On Tue, Apr 21, 2015 at 05:58:39PM +0200, Jakub Jelinek wrote: Attached is a minimal patch to get at least a trivial OpenMP 4.0 testcase offloading to NVPTX (the first patch). The second patch is WIP, just first few needed changes to make libgomp to build for NVPTX (several weeks of work at

RE: [PATCH v3][MIPS] fix CRT_CALL_STATIC_FUNCTION macro

2015-04-23 Thread Maciej W. Rozycki
On Wed, 22 Apr 2015, Petar Jovanovic wrote: I think this will best be reduced to a link-only test on bare iron, hoping for a link failure. I am not sure how we can reduce the test to a link failure (today), if ld will not report an error (today). Me neither, offhand; hopefully someone

Re: C++ PATCH for c++/65646 (ICE after error specializing missing static data member)

2015-04-23 Thread Jason Merrill
On 04/01/2015 12:36 PM, Jason Merrill wrote: This testcase started crashing because the added call to check_explicit_specialization does a SET_DECL_TEMPLATE_SPECIALIZATION which wasn't happening previously, and then determine_visibility assumes that if DECL_USE_TEMPLATE is set, so is

[Bug c++/65646] [5 Regression] ICE in invalid syntax

2015-04-23 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65646 --- Comment #7 from Jason Merrill jason at gcc dot gnu.org --- Author: jason Date: Thu Apr 23 15:55:11 2015 New Revision: 222376 URL: https://gcc.gnu.org/viewcvs?rev=222376root=gccview=rev Log: PR c++/65646 * pt.c

[Bug rtl-optimization/65862] New: [MIPS] IRA/LRA issue: integers spilled to floating-point registers

2015-04-23 Thread robert.suchanek at imgtec dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65862 Bug ID: 65862 Summary: [MIPS] IRA/LRA issue: integers spilled to floating-point registers Product: gcc Version: 5.1.1 Status: UNCONFIRMED Severity: normal

[Bug target/26702] .size is not emitted for BSS variables

2015-04-23 Thread ramana at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=26702 Ramana Radhakrishnan ramana at gcc dot gnu.org changed: What|Removed |Added Status|NEW |RESOLVED

[Bug libstdc++/65861] libstdc++ is silently generating wrong code when its std::search is given an input iterator

2015-04-23 Thread glisse at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65861 --- Comment #1 from Marc Glisse glisse at gcc dot gnu.org --- https://gcc.gnu.org/onlinedocs/libstdc++/manual/ext_compile_checks.html

Re: [PATCH] Fix LTO option streaming

2015-04-23 Thread Richard Biener
On Thu, 23 Apr 2015, Jan Hubicka wrote: It looks like when transitioning to using target and optimization option nodes for compile-time to link-time option streaming you didn't adjust lto-opts.c nor lto-wrapper.c. The following fixes Yep, I assumed that lto-wrapper's merging is now

Re: [PATCH] Tidy up locking for libgomp OpenACC entry points

2015-04-23 Thread Thomas Schwinge
Hi! On Wed, 22 Apr 2015 19:42:43 +0100, Julian Brown jul...@codesourcery.com wrote: This patch is an attempt to fix some potential race conditions with accesses to shared data structures from multiple concurrent threads in libgomp's OpenACC entry points. The main change is to move locking out

Re: [Patch, Fortran, PR58586, v2] ICE with derived type with allocatable component passed by value

2015-04-23 Thread Andre Vehreschild
Hi Mikael, hi all, thanks for the review. I have made some changes. Answers to your questions are inline below. On Sun, 19 Apr 2015 12:01:23 +0200 Mikael Morin mikael.mo...@sfr.fr wrote: snip I was pointed to the patch in comment #44 of pr61831 which seemingly fixes the #3 comment of

[Bug c++/65856] -Wsuggest-override shall not report a warning on final method

2015-04-23 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65856 --- Comment #8 from Markus Trippelsdorf trippels at gcc dot gnu.org --- (In reply to Helfer Thomas from comment #7) can you give me the rationale of this usage ? Performance. See also -Wsuggest-final-types -Wsuggest-final-methods in the gcc

[PATCH][AArch64] Properly handle mvn-register and add EON+shift pattern and cost appropriately

2015-04-23 Thread Kyrill Tkachov
[resending due to mail client messing up.] Hi all, The EON instruction can be expressed either by (xor (not a) b) or (not (xor a b)), simplify-rtx canonicalizes to the second form and we have a pattern for it (*xor_one_cmplmode3) but we don't have a pattern for the shifted operand version. This

[Bug bootstrap/65863] GCC does not compile with weird error messages

2015-04-23 Thread porton at narod dot ru
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65863 --- Comment #1 from Victor Porton porton at narod dot ru --- Linux victor.local 3.10-2-amd64 #1 SMP Debian 3.10.7-1 (2013-08-17) x86_64 GNU/Linux with 32 bit userland

[Bug bootstrap/65865] [6 Regression] Bootstrap failure on x86

2015-04-23 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65865 H.J. Lu hjl.tools at gmail dot com changed: What|Removed |Added Status|UNCONFIRMED |NEW Last

[PATCH, rs6000, committed] Change documentation of -mcrypto to reflect ISA 2.07B changes

2015-04-23 Thread Bill Schmidt
Hi, The recently released POWER ISA 2.07B replaced Category:Vector.Crypto with Category:Vector.AES and Category:Vector.SHA2, which outdated the description of the -mcrypto option. This patch fixes that. Verified on powerpc64le-linux-gnu, committed as obvious. Thanks, Bill 2015-04-23 Bill

Re: [PATCH] Fix a -fcheck-pointer-bounds -mmpx ICE (PR target/65523)

2015-04-23 Thread H.J. Lu
On Mon, Mar 23, 2015 at 1:01 PM, Richard Biener rguent...@suse.de wrote: On March 23, 2015 8:54:54 PM GMT+01:00, Jakub Jelinek ja...@redhat.com wrote: Hi! On the following testcase we ICE, because we don't verify we have the ERF_RETURNS_ARG argument, on non-verified builtins that is possible.

[Bug c++/65856] -Wsuggest-override shall not report a warning on final method

2015-04-23 Thread thomas.helfer at cea dot fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65856 --- Comment #7 from Helfer Thomas thomas.helfer at cea dot fr --- can you give me the rationale of this usage ?

Re: [PATCH] Improve targetm.binds_local_p for common symbols on s390*/arm/aarch64 (PR target/65780)

2015-04-23 Thread Ramana Radhakrishnan
On 23/04/15 17:36, Jakub Jelinek wrote: Hi! This patch undoes the PR65780 performance regressions on a few targets I have tested to work fine. This PR was about an access to uninitialized COMMON symbol defined in executable (or PIE) where there is a normal symbol definition in a shared

Assembly 3.2 GCC

2015-04-23 Thread Lesiano 16
I know that this maybe a silly question, but I have a question about the assembly generated by your 3.2 compiler, it has to do with the reasoning behind a certain instruction related with the loop transformation: All the details are here:

[Bug rtl-optimization/65862] [MIPS] IRA/LRA issue: integers spilled to floating-point registers

2015-04-23 Thread robert.suchanek at imgtec dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65862 --- Comment #2 from Robert Suchanek robert.suchanek at imgtec dot com --- That's correct. It was just easier to expose this problem by compiling the kernel.

[Bug bootstrap/65865] New: [6 Regression] Bootstrap failure on x86

2015-04-23 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65865 Bug ID: 65865 Summary: [6 Regression] Bootstrap failure on x86 Product: gcc Version: 6.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: bootstrap

[Bug c/65864] Consider emitting -Wswitch-bool less aggressively?

2015-04-23 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65864 --- Comment #1 from Marek Polacek mpolacek at gcc dot gnu.org --- I think they should simply cast the controlling expression to int--that should quash the warning. Does this workaround sound reasonable to them?

Re: C++ PATCH for c++/50800 (ICE with may_alias and templates)

2015-04-23 Thread H.J. Lu
On Thu, Apr 23, 2015 at 8:52 AM, Jason Merrill ja...@redhat.com wrote: We try to strip attributes that aren't reflected in mangling from template arguments, but were failing to do that in this case. Fixed by making strip_typedefs strip such attributes as well. Tested x86_64-pc-linux-gnu,

[PATCH] Improve targetm.binds_local_p for common symbols on s390*/arm/aarch64 (PR target/65780)

2015-04-23 Thread Jakub Jelinek
Hi! This patch undoes the PR65780 performance regressions on a few targets I have tested to work fine. This PR was about an access to uninitialized COMMON symbol defined in executable (or PIE) where there is a normal symbol definition in a shared library. The PR65780 fix that got committed

[Bug ipa/65858] ICE in varpool_node::get_constructor during chromium build on arm-linux-gnueabihf with LTO during LINK chrome

2015-04-23 Thread hubicka at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65858 --- Comment #1 from Jan Hubicka hubicka at gcc dot gnu.org --- Those two bugs seems to be caused by same issue. Unforutnately it is bit hard to guess what it is coming from - it seems that we manage to store error_mark_node into the LTO object

[Bug c/65864] New: Consider emitting -Wswitch-bool less aggressively?

2015-04-23 Thread ktkachov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65864 Bug ID: 65864 Summary: Consider emitting -Wswitch-bool less aggressively? Product: gcc Version: 6.0 Status: UNCONFIRMED Keywords: diagnostic Severity: enhancement

[Bug bootstrap/65863] GCC does not compile with weird error messages

2015-04-23 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65863 --- Comment #2 from H.J. Lu hjl.tools at gmail dot com --- (In reply to Victor Porton from comment #1) Linux victor.local 3.10-2-amd64 #1 SMP Debian 3.10.7-1 (2013-08-17) x86_64 GNU/Linux with 32 bit userland Try configure GCC with

[Bug c/65345] ICE with _Generic selection on _Atomic int

2015-04-23 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65345 --- Comment #10 from Marek Polacek mpolacek at gcc dot gnu.org --- Author: mpolacek Date: Thu Apr 23 14:35:12 2015 New Revision: 222370 URL: https://gcc.gnu.org/viewcvs?rev=222370root=gccview=rev Log: PR c/65345 * c-decl.c

Re: [PATCH] Fix for PR26702: Emit .size for BSS variables on arm-eabi

2015-04-23 Thread Ramana Radhakrishnan
On Mon, Mar 30, 2015 at 9:25 PM, Kwok Cheung Yeung k...@codesourcery.com wrote: This is a simple patch that ensures that a .size directive is emitted when space is allocated for a static variable in the BSS on bare-metal ARM targets. This allows other tools such as GDB to look up the size of

[Bug libstdc++/65861] libstdc++ is silently generating wrong code when its std::search is given an input iterator

2015-04-23 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65861 --- Comment #5 from Jonathan Wakely redi at gcc dot gnu.org --- (In reply to Jack Howarth from comment #3) Is the use of _GLIBCXX_CONCEPT_CHECKS well enough advertised in the documentation that we can expect the average developer to know to

[Bug bootstrap/65863] New: GCC does not compile with weird error messages

2015-04-23 Thread porton at narod dot ru
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65863 Bug ID: 65863 Summary: GCC does not compile with weird error messages Product: gcc Version: 5.1.0 Status: UNCONFIRMED Severity: blocker Priority: P3 Component:

[Bug target/65296] [avr] fix various issues with specs file generation

2015-04-23 Thread gjl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65296 Georg-Johann Lay gjl at gcc dot gnu.org changed: What|Removed |Added Keywords||link-failure

GCC 5.1 successful avr target build

2015-04-23 Thread Ralph Doncaster
I repeated the toolchain build I did for RC-20150418, this time with the 5.1 release. The texinfo build dependency bug that I noticed in RC-20150418 appears to be fixed, and the code generated for a small (300 line) program was identical to the code generated by RC-20150418 and RC-20150412. I

[Bug c++/65866] New: Wrong warning when using list-initialization: operation on 'b' may be undefined [-Wsequence-point]

2015-04-23 Thread o_kniemeyer at maxon dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65866 Bug ID: 65866 Summary: Wrong warning when using list-initialization: operation on 'b' may be undefined [-Wsequence-point] Product: gcc Version: 5.1.0 Status: UNCONFIRMED

Re: [PATCH] Emit -Waddress warnings for comparing address of reference against NULL

2015-04-23 Thread Jason Merrill
On 04/23/2015 11:34 AM, Manuel López-Ibáñez wrote: It seems also weird we do not warn directly for '*(int *)0' in the C/C++ FE. Agreed. Using decl_with_nonnull_addr_p doesn't make sense for reference variables, since we're using their pointer value rather than their address. Is an extra

[Bug ada/65868] cannot find -lstdc++ for GNAT compilation

2015-04-23 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65868 Andrew Pinski pinskia at gcc dot gnu.org changed: What|Removed |Added Severity|blocker |normal ---

[debug-early] branch merged with trunk@222380

2015-04-23 Thread Aldy Hernandez
There was one minor regression which I've fixed. Tested on x86-64 Linux with the GCC and GDB testsuites. Next on my plate is (finally) a full bootstrap now that GCC's guality.exp, dwarf2.exp, and debug.exp are down to 1 regression versus mainline. And finally... submitting the branch for

[PING][PATCH] libgcc: Add CFI directives to the soft floating point support code for ARM

2015-04-23 Thread Martin Galvan
This patch adds CFI directives to the soft floating point support code for ARM. Previously, if we tried to do a backtrace from that code in a debug session we'd get something like this: (gdb) bt #0 __nedf2 () at ../../../../../../gcc-4.9.2/libgcc/config/arm/ieee754-df.S:1082 #1 0x0db6 in

[Bug target/65867] New: [5 Regression] bootstrap fails for mingw32 due to missing header in ssp.c

2015-04-23 Thread daniel.f.starke at freenet dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65867 Bug ID: 65867 Summary: [5 Regression] bootstrap fails for mingw32 due to missing header in ssp.c Product: gcc Version: 5.1.0 Status: UNCONFIRMED Severity:

Re: [PATCH] Fix LTO option streaming

2015-04-23 Thread Jakub Jelinek
On Thu, Apr 23, 2015 at 05:16:19PM +0200, Jan Hubicka wrote: the target option side (for SWITCHABLE_TARGET). Do not record any target options in the lto_opts section. Honza - I suppose we don't have any testcase that this works, I'll try to come up with sth. This also looks like a

[Bug bootstrap/65863] GCC does not compile with weird error messages

2015-04-23 Thread porton at narod dot ru
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65863 Victor Porton porton at narod dot ru changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED

Re: [PATCH v2 2/4] libcc1: set debug compile: Display GCC driver filename

2015-04-23 Thread Jan Kratochvil
On Thu, 23 Apr 2015 22:38:34 +0200, Jan Kratochvil wrote: Unfortunately this changes libcc1 API in an incompatible way. There is a possibility of a hack to keep the API the same - one could pass -v option explicitly to set_arguments(), set_arguments() could compare the -v string and print the

[Bug fortran/52251] Nonadvancing I/O and the t edit descriptor

2015-04-23 Thread jvdelisle at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52251 Jerry DeLisle jvdelisle at gcc dot gnu.org changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org

[Bug c++/65866] Wrong warning when using list-initialization: operation on 'b' may be undefined [-Wsequence-point]

2015-04-23 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65866 Jonathan Wakely redi at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |NEW Last

[Bug ada/65868] New: cannot find -lstdc++ for GNAT compilation

2015-04-23 Thread porton at narod dot ru
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65868 Bug ID: 65868 Summary: cannot find -lstdc++ for GNAT compilation Product: gcc Version: 5.1.0 Status: UNCONFIRMED Severity: blocker Priority: P3 Component: ada

C PATCH for c/52085 (enum forward declarations and attribute packed)

2015-04-23 Thread Marek Polacek
This PR points out a problem with enum forward declarations (so C++ is out as these are forbidden in C++). If we forward declare an enum, and later on declare the enum with __attribute__ ((packed)), the attribute is ignored. The reason is that when we first see the forward declaration,

[Bug rtl-optimization/65805] [5 Regression] Chromium gets miscompiled

2015-04-23 Thread vmakarov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65805 --- Comment #6 from Vladimir Makarov vmakarov at gcc dot gnu.org --- Author: vmakarov Date: Thu Apr 23 19:07:52 2015 New Revision: 222383 URL: https://gcc.gnu.org/viewcvs?rev=222383root=gccview=rev Log: 2015-04-23 Vladimir Makarov

[Bug bootstrap/65865] [6 Regression] Bootstrap failure on x86

2015-04-23 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65865 --- Comment #2 from H.J. Lu hjl.tools at gmail dot com --- [hjl@gnu-6 gcc]$ cat /tmp/x.cc extern unsigned long indirect_call_wrapper; typedef unsigned long size_t; template typename F F IndirectExternCall(F f) { typedef F (*WrapF)(F);

patch for PR65805 was backported to gcc-5 branch

2015-04-23 Thread Vladimir Makarov
The following patch was backported to gcc-5 branch. The patch was bootstrapped on x86-64. Committed as rev. 222383. 2015-04-23 Vladimir Makarov vmaka...@redhat.com Backport from trunk r23. 2015-04-19 Vladimir Makarov vmaka...@redhat.com PR

Re: [PATCH, rs6000, testsuite] Fix PR target/64579, __TM_end __builtin_tend failed to return transactional state

2015-04-23 Thread Peter Bergner
On Wed, 2015-04-22 at 17:16 -0500, Segher Boessenkool wrote: On Wed, Apr 22, 2015 at 08:43:10AM -0500, Peter Bergner wrote: Maybe you can fold tabortdc with tabortwc now? Use one UNSPEC name for both, :GPR and wd? Wouldn't that change the tabortwc pattern to use DImode rather than

[PATCH] PR target/65849 -- Add additional options to powerpc #pragma/attribute target support

2015-04-23 Thread Michael Meissner
Steve Munroe was tuning an application on PowerPC, and needed to set the -msave-toc-indirect option for only one function, and it wasn't available via the #praga/attribute target options. This patch adds support for the additional options that don't involve an ABI change to the list of options

[Bug bootstrap/65865] [6 Regression] Bootstrap failure on x86

2015-04-23 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65865 Jason Merrill jason at gcc dot gnu.org changed: What|Removed |Added CC||jason at gcc dot

[Bug libquadmath/65757] gfortran gives incorrect result for anint with real*16 argument

2015-04-23 Thread bugs at dhbailey dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65757 --- Comment #8 from dhbbugs bugs at dhbailey dot com --- Jerry DeLisl'e output is certainly not correct -- anint should invariably return the nearest whole number. It should be the equivalent of this code: if (x = 0.0) then anint = aint (x +

[PATCH v2 1/4] libcc1: Introduce GCC_FE_VERSION_1

2015-04-23 Thread Jan Kratochvil
Hi, in mail thread https://sourceware.org/ml/gdb-patches/2015-04/msg00804.html the idea of breaking libcc1.so compatibility was rejected. Therefore this patch series implements full backward/forward GCC/GDB ABI compatibility. Jan include/ChangeLog 2015-04-23 Jan Kratochvil

[PATCH v2 3/4] libcc1: Add 'set compile-gcc'

2015-04-23 Thread Jan Kratochvil
Hi, already approved, reposting just to keep it a part of the series: https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01299.html As discussed in How to use compile execute function in GDB https://sourceware.org/ml/gdb/2015-04/msg00026.html GDB currently searches for

[PATCH v2 2/4] libcc1: set debug compile: Display GCC driver filename

2015-04-23 Thread Jan Kratochvil
Hi, already approved, reposting just to keep it a part of the series: https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01301.html As discussed in How to use compile execute function in GDB https://sourceware.org/ml/gdb/2015-04/msg00026.html GDB currently searches for

[PATCH v2 4/4] libcc1: 'set debug compile': Display absolute GCC driver filename

2015-04-23 Thread Jan Kratochvil
Hi, already approved, maybe it could be already checked in outside of the series: https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01298.html With the patches so far after (gdb) set debug compile 1 one would get: searching for compiler matching regex

Re: [PATCH v2 1/4] libcc1: Introduce GCC_FE_VERSION_1

2015-04-23 Thread Jeff Law
On 04/23/2015 02:38 PM, Jan Kratochvil wrote: Hi, in mail thread https://sourceware.org/ml/gdb-patches/2015-04/msg00804.html the idea of breaking libcc1.so compatibility was rejected. Therefore this patch series implements full backward/forward GCC/GDB ABI compatibility. Jan

[Bug c++/65870] New: Explicit function instantiation with default valued lambda causes duplicate symbol

2015-04-23 Thread w.shane.grant at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65870 Bug ID: 65870 Summary: Explicit function instantiation with default valued lambda causes duplicate symbol Product: gcc Version: 4.9.1 Status: UNCONFIRMED

Re: [wwwdocs] PATCH for Re: Confirmation Mirror Changes

2015-04-23 Thread Tim Semeijn
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Dear Gerald, Thanks for processing the patch! Best regards, On 4/23/15 11:49 PM, Gerald Pfeifer wrote: On Fri, 17 Apr 2015, Tim Semeijn wrote: We have changed our company name, hostnames and contact information. Please remove the current BBLN

Re: [C PATCH] Make -Wno-shift-count-negative -Wno-shift-count-overflow work for const ints (PR c/65830)

2015-04-23 Thread Jeff Law
On 04/21/2015 04:00 PM, Marek Polacek wrote: A trivial patch to use OPT_* where they belong. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2015-04-21 Marek Polacek pola...@redhat.com PR c/65830 * c-common.c (c_fully_fold_internal): Use OPT_Wshift_count_negative

[Bug target/65456] powerpc64le autovectorized copy loop missed optimization

2015-04-23 Thread wschmidt at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65456 --- Comment #18 from Bill Schmidt wschmidt at gcc dot gnu.org --- Author: wschmidt Date: Thu Apr 23 21:03:40 2015 New Revision: 222386 URL: https://gcc.gnu.org/viewcvs?rev=222386root=gccview=rev Log: [gcc] 2015-04-23 Bill Schmidt

Re: [PATCH v2 2/4] libcc1: set debug compile: Display GCC driver filename

2015-04-23 Thread Jeff Law
On 04/23/2015 02:38 PM, Jan Kratochvil wrote: Hi, already approved, reposting just to keep it a part of the series: https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01301.html As discussed in How to use compile execute function in GDB

[Bug c++/65869] New: Incorrect overload resolution in function return statement

2015-04-23 Thread botond at mozilla dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65869 Bug ID: 65869 Summary: Incorrect overload resolution in function return statement Product: gcc Version: 4.9.2 Status: UNCONFIRMED Severity: normal

Re: [PATCH v2 3/4] libcc1: Add 'set compile-gcc'

2015-04-23 Thread Jeff Law
On 04/23/2015 02:38 PM, Jan Kratochvil wrote: Hi, already approved, reposting just to keep it a part of the series: https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01299.html As discussed in How to use compile execute function in GDB

[Patch, MIPS] Minor cleanup in mips.md

2015-04-23 Thread Steve Ellcey
This is a minor cleanup patch for MIPS. A number of floating point madd type instructions set the accum_in attribute. But this attribute is only used for integer madd instructions, so this patch removes it from the floating point madd instructions where it is not needed or used. The 'accum_in'

C PATCH to add -Woverride-init-side-effects (PR c/64918)

2015-04-23 Thread Marek Polacek
As discussed in the PR, the initialized field with side-effects overwritten warning is sometimes not so useful, so it probably makes sense to provide an option so that users are able to specifically enable/disable it. Since the warning is enabled by default at present, it is enabled by default

Re: [wwwdocs] PATCH for Re: Confirmation Mirror Changes

2015-04-23 Thread Tim Semeijn
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Dear Gerald, Thanks for processing the patch! Best regards, On 4/23/15 11:49 PM, Gerald Pfeifer wrote: On Fri, 17 Apr 2015, Tim Semeijn wrote: We have changed our company name, hostnames and contact information. Please remove the current BBLN

Re: [PATCH] Quiet down -Wlogical-op a bit (PR c/61534)

2015-04-23 Thread Jeff Law
On 04/22/2015 07:56 AM, Marek Polacek wrote: This patch stifles -Wlogical-op a bit: don't warn if either operand comes from a macro expansion. As the comment says, it doesn't fix the bug completely, but it's a simple improvement. I did this by introducing a new macro. Bootstrapped/regtested

Re: [Patch] OPT: Update heuristics for loop-invariant for address arithmetic.

2015-04-23 Thread Jeff Law
On 03/23/2015 10:35 AM, Ajit Kumar Agarwal wrote: Hello All: Did you get a chance to look at the below patch. Thanks Regards Ajit -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On Behalf Of Ajit Kumar Agarwal Sent: Wednesday, March 04,

Re: [C/C++ PATCH] Improve -Wlogical-op (PR c/63357)

2015-04-23 Thread Jeff Law
On 04/21/2015 05:16 AM, Marek Polacek wrote: This patch improves -Wlogical-op so that it also warns about cases such as P P or P || P. I made use of what merge_ranges computes: if we have equal operands with the same ranges, warn -- that seems to work well. (-Wlogical-op still isn't enabled

[Bug c/64918] invalid (?) warning when initializing structure

2015-04-23 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64918 Marek Polacek mpolacek at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED

[wwwdocs] PATCH for Re: Confirmation Mirror Changes

2015-04-23 Thread Gerald Pfeifer
On Fri, 17 Apr 2015, Tim Semeijn wrote: We have changed our company name, hostnames and contact information. Please remove the current BBLN mirror (mirror.bbln.org) and replace it with our three new ones: The patch below implements those changes: - Replace mirror.bbln.org by

[wwwdocs] PATCH for Re: Confirmation Mirror Changes

2015-04-23 Thread Gerald Pfeifer
On Fri, 17 Apr 2015, Tim Semeijn wrote: We have changed our company name, hostnames and contact information. Please remove the current BBLN mirror (mirror.bbln.org) and replace it with our three new ones: The patch below implements those changes: - Replace mirror.bbln.org by

Re: [PATCH v2 4/4] libcc1: 'set debug compile': Display absolute GCC driver filename

2015-04-23 Thread Jeff Law
On 04/23/2015 02:38 PM, Jan Kratochvil wrote: Hi, already approved, maybe it could be already checked in outside of the series: https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01298.html With the patches so far after (gdb) set debug compile 1 one would get: searching

Re: [gcc libcc1] build_qualified_type for self-referencing/incomplete types

2015-04-23 Thread Jeff Law
On 04/18/2015 04:19 AM, Jan Kratochvil wrote: On Fri, 17 Apr 2015 17:22:13 +0200, Jan Kratochvil wrote: How to get 'volatile struct sv' GCC 'tree' type for: volatile struct sv { volatile struct sv *p; }; I have found out how it can work, even with no change on the GCC side: Instead

  1   2   3   >