Re: Vandalised wiki page

2013-08-26 Thread Dodji Seketeli
Jonathan Wakely jwakely@gmail.com a écrit: I've reverted or deleted all the spam I could find (and that you hadn't already done) and have added lots of people to the EditorGroup who had made good changes in the past. Thank you for doing this. It's appreciated. -- Dodji

Re: Vandalised wiki page

2013-08-26 Thread Joel Sherrill
For the RTEMS wiki, we require a 50 word bio and manually approve accounts. It is effort to approve accounts but less than the spam clean up was. It is sad either way. --joel Dodji Seketeli do...@seketeli.org wrote: Jonathan Wakely jwakely@gmail.com a écrit: I've reverted or deleted

--with-dwarf2 and default DWARF version

2013-08-26 Thread Senthil Kumar Selvaraj
Hi, The help text for the --with-dwarf2 configure option says it forces the default debug format to be DWARF 2. However, the built compiler still produces DWARF 4, unless -gdwarf-2 is explicitly specified when compiling. I see that the darwin target gets around this by using the

Re: --with-dwarf2 and default DWARF version

2013-08-26 Thread Jakub Jelinek
On Mon, Aug 26, 2013 at 01:42:18PM +0530, Senthil Kumar Selvaraj wrote: The help text for the --with-dwarf2 configure option says it forces the default debug format to be DWARF 2. However, the built compiler still produces DWARF 4, unless -gdwarf-2 is explicitly specified when

Re: --with-dwarf2 and default DWARF version

2013-08-26 Thread Senthil Kumar Selvaraj
On Mon, Aug 26, 2013 at 10:23:24AM +0200, Jakub Jelinek wrote: On Mon, Aug 26, 2013 at 01:42:18PM +0530, Senthil Kumar Selvaraj wrote: The help text for the --with-dwarf2 configure option says it forces the default debug format to be DWARF 2. However, the built compiler still produces

[ping] [buildrobot] gcc/config/linux-android.c:40:7: error: ‘OPTION_BIONIC’ was not declared in this scope

2013-08-26 Thread Jan-Benedict Glaw
Hi! On Tue, 2013-08-20 11:24:31 +0400, Alexander Ivchenko aivch...@gmail.com wrote: Hi, thanks for cathing this. I certainly missed that OPTION_BIONIC is not defined for linux targets that do not include config/linux.h in their tm.h. This patch fixed build for powerpc64le-linux and

Re: [RFC] Offloading Support in libgomp

2013-08-26 Thread Michael V. Zolotukhin
No need for the device and handler IMHO, each vector would correspond to one function call (GOMP_target, GOMP_target_data or GOMP_target_update) and all those calls would be called with device id. Probably yes. Let's talk about some concrete example (though, I see the gimplifier doesn't

Re: [RFC] Offloading Support in libgomp

2013-08-26 Thread Jakub Jelinek
On Mon, Aug 26, 2013 at 03:59:11PM +0400, Michael V. Zolotukhin wrote: As I currently see it, the given code would be expanded to something like this: // Create two versions of V: for host and for target int v; int v_target __attribute(target); // The same for TGT function int

Re: [RFC] Offloading Support in libgomp

