Re: [gomp4] OpenACC routine directive

Cesar Philippidis wrote: > This patch adds initial support for the OpenACC routine directive. It's > not complete just yet because it doesn't implement any of the optional > clauses, except for the optional function/subroutine name. As such, it > doesn't go beyond marking functions with the "omp de

Re: [PATCH i386 AVX512] [41/n] Extend extract insn patterns.

On Tue, Sep 16, 2014 at 7:26 AM, Kirill Yukhin wrote: > Hello, > This patch extends extract insn patterns. > It also fixes ICE on testsuite when F16C switched off. > Also it fixes condition in old xtract pattern. > > Bootstrapped. > AVX-512* tests on top of patch-set all pass > under simulator. >

Re: [PR libfortran/62768] Handle filenames with embedded nulls

>> 2014-09-05 Janne Blomqvist >> >>PR libfortran/62768 >>* io/io.h (gfc_unit): Store C string for the filename. >>* io/close.c (st_close): Use gfc_unit.filename. >>* io/inquire.c (inquire_via_unit): Likewise. >>* io/open.c (new_unit): Likewise. >>(already_open): Likewise

RE: [PATCH] Avoid inter-test dependencies in gfortran.dg (PR fortran/56408)

Hi Jakub, thanks! > +dg-test $gfortran_test_path/[lindex $args 1] "" > $gfortran_aux_module_flags > +# cleanup-modules isn't intentionally invoked here. should this be 'is intentionally not invoked here' ? I'm currently seeing a lot of errors in the log of make -j32 -k check. Similar

Re: [PATCH] Avoid inter-test dependencies in gfortran.dg (PR fortran/56408)

On Tue, Sep 16, 2014 at 08:28:48AM +, VandeVondele Joost wrote: > > +dg-test $gfortran_test_path/[lindex $args 1] "" > > $gfortran_aux_module_flags > > +# cleanup-modules isn't intentionally invoked here. > > should this be 'is intentionally not invoked here' ? Isn't that the same?

Re: [PATCH] Avoid inter-test dependencies in gfortran.dg (PR fortran/56408)

On Tue, Sep 16, 2014 at 10:42:46AM +0200, Jakub Jelinek wrote: > > I'm currently seeing a lot of errors in the log of make -j32 -k check. > > Similar to the ones below. > > Weird. Works just fine for me, on two different boxes, > one with dejagnu 1.5 and one with dejagnu 1.5.1. There is > no d

[ping] define CROSS = @CROSS@ in gcc/Makefile.in

Hello, ping on https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00056.html Thanks in advance, With Kind Regards, Olivier On Sep 1, 2014, at 17:26 , Olivier Hainque wrote: > Hello, > > This patch is necessary for proper operation of a piece > of the Ada Makefile fragment which tests the value o

[ping*3] fix build failure of x86_64-mingw32, missing crtbegin/crtend.o

Hello, Ping #3 for https://gcc.gnu.org/ml/gcc-patches/2014-07/msg00237.html Thanks in advance for your feedback, With Kind Regards, Olivier On Jul 3, 2014, at 16:57 , Olivier Hainque wrote: > From gcc/i386/config/mingw32.h, STARTFILE_SPEC and ENDFILE_SPEC include > crtbegin.o and crtend.o unc

Re: [PATCH] Avoid inter-test dependencies in gfortran.dg (PR fortran/56408)

Hi, On 09/16/2014 10:42 AM, Jakub Jelinek wrote: On Tue, Sep 16, 2014 at 08:28:48AM +, VandeVondele Joost wrote: +dg-test $gfortran_test_path/[lindex $args 1] "" $gfortran_aux_module_flags +# cleanup-modules isn't intentionally invoked here. should this be 'is intentionally not in

RE: [PATCH] Avoid inter-test dependencies in gfortran.dg (PR fortran/56408)

