Re: [Patch, Fortran] PR 57639: [OOP] ICE with polymorphism (and illegal code)

2013-07-25 Thread Janus Weil
2013/7/24 Steve Kargl : > On Wed, Jul 24, 2013 at 11:53:09PM +0200, Janus Weil wrote: >> Hi all, >> >> here is a straightforward patch for an ICE-on-invalid problem, which >> basically adds some checks for 'class_ok'. >> >> Regtested on x86_64-unknown-linux-gnu. Ok for trunk? >> > > OK. Thanks, St

[Patch ARM] Fix PR19599 tail

2013-07-25 Thread Ramana Radhakrishnan
Hi, This fixes up the issues with PR target/19599 and the issues we've had around it. This fixes up some of the current issues we have around this. I'll follow up with a separate patch around longjmps. This puts in predicates for bx where needed - missed out CCFSM (sigh :( ), replaces uses

[PATCH] Small OpenBSD/hppa cleanup

2013-07-25 Thread Mark Kettenis
Hi, Dave spotted this leftover a while ago. The pa/t-openbsd file doesn't exist anymore. Seems to have no ill effect, but would be good to remove this wart anyway. Dave, can you commit this? Thanks, Mark 2013-07-25 Mark Kettenis * config.gcc (hppa-*-openbsd*): Don't set tmake_fi

[PATCH] Fix nested function support on OpenBSD/hppa

2013-07-25 Thread Mark Kettenis
On OpenBSD/hppa the stack is protected against execution in the same way as on many other architectures. The diff below makes GCC emit the call that's needed to "unprotect" the stack such that the magic trampolines needed for execution of nested functions work. 2013-07-25 Mark Kettenis

Re: [PATCH 0/6] Contributing new target port: Andes 'nds32'.

2013-07-25 Thread Chung-Ju Wu
On 7/8/13 7:13 PM, Chung-Ju Wu wrote: Hi, GCC Steering Committee, reviewers, and developers, On behalf of Andes Technology Corp., we are submitting a new port 'nds32' for GCC contribution. In this contribution, we use the design and strategy as modern as possible, such as having LRA enabled and

[PING] [Patch RX] Add assembler option "-mcu" for generating assembler

2013-07-25 Thread Sandeep Kumar Singh
Hi, Please find the patch to add assembler option "-mcu" for generating assembler error messages when target not supporting hardware FPU were seeing FPU code, namely RX100 and RX200. KPIT has recently submitted a patch to add warnings of RX variants that do not have hardware FPU support, http://w

[C++ Patch] PR 57981

2013-07-25 Thread Paolo Carlini
Hi, this is in fact a Regression in the 4_8-branch too. For the testcase, when we try to produce at instantiation time the warning about the unused parameter we re-enter the diagnostic routines because cp_convert_to_pointer doesn't get, as it should, a tf_none - the propagation breaks between

Re: [C++ Patch] PR 57981

2013-07-25 Thread Paolo Carlini
... sorry, attached the wrong ChangeLog entry. Fixed below. Paolo. // /cp 2013-07-25 Paolo Carlini PR c++/57981 * decl.c (check_default_argument): Take a tsubst_flags_t parameter. (grokparms): Adjust. * parser.c (cp_parser_late_parse_one_defaul

Re: [testsuite] Adding target nonpic to certain tests part. 2

2013-07-25 Thread Alexander Ivchenko
*Ping* Richard, you approved adding "target nonpic" last time, could you please take quick a look? I regularly encounter failing due to "AVAIL_OVERWRITABLE" tests on Android.. When people write tests they don't consider pic targets. thanks, Alexander 2013/7/8 Alexander Ivchenko : > *Ping* > >>>

Re: [PATCH i386 1/8] [AVX512] Adjust register classes.

2013-07-25 Thread Uros Bizjak
On Wed, Jul 24, 2013 at 8:27 PM, Kirill Yukhin wrote: > Hello, > By this patch I am starting series of patches toward Intel (R) AVX-512 and > SHA (see [1]) > extensions enabling in GCC. > I've already submitted corresponding patches to BinUtils (see [2],[3]). > > This patch adds comand-line optio

Re: [Patch ARM] Fix PR19599 tail

2013-07-25 Thread Bernd Edlinger
Hi Ramana, why did you leave the space before the bx? this ends up in the .s file making it look ugly.. + return \" bx%?\\t%0\\t%@ indirect register sibling call\"; Regards Bernd.

Re: [PATCH i386 1/8] [AVX512] Adjust register classes.

2013-07-25 Thread Uros Bizjak
On Wed, Jul 24, 2013 at 8:27 PM, Kirill Yukhin wrote: > Hello, > By this patch I am starting series of patches toward Intel (R) AVX-512 and > SHA (see [1]) > extensions enabling in GCC. > I've already submitted corresponding patches to BinUtils (see [2],[3]). > > This patch adds comand-line optio

Re: [PATCH 2/2] Introduce beginnings of a pipeline class.

2013-07-25 Thread Martin Jambor
Hi, On Wed, Jul 24, 2013 at 08:27:43PM -0400, David Malcolm wrote: > On Wed, 2013-07-24 at 19:10 -0400, Diego Novillo wrote: > > On Wed, Jul 24, 2013 at 6:56 PM, Diego Novillo wrote: > > > Could you please add a description of what this does? > > > > Sorry. You did, but in a previous message th

[ubsan] Move ubsan CL entries to separate CLs

2013-07-25 Thread Marek Polacek
This patch only moves ubsan ChangeLog entries to its own ChangeLogs, to make merging easy. This is only makeshift; when doing the final merge into trunk I'll prepend the ubsan CL entries into regular CLs. Applying to ubsan branch. --- gcc/ChangeLog | 117

Re: [Patch ARM] Fix PR19599 tail

2013-07-25 Thread Ramana Radhakrishnan
On Thu, Jul 25, 2013 at 12:40 PM, Bernd Edlinger wrote: > Hi Ramana, > > why did you leave the space before the bx? > this ends up in the .s file making it look ugly.. Ooops - thanks for noticing - wasn't deliberate - fixed as obvious with this. Ramana Index: gcc/config/arm/arm.md =

Re: [PATCH 2/2] Introduce beginnings of a pipeline class.

2013-07-25 Thread Martin Jambor
Hi, I don't know why it's me again but again I do have a few comments. One global remark first: If we are going to start using the gcc namespace (I understand it you need for isolation of symbols once you use gcc as library, right?), I'm wondering whether mixing "using namespace gcc" and explicit

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

2013-07-25 Thread Ilya Enkovich
2013/7/25 Joseph S. Myers : > On Wed, 24 Jul 2013, Ilya Enkovich wrote: > >> Well, this patch does not introduce any changes on user-visible level. >> It just adds MPX instructions support to i386 target. Usually each new >> x86 instruction has corresponding builtin function and therefore is >> pro

Symtab cleanup 7/17 make releasing of function bodies more consistent

2013-07-25 Thread Jan Hubicka
Hi, this is first patch that makes function body management more consistent to what happens with variables. We now no longer incorrectly call cgraph_release_function_body for decls that are shared during LTO streaming and I fixed ipa.c to make proper reachability analysis of partial symtabs seen

Re: [C++ Patch] PR 57981

2013-07-25 Thread Jason Merrill
OK. Jason

Re: [C++ Patch] PR 57880

2013-07-25 Thread Jason Merrill
OK. Jason

[Patch] Add documents and comments for regex

2013-07-25 Thread Tim Shen
Documents and comments, along with a testcase for not-entirely-correct basic_regex::position() behavior(when regex_iterator::operator++() is called). Thanks! -- Tim Shen comments.patch Description: Binary data

Re: [ping] Re: [PATCH] Remove old x86 builtins from documentation

2013-07-25 Thread Chung-Ju Wu
2013/7/17 Cameron McInally : > Ping. > > I do realize that patches of this nature are of low priority, but I > would really like to see it committed to save my users from tripping > over the docs. > > Thanks again, > Cameron > > On Thu, Jun 13, 2013 at 9:58 PM, Cameron McInally > wrote: >> Hey guy

Re: [ping] Re: [PATCH] Fix documentation for x86 __builtin_ia32_cmp*s

2013-07-25 Thread Chung-Ju Wu
2013/7/17 Cameron McInally : > Ping. > > On Fri, Jun 14, 2013 at 1:41 PM, Cameron McInally > wrote: >> Hey guys, >> >> The documentation states that the return types on the x86 >> __builtin_ia32_cmp*s builtins are integer vectors. This is >> inconsistent with the source. These builtins actually re

Re: [Patch] Add documents and comments for regex

2013-07-25 Thread Jonathan Wakely
On 25 July 2013 15:09, Tim Shen wrote: > Documents and comments, along with a testcase for not-entirely-correct > basic_regex::position() behavior(when regex_iterator::operator++() is > called). This is OK to commit, thanks.

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

2013-07-25 Thread Joseph S. Myers
On Thu, 25 Jul 2013, Ilya Enkovich wrote: > > Usually also new instructions have a -m option to enable them, but you > > don't have that here either. I realise the instructions are NOPs on > > processors not supporting them (all processors not supporting them?), but > > given that the availabilit

Re: [ping] Re: [PATCH] Remove old x86 builtins from documentation

2013-07-25 Thread Uros Bizjak
On Thu, Jul 25, 2013 at 4:25 PM, Chung-Ju Wu wrote: > 2013/7/17 Cameron McInally : >> Ping. >> >> I do realize that patches of this nature are of low priority, but I >> would really like to see it committed to save my users from tripping >> over the docs. Please provide ChangeLog entries for both

patch to fix PR57960

2013-07-25 Thread Vladimir Makarov
The following patch fixes http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57960 The patch was successfully bootstrapped and tested on s390 and x86/x86-64. Committed as rev. 201243. 2013-07-25 Vladimir Makarov PR rtl-optimization/57960 * lra-constraints.c (process_alt_operands):

[PATCH][ARM] Fix bug in add patterns due to commutativity modifier

2013-07-25 Thread Kyrylo Tkachov
Hi all, Some of the new alternatives added to the add patterns in arm.md recently to accommodate 16-bit encodings have the output template "add %0, %2" if operand 1 is the same register as the destination (operand 0). This would be fine, except that the two source operands for the add patterns are

[PATCH][ARM] Update two fixed point arithmetic patterns for -mrestrict-it

2013-07-25 Thread Kyrylo Tkachov
Hi all, The ssmulsa3 and usmulusa3 fixed point arithmetic patterns contain explicit IT blocks in their output template that do not conform to the ARMv8 IT block rules. This patch fixes that. Also cleans up various whitespace issues in the arm-fixed.md file. Tested arm-none-eabi on qemu with arch

Request to merge Undefined Behavior Sanitizer in

2013-07-25 Thread Marek Polacek
Hi! I'd like to announce first version of the Undefined Behavior Sanitizer, a tool I've spent this June/July hacking on. It is an undefined behavior detector for the C family FEs and works by creating a COMPOUND_EXPR around original expression, for slightly more information see my slides about u

Re: [PATCH] Small OpenBSD/hppa cleanup

2013-07-25 Thread John David Anglin
Committed. Dave On 25-Jul-13, at 5:50 AM, Mark Kettenis wrote: Hi, Dave spotted this leftover a while ago. The pa/t-openbsd file doesn't exist anymore. Seems to have no ill effect, but would be good to remove this wart anyway. Dave, can you commit this? Thanks, Mark 2013-07-25 Mark Ke

Re: [PATCH] Fix nested function support on OpenBSD/hppa

2013-07-25 Thread John David Anglin
Committed. On 25-Jul-13, at 6:00 AM, Mark Kettenis wrote: On OpenBSD/hppa the stack is protected against execution in the same way as on many other architectures. The diff below makes GCC emit the call that's needed to "unprotect" the stack such that the magic trampolines needed for execution

[PATCH] docs/install.texi: Add note about not relying on gawk extensions

2013-07-25 Thread David Malcolm
OK for trunk? ("make info" and "make dvi" seem to work, though I don't have a dvi viewer handy). commit fb34f52e9f9f1dcce416cb45dab23ec10625ad16 Author: David Malcolm Date: Thu Jul 25 12:07:26 2013 -0400 gcc/ * doc/install.texi (A POSIX or SVR4 awk): Add note about not relying on

[PATCH][ARM][combine] Improve opportunities to generate cmn instructions

2013-07-25 Thread Kyrylo Tkachov
Hi all, Currently on arm we miss some opportunities to generate the cmn instruction (cmn r1, r2 ~ r1 + r2 == 0) For example, for code: int foo (int x, int y) { if (x + y == 0) return 25; else return 5; } we generate add r0, r0, r1 cmp r0, #0 movn

Re: [PATCH][ARM] Fix bug in add patterns due to commutativity modifier

2013-07-25 Thread Richard Earnshaw
On 25/07/13 16:31, Kyrylo Tkachov wrote: Hi all, Some of the new alternatives added to the add patterns in arm.md recently to accommodate 16-bit encodings have the output template "add %0, %2" if operand 1 is the same register as the destination (operand 0). This would be fine, except that the t

Re: [PATCH][ARM][combine] Improve opportunities to generate cmn instructions

2013-07-25 Thread Richard Earnshaw
On 25/07/13 17:13, Kyrylo Tkachov wrote: Hi all, Currently on arm we miss some opportunities to generate the cmn instruction (cmn r1, r2 ~ r1 + r2 == 0) For example, for code: int foo (int x, int y) { if (x + y == 0) return 25; else return 5; } we generate add r

Re: [PATCH][ARM] Update two fixed point arithmetic patterns for -mrestrict-it

2013-07-25 Thread Richard Earnshaw
On 25/07/13 16:30, Kyrylo Tkachov wrote: Hi all, The ssmulsa3 and usmulusa3 fixed point arithmetic patterns contain explicit IT blocks in their output template that do not conform to the ARMv8 IT block rules. This patch fixes that. Also cleans up various whitespace issues in the arm-fixed.md fil

Re: [ping] Re: [PATCH] Fix documentation for x86 __builtin_ia32_cmp*s

2013-07-25 Thread Cameron McInally
On Thu, Jul 25, 2013 at 10:28 AM, Chung-Ju Wu wrote: > 2013/7/17 Cameron McInally : >> Ping. >> >> On Fri, Jun 14, 2013 at 1:41 PM, Cameron McInally >> wrote: >>> Hey guys, >>> >>> The documentation states that the return types on the x86 >>> __builtin_ia32_cmp*s builtins are integer vectors. Thi

Re: [ping] Re: [PATCH] Remove old x86 builtins from documentation

2013-07-25 Thread Cameron McInally
On Thu, Jul 25, 2013 at 10:57 AM, Uros Bizjak wrote: > On Thu, Jul 25, 2013 at 4:25 PM, Chung-Ju Wu wrote: >> 2013/7/17 Cameron McInally : >>> Ping. >>> >>> I do realize that patches of this nature are of low priority, but I >>> would really like to see it committed to save my users from tripping

[PATCH] Fix missing deps for tree-sra.o

2013-07-25 Thread David Malcolm
Successfully bootstrapped on x86_64-unknown-linux-gnu OK for trunk? commit 4d7b6e5cf8e7f2613f516c9b9fe1f888b1193f8d Author: David Malcolm Date: Tue Jul 23 16:11:14 2013 -0400 Add missing deps to tree-sra.o gcc/ * Makefile.in (tree-sra.o): Add missing deps on TREE_PASS_H and

Re: [PATCH] Fix missing deps for tree-sra.o

2013-07-25 Thread Marek Polacek
On Thu, Jul 25, 2013 at 01:21:27PM -0400, David Malcolm wrote: > Successfully bootstrapped on x86_64-unknown-linux-gnu > > OK for trunk? > commit 4d7b6e5cf8e7f2613f516c9b9fe1f888b1193f8d > Author: David Malcolm > Date: Tue Jul 23 16:11:14 2013 -0400 > > Add missing deps to tree-sra.o >

[Patch, Fortran OOP] PR 57966: Using a TBP to specify the shape of a dummy argument

2013-07-25 Thread Janus Weil
Hi all, here is a small patch for a rejects-valid OOP problem, where a type-bound call was resolved before the base type and its bindings. I'm adding a call to resolve_fl_derived, in order to make sure that the base type is resolved beforehand. Regtested on x86_64-unknown-linux-gnu. Ok for trunk?

Re: [Patch, Fortran OOP] PR 57966: Using a TBP to specify the shape of a dummy argument

2013-07-25 Thread Tobias Burnus
Janus Weil wrote: here is a small patch for a rejects-valid OOP problem, where a type-bound call was resolved before the base type and its bindings. I'm adding a call to resolve_fl_derived, in order to make sure that the base type is resolved beforehand. Regtested on x86_64-unknown-linux-gnu. Ok

Aw: Re: [PATCH] FPU IEEE 754 for MIPS r5900

2013-07-25 Thread Jürgen Urban
Hello Richard, Sorry in the last days, I was not at home. So I couldn't test it until now. > "Jürgen Urban" writes: > > Index: gcc/config.gcc > > === > > --- gcc/config.gcc (Revision 200583) > > +++ gcc/config.gcc (Arbeitskopie) >

Re: [PATCH][ARM][combine] Improve opportunities to generate cmn instructions

2013-07-25 Thread Jeff Law
On 07/25/2013 10:13 AM, Kyrylo Tkachov wrote: Hi all, Currently on arm we miss some opportunities to generate the cmn instruction (cmn r1, r2 ~ r1 + r2 == 0) For example, for code: int foo (int x, int y) { if (x + y == 0) return 25; else return 5; } we generate add

Re: Aw: Re: [PATCH] FPU IEEE 754 for MIPS r5900

2013-07-25 Thread Richard Sandiford
"Jürgen Urban" writes: >> "Jürgen Urban" writes: >> > Index: gcc/config.gcc >> > === >> > --- gcc/config.gcc (Revision 200583) >> > +++ gcc/config.gcc (Arbeitskopie) >> > @@ -3080,7 +3080,7 @@ >> >esac >> > fi >> > >> > -# Infer

[Dwarf patch] Refactor logic to decide if a pubname should be output.

2013-07-25 Thread Sterling Augustine
The enclosed patch refactors the logic that decides if a pubname belongs in the output, taking into account type pruning in the presence of debug types. When the logic was separate, it didn't always agree, creating issues with the size output to the object file. Older editions of Gold didn't use t

Re: [Patch, Fortran OOP] PR 57966: Using a TBP to specify the shape of a dummy argument

2013-07-25 Thread Janus Weil
2013/7/25 Tobias Burnus : > Janus Weil wrote: >> >> here is a small patch for a rejects-valid OOP problem, where a >> type-bound call was resolved before the base type and its bindings. >> I'm adding a call to resolve_fl_derived, in order to make sure that >> the base type is resolved beforehand. >

Re: [Patch, Fortran OOP] PR 57966: Using a TBP to specify the shape of a dummy argument

2013-07-25 Thread Tobias Burnus
Janus Weil wrote: 2013/7/25 Tobias Burnus : Looks fine to me. (If I tracked the PR correctly, it only solves the original problem but not yet the one of comment 5, does it?) No, this one should actually fix both. I can also add the variant from comment 5 to the test case, if you prefer. I thi

Re: [Dwarf patch] Refactor logic to decide if a pubname should be output.

2013-07-25 Thread Cary Coutant
> gcc/ChangeLog > 013-07-25 Sterling Augustine > > * dwarf2out.c (size_of_pubnames): Move code to... > (include_pubname_in_output): ...here. New. > (want_pubnames): Rearrange. > (output_pubnames): Call include_pubname_in_output. Move assertion. This is OK. Thanks! -cary

Re: [Patch, Fortran OOP] PR 57966: Using a TBP to specify the shape of a dummy argument

2013-07-25 Thread Janus Weil
2013/7/25 Tobias Burnus : > Janus Weil wrote: >> >> 2013/7/25 Tobias Burnus : >> >>> Looks fine to me. (If I tracked the PR correctly, it only solves the >>> original problem but not yet the one of comment 5, does it?) >> >> No, this one should actually fix both. I can also add the variant from >>

Re: [c++-concepts] requires expressions

2013-07-25 Thread Jason Merrill
On 07/25/2013 10:19 AM, Andrew Sutton wrote: + pp_cxx_parameter_declaration_clause(pp, TREE_OPERAND (t, 0)); Space before (. Otherwise, looks good. Jason

[v3] (Re-)enable pow(const complex<>&, int) in C++11 mode

2013-07-25 Thread Paolo Carlini
Hi, as discussed once more in the audit trail of PR57974 and agreed with Gaby I'm going to re-enable the overload. Tested x86_64-linux. Thanks, Paolo. // 2013-07-25 Paolo Carlini * include/std/complex (pow(const complex<>&, int)): Enable in C++11 mode t

Re: Request to merge Undefined Behavior Sanitizer in

2013-07-25 Thread Jason Merrill
On 07/25/2013 11:32 AM, Marek Polacek wrote: + vec_alloc (v, 3); + tree ctor = build_constructor (dtype, v); You might use build_constructor_va instead of managing a vector directly. Otherwise, looks fine. If nobody else has comments, go ahead and check it in next week. Jason

Re: Request to merge Undefined Behavior Sanitizer in

2013-07-25 Thread Marek Polacek
On Thu, Jul 25, 2013 at 04:13:29PM -0400, Jason Merrill wrote: > On 07/25/2013 11:32 AM, Marek Polacek wrote: > >+ vec_alloc (v, 3); > >+ tree ctor = build_constructor (dtype, v); > > You might use build_constructor_va instead of managing a vector directly. Thanks, will give it a try. > Otherw

[PATCH 03/18] move generated_files order-only dependency later

2013-07-25 Thread Tom Tromey
There is an order-only dependency in gcc/Makefile.in that tries to build the generated files before compiling regular objects. However, this appears too early, and so at the time it is seen by make, GCOV_OBJS and GCOV_DUMP_OBJS have not yet been set. This patch fixes the problem and prevents any

[PATCH 01/18] clean up SHLIB so subshells are not needed

2013-07-25 Thread Tom Tromey
This changes the handling of SHLIB so that it is inlined into DRIVER_DEFINES. This is ok because SHLIB is defined in a Makefile fragment that is included by the generated Makefile. The rationale for this is that it simplifies some .o targets, so that we can share more code. * Makefile.in

[PATCH 02/18] update generated_files

2013-07-25 Thread Tom Tromey
A few generated files were not mentioned in the generated_files variable. This fixes the problem. * Makefile.in (generated_files): Add options.h, target-hooks-def.h, insn-opinit.h, common/common-target-hooks-def.h. --- gcc/Makefile.in | 4 +++- 1 file changed, 3 insertion

[PATCH 05/18] convert the C front end to automatic dependencies

2013-07-25 Thread Tom Tromey
This converts the C front end. Note that this fixes a latent bug in gcc/Makefile.in's definition of C_TREE_H. This is needed to avoid breaking this build with this patch. * Makefile.in (C_TREE_H): Reference c/c-tree.h. * Make-lang.in (c/gccspec.o): Remove. (CFLAGS-c/gccs

[PATCH 06/18] convert the C++ front end to automatic dependencies

2013-07-25 Thread Tom Tromey
This converts the C++ front end. This renames g++spec.o to cp/g++spec.o for uniformity. This lets us remove an explicit rule. This patch does not remove various *_H macros from cp/Make-lang.in. These are still needed by ObjC++. They're removed by a later patch. * Make-lang.in (g++spec.o

[PATCH 08/18] convert the ObjC++ front end to automatic dependencies

2013-07-25 Thread Tom Tromey
This converts the ObjC++ front end. Now we can finally remove the *_H macros from cp/Make-lang.in. * Make-lang.in (CXX_TREE_H, CXX_PARSER_H, CXX_PRETTY_PRINT_H): Remove. * Make-lang.in (START_HDRS, cc1objplus-checksum.o) (objcp/objcp-lang.o, objcp/objcp-decl.o

[PATCH 07/18] convert the ObjC front end to automatic dependencies

2013-07-25 Thread Tom Tromey
This converts the ObjC front end. Note that there is a latent possible bug in this code -- both ObjC and ObjC++ define START_HDRS. Whichever is included last, wins; if they are out of sync, then something could break. This possibility is eliminated by this series. * Make-lang.in (START_

[PATCH 04/18] add configury

2013-07-25 Thread Tom Tromey
This adds the configury needed for automatic dependency tracking. It also adds some bits to the Makefile so we can begin converting (removing) explicit dependencies. * Makefile.in (CCDEPMODE, DEPDIR, depcomp, COMPILE.base) (COMPILE): New variables. (.cc.o .c.o): Use COMPIL

[PATCH 09/18] convert the Java front end to automatic dependencies

2013-07-25 Thread Tom Tromey
This converts the Java front end. We also rename jvspec.o to java/jvspec.o, for uniformity; this lets us remove an explicit rule. * Make-lang.in (jvspec.o): Remove. (CFLAGS-java/jvspec.o): New variable. ($(XGCJ)$(exeext), java_OBJS): Use java/jvspec.o (java/jvspec.

[PATCH 10/18] Fix up c-family targets

2013-07-25 Thread Tom Tromey
This removes manual dependencies for the c-family .o files. * Makefile.in (c-family/cppspec.o, c-family/c-common.o) (c-family/c-cppbuiltin.o, c-family/c-dump.o, c-family/c-format.o) (c-family/c-gimplify.o, c-family/c-lex.o, c-family/c-omp.o) (c-family/c-opts.o, c-fa

[PATCH 11/18] convert the Fortran front end to automatic dependencies

2013-07-25 Thread Tom Tromey
This convert fortran. It renames gfortranspec.o to fortran/gfortranspec.o, for uniformity and to allow removing an explicit rule. * Make-lang.in (fortran_OBJS): Use fortran/gfortranspec.o. (gfortranspec.o): Remove. (CFLAGS-fortran/gfortranspec.o): New variable. (GF

[PATCH 16/18] remove last reference to TREE_GIMPLE_H

2013-07-25 Thread Tom Tromey
There is a single reference to TREE_GIMPLE_H in the source tree. Since it is not defined anywhere, we might as well remove the use. * config/i386/t-i386 (i386.o): Don't use TREE_GIMPLE_H. --- gcc/config/i386/t-i386 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/co

[PATCH 12/18] convert the Go front end to automatic dependencies

2013-07-25 Thread Tom Tromey
This converts Go. It renames gospec.o to go/gospec.o, for uniformity and so we can remove an explicit rule. It defines go_OBJS, to conform to the documented Make-lang.in conventions, and to ensure that Go objects are given the correct order-only dependencies on generated files. * Make-la

[PATCH 15/18] make out_object_file rule use automatic dependencies

2013-07-25 Thread Tom Tromey
This is a small change to make out_object_file use automatic dependencies. * Makefile.in ($(out_object_file)): Use COMPILE. --- gcc/Makefile.in | 11 ++- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/gcc/Makefile.in b/gcc/Makefile.in index 2da93021..b689442 100644

[PATCH 13/18] convert LTO to automatic dependencies

2013-07-25 Thread Tom Tromey
This converts LTO. This also fixes a latent buglet in lto/Make-lang.in. lto_OBJS should hold all the objects for a language, but LTO never defined this. * Make-lang.in (LTO_H, LINKER_PLUGIN_API_H, LTO_TREE_H) (lto/lto-lang.o, lto/lto.o, lto/lto-partition.o) (lto/lto-objec

[PATCH 18/18] remove unused macros

2013-07-25 Thread Tom Tromey
I used this perl script to find unused _H macros in the Makefile. I deleted the definitions it reported and re-ran the script, until there was no more output. The script also makes note of _H variables which are used but never defined. That is how I found the TREE_GIMPLE_H use, fixed earlier in

[PATCH 17/18] remove last definition of CROSS_FLOAT_H

2013-07-25 Thread Tom Tromey
There is a single definition of CROSS_FLOAT_H in the source. As far as I can tell, this is never used anywhere. So, this patch removes it. * config/mcore/t-mcore (CROSS_FLOAT_H): Remove. --- gcc/config/mcore/t-mcore | 3 --- 1 file changed, 3 deletions(-) diff --git a/gcc/config/mcore/t-

[PATCH 00/18] resurrect automatic dependency tracking

2013-07-25 Thread Tom Tromey
Somehow "patch #0" of my series didn't go out. So, I'm sending it separately. Sorry about that. I don't know what happened, so I wouldn't be totally surprised if it made its way to the list some day :) This patch series resurrects my automatic dependency tracking patch from eons ago. That pat

Re: [ping] Re: [PATCH] Remove old x86 builtins from documentation

2013-07-25 Thread Uros Bizjak
On Thu, Jul 25, 2013 at 7:07 PM, Cameron McInally wrote: I do realize that patches of this nature are of low priority, but I would really like to see it committed to save my users from tripping over the docs. >> >> Please provide ChangeLog entries for both patches, and I'll commit

Re: [Patch, Fortran OOP] PR 57966: Using a TBP to specify the shape of a dummy argument

2013-07-25 Thread Janus Weil
Committed as r201254. Cheers, Janus 2013/7/25 Janus Weil : > 2013/7/25 Tobias Burnus : >> Janus Weil wrote: >>> >>> 2013/7/25 Tobias Burnus : >>> Looks fine to me. (If I tracked the PR correctly, it only solves the original problem but not yet the one of comment 5, does it?) >>> >>> N

Re: [google][4.8] Expose all target specific builtins unconditionally for LIPO builds

2013-07-25 Thread Sriraman Tallam
On Wed, Jul 24, 2013 at 3:21 PM, Xinliang David Li wrote: > Can you collect some number on ggc_memory increase with this change > in profile_gen build -- the value is recorded gcov_module_info object > in coverage.c. The ggc_memory increase if < 0.2% for large modules (initial usage around 100MB

Re: [google][4.8] Expose all target specific builtins unconditionally for LIPO builds

2013-07-25 Thread Xinliang David Li
the patch is ok. thanks, David On Thu, Jul 25, 2013 at 2:53 PM, Sriraman Tallam wrote: > On Wed, Jul 24, 2013 at 3:21 PM, Xinliang David Li wrote: >> Can you collect some number on ggc_memory increase with this change >> in profile_gen build -- the value is recorded gcov_module_info object >>

[Patch, Fortran] PR57530 (Part 2 of 3) Support TYPE => CLASS

2013-07-25 Thread Tobias Burnus
This patch is a follow up to the resolve part, which permits TYPE=>CLASS. That approved patch is at http://gcc.gnu.org/ml/fortran/2013-06/msg00049.html (I didn't want to commit it without this trans*.c patch.) The attached patch adds support for: TYPE => CLASS additionally, it fixes some i

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

2013-07-25 Thread Bernd Edlinger
On Tue, 23 Jul 2013, Hans-Peter Nilsson wrote: > Please put the "as it would" parts of the changelog entries as > comments in the code instead. (ChangeLog says "what", not "why".) > > I'd also tweak the head comment of warn_portable_volatility_p > (like the documentation change) to not refer to > -

Re: Request to merge Undefined Behavior Sanitizer in

2013-07-25 Thread Joseph S. Myers
On Thu, 25 Jul 2013, Marek Polacek wrote: > So far it sanitizes division-by-zeros, shifts and > __builtin_unreachable calls. This is of course far from being > complete; I intend to write more features during this 4.9 stage. Such as everything needed for it to replace -ftrapv (for -ftrapv to bec

Re: Request to merge Undefined Behavior Sanitizer in

2013-07-25 Thread Joseph S. Myers
On Thu, 25 Jul 2013, Marek Polacek wrote: > +@item -fsanitize=undefined > +Enable UndefinedBehaviorSanitizer, a fast undefined behavior detector > +Various computations will be instrumented to detect > +undefined behavior, e.g.@: division by zero or various overflows. The same issues applies as f

Re: [PATCH] docs/install.texi: Add note about not relying on gawk extensions

2013-07-25 Thread Joseph S. Myers
On Thu, 25 Jul 2013, David Malcolm wrote: > OK for trunk? ("make info" and "make dvi" seem to work, though I don't > have a dvi viewer handy). This seems the wrong place. A statement about "awk scripts should not" is information for people modifying GCC, but install.texi is documentation for

Re: [PATCH 00/18] resurrect automatic dependency tracking

2013-07-25 Thread Joseph S. Myers
On Thu, 25 Jul 2013, Tom Tromey wrote: > That patch was ultimately reverted due to a GNU make bug. This time > around, thanks to git, I chose to make a patch series. This way, even > if we stumble across the bug again (it is not clear it was ever > fixed), we can determine which patch triggers i

Re: msp430 port

2013-07-25 Thread DJ Delorie
I tried to reproduce the original bugs that led to these patterns, but was unable. Testsuite results are the same with and without, and eembc code size doesn't change. So, I'm removing these patterns from the port. The remaining (subreg...) patterns are just optimizations. ; This pattern is i

Re: [PATCH 06/18] convert the C++ front end to automatic dependencies

2013-07-25 Thread Gabriel Dos Reis
On Thu, Jul 25, 2013 at 3:31 PM, Tom Tromey wrote: > This converts the C++ front end. > > This renames g++spec.o to cp/g++spec.o for uniformity. > This lets us remove an explicit rule. > > This patch does not remove various *_H macros from cp/Make-lang.in. > These are still needed by ObjC++. They

Re: [PATCH 00/18] resurrect automatic dependency tracking

2013-07-25 Thread Tom Tromey
Joseph> Thus, it would seem appropriate, for each of the 18 successive Joseph> states after an initial subsequence of the patches is applied, Joseph> to verify that, if you build with make 3.81, touch Makefile.in Joseph> and then do make -j2 cc1, it does not produce the hang Joseph> previously obse

Re: Request to merge Undefined Behavior Sanitizer in

2013-07-25 Thread Jeff Law
On 07/25/2013 04:40 PM, Joseph S. Myers wrote: On Thu, 25 Jul 2013, Marek Polacek wrote: So far it sanitizes division-by-zeros, shifts and __builtin_unreachable calls. This is of course far from being complete; I intend to write more features during this 4.9 stage. Such as everything needed

Re: Request to merge Undefined Behavior Sanitizer in

2013-07-25 Thread Andrew Pinski
On Thu, Jul 25, 2013 at 9:50 PM, Jeff Law wrote: > On 07/25/2013 04:40 PM, Joseph S. Myers wrote: >> >> On Thu, 25 Jul 2013, Marek Polacek wrote: >> >>> So far it sanitizes division-by-zeros, shifts and >>> __builtin_unreachable calls. This is of course far from being >>> complete; I intend to wr

Re: Request to merge Undefined Behavior Sanitizer in

2013-07-25 Thread Marc Glisse
On Thu, 25 Jul 2013, Joseph S. Myers wrote: On Thu, 25 Jul 2013, Marek Polacek wrote: So far it sanitizes division-by-zeros, shifts and __builtin_unreachable calls. This is of course far from being complete; I intend to write more features during this 4.9 stage. Such as everything needed fo

Re: [PATCH] Fix illegal cast to rtx (*insn_gen_fn) (rtx, ...)

2013-07-25 Thread Uros Bizjak
Hello! > The (static arg) generator functions are casted to a var arg > function pointer, making the assumption that the ABI for passing > the arguments will be the same as for static arguments. > This isn't a valid assumption on all architectures, var args might for > example be passed on the sta