2013-08-26 Thread Michael V. Zolotukhin
Actually, not two versions of those during the compilation, you have just one v and one tgt, both have __attribute__((omp declare target)) on them (note, you can't specify that attribute manually). And just when streaming into .gnu.target_lto_* sections you only stream everything that has

Re: [RFC] Offloading Support in libgomp

2013-08-26 Thread Jakub Jelinek
On Mon, Aug 26, 2013 at 05:29:36PM +0400, Michael V. Zolotukhin wrote: Nope, there is only one target data pragma, so you would use here just: struct data_descriptor data_desc2[2] = { ... }; GOMP_target (-1, bar.omp_fn.1, bar.omp_fn.1, data_desc2, 2); This 'pragma target' is placed

MPX ABI extension

2013-08-26 Thread Ilya Enkovich
Hi, Here is a patch to extend x86-64 psABI to support MPX [1]. A short description of changes: - There are 4 bound registers (bnd0-bnd3) added; all of them are caller-save - When we pass (return) pointer on register, we use the next available bound register to pass (return) bounds - If there

Re: --with-dwarf2 and default DWARF version

2013-08-26 Thread Michael Eager
On 08/26/13 01:40, Senthil Kumar Selvaraj wrote: It isn't any kind of workaround the above. Apple tools are simply so buggy that they can't grok DWARF later than 2, crash in lots of ways on it. Why do you want DWARF2 rather than DWARF3 or 4? Same reason, some tools (proprietary debugger

peliculas series de tv musica recitales juegos

2013-08-26 Thread Yanina
peliculas series de tv recitales discografias completas juegos para todas las consolas aplicaciones para todos los operativos buscanos con el nombre xtop

[Bug c/58240] GCC optimize strncmp when N=1 incorrectly

2013-08-26 Thread pinskia at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58240 --- Comment #10 from Andrew Pinski pinskia at gcc dot gnu.org --- (In reply to shiyan from comment #7) (In reply to Marc Glisse from comment #4) -fno-builtin (or some better such option)? strncmp is a standard function, your code redefining

[Bug c/58240] GCC optimize strncmp when N=1 incorrectly

2013-08-26 Thread glisse at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58240 --- Comment #11 from Marc Glisse glisse at gcc dot gnu.org --- (In reply to shiyan from comment #7) Thank you for the suggestion. Yes, I know -fno-builtin can work around it. I can think of many possible ways to work around itbut whatever,

[Bug c/58240] GCC optimize strncmp when N=1 incorrectly

2013-08-26 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58240 Jakub Jelinek jakub at gcc dot gnu.org changed: What|Removed |Added CC||jakub at gcc dot

[Bug bootstrap/58242] New: [4.9 regression] linux-android.c:40:7: error: 'OPTION_BIONIC' was not declared in this scope breaks bootstrap on powerpc64-linux

2013-08-26 Thread mikpe at it dot uu.se
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58242 Bug ID: 58242 Summary: [4.9 regression] linux-android.c:40:7: error: 'OPTION_BIONIC' was not declared in this scope breaks bootstrap on powerpc64-linux Product: gcc

[Bug c/58240] GCC optimize strncmp when N=1 incorrectly

2013-08-26 Thread shiyan2016 at 126 dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58240 --- Comment #13 from shiyan shiyan2016 at 126 dot com --- Hi all, Thank you for all your explanation. This seems more like a philosophical discussion :) I do can understand what GCC is doing (as I mentioned, it may be optimized to *s1-*s2).

[Bug bootstrap/58242] [4.9 regression] linux-android.c:40:7: error: 'OPTION_BIONIC' was not declared in this scope breaks bootstrap on powerpc64-linux

2013-08-26 Thread ubizjak at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58242 Uroš Bizjak ubizjak at gmail dot com changed: What|Removed |Added CC||aivchenk at gmail

[Bug tree-optimization/58243] New: Suboptimal structure initialization with tree-sra

2013-08-26 Thread hp at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58243 Bug ID: 58243 Summary: Suboptimal structure initialization with tree-sra Product: gcc Version: 4.9.0 Status: UNCONFIRMED Keywords: missed-optimization Severity: normal

[Bug target/58208] dequestd::string 32-bit -O3 bug

2013-08-26 Thread tammy at Cadence dot COM
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58208 --- Comment #9 from Tammy Hsu tammy at Cadence dot COM --- I tried to run the g++ build on RHEL 5.5 on a RHEL 6.3 system, import seg fault. I then tried to rebuild gcc481 on RHEL 6.3 and rerun the testcase, it still crash. The glibc on RHEL 6.3

[Bug c/35649] Incorrect printf warning: expect double has float

2013-08-26 Thread amylaar at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35649 Jorn Wolfgang Rennecke amylaar at gcc dot gnu.org changed: What|Removed |Added Status|NEW

[Bug regression/58244] New: global variable: many THOUSANDS times slower execution

2013-08-26 Thread martin.konopka at stuba dot sk
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58244 Bug ID: 58244 Summary: global variable: many THOUSANDS times slower execution Product: gcc Version: 4.7.3 Status: UNCONFIRMED Severity: normal Priority: P3

[Bug regression/58244] global variable: many THOUSANDS times slower execution

2013-08-26 Thread pinskia at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58244 --- Comment #1 from Andrew Pinski pinskia at gcc dot gnu.org --- Why do you think this is a bug?

[Bug regression/58244] global variable: many THOUSANDS times slower execution

2013-08-26 Thread martin.konopka at stuba dot sk
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58244 --- Comment #2 from Martin Konôpka martin.konopka at stuba dot sk --- It it is confirmed, it is a very serious performance issue. In my test the code with the global declaration executed about 6 times slower than the code with the local

[Bug regression/58244] global variable: many THOUSANDS times slower execution

2013-08-26 Thread glisse at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58244 --- Comment #3 from Marc Glisse glisse at gcc dot gnu.org --- With auxval local, the compiler knows that computing suma, etc is useless and removes all that code, including the calls to sin. So you would like the compiler to do the same for a

[Bug regression/58244] global variable: many THOUSANDS times slower execution

2013-08-26 Thread sch...@linux-m68k.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58244 --- Comment #4 from Andreas Schwab sch...@linux-m68k.org --- gcc was able to optimize your code to make it 6000 times faster. How is that a regression?

[Bug regression/58244] global variable: many THOUSANDS times slower execution

2013-08-26 Thread martin.konopka at stuba dot sk
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58244 --- Comment #5 from Martin Konôpka martin.konopka at stuba dot sk --- I do not fully understand the question. In both cases I used the same optimisation (-O2). My other comment a while ago was lost. I must retype it.

[Bug target/58208] dequestd::string 32-bit -O3 bug

2013-08-26 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58208 Jakub Jelinek jakub at gcc dot gnu.org changed: What|Removed |Added CC||jakub at gcc dot

[Bug regression/58244] global variable: many THOUSANDS times slower execution

2013-08-26 Thread martin.konopka at stuba dot sk
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58244 --- Comment #6 from Martin Konôpka martin.konopka at stuba dot sk --- (I am sorry that I do not understand internals of compilers.) I first hit the issue in a bigger code having more than 11000 lines. First I did not understand at all what is

[Bug regression/58244] global variable: many THOUSANDS times slower execution

2013-08-26 Thread sch...@linux-m68k.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58244 --- Comment #7 from Andreas Schwab sch...@linux-m68k.org --- Now try the same code with -O0.

[Bug target/58208] dequestd::string 32-bit -O3 bug

2013-08-26 Thread tammy at Cadence dot COM
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58208 --- Comment #11 from Tammy Hsu tammy at Cadence dot COM --- Thank you. Yes, on the fedora 19 systems, I don't have these 3 i686 rpms installed. I will add them. Do you have any comments on the crash issue we have on CentOS 5.8 or RHEL 5.5/RHEL

[Bug c++/39057] ICE with default argument in friend declaration

2013-08-26 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39057 Paolo Carlini paolo.carlini at oracle dot com changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug libstdc++/58153] unordered_multimap::erase(iterator) is not constant-time when many entries have the same key

2013-08-26 Thread fdumont at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58153 --- Comment #5 from François Dumont fdumont at gcc dot gnu.org --- And your remark is good too and will avoid me to spend some time on this idea. Standard requirements regarding validity of iterators won't let us have iterators invalidated because

[Bug regression/58244] global variable: many THOUSANDS times slower execution

2013-08-26 Thread martin.konopka at stuba dot sk
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58244 --- Comment #8 from Martin Konôpka martin.konopka at stuba dot sk --- Yes, I understand now. Thanks. The lines with the sin() functions were not evaluated with the local declaration. My apologies for reporting the false bug. I will try to close

[Bug other/58238] cc1 crashes when built for ms-dos cross-compiling

2013-08-26 Thread dj at redhat dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58238 --- Comment #2 from DJ Delorie dj at redhat dot com --- Please try the attached patch. I tested it with a simple #include stdint.h but we made the type names exact matches (way back when) for a reason...

[Bug other/58238] cc1 crashes when built for ms-dos cross-compiling

2013-08-26 Thread dj at redhat dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58238 DJ Delorie dj at redhat dot com changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last

[Bug middle-end/58245] New: -fstack-protector[-all] does not protect functions that call noreturn functions

2013-08-26 Thread bugdal at aerifal dot cx
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58245 Bug ID: 58245 Summary: -fstack-protector[-all] does not protect functions that call noreturn functions Product: gcc Version: unknown Status: UNCONFIRMED

[Bug middle-end/58245] -fstack-protector[-all] does not protect functions that call noreturn functions

2013-08-26 Thread bugdal at aerifal dot cx
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58245 --- Comment #1 from Rich Felker bugdal at aerifal dot cx --- One more thing: I would be happy with either of two solutions, either: (1) Checking the canary before calling a noreturn function, just like performing a check before a tail-call, or

[Bug middle-end/58245] -fstack-protector[-all] does not protect functions that call noreturn functions

2013-08-26 Thread pinskia at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58245 --- Comment #2 from Andrew Pinski pinskia at gcc dot gnu.org --- The best solution: Don't use the same triplet as the GNU (glibc) one. Have musl have its own triplet.

[Bug middle-end/58245] -fstack-protector[-all] does not protect functions that call noreturn functions

2013-08-26 Thread bugdal at aerifal dot cx
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58245 --- Comment #3 from Rich Felker bugdal at aerifal dot cx --- We already do that; the patch is in the musl-cross repo here: https://bitbucket.org/GregorR/musl-cross or https://github.com/GregorR/musl-cross However, we want the stack-protector

[Bug libstdc++/58153] unordered_multimap::erase(iterator) is not constant-time when many entries have the same key

2013-08-26 Thread temporal at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58153 --- Comment #6 from Kenton Varda temporal at gmail dot com --- Yep, I realize that erase_after would need to be added to the standard. I was just speculating that it may be something the standard committee should consider. I've long since solved

[Bug target/57927] -march=core-avx2 different than -march=native on INTEL Haswell (i7-4700K)

2013-08-26 Thread shadow at umbrox dot de
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57927 Christian Widmer shadow at umbrox dot de changed: What|Removed |Added CC||shadow at

[Bug tree-optimization/58246] New: wrong code at -O1 and above (affecting 4.6, 4.7, 4.8, and trunk)

2013-08-26 Thread su at cs dot ucdavis.edu
and above in both 32-bit and 64-bit modes. $ gcc-trunk -v gcc version 4.9.0 20130826 (experimental) [trunk revision 201986] (GCC) $ gcc-trunk -O0 small.c $ a.out 1 $ gcc-trunk -O1 small.c $ a.out 0 $ gcc-4.8 -O1 small.c $ a.out 0 $ gcc-4.7 -O1 small.c $ a.out 0 $ gcc-4.6 -O1 small.c $ a.out 0

[Bug tree-optimization/58247] New: ICE in tree_unroll_loops_completely at -O3 (both 32-bit and 64-bit modes)

2013-08-26 Thread su at cs dot ucdavis.edu
. This is a regression from 4.8.x. It is likely a duplicate of 57592, although the stack traces are somewhat different. $ gcc-trunk -v gcc version 4.9.0 20130826 (experimental) [trunk revision 201986] (GCC) $ gcc-trunk -O2 -c reduced.c $ gcc-4.8 -O3 -c reduced.c $ gcc-trunk -O3 -c reduced.c reduced.c

Re: [PATCH, vtv update] Fix /tmp directory issues in libvtv

2013-08-26 Thread Florian Weimer
On 08/25/2013 09:33 PM, Gerald Pfeifer wrote: On Tue, 20 Aug 2013, Florian Weimer wrote: As the libvtv reviewer, you don't need permission to commit your changes. :-) Actually, reviewers do need someone else's approval for their own changes (unlike maintainers and of course not for trivial

Re: Fix OBJ_TYPE_REF handling in ipa-cp

2013-08-26 Thread Martin Jambor
Hi, On Thu, Aug 22, 2013 at 05:33:50PM +0200, Jan Hubicka wrote: Hi, this problem was noticed by my verifier that binfo walks are not across type hiearchy. ipa_intraprocedural_devirtualization is one remaining place where we take class of object from OBJ_TYPE_REF_OBJECT instead of

Re: [PATCH 0/2] Port symtab/cgraph/varpool nodes to use C++ inheritance

2013-08-26 Thread David Malcolm
On Tue, 2013-08-20 at 23:01 +0200, Jan Hubicka wrote: [...] In summary, struct GTY(()) symtab_node_base becomes: class GTY((user)) symtab_node_base and the subclasses: struct GTY(()) cgraph_node and: struct GTY(()) varpool_node become (respectively): struct

Re: [PATCH, vtv update] Fix /tmp directory issues in libvtv

2013-08-26 Thread Caroline Tice
Thank you for pointing out my error. I will commit the following patch. -- Caroline Tice cmt...@google.com 2013-08-26 Caroline Tice cmt...@google.com * MAINTAINERS: Correct earliers update: Move myself from libvtv Various Reviewers to libvtv Various Maintainers. Index: MAINTAINERS

Re: ELF interposition and One Definition Rule

2013-08-26 Thread Mike Stump
On Aug 26, 2013, at 8:21 AM, Jan Hubicka hubi...@ucw.cz wrote: My understanding of C++ One Definition Rule, in a strict sense, does not a allow in to define two functions of the same name and different semantics in a valid program . I also think that all DSOs eventually linked together or

Re: [GOOGLE] Update AutoFDO annotation

2013-08-26 Thread Xinliang David Li
Can you add missing documentation on functions like ...:get_count_info -- documenting return value etc. Also it might be better to avoid using 'set' as the local variable name. Change it to something more specific. thanks, David On Thu, Aug 22, 2013 at 3:56 PM, Dehao Chen de...@google.com

Re: ELF interposition and One Definition Rule

2013-08-26 Thread Jason Merrill
On 08/26/2013 11:21 AM, Jan Hubicka wrote: Our default behaviour special case inline functions that are always AVAIL_AVAILABLE and, via decl_replaceable_p, also any COMDAT (that may be for functions since all COMDATs are also inlines, but makes difference for variables I think). Not all COMDAT

Re: [C++ Patch] Remove old bison hack?!

2013-08-26 Thread Jason Merrill
OK. Jason

Re: Make some comdats implicitly hidden

2013-08-26 Thread Jason Merrill
On 08/26/2013 03:57 PM, Jan Hubicka wrote: While analyzing the relocations of libreoffice I noticed that I can play the same game w/o LTO at linker level. Making those symbols hidden truns external relocations to internal and should improve runtime, too: comdat sharing by dynamic linker is

Re: ELF interposition and One Definition Rule

2013-08-26 Thread Gabriel Dos Reis
On Mon, Aug 26, 2013 at 7:58 PM, Jason Merrill ja...@redhat.com wrote: On 08/26/2013 11:21 AM, Jan Hubicka wrote: Our default behaviour special case inline functions that are always AVAIL_AVAILABLE and, via decl_replaceable_p, also any COMDAT (that may be for functions since all COMDATs are

Re: [C++ patch] Set attributes for C++ runtime library calls

2013-08-26 Thread Jason Merrill
On 08/26/2013 10:45 AM, Gabriel Dos Reis wrote: Hmm, let's not make it a default. Replacing global operator new (e.g. for tracing purposes) is a valid C++ programming idiom. Absolutely. What strikes me as vanishingly unlikely is the idea that the replacement operator new would expose

Re: [C++ patch] Set attributes for C++ runtime library calls

2013-08-26 Thread Gabriel Dos Reis
On Mon, Aug 26, 2013 at 8:17 PM, Jason Merrill ja...@redhat.com wrote: On 08/26/2013 10:45 AM, Gabriel Dos Reis wrote: Hmm, let's not make it a default. Replacing global operator new (e.g. for tracing purposes) is a valid C++ programming idiom. Absolutely. What strikes me as vanishingly

Re: [C++ patch] Set attributes for C++ runtime library calls

2013-08-26 Thread Jason Merrill
On 08/22/2013 09:19 AM, Jan Hubicka wrote: - I tried to track functions that lead to terminate() and not mark them as ECF_LEAF. This is because user can set handler. If the handler can resonably expect the static vars defined in its unit to be in the final form, we can not

[patch/djgpp]: Add ASM_DECLARE_FUNCTION_NAME

2013-08-26 Thread DJ Delorie
Needed to build i586-pc-msdosdjgpp target, committed. * config/i386/djgpp.h (ASM_DECLARE_FUNCTION_NAME): New. Index: config/i386/djgpp.h === --- config/i386/djgpp.h (revision 202015) +++ config/i386/djgpp.h (working copy)

Re: [PATCH, libgcc] Fix licenses on several files

2013-08-26 Thread Maxim Kuvyrkov
On 29/07/2013, at 10:03 AM, Maxim Kuvyrkov wrote: While verifying license compliance for GCC and its libraries I noticed that several libgcc files that end up in the final library are licensed under GPL-3.0+ instead of GPL-3.0-with-GCC-exception. This is, obviously, was not the intention

RE: [ping**4] Re: [patch 0/4] reimplement -fstrict-volatile-bitfields, v3

2013-08-26 Thread Bernd Edlinger
PING! This issue is really important. It does not only affect bitfields but all kinds of packed structures. Starting from gcc 4.6.0 there is not a single released version that handles the packed structures correctly. So could some one please approve Sandra's patch now? Thanks Bernd.

Re: powerpc64le multilibs and multiarch dir

2013-08-26 Thread Alan Modra
On Sun, Aug 25, 2013 at 10:40:30PM -0700, Mike Stump wrote: On Aug 25, 2013, at 8:32 PM, Alan Modra amo...@gmail.com wrote: We (IBM) don't intend to support running both big and little-endian processes on the same system in the near future. Perhaps I'm jumping the gun in defining the

Re: [C++ patch] Set attributes for C++ runtime library calls

2013-08-26 Thread Mike Stump
On Aug 23, 2013, at 9:36 PM, Gabriel Dos Reis g...@integrable-solutions.net wrote: You made a statement. It was not clear whether it was what you want or whether it is what the standards mandate. Both. (2) what you believe the standards mandate, with appropriate quote; and The life of

Re: [C++ patch] Set attributes for C++ runtime library calls

2013-08-26 Thread Gabriel Dos Reis
On Mon, Aug 26, 2013 at 2:44 AM, Mike Stump mikest...@comcast.net wrote: but I fear you won't understand this and how it applies. You must be right. I cannot match the entertaining value of your message, so you win. -- Gaby

Clean up pretty printers [18/n]

2013-08-26 Thread Gabriel Dos Reis
Same topic as patch 17/n. For more expressing printers. Tested on an x86_64-suse-linux. Applied to mainline. -- Gaby 2013-08-26 Gabriel Dos Reis g...@integrable-solutions.net c-family/ * c-pretty-print.h (c_pretty_printer::unary_expression): Now a virtual member function.

[PATCH, i386]: Remove unneeded ATTRIBUTE_UNUSED decorations

2013-08-26 Thread Uros Bizjak
Hello! 2013-08-26 Uros Bizjak ubiz...@gmail.com * config/i386/i386.c (ix86_debug_options): Remove prototype. (x86_64_elf_select_section): Ditto. (ix86_handle_tm_regparm_attribute): Remove ATTRIBUTE_UNUSED on used arguments. (ix86_pass_by_reference): Ditto.

Re: [PATCH, i386]: Remove unneeded ATTRIBUTE_UNUSED decorations

2013-08-26 Thread Jakub Jelinek
On Mon, Aug 26, 2013 at 11:04:42AM +0200, Uros Bizjak wrote: static bool -ix86_return_in_memory (const_tree type, const_tree fntype ATTRIBUTE_UNUSED) +ix86_return_in_memory (const_tree type, const_tree fntype) { #ifdef SUBTARGET_RETURN_IN_MEMORY return SUBTARGET_RETURN_IN_MEMORY (type,

Re: [PATCH, i386]: Remove unneeded ATTRIBUTE_UNUSED decorations

2013-08-26 Thread Uros Bizjak
On Mon, Aug 26, 2013 at 11:13 AM, Jakub Jelinek ja...@redhat.com wrote: On Mon, Aug 26, 2013 at 11:04:42AM +0200, Uros Bizjak wrote: static bool -ix86_return_in_memory (const_tree type, const_tree fntype ATTRIBUTE_UNUSED) +ix86_return_in_memory (const_tree type, const_tree fntype) {

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Gabriel Dos Reis
Paolo Carlini paolo.carl...@oracle.com writes: | Hi Gaby. | | On 08/26/2013 10:42 AM, Gabriel Dos Reis wrote: | Same topic as patch 17/n. For more expressing printers. | Tested on an x86_64-suse-linux. Applied to mainline. | Just got this: | |

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Gabriel Dos Reis
Paolo Carlini paolo.carl...@oracle.com writes: | Hi Gaby, | | bootstrap is currently broken: | | /scratch/Gcc/svn-dirs/trunk/gcc/c/c-objc-common.c: In function ‘bool | c_tree_printer(pretty_printer*, text_info*, const char*, int, bool, | bool, bool)’: |

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Paolo Carlini
On 08/26/2013 11:20 AM, Gabriel Dos Reis wrote: You need more than that. The rest of patch is in trunk now. Sorry for the breakage -- as you probably guessed, I specified only c-family directory on the commit command line. Thanks! Paolo.

[c++-concepts] MErge from trunk

2013-08-26 Thread Gabriel Dos Reis
at revision 201992. There were conflicts with the recent pretty printing patches. Fixed. (Mostly due to the new concept-specific nodes printing.) Modulo the asan stuff. -- Gaby

Re: [PATCH] Create pass through and ancestor jump functions even there is dynamic type change

2013-08-26 Thread Jan Hubicka
2013-08-23 Martin Jambor mjam...@suse.cz * ipa-prop.h (ipa_pass_through_data): New field type_preserved. (ipa_ancestor_jf_data): Likewise. (ipa_get_jf_pass_through_agg_preserved): Fix comment typo. (ipa_get_jf_pass_through_type_preserved): New function.

[ubsan] Introduce pointer_sized_int_node

2013-08-26 Thread Marek Polacek
I noticed I forgot to apply this old patch, already acked by Jason. It introduces new pointer_sized_int_node, thus we can get rid of uptr_type function in ubsan, and it allows us to do some clean-up in asan.c, too. Tested x86_64-linux, applying to ubsan branch. 2013-08-26 Marek Polacek

Committed: fix i686 bootstrap: x86_64_elf_select_section is unused

2013-08-26 Thread Joern Rennecke
bootstrap for i686-pc-linux-gnu was failing with: /ssd/fsf/boot-201992/./prev-gcc/xg++ -B/ssd/fsf/boot-201992/./prev-gcc/ -B/usr/local/i686-pc-linux-gnu/bin/ -nostdinc++ -B/ssd/fsf/boot-201992/prev-i686-pc-linux-gnu/libstdc++-v3/src/.libs

Re: [Patch, Fortran, OOP] PR 57305: ICE when calling SIZEOF on an unlimited polymorphic variable

2013-08-26 Thread Janus Weil
Hi Mikael, the SIZEOF intrinsic currently returns the size according to the *declared* type for polymorphic variables. I think this doesn't really make much sense and it also causes ICEs when SIZEOF is called on CLASS(*) variables (which don't really have a declared type). Therefore I'm

Fix some issues with speculative devirtualization machinery

2013-08-26 Thread Jan Hubicka
Hi, by making speculative indirect call machinery to be used by ipa-devirt I put it on wild on firefox converting about 30% of all polymorphic calls in the progrma (3). This has provoked some issues I missed previously this patch fixes: 1) dumping - we no longer do only indirect calls from

RFA: fix avr gcc.dg/fixed-point/convert-accum-neg.c execution failure

2013-08-26 Thread Joern Rennecke
The gcc.dg/fixed-point/convert-accum-neg.c execution test fails for avr because for fractional integer to accumulator / integer conversions, the avr target rounds towards -infinity, whereas we are supposed to round towards 0. The attached patch implements rounding towards 0, and adds an option

Re: [PATCH] Intrinsics for fxsave[,64], xsave[,64], xsaveopt[,64]

2013-08-26 Thread Alexander Ivchenko
Hi, I've added references to fxsr, xsave and xsaveopt options and builtins to doc/[invoke,extend].texi. Is it OK? diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 29a30ee..3aad294 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2013-08-23 Alexander Ivchenko

wide-int branch updated

2013-08-26 Thread Kenneth Zadeck
fixed fits_uhwi_p. tested on x86-64. kenny Index: gcc/wide-int.h === --- gcc/wide-int.h (revision 201985) +++ gcc/wide-int.h (working copy) @@ -1650,7 +1650,7 @@ wide_int_ro::fits_shwi_p () const inline bool

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Jan Hubicka
Hi, it seems to be couple weeks I am not able to compile big ltrans unit with -fdump-tree-all because the compiler eventually runs out of memory. I think it is one of your patches introducing serious memory leak. Can you, please, take a look on this? It makes my life harder - it is not fun to

Re: [Patch, Fortran, OOP] PR 57305: ICE when calling SIZEOF on an unlimited polymorphic variable

2013-08-26 Thread Mikael Morin
Le 26/08/2013 13:22, Janus Weil a écrit : Hi Mikael, the SIZEOF intrinsic currently returns the size according to the *declared* type for polymorphic variables. I think this doesn't really make much sense and it also causes ICEs when SIZEOF is called on CLASS(*) variables (which don't

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Gabriel Dos Reis
On Mon, Aug 26, 2013 at 7:24 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, it seems to be couple weeks I am not able to compile big ltrans unit with -fdump-tree-all because the compiler eventually runs out of memory. I think it is one of your patches introducing serious memory leak. Can you,

Re: [ping**4] Re: [patch 0/4] reimplement -fstrict-volatile-bitfields, v3

2013-08-26 Thread Sandra Loosemore
On 08/26/2013 01:16 AM, Bernd Edlinger wrote: PING! This issue is really important. It does not only affect bitfields but all kinds of packed structures. Starting from gcc 4.6.0 there is not a single released version that handles the packed structures correctly. So could some one please

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Jan Hubicka
On Mon, Aug 26, 2013 at 7:24 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, it seems to be couple weeks I am not able to compile big ltrans unit with -fdump-tree-all because the compiler eventually runs out of memory. I think it is one of your patches introducing serious memory leak.

Re: [C++ patch] Set attributes for C++ runtime library calls

2013-08-26 Thread Jason Merrill
On 08/22/2013 12:45 PM, Gabriel Dos Reis wrote: If the user-supplied operator new returns a, then it must also ensure that 'a' is not used anywhere else -- e.g. I you can't do lvalue-to-value conversion on 'a' to see what is written there. Because its storage has been reused. That is, aliasing

Re: [PATCH, i386, MPX 1/X] Support of Intel MPX ISA

2013-08-26 Thread Ilya Enkovich
Ping 2013/8/19 Ilya Enkovich enkovich@gmail.com: Ping 2013/8/12 Ilya Enkovich enkovich@gmail.com: 2013/8/10 Joseph S. Myers jos...@codesourcery.com: On Mon, 29 Jul 2013, Ilya Enkovich wrote: Hi, Here is updated version of the patch. I removed redundant mode_for_bound, added

Re: [Patch, Fortran, OOP] PR 57305: ICE when calling SIZEOF on an unlimited polymorphic variable

2013-08-26 Thread Janus Weil
I'm slightly inclined to kindly invite the user to switch to STORAGE_SIZE+SIZE instead. Any other opinion? Since the SIZEOF intrinsic has been around for some time in gfortran (before STORAGE_SIZE was available), I would say we should at least continue to support it for backward

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Gabriel Dos Reis
On Mon, Aug 26, 2013 at 9:00 AM, Jan Hubicka hubi...@ucw.cz wrote: On Mon, Aug 26, 2013 at 7:24 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, it seems to be couple weeks I am not able to compile big ltrans unit with -fdump-tree-all because the compiler eventually runs out of memory. I think

Re: Add overload for register_pass

2013-08-26 Thread Richard Henderson
On 08/24/2013 02:33 PM, Oleg Endo wrote: gcc/ChangeLog: * passes.c (register_pass): Add overload. * tree-pass.h (register_pass): Forward declare it. Add comment. Ok. r~

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Jan Hubicka
On Mon, Aug 26, 2013 at 9:00 AM, Jan Hubicka hubi...@ucw.cz wrote: On Mon, Aug 26, 2013 at 7:24 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, it seems to be couple weeks I am not able to compile big ltrans unit with -fdump-tree-all because the compiler eventually runs out of memory. I

Re: [C++ patch] Set attributes for C++ runtime library calls

2013-08-26 Thread Jan Hubicka
On 08/22/2013 12:45 PM, Gabriel Dos Reis wrote: If the user-supplied operator new returns a, then it must also ensure that 'a' is not used anywhere else -- e.g. I you can't do lvalue-to-value conversion on 'a' to see what is written there. Because its storage has been reused. That is,

Re: [C++ patch] Set attributes for C++ runtime library calls

2013-08-26 Thread Gabriel Dos Reis
On Mon, Aug 26, 2013 at 9:38 AM, Jan Hubicka hubi...@ucw.cz wrote: On 08/22/2013 12:45 PM, Gabriel Dos Reis wrote: If the user-supplied operator new returns a, then it must also ensure that 'a' is not used anywhere else -- e.g. I you can't do lvalue-to-value conversion on 'a' to see what is

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Gabriel Dos Reis
On Mon, Aug 26, 2013 at 9:32 AM, Jan Hubicka hubi...@ucw.cz wrote: On Mon, Aug 26, 2013 at 9:00 AM, Jan Hubicka hubi...@ucw.cz wrote: On Mon, Aug 26, 2013 at 7:24 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, it seems to be couple weeks I am not able to compile big ltrans unit with

Re: [C++ patch] Set attributes for C++ runtime library calls

2013-08-26 Thread Gabriel Dos Reis
On Mon, Aug 26, 2013 at 9:07 AM, Jason Merrill ja...@redhat.com wrote: On 08/22/2013 12:45 PM, Gabriel Dos Reis wrote: If the user-supplied operator new returns a, then it must also ensure that 'a' is not used anywhere else -- e.g. I you can't do lvalue-to-value conversion on 'a' to see what

RFA: fix some avr stdint issues

2013-08-26 Thread Joern Rennecke
This patch fixes the gcc.dg/c99-stdint-5.c and gcc.dg/c99-stdint-6.c excess error failures. Tested for atmega128-sim. OK to apply? 2013-05-26 Joern Rennecke joern.renne...@embecosm.com * config/avr/avr-stdint.h (INT16_TYPE): Change default to int. (UINT16_TYPE): Change

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Gabriel Dos Reis
On Mon, Aug 26, 2013 at 9:32 AM, Jan Hubicka hubi...@ucw.cz wrote: modify ALL_LINKERFLAGS or BUILD_LINKERFLAGS? configure --with-build-config=bootstrap-lto . You need plugin enabled buinutils for that. I think there however should be a lot easier way to reproduce it by just dropping some

Re: RFA: fix find_valid_class to accept classes w/out last hard reg

2013-08-26 Thread Ulrich Weigand
Joern Rennecke wrote: 2013-05-02 Joern Rennecke joern.renne...@embecosm.com * reload.c (find_valid_class): Allow classes that do not include FIRST_PSEUDO_REGISTER - 1. This is OK. Thanks, Ulrich -- Dr. Ulrich Weigand GNU/Linux compilers and toolchain

ELF interposition and One Definition Rule

2013-08-26 Thread Jan Hubicka
Hi, cgraph_function_body_availability determine if body of a given function is available in current compilation unit and if so, if it may be overwritten by completely different semantic by (dynamic)linker. Function that is AVAIL_AVAILABLE can still be repleaced by a function fron different unit,

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Jan Hubicka
On Mon, Aug 26, 2013 at 9:32 AM, Jan Hubicka hubi...@ucw.cz wrote: modify ALL_LINKERFLAGS or BUILD_LINKERFLAGS? configure --with-build-config=bootstrap-lto . You need plugin enabled buinutils for that. I think there however should be a lot easier way to reproduce it by just

  1   2   >