>> What dejagnu version are you using? > runtest --version WARNING: Couldn't find the global config file. Expect version is 5.44.1.15 Tcl version is 8.5 Framework version is1.4.4

Re: [PATCH, i386, Pointer Bounds Checker 30/x] Size relocation

Hello! > This patch adds size relocation support for i386 target. Relocation is used > to compute bounds for static objects with incomplete type. > > Thanks, > Ilya > -- > gcc/ > > 2014-06-11 Ilya Enkovich > > * config/i386/i386.md (UNSPEC_SIZEOF): New. > (move_size_reloc_si):

Re: [PATCH, Pointer Bounds Checker 23/x] Function split

2014-09-16 1:08 GMT+04:00 Jeff Law : > On 09/15/14 10:20, Ilya Enkovich wrote: A problem I'm trying to avoid is that bounds in return statement are not taken into account when checking for data dependencies between parts. It means we may have a case when return statement w

[PATCH][match-and-simplify] Fix comparison operator type handling

Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2014-09-16 Richard Biener * genmatch.c (operator_id): Add tcc member. (get_operand_type): New function, split out from ... (expr::gen_transform): ... here. Treat comparisons properly.

Re: [PATCH] Avoid inter-test dependencies in gfortran.dg (PR fortran/56408)

"VandeVondele Joost" writes: >>> What dejagnu version are you using? > >> runtest --version > WARNING: Couldn't find the global config file. > Expect version is 5.44.1.15 > Tcl version is8.5 > Framework version is 1.4.4 You need at least dejagnu 1.5, which includes this fix

Re: [PATCH] Avoid inter-test dependencies in gfortran.dg (PR fortran/56408)

On Tue, Sep 16, 2014 at 10:47:17AM +0200, Jakub Jelinek wrote: > On Tue, Sep 16, 2014 at 10:42:46AM +0200, Jakub Jelinek wrote: > > > I'm currently seeing a lot of errors in the log of make -j32 -k check. > > > Similar to the ones below. > > > > Weird. Works just fine for me, on two different b

RE: [PATCH] Avoid inter-test dependencies in gfortran.dg (PR fortran/56408)

> > Framework version is 1.4.4 > You need at least dejagnu 1.5, which includes this fix: I see, but that's contrary to : https://gcc.gnu.org/install/prerequisites.html

Re: [PATCH] gcc parallel make check

On Mon, Sep 15, 2014 at 7:44 PM, Mike Stump wrote: > On Sep 15, 2014, at 9:05 AM, Jakub Jelinek wrote: > > All the updates sound good. > >> Regtested on x86_64-linux, without the patch toplevel make -k check >> took 8hrs3minutes (don't have time data for that run), > > This confuses me, but, no m

Re: [PATCH] Avoid inter-test dependencies in gfortran.dg (PR fortran/56408)

On Tue, Sep 16, 2014 at 11:14:33AM +0200, Andreas Schwab wrote: > "VandeVondele Joost" writes: > > >>> What dejagnu version are you using? > > > >> runtest --version > > WARNING: Couldn't find the global config file. > > Expect version is 5.44.1.15 > > Tcl version is 8.5 > > Frame

Re: [match-and-simplify] remove dt_node::level_max

On Mon, Sep 15, 2014 at 8:51 PM, Prathamesh Kulkarni wrote: > * genmatch.c (dt_node::level_max): Remove. Applied. Thanks, Richard. > Thanks, > Prathamesh

Re: [PATCH] gcc parallel make check

On Tue, Sep 16, 2014 at 11:20:37AM +0200, Richard Biener wrote: > > This confuses me, but, no matter. Isn’t 8hrs time data? :-) It is, but not time(1) data, just wall clock computed from subtracting mtimes of my make check output log and make -j48 bootstrap log. > >> patch toplevel make -j48 -k

RE: [PATCH] Avoid inter-test dependencies in gfortran.dg (PR fortran/56408)

> Does the following patch fix this? Works for me with dejagnu 1.5.1. and works for me with 1.4.4 Joost

Re: [PATCH, i386, Pointer Bounds Checker 30/x] Size relocation

2014-09-16 13:08 GMT+04:00 Uros Bizjak : > Hello! > >> This patch adds size relocation support for i386 target. Relocation is used >> to compute bounds for static objects with incomplete type. >> >> Thanks, >> Ilya >> -- >> gcc/ >> >> 2014-06-11 Ilya Enkovich >> >> * config/i386/i386.m

Re: [PATCH] Avoid inter-test dependencies in gfortran.dg (PR fortran/56408)

On Tue, Sep 16, 2014 at 09:31:04AM +, VandeVondele Joost wrote: > > > Does the following patch fix this? Works for me with dejagnu 1.5.1. > > and works for me with 1.4.4 Ok, I've committed this to trunk then, so that testing works again for those with old dejagnu. 2014-09-16 Jakub Jeline

Re: [match-and-simplify] CSE with expression captures

On Tue, Sep 16, 2014 at 8:25 AM, Prathamesh Kulkarni wrote: > On Tue, Sep 16, 2014 at 2:15 AM, Marc Glisse wrote: >> On Tue, 16 Sep 2014, Prathamesh Kulkarni wrote: >> >>> --- gcc/match-builtin.pd(revision 215271) >>> +++ gcc/match-builtin.pd(working copy) >>> @@ -44,8 +44,8 @@ >>

Re: [PATCH] gcc parallel make check

On Tue, Sep 16, 2014 at 11:28 AM, Jakub Jelinek wrote: > On Tue, Sep 16, 2014 at 11:20:37AM +0200, Richard Biener wrote: >> > This confuses me, but, no matter. Isn’t 8hrs time data? :-) > > It is, but not time(1) data, just wall clock computed from subtracting > mtimes of my make check output lo

Re: [PATCH, i386, Pointer Bounds Checker 31/x] Pointer Bounds Checker builtins for i386 target

> 2014-06-11 Ilya Enkovich > > * config/i386/i386-builtin-types.def (BND): New. > (ULONG): New. > (BND_FTYPE_PCVOID_ULONG): New. > (VOID_FTYPE_BND_PCVOID): New. > (VOID_FTYPE_PCVOID_PCVOID_BND): New. > (BND_FTYPE_PCVOID_PCVOID): New. > (BND

Re: [PATCH i386 AVX512] [41/n] Extend extract insn patterns.

Hello Uroš, On 16 Sep 09:47, Uros Bizjak wrote: > > + "TARGET_AVX512DQ && (INTVAL (operands[2]) = INTVAL (operands[3]) - 1)" > > Ouch, you have assignment instead of comparison here! Thanks, fixed! > > + (set (attr "memory") > > + (if_then_else (match_test "MEM_P (operands[0])") > > +

Re: [PATCH, i386, Pointer Bounds Checker 30/x] Size relocation

On Tue, Sep 16, 2014 at 11:37 AM, Ilya Enkovich wrote: > 2014-09-16 13:08 GMT+04:00 Uros Bizjak : >> Hello! >> >>> This patch adds size relocation support for i386 target. Relocation is >>> used to compute bounds for static objects with incomplete type. >>> >>> Thanks, >>> Ilya >>> -- >>> gcc/ >

[PATCH] Fix PR63266: Keep track of impact of sign extension in bswap

Hi all, The fix for PR61306 disabled bswap when a sign extension is detected. However this led to a test case regression (and potential performance regression) in case where a sign extension happens but its effect is canceled by other bit manipulation. This patch aims to fix that by having a sp

[PATCH][match-and-simplify] Make predicate IDs explicit

The following patch makes the parser know known predicates via us defining them in match.pd with a new (define_predicates ...) construct. This will allow (in a followup patch) to define our own predicates via sth like (match negate_expr_p (negate @0)) (match negate_expr_p INTEGER_CST@0 (if

Re: [PATCH i386 AVX512] [41/n] Extend extract insn patterns.

On Tue, Sep 16, 2014 at 12:10 PM, Kirill Yukhin wrote: > Updated ChangeLog entry: > gcc/ > * config/i386/i386.c > (ix86_expand_vector_extract): Handle V32HI and V64QI modes. > * config/i386/sse.md > (define_mode_iterator VI48F_256): New. > (define_mode_attr

Re: ptx preliminary address space fixes [1/4]

On 09/12/2014 01:48 PM, Richard Biener wrote: On Fri, Sep 12, 2014 at 1:15 PM, Bernd Schmidt wrote: On 09/11/2014 01:29 PM, Richard Biener wrote: + if (TREE_CODE (type) == ARRAY_TYPE) +TREE_TYPE (type) = apply_as_to_type (TREE_TYPE (type), as); why is this necessary for ARRAY_TYPE but n

Extract and insert merging patch

Hi, This patch allows merging of extract and insert. Please have a look. 2014-09-16 Petr Murzin * simplify-rtx.c (simplify_ternary_operation): Allow extract and insert merging. extract_insert_patch Description: Binary data

Re: Extract and insert merging patch

On Tue, 16 Sep 2014, Petr Murzin wrote: Hi, This patch allows merging of extract and insert. Please have a look. 2014-09-16 Petr Murzin * simplify-rtx.c (simplify_ternary_operation): Allow extract and insert merging. Hello, no testcase? You could probably generalize 0 and 1 to i and 1

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

On 09/15/14 15:53, Chen Gang wrote: On 09/15/2014 11:30 PM, Michael Eager wrote: Configuring DejaGNU for cross-target testing requires creating a configuration file describing the target board. OK, thank you very much. And could you share your configuration file, which I can reference to?

Re: ptx preliminary address space fixes [1/4]

On Tue, Sep 16, 2014 at 1:24 PM, Bernd Schmidt wrote: > On 09/12/2014 01:48 PM, Richard Biener wrote: >> >> On Fri, Sep 12, 2014 at 1:15 PM, Bernd Schmidt >> wrote: >>> >>> On 09/11/2014 01:29 PM, Richard Biener wrote: + if (TREE_CODE (type) == ARRAY_TYPE) +TREE_TYPE (typ

Re: Fix pr61848, linux kernel miscompile

On Tue, 16 Sep 2014, Alan Modra wrote: > gcc testsuite additions? I decline. It is too soon. If you had read > my patch submission you'll see that at some stage gcc was supposed to > warn on conflicting section attributes, but hasn't done so for a very > long time. There needs to be some agree

Re: [PATCH 1/2, x86] Add palignr support for AVX2.

PING 2 On Mon, Sep 8, 2014 at 2:03 PM, Evgeny Stupachenko wrote: > PING > > On Wed, Aug 27, 2014 at 7:50 PM, Evgeny Stupachenko > wrote: >> The rotate insn appeared right after expand. >> I've done it similar to define_insn_and_split "*avx_vperm_broadcast_". >> I don't see any potential losses

[PATCH][match-and-simplify] User defined predicates

The following adds the ability to write predicates using patterns with an example following negate_expr_p which already has a use in comparison folding (via its if c-expr). The syntax is as follows: (match negate_expr_p INTEGER_CST (if (TYPE_OVERFLOW_WRAPS (type) || may_negate_without_ov

[PATCH] Fix PR63258

We no longer even consider applying peeling for alignment on gcc.dg/vect/costmodel/ppc/costmodel-slp-33.c, so the cost model no longer triggers. Committed. Richard. 2014-09-16 Richard Biener PR testsuite/63258 * gcc.dg/vect/costmodel/ppc/costmodel-slp-33.c: Remove scan

[RFC PATCH] -fsanitize=vptr instrumentation

Hi! This patch attempts to instrument C++ when a pointer might not point to object of the right dynamic type. clang++ apparently instruments: 1) member accesses 2) method calls 3) downcasts (both pointer and reference downcasts) The vptr-1.C (and from there derived -2 and -3 testcases) attempts

Re: [GOOGLE] Fix LIPO COMDAT fixup and gcov-tool interactions

On Mon, Sep 15, 2014 at 9:29 PM, Xinliang David Li wrote: > Is it necessary to declare zero_counts array at all? Can a flag field > be added to dyn_cgraph_node structure to indicate if it is fixed up? The zero_counts array is used to pass info back to the caller in libgcov-driver.cc (dyn_cgraph_

[PINGv2][PATCH] Asan optimization for aligned accesses.

On 09/10/2014 04:30 PM, Marat Zakirov wrote: On 09/02/2014 07:09 PM, Marat Zakirov wrote: Hi all! Here's a simple optimization patch for Asan. It stores alignment information into ASAN_CHECK which is then extracted by sanopt to reduce number of "and 0x7" instructions for sufficiently aligned

Re: [PATCH 1/2, x86] Add palignr support for AVX2.

On Tue, Sep 16, 2014 at 6:15 AM, Evgeny Stupachenko wrote: > PING 2 > > On Mon, Sep 8, 2014 at 2:03 PM, Evgeny Stupachenko wrote: >> PING >> >> On Wed, Aug 27, 2014 at 7:50 PM, Evgeny Stupachenko >> wrote: >>> The rotate insn appeared right after expand. >>> I've done it similar to define_insn_

Re: [GOOGLE] Fix LIPO COMDAT fixup and gcov-tool interactions

The zero_counts array is passed to gcov_build_callgraph but not used until the dyn-cgraph is initialized. We should avoid increasing runtime memory overhead by not creating it if possible. David On Tue, Sep 16, 2014 at 7:57 AM, Teresa Johnson wrote: > On Mon, Sep 15, 2014 at 9:29 PM, Xinliang D

Re: [PINGv2][PATCH] Asan optimization for aligned accesses.

On Tue, Sep 16, 2014 at 06:59:57PM +0400, Marat Zakirov wrote: > --- a/gcc/asan.c > +++ b/gcc/asan.c > @@ -1639,9 +1639,11 @@ build_check_stmt (location_t loc, tree base, tree len, >if (end_instrumented) > flags |= ASAN_CHECK_END_INSTRUMENTED; > > - g = gimple_build_call_internal (IFN_A

Re: [PATCH 2/2] Add some more test cases for fentry and pg

On Mon, Sep 15, 2014 at 8:59 AM, Andi Kleen wrote: > On Mon, Sep 15, 2014 at 11:29:50AM +0200, Richard Biener wrote: >> On Fri, Sep 12, 2014 at 4:32 PM, Andi Kleen wrote: >> > From: Andi Kleen >> > >> > Test fentry and no_instrument_function overriding. >> > >> > No test cases for the LTO test f

Re: [debug-early] reuse variable DIEs and fix their context

On 09/12/14 10:56, Jason Merrill wrote: On 09/12/2014 01:48 PM, Aldy Hernandez wrote: Unless I'm misunderstanding something, validate_phases() verifies that the numbers add up by looking at the actual string name of the phase, irregardless of if you timevar_push/pop'ed it: Yes, but why wouldn'

RE: [PATCH] gcc parallel make check

>> > These numbers are useful to try and ensure the overhead (scaling factor) >> > is reasonable, thanks. >> >> A nice improvement indeed. The patched result is 15 times faster >> than the serial unpatched run. So there is room for improvement > > Note, the box used was oldish AMD 16-core, no ht

[patch] allowing configure --target=e500v[12]-etc

Hello, The attached patch is something we have been using for a while to facilitate the production of toolchains targeting the e500 series of powerpc's. The general idea is to have e500 treated essentially as powerpc except selecting a default with_cpu and using a specific "spe" os extension when

[PATCHv3] Vimrc config with GNU formatting

Hi all, This is the third version of the patch. A list of changes since last version: * move config to contrib so that it's _not_ enabled by default (current score is 2/1 in favor of no Vim config by default) * update Makefile.in to make .local.vimrc if developer asks for it * disable autoform

[patch] flag .persistent.bss sections as bss

Hello, Some target loaders (on VxWorks 653 for example) handle the ".persistent.bss" section as an area to reset during the initial startup of applications but not during warm restarts. We need these to be treated as bss sections by the compiler, in particular to prevent the use of executable fil

Re: [GOOGLE] Fix LIPO COMDAT fixup and gcov-tool interactions

On Tue, Sep 16, 2014 at 8:07 AM, Xinliang David Li wrote: > The zero_counts array is passed to gcov_build_callgraph but not used > until the dyn-cgraph is initialized. We should avoid increasing > runtime memory overhead by not creating it if possible. We could delay creation a little bit, until

Re: [PATCH 2/2] Add some more test cases for fentry and pg

On Tue, Sep 16, 2014 at 8:36 AM, H.J. Lu wrote: > On Mon, Sep 15, 2014 at 8:59 AM, Andi Kleen wrote: >> On Mon, Sep 15, 2014 at 11:29:50AM +0200, Richard Biener wrote: >>> On Fri, Sep 12, 2014 at 4:32 PM, Andi Kleen wrote: >>> > From: Andi Kleen >>> > >>> > Test fentry and no_instrument_functio

Re: Move unwind info to read-only section on AIX

On Thu, Sep 11, 2014 at 8:11 PM, Andrew Dixie wrote: > Hi David, > > As discussed, I have updated the to use SYMBOL-$ syntax for > PC-relative addressing and to use a new symbol, __gcc_unwind_dbase, > rather than overloading __dso_handle. > > Regards, > Andrew > > 2014-09-09 Andrew Dixie > >

Re: [GOOGLE] Fix LIPO COMDAT fixup and gcov-tool interactions

1.5M increase is not big concern. Ok with that. David On Tue, Sep 16, 2014 at 10:51 AM, Teresa Johnson wrote: > On Tue, Sep 16, 2014 at 8:07 AM, Xinliang David Li wrote: >> The zero_counts array is passed to gcov_build_callgraph but not used >> until the dyn-cgraph is initialized. We should a

Flatten function.h

Im currently focusing on flattening middle/backend files which are included from front-ends files so we can clean up the include interface a bit. This flattens function.h. It wasn't too bad, there were a few prototypes and defines in expr.h and rtl.h that belong in function.h, and a couple o

Re: Extract and insert merging patch

On Tue, Sep 16, 2014 at 4:40 AM, Petr Murzin wrote: > Hi, > This patch allows merging of extract and insert. Please have a look. > > 2014-09-16 Petr Murzin > > * simplify-rtx.c (simplify_ternary_operation): Allow extract and > insert merging. Besides no testcase. Can your changelog mention ve

Re: [PATCH][match-and-simplify] User defined predicates

On Tue, 16 Sep 2014, Richard Biener wrote: The following adds the ability to write predicates using patterns with an example following negate_expr_p which already has a use in comparison folding (via its if c-expr). The syntax is as follows: (match negate_expr_p INTEGER_CST (if (TYPE_OVERFLOW_

Re: ptx preliminary address space fixes [1/4]

On Tue, 16 Sep 2014, Richard Biener wrote: > Hmm. How is it with other compositive types like vectors and complex? > It's bad that the middle-end needs to follow a specific frontends need. > Why's the representation tied so closely together? Complex types aren't derived types in C terms; they do

Re: [patch] allowing configure --target=e500v[12]-etc

On Tue, 16 Sep 2014, Olivier Hainque wrote: > 2014-09-16 Olivier Hainque > > toplevel/ > * config.sub: Accept e500v[12] cpu names. Canonicalize to > powerpc and add a "spe" suffix to the os name when required > to select the proper ABI and not already there. config.sub

Re: Move unwind info to read-only section on AIX

Hi David, On AIX, mixing objects from different GCC versions should work as long as the newest GCC is used for linking. I tested a library with some objects compiled by GCC-4.4 and some objects compiled by a patched GCC-5.0. Exceptions passed through the mixed objects without issue. Did you have

Re: [PATCH] PowerPC: Implement TARGET_ATOMIC_ASSIGN_EXPAND_FENV

Hi, Adhemerval I cornered Honza during his visit to IBM Research to help me understand my concerns with the function. The code for *hold does: + tree fenv_var = create_tmp_var (double_type_node, NULL); + + tree hold_mffs = build2 (MODIFY_EXPR, void_type_node, fenv_var, call_mffs); + + tree fe

Re: ptx preliminary address space fixes [1/4]

On 09/16/2014 10:56 PM, Joseph S. Myers wrote: On Tue, 16 Sep 2014, Richard Biener wrote: Hmm. How is it with other compositive types like vectors and complex? It's bad that the middle-end needs to follow a specific frontends need. Why's the representation tied so closely together? Complex t

Re: Move unwind info to read-only section on AIX

Hi, Andrew I can understand the requirement of linking with the newer version GCC to provide the dbase symbol. I'm just trying to understand how EH will work when some objects have EH with absolute references in the data section and some have relative references in the text section. I guess the in

Re: Flatten function.h

On Tue, 16 Sep 2014, Andrew MacLeod wrote: > I did an include file reduction on all the language/*.[ch] and core *.[ch] > files, but left the target files with the full complement of 7 includes that > function.h use to have. Its probably easier when this is all done to fully > reduce the targets

Re: ptx preliminary address space fixes [1/4]

On Tue, 16 Sep 2014, Bernd Schmidt wrote: > > It's not an implementation detail, it's how TR 18037 defines them, and > > thus how the C front end should represent them in order to follow the > > requirements of TR 18037. > > My position is that standards do not mandate how our internal data struc

Re: Flatten function.h

On 09/16/2014 05:12 PM, Joseph S. Myers wrote: On Tue, 16 Sep 2014, Andrew MacLeod wrote: I did an include file reduction on all the language/*.[ch] and core *.[ch] files, but left the target files with the full complement of 7 includes that function.h use to have. Its probably easier when thi

Re: [PATCH v2 AArch64]: Re: [PATCH AArch64]: Add constraint letter for stack_protect_test pattern.

On Thu, Sep 4, 2014 at 1:18 AM, James Greenhalgh wrote: > On Thu, Sep 04, 2014 at 08:42:31AM +0100, Venkataramanan Kumar wrote: >> Hi maintainers, >> >> I just added "=r" and retested it. > > I had a very similar patch to this sitting in my local tree. However, > I am surprised you have left opera

Remove LIBGCC2_TF_CEXT target macro

This patch removes the (undocumented) LIBGCC2_TF_CEXT target macro, replacing it by -fbuilding-libgcc predefines (and thereby gets rid of another LIBGCC2_LONG_DOUBLE_TYPE_SIZE conditional, though some more patches are needed before that target macro can be eliminated). This macro indicated the suf

Re: [PR libfortran/62768] Handle filenames with embedded nulls

On Tue, Sep 16, 2014 at 11:17 AM, FX wrote: >>> 2014-09-05 Janne Blomqvist >>> >>>PR libfortran/62768 >>>* io/io.h (gfc_unit): Store C string for the filename. >>>* io/close.c (st_close): Use gfc_unit.filename. >>>* io/inquire.c (inquire_via_unit): Likewise. >>>* io/open.c (

Re: [PATCHv3] Vimrc config with GNU formatting

On Tue, Sep 16, 2014 at 08:38:58PM +0400, Yury Gribov wrote: > Hi all, > > This is the third version of the patch. A list of changes since last > version: > * move config to contrib so that it's _not_ enabled by default (current > score is 2/1 in favor of no Vim config by default) fwiw, I think e

Re: [PATCH v2 AArch64]: Re: [PATCH AArch64]: Add constraint letter for stack_protect_test pattern.

Hi Andrew, Thanks for pointing that. I thought "&" modifier is enough to say that operand is early clobbered and so GCC will use a different register and it will not allocate same register that was given to a input operand. Lookign at the the bug it looks like "=" is needed for the clobber, so t

Re: [PATCH v2 AArch64]: Re: [PATCH AArch64]: Add constraint letter for stack_protect_test pattern.

On Tue, Sep 16, 2014 at 10:36:08PM +0100, Andrew Pinski wrote: > On Thu, Sep 4, 2014 at 1:18 AM, James Greenhalgh > wrote: > > On Thu, Sep 04, 2014 at 08:42:31AM +0100, Venkataramanan Kumar wrote: > >> Hi maintainers, > >> > >> I just added "=r" and retested it. > > > > I had a very similar patch

Re: ptx preliminary address space fixes [1/4]

On 09/16/2014 11:18 PM, Joseph S. Myers wrote: (That TYPE_MAIN_VARIANT maps an array of qualified type to an array of corresponding unqualified type necessitates lots of special cases in the front end to avoid applying TYPE_MAIN_VARIANT to array types, since in C terms array types are always unq

Re: ptx preliminary address space fixes [1/4]

On Wed, 17 Sep 2014, Bernd Schmidt wrote: > On 09/16/2014 11:18 PM, Joseph S. Myers wrote: > > > (That TYPE_MAIN_VARIANT maps an array of qualified type to an array of > > corresponding unqualified type necessitates lots of special cases in the > > front end to avoid applying TYPE_MAIN_VARIANT to

Re: [PATCH v2 AArch64]: Re: [PATCH AArch64]: Add constraint letter for stack_protect_test pattern.

On Tue, Sep 16, 2014 at 3:03 PM, James Greenhalgh wrote: > On Tue, Sep 16, 2014 at 10:36:08PM +0100, Andrew Pinski wrote: >> On Thu, Sep 4, 2014 at 1:18 AM, James Greenhalgh >> wrote: >> > On Thu, Sep 04, 2014 at 08:42:31AM +0100, Venkataramanan Kumar wrote: >> >> Hi maintainers, >> >> >> >> I ju

Re: [C PATCH] Better diagnostics for C++ comments in C90 (PR c/61854)

On Mon, 15 Sep 2014, Marek Polacek wrote: > On Mon, Sep 15, 2014 at 05:49:25PM +, Joseph S. Myers wrote: > > On Mon, 15 Sep 2014, Marek Polacek wrote: > > > > > We must be careful to properly handle code such as "1 //**/ 2", which > > > has a different meaning in C90 and GNU90 mode. New test

[PING^2][PATCH] Power/GCC: Fix e500 vs non-e500 register save slot issue

David, This patch: https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00051.html is still waiting, please review. Thanks, Maciej

[committed] Fix problem handling return value aggregates on PA

The attached problem fixes a regression in handling DFmode aggregates in pa_function_value. Sometimes, an aggregrate with a single DFmode field could not be handled by the existing code which used a BLKmode parallel. We can just use a register when the aggregate fits exactly in a single or

[gomp4] various OpenACC/PTX built-ins and a reduction tweak

The patch introduces the following OpenACC/PTX-specific built-ins: * GOACC_ntid * GOACC_tid * GOACC_nctaid * GOACC_ctaid * acc_on_device * GOACC_get_thread_num * GOACC_get_num_threads Of these functions, the only one part of the OpenACC spec is acc_on_device. The other functions are