Re: [PATCH] SH FDPIC backend support

On Sat, Oct 03, 2015 at 06:57:56AM +0900, Kaz Kojima wrote: > Rich Felker wrote: > > I worked around it and opened an issue for it: > > > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67812 > > > > But trying the patch on vanilla GCC trunk without my usual J2 target > > setup revealed some addi

[patch] Fix testsuite failures with --disable-wchar_t

This fixes some FAILs seen with --disable-wchar_t, and for targets where that's the default. Tested powerpc64le-linux, committed to trunk. commit e559cc2e962b0d7ec394d38c767f7d212da276aa Author: Jonathan Wakely Date: Sat Oct 3 00:28:00 2015 +0100 Fix testsuite failures with --disable-wc

[patch] Enable dual ABI for Filesystem TS

Currently the Filesystem TS library, libstdc++fs.a, only has symbols for one ABI, whichever is configured as the default when GCC is built. This adds dual ABI support, by building the relevant files twice, once with each ABI (as we do already for libstdc++.so and libstdc++.a). Tested powerpc64le

[gomp4.1] fix dependency scheduling problem

Hi. As I may have mentioned earlier, I'm implementing OpenMP 4.1's task priorities for GCC's libgomp. This has had me looking at how we currently implement scheduling, and I believe we have a problem in how we rearrange dependencies. Currently in gomp_task_maybe_wait_for_dependencies(), we

[patch] Fix tests that fail with old std::string

These tests only work with the new std::string, but the preprocessor test was in the wrong place. Tested powerpc64le-linux, committed to trunk. commit 482100af94e90eaa6fb3c865e30bc907f6691f59 Author: Jonathan Wakely Date: Fri Oct 2 23:16:27 2015 +0100 * testsuite/21_strings/basic_strin

[patch] Use noexcept instead of _GLIBCXX_NOEXCEPT

There's no need to use the _GLIBCXX_NOEXCEPT macro in code that is only compiled when __cplusplus >= 201103L, just use noexcept directly. Tested powerpc64le-linux (both ABIs), committed to trunk. commit 5c1cf3fcbfa60abf8d8a174336982a548e23c64e Author: Jonathan Wakely Date: Fri Oct 2 22:55:39

Re: [PATCH 3/3] increase the number of parameters

On October 2, 2015 10:44:00 PM GMT+01:00, Sebastian Pop wrote: >--- > gcc/params.def | 2 +- > gcc/testsuite/gcc.dg/graphite/scop-sor.c | 3 +-- > 2 files changed, 2 insertions(+), 3 deletions(-) > >diff --git a/gcc/params.def b/gcc/params.def >index 3f91992..da2c6a3 10064

Re: RFC: PATCH for front end parts of C++ transactional memory TS

On 10/02/2015 03:06 PM, Torvald Riegel wrote: On Fri, 2015-10-02 at 14:13 -0400, Jason Merrill wrote: The patch also doesn't attempt to do anything about the library. The second patch sets transaction_safe on various built-ins, but without the library support this just means references to undef

Re: [PATCH] SH FDPIC backend support

Rich Felker wrote: > I worked around it and opened an issue for it: > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67812 > > But trying the patch on vanilla GCC trunk without my usual J2 target > setup revealed some additional issues I need to address. I'm getting > ICE in the code that genera

Re: Do not use TYPE_CANONICAL in useless_type_conversion

Hi, so this is variant without the BLKmode subreg that passes testing on ppc64-linux. The case of parlallel and copy_blkmode_to_reg appears to be handled upstream in expand_assignment. Honza * expr.c (store_expr_with_bounds): Handle aggregate moves from BLKmode. * gimple-

[patch] Fix comment in libstdc++-v3/acinclude.m4

Fix an incorrectly spelled option in a comment. Committed to trunk. commit be72d20163bef9efd626d6c5337fef6e652683ae Author: Jonathan Wakely Date: Fri Oct 2 22:44:50 2015 +0100 * acinclude.m4 (GLIBCXX_ENABLE_DEBUG_FLAGS): Fix comment. diff --git a/libstdc++-v3/acinclude.m4 b/libstdc++-v

[PATCH 2/3] move param detection to scop detection

--- gcc/graphite-poly.c | 68 ++- gcc/graphite-poly.h | 11 +- gcc/graphite-scop-detection.c| 217 ++--- gcc/graphite-sese-to-poly.c | 324 --- gcc/graphite-sese-to-poly.h

[PATCH 1/3] move scop detection related stuff to graphite-scop-detection.c

From: hiraditya --- gcc/graphite-scop-detection.c | 1943 + gcc/graphite-sese-to-poly.c | 184 +--- 2 files changed, 1207 insertions(+), 920 deletions(-) diff --git a/gcc/graphite-scop-detection.c b/gcc/graphite-scop-detection.c index 77fe25f..1704265

[PATCH 3/3] increase the number of parameters

--- gcc/params.def | 2 +- gcc/testsuite/gcc.dg/graphite/scop-sor.c | 3 +-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/gcc/params.def b/gcc/params.def index 3f91992..da2c6a3 100644 --- a/gcc/params.def +++ b/gcc/params.def @@ -835,7 +835,7 @@ DEFPARAM

Re: Do not use TYPE_CANONICAL in useless_type_conversion

> > Index: expr.c > > === > > --- expr.c (revision 228267) > > +++ expr.c (working copy) > > @@ -5462,7 +5462,12 @@ store_expr_with_bounds (tree exp, rtx ta > > { > >if (GET_MODE (temp) != GET_MODE (target) && GET_MODE (

Re: Do not use TYPE_CANONICAL in useless_type_conversion

> actually I do not quite understand why we need a TYPE_ALIGN_OK flag that is > only used in Ada. Somehow other languages seem to have no problem of that > kind. It's related to a known difficulty with alignment and inheritance, and other languages are affected by variant of it, see e.g. PR c++/3

Help the offload gcc driver find the right assembler (was: [PATCH 4/n] OpenMP 4.0 offloading infrastructure: lto-wrapper)

Hi! On Tue, 12 May 2015 19:05:17 +0200, Bernd Schmidt wrote: > > @@ -4266,7 +4266,7 @@ process_command (unsigned int decoded_op > > } > > > > gcc_assert (!IS_ABSOLUTE_PATH (tooldir_base_prefix)); > > - tooldir_prefix2 = concat (tooldir_base_prefix, spec_host_machine, > > + tooldir_pr

Re: [libgo PATCH] Request for a backport to gcc-5

On Fri, Oct 2, 2015 at 1:05 PM, Marek Polacek wrote: > > I'd like to backport the following patch to the gcc-5 branch. The > problem here is that there's a bug in the gcc-5 branch due to which > we aren't able to warn for > int foo (void) { return NULL; } > in C. I have a patch for this issue,

Re: Do not use TYPE_CANONICAL in useless_type_conversion

> Index: expr.c > === > --- expr.c(revision 228267) > +++ expr.c(working copy) > @@ -5462,7 +5462,12 @@ store_expr_with_bounds (tree exp, rtx ta > { >if (GET_MODE (temp) != GET_MODE (target) && GET_MODE (temp) != >

RFC: Patch to allow spill slot alignment greater than the stack alignment

I have spent some time trying to do dynamic stack alignment on MIPS and had considerable trouble. The problems are mainly due to the dwarf based stack unwinding and setjmp/longjmp usages where the code does not go through the normal function prologue and epilogue code. Since the only need I have

Re: [PATCH, rs6000] Fix PR target/67808, LRA ICE on double to long double conversion

On Fri, Oct 2, 2015 at 3:04 PM, Peter Bergner wrote: > PR67808 exposes a problem with the constraints in the *extenddftf2_internal > pattern, in that it allows TFmode operands to occupy Altivec registers > which they are not allowed to do. Reload was able to work around the > problem, but LRA is

[libgo PATCH] Request for a backport to gcc-5

Hi Ian, I'd like to backport the following patch to the gcc-5 branch. The problem here is that there's a bug in the gcc-5 branch due to which we aren't able to warn for int foo (void) { return NULL; } in C. I have a patch for this issue, but I can't backport it without the following patch, bec

Re: RFC: PATCH for front end parts of C++ transactional memory TS

On Fri, 2 Oct 2015, Jason Merrill wrote: > Since the transaction_safe attribute now affects type identity, I needed to > change the C front end to allow declarations of built-ins to change whether > the function is declared transaction_safe. Joseph, is that hunk (copied at > the bottom) OK? OK.

nvptx offloading linking

Hi! On Wed, 13 May 2015 22:11:36 +0200, I wrote: > On Wed, 22 Apr 2015 17:08:26 +0200, Bernd Schmidt > wrote: > > On 04/21/2015 05:58 PM, Jakub Jelinek wrote: > > > > > suggests that while it is nice that when building nvptx accel compiler > > > we build libgcc.a, libc.a, libm.a, libgfortran.a

[patch] Allow Filesystem TS to compile without wchar_t

This guards all uses of wchar_t with _GLIBCXX_USE_WCHAR_T. There's also a second patch that improves a testsuite helper function slightly. Tested powerpc64le-linux, x86_64dragonfly, powerpc-aix. Committed to trunk. commit e765e9d79405c7979f656460b995e6237900eea3 Author: Jonathan Wakely Date:

Re: [PATCH] Improve DOM's optimization of control statements

On 10/02/2015 05:15 AM, Renlin Li wrote: Hi Jeff, Your patch causes an ICE regression. The test case is " gcc.c-torture/compile/pr27087.c", I observed it on aarch64-none-elf target when compiling the test case with '-Os' flag. A quick check shows, the cfg has been changed, but the loop informat

[PR target/67822] OpenMP offloading to nvptx fails (was: [gomp4 2/8] nvptx mkoffload: do not restrict to OpenACC)

Hi! On Thu, 24 Sep 2015 09:25:54 +0200, Jakub Jelinek wrote: > On Wed, Sep 23, 2015 at 08:22:16PM +0300, Alexander Monakov wrote: > > This patch allows to meaningfully invoke mkoffload with -fopenmp. The check > > for -fopenacc flag is specific to gomp4 branch: trunk does not have it. > > > >

Re: [gomp4.1] depend nowait support for target {update,{enter,exit} data}

Hi! On Tue, Sep 08, 2015 at 11:20:14 +0200, Jakub Jelinek wrote: > nowait support for #pragma omp target is not implemented yet, supposedly we > need to mark those somehow (some flag) already in the struct gomp_task > structure, essentially it will need either 2 or 3 callbacks > (the current one,

Re: RFC: PATCH for front end parts of C++ transactional memory TS

On Fri, 2015-10-02 at 14:13 -0400, Jason Merrill wrote: > The patch also doesn't attempt to do anything about the library. The > second patch sets transaction_safe on various built-ins, but without the > library support this just means references to undefined symbols. For some of the builtins,

[PATCH, rs6000] Fix PR target/67808, LRA ICE on double to long double conversion

PR67808 exposes a problem with the constraints in the *extenddftf2_internal pattern, in that it allows TFmode operands to occupy Altivec registers which they are not allowed to do. Reload was able to work around the problem, but LRA is more pedantic and it caused it to go into an infinite spill lo

RFC: PATCH for front end parts of C++ transactional memory TS

This patch implements the front end bits of the transactional memory technical specification: apart from parsing, that means treating transaction_safe as part of the type and dealing with conversions, mangling and such. Since the transaction_safe attribute now affects type identity, I needed

Re: [patch] libstdc++/67747 Allocate space for dirent::d_name

On 10/02/2015 07:53 PM, Martin Sebor wrote: > Here's the older page: > http://docs.oracle.com/cd/E19683-01/816-0213/6m6ne3895/index.html > > With that, I agree that using readdir is thread-safe on Solaris 11.2. > It is not safe on Solaris 9, and it would not be safe on other systems > that don't

Re: [PATCH] Introduce ROUND_UP and ROUND_DOWN macros

On 10/02/2015 06:42 PM, Uros Bizjak wrote: Attached patch introduces ROUND_UP and ROUND_DOWN global macros. These come handy to round and align various values, as shown by the usage in config/i386/ files. * system.h (ROUND_UP): New macro definition. (ROUND_DOWN): Ditto. * ggc-page

Re: Do not use TYPE_CANONICAL in useless_type_conversion

> > On Fri, 2 Oct 2015, Jan Hubicka wrote: > > > > > > There must be a reason why I allowed modes to differ there btw ;) > > > > > > Thinking about it, I guess reason is that incomplete types do not have > > > resonable modes set, so requiring modes to match will prevent complete > > > and incomp

Re: [patch] libstdc++/67747 Allocate space for dirent::d_name

On 10/02/2015 11:43 AM, Florian Weimer wrote: On 10/02/2015 07:37 PM, Martin Sebor wrote: I'm not sure what you are basing this assertion on. In the man pages I have looked at, memcpy is documented as MT-Safe. readdir is documented as MT-Unsafe. The Unsafe definition is clear: contains global a

Re: [patch] libstdc++/67747 Allocate space for dirent::d_name

On 10/02/2015 07:37 PM, Martin Sebor wrote: > I'm not sure what you are basing this assertion on. In the man > pages I have looked at, memcpy is documented as MT-Safe. readdir > is documented as MT-Unsafe. The Unsafe definition is clear: > contains global and static data that is not protected. I

Re: [PATCH] x86 interrupt attribute

Fixed it. Thanks. On Fri, Oct 2, 2015 at 6:45 PM, Uros Bizjak wrote: > On Fri, Oct 2, 2015 at 2:51 PM, Yulia Koval wrote: >> Hi, >> Here is a new patch. Added HJ's changes and review changes. >> >> Implement x86 interrupt attribute > > + incoming_stack_boundary > + = (crtl->parm_stack_bound

Re: [patch] libstdc++/67747 Allocate space for dirent::d_name

On 10/02/2015 11:09 AM, Florian Weimer wrote: On 10/02/2015 06:57 PM, Martin Sebor wrote: Readdir isn't required to be thread-safe (it may reference global data) so calling it in multiple threads even with a different dirp argument is undefined. A thread-unsafe implementation can meet the POSIX

Re: [PATCH] Convert SPARC to LRA

On 09/30/2015 10:15 AM, Segher Boessenkool wrote: On Wed, Sep 30, 2015 at 09:15:17AM -0600, Jeff Law wrote: I guess the support of cc0 can be implemented for reasonable amount of time. It is just a priority issue. I still have a lot PRs for the targets already using LRA. I wouldn't suggest ma

Re: [patch] libstdc++/67747 Allocate space for dirent::d_name

On 10/02/2015 06:57 PM, Martin Sebor wrote: > Readdir isn't required to be thread-safe (it may reference global > data) so calling it in multiple threads even with a different dirp > argument is undefined. A thread-unsafe implementation can meet the > POSIX requirement and still access global data

Re: C PATCH for c/65345 (file-scope _Atomic expansion with floats)

On 01/10/15 17:18, Marek Polacek wrote: > On Thu, Oct 01, 2015 at 11:02:09AM -0400, David Edelsohn wrote: >> On Thu, Oct 1, 2015 at 10:49 AM, Marek Polacek wrote: >>> Joseph reminded me that I had forgotten about this patch. As mentioned >>> here

Re: [patch] libstdc++/67747 Allocate space for dirent::d_name

On 02/10/15 10:57 -0600, Martin Sebor wrote: On 10/02/2015 06:34 AM, Jonathan Wakely wrote: On 02/10/15 14:16 +0200, Florian Weimer wrote: On 09/29/2015 01:37 PM, Jonathan Wakely wrote: POSIX says that dirent::d_name has an unspecified length, so calls to readdir_r must pass a buffer with enou

Re: [patch] libstdc++/67747 Allocate space for dirent::d_name

On 10/02/2015 06:34 AM, Jonathan Wakely wrote: On 02/10/15 14:16 +0200, Florian Weimer wrote: On 09/29/2015 01:37 PM, Jonathan Wakely wrote: POSIX says that dirent::d_name has an unspecified length, so calls to readdir_r must pass a buffer with enough trailing space for {NAME_MAX}+1 characters.

Re: [Patch AArch64] Use default_elf_asm_named_section instead of special cased hook

On Fri, Oct 2, 2015 at 5:40 PM, Christophe Lyon wrote: > On 2 October 2015 at 15:05, Marcus Shawcroft > wrote: >> On 2 October 2015 at 14:01, Ramana Radhakrishnan >> wrote: >> > #undef TARGET_ASM_NAMED_SECTION > -#define TARGET_ASM_NAMED_SECTION aarch64_elf_asm_named_section > +#d

Re: [PATCH] fortran/67758 -- Prevent ICE caused by misplaced COMMON

On Fri, Oct 02, 2015 at 11:28:06AM +0200, Mikael Morin wrote: > Le 01/10/2015 18:30, Steve Kargl a écrit : > > I'm fine with your patch, although I find the error message > > to be somewhat confusing as no procedure appears in COMMON. > > Well, XX is implicitly a procedure. > Yes, I understamf w

[PATCH] Introduce ROUND_UP and ROUND_DOWN macros

Hello! Attached patch introduces ROUND_UP and ROUND_DOWN global macros. These come handy to round and align various values, as shown by the usage in config/i386/ files. In addition to target independent code, many targets could benefit from these universal macros (e.g. arm and aarch64 can immedia

Re: [Patch AArch64] Use default_elf_asm_named_section instead of special cased hook

On 2 October 2015 at 15:05, Marcus Shawcroft wrote: > On 2 October 2015 at 14:01, Ramana Radhakrishnan > wrote: > #undef TARGET_ASM_NAMED_SECTION -#define TARGET_ASM_NAMED_SECTION aarch64_elf_asm_named_section +#define TARGET_ASM_NAMED_SECTION default_elf_asm_named_section >>> >

Re: [PATCH] Disable -Wduplicated-cond for the time being

On Fri, Oct 02, 2015 at 06:27:37PM +0200, Marek Polacek wrote: > Given that the new warning currently breaks bootstrap [*] and I see > no simple way how to resolve PR67819 derived out of the bootstrap > failure, I'm moving -Wduplicated-cond out of -Wall and skipping one > of the tests for the time

Re: [Patch, fortran] PR67721 deep copy missing when assigning derived type constructor to an array

On Fri, Oct 02, 2015 at 09:33:33AM -0700, H.J. Lu wrote: > On Sat, Sep 26, 2015 at 6:10 AM, Mikael Morin wrote: > > Hello, > > > > I've just submitted this PR, and the patch as well, which passes the > > testsuite. > > > > The problem is a missing deep copy when the rhs is a (scalar) derived type

Re: [patch] libstdc++/67747 Allocate space for dirent::d_name

On 02/10/15 14:41 +0200, Florian Weimer wrote: On 10/02/2015 02:34 PM, Jonathan Wakely wrote: On 02/10/15 14:16 +0200, Florian Weimer wrote: On 09/29/2015 01:37 PM, Jonathan Wakely wrote: POSIX says that dirent::d_name has an unspecified length, so calls to readdir_r must pass a buffer with en

Re: [Patch, fortran] PR67721 deep copy missing when assigning derived type constructor to an array

On Sat, Sep 26, 2015 at 6:10 AM, Mikael Morin wrote: > Hello, > > I've just submitted this PR, and the patch as well, which passes the > testsuite. > > The problem is a missing deep copy when the rhs is a (scalar) derived type > constructor (with allocatable components) and the lhs an array. > The

[PATCH] Disable -Wduplicated-cond for the time being

Given that the new warning currently breaks bootstrap [*] and I see no simple way how to resolve PR67819 derived out of the bootstrap failure, I'm moving -Wduplicated-cond out of -Wall and skipping one of the tests for the time being. I'm also reverting the hack in genemit.c. Oh well. [*] https:

Re: [PATCH] Improve DOM's optimization of control statements

On 10/02/2015 05:15 AM, Renlin Li wrote: Hi Jeff, Your patch causes an ICE regression. The test case is " gcc.c-torture/compile/pr27087.c", I observed it on aarch64-none-elf target when compiling the test case with '-Os' flag. A quick check shows, the cfg has been changed, but the loop informat

Re: [PATCH] Cleanup of IPA-CP alignment lattices

> Hi, > > I have already proposed this patch many months ago but it got > forgotten, for a number of reasons. It does not change functionality, > it only cleans up the alignment lattices, which are currently a bit > clumsy. > > I have refreshed the patch and made sure it still bootstraps and tes

Re: [PATCH, i386] Introduce switch for Skylake Server CPU.

Hello, Patch in the bottom introduces cpuid detection for Skylake CPU supporting AVX-512. Bootstrapped. Changed test pass. Is it ok for trunk? libgcc/ * libgcc/config/i386/cpuinfo.c (get_intel_cpu): Detect "skylake-avx512". gcc/testsuite/ * gcc.target/i386/builtin_target.c: Add

[PATCH] Cleanup of IPA-CP alignment lattices

Hi, I have already proposed this patch many months ago but it got forgotten, for a number of reasons. It does not change functionality, it only cleans up the alignment lattices, which are currently a bit clumsy. I have refreshed the patch and made sure it still bootstraps and tests fine on an x8

Re: Do not use TYPE_CANONICAL in useless_type_conversion

> On Fri, 2 Oct 2015, Jan Hubicka wrote: > > > > There must be a reason why I allowed modes to differ there btw ;) > > > > Thinking about it, I guess reason is that incomplete types do not have > > resonable modes set, so requiring modes to match will prevent complete > > and incomplete types to

Re: [C PATCH] Another fix for missing diagnostics (PR c/67730)

On Fri, 2 Oct 2015, Marek Polacek wrote: > This patch continues fixing up this PR, where we were failing to give > diagnostics for attached testcases, because of NULL being defined in > a system header. Probably the best we can do at this time is just to > use expansion_point_location_if_in_syste

Re: [C/C++ PATCH] RFC: Implement -Wduplicated-cond (PR c/64249) (version 2)

On Fri, Oct 02, 2015 at 08:43:30AM -0700, H.J. Lu wrote: > On Fri, Oct 2, 2015 at 3:23 AM, Marek Polacek wrote: > > On Wed, Sep 30, 2015 at 12:45:35PM -0600, Jeff Law wrote: > >> On 09/30/2015 09:47 AM, Joseph Myers wrote: > >> >The C front-end changes are OK. > >> The rest are OK as well. > > > >

Re: [PATCH] x86 interrupt attribute

On Fri, Oct 2, 2015 at 2:51 PM, Yulia Koval wrote: > Hi, > Here is a new patch. Added HJ's changes and review changes. > > Implement x86 interrupt attribute + incoming_stack_boundary + = (crtl->parm_stack_boundary > ix86_incoming_stack_boundary + ? crtl->parm_stack_boundary : ix86_incoming

Re: [C/C++ PATCH] RFC: Implement -Wduplicated-cond (PR c/64249) (version 2)

On Fri, Oct 2, 2015 at 3:23 AM, Marek Polacek wrote: > On Wed, Sep 30, 2015 at 12:45:35PM -0600, Jeff Law wrote: >> On 09/30/2015 09:47 AM, Joseph Myers wrote: >> >The C front-end changes are OK. >> The rest are OK as well. > > Thanks Jeff & Joseph. > > I'm going to apply the patch soon; should it

Re: Do not describe -std=c11 etc. as experimental in c.opt help text

On Fri, Oct 02, 2015 at 05:35:39PM +0200, Marek Polacek wrote: > On Thu, Oct 01, 2015 at 05:01:26PM +, Joseph Myers wrote: > > I noticed that c.opt still described -std=c11 and related options as > > experimental in the --help text. This patch fixes this. > > > > Jason, note that -std=gnu++11

Re: Do not describe -std=c11 etc. as experimental in c.opt help text

On Fri, Oct 02, 2015 at 05:35:39PM +0200, Marek Polacek wrote: > On Thu, Oct 01, 2015 at 05:01:26PM +, Joseph Myers wrote: > > I noticed that c.opt still described -std=c11 and related options as > > experimental in the --help text. This patch fixes this. > > > > Jason, note that -std=gnu++11

Re: [PATCH] SH FDPIC backend support

On Fri, 2015-10-02 at 11:18 -0400, Rich Felker wrote: > Thanks! This is very helpful. gcc style has changed a lot since the > old patch was submitted so I think it makes sense to update it to > match current practices rather than just making it work. I'll try to > focus on any functional problems

Re: Do not describe -std=c11 etc. as experimental in c.opt help text

On Thu, Oct 01, 2015 at 05:01:26PM +, Joseph Myers wrote: > I noticed that c.opt still described -std=c11 and related options as > experimental in the --help text. This patch fixes this. > > Jason, note that -std=gnu++11 and -std=gnu++14 still have that text, > contrary to the descriptions of

[PATCH, obvious, AVX-512] Add missing AVX-512 features detection.

Hello, Patch in the bottom adds missing AVX-512VBMI,IFMA features to libgcc/config/i386/cpuinfo.c and, built-in expansion and test. Comitted to main trunk as obvious. gcc/ * config/i386/i386.c (processor_features): Add F_AVX512VBMI, F_AVX512IFMA. (isa_names_table): Handle

Re: [PATCH] Add verifier for leaked SSA names

On 10/02/2015 01:37 AM, Richard Biener wrote: The following patch doesn't pass bootstrap & regtest. It did at some point though and its comment hints that fixing leaks after inlining was too interesting a problem to solve ;) Thus patch is FYI. Richard. Index: tree-ssa.c =

Re: [PATCH] Improve DOM's optimization of control statements

On 10/02/2015 05:15 AM, Renlin Li wrote: Hi Jeff, Your patch causes an ICE regression. The test case is " gcc.c-torture/compile/pr27087.c", I observed it on aarch64-none-elf target when compiling the test case with '-Os' flag. A quick check shows, the cfg has been changed, but the loop informat

Re: Do not describe -std=c11 etc. as experimental in c.opt help text

On Fri, 2 Oct 2015, Matthias Klose wrote: > On 01.10.2015 19:01, Joseph Myers wrote: > > I noticed that c.opt still described -std=c11 and related options as > > experimental in the --help text. This patch fixes this. > > this seems to be true for the gcc-5 branch as well. Could you fix it there

Re: [PATCH] SH FDPIC backend support

On Fri, Oct 02, 2015 at 10:51:03PM +0900, Oleg Endo wrote: > On Thu, 2015-10-01 at 21:30 -0400, Rich Felker wrote: > > > If you have any other general comments on the patch in the mean time > > I'd be happy to hear them. > > Below are some comments. Might be a bit unstructured, I was hopping > t

[C PATCH] Another fix for missing diagnostics (PR c/67730)

This patch continues fixing up this PR, where we were failing to give diagnostics for attached testcases, because of NULL being defined in a system header. Probably the best we can do at this time is just to use expansion_point_location_if_in_system_header. That also means we will warn for "RETUR

patch to fix PR657756

The following patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67756 The patch was bootstrapped and tested on x86-64. Committed as rev. 228396. Index: ChangeLog === --- ChangeLog (revision 228395) +++ ChangeLog (workin

[PATCH] Reject loops early where ivs cannot be represented.

During scop detection we can figure out if loop IVs cannot be represeted in the polyhedral model. We now bail out early instead of waiting until graphite-sese-to-poly.c Passes regtest and bootstrap with BOOT_CFLAGS=="-g -O2 -fgraphite-identity -floop-nest-optimize" with ISL-0.15. gcc/ChangeLog:

Re: [PATCH, i386, AVX-512] Update extract_even_odd w/ AVX-512BW insns.

On 01 Oct 14:11, Kirill Yukhin wrote: > Bootstrapped. New tests pass (fail w/o the change). Regtesting is in progress. > > Is it ok for trunk? > > gcc/ > * config/i386/i386.c (expand_vec_perm_even_odd_trunc): New. > (expand_vec_perm_even_odd_1): Handle V64QImode. > (ix86_expand_

Re: [PATCH, i386, AVX-512, doc] Mention all AVX-512 switches in invoke.texi.

On 01 Oct 17:51, Kirill Yukhin wrote: > `make pdf` looks ok. > Is it ok for trunk and gcc-5-branch (a week after check in to trunk)? > > gcc/ > * doc/invoke.texi: Mention -mavx512vl, -mavx512bw, -mavx512dq, > -mavx521vbmi, -mavx512ifma. Add missing opindex-es. Checked into main trunk.

Re: [AArch64/testsuite] Add more TLS local executable testcases

On 22 September 2015 at 17:49, Jiong Wang wrote: > 2015-09-22 Jiong Wang > > gcc/testsuite/ >* gcc.target/aarch64/tlsle12_tiny_1.c: New testcase for tiny model. >* gcc.target/aarch64/tlsle24_tiny_1.c: Likewise. >* gcc.target/aarch64/tlsle_sizeadj_tiny_1.c: TLS size truncation test

[[Boolean Vector, patch 5/5] Support boolean vectors in vector lowering

Hi, This patch supports boolean vectors in vector lowering. Main change is to lower vector comparison into comparisons, not cond_exprs. Thanks, Ilya -- 2015-10-02 Ilya Enkovich * tree-vect-generic.c (elem_op_func): Add new operand to hold vector type. (do_unop): Adju

[Boolean Vector, patch 4/5] Use boolean vectors in VEC_COND_EXPR

Hi, This patch forces boolean vector usage in VEC_COND_EXPR generated by vectorizer. VEC_COND_EXPR expand is fixed appropriately. Thanks, Ilya -- gcc/ 2015-10-02 Ilya Enkovich * optabs.c (expand_vec_cond_expr): Accept boolean vector as condition operand. * tree-vect

[Boolean Vector, patch 3/5] Use boolean vector in C/C++ FE

Hi, This patch makes C/C++ FE to use boolean vector as a resulting type for vector comparison. As a result vector comparison in source code now parsed into VEC_COND_EXPR, it required a testcase fix-up. Thanks, Ilya -- gcc/c 2015-10-02 Ilya Enkovich * c-typeck.c (build_conditional_

[Boolean Vector, patch 2/5] Change vector comparison IL requirement

Hi, This patch change vector comparison to require boolean vector resulting type. Thanks, Ilya -- gcc/ 2015-10-02 Ilya Enkovich * tree-cfg.c (verify_gimple_comparison) Require boolean vector type for vector comparison. (verify_gimple_assign_ternary): Likewise. diff

Re: [AArch64/testsuite] Add more TLS local executable testcases

Jiong Wang writes: > Marcus Shawcroft writes: > >> On 26 August 2015 at 14:58, Jiong Wang wrote: >>> >>> This patch cover tlsle tiny model tests, tls size truncation for tiny & >>> small model included also. >>> >>> All testcases pass native test. >>> >>> OK for trunk? >>> >>> 2015-08-26 Jiong

[Boolean Vector, patch 1/5] Introduce boolean vector to be used as a vector comparison type

Hi, This patch starts the first series to introduce vec as a vector comparison type. This series introduces the new vec type and force its usage for all vector comparisons. This series doesn't intoroduce any new vectorization features. I split it into five small patches but will commit in a

Re: [PATCH] SH FDPIC backend support

On Thu, 2015-10-01 at 21:30 -0400, Rich Felker wrote: > If you have any other general comments on the patch in the mean time > I'd be happy to hear them. Below are some comments. Might be a bit unstructured, I was hopping through the patch file. Sorry about that. > +function_symbol (rtx target

[PATCH] Stop including in gcc/system.h

is deprecated on some systems and we gets hundreds of warnings due to including it on (at least) dragonfly and openbsd. We already do this earlier in system.h: #ifdef HAVE_STDLIB_H # include #endif /* When compiling C++ we need to include as well as so that it is processed before we poiso

[wwwdocs] Mention -Wduplicated-cond

Applied. Index: changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-6/changes.html,v retrieving revision 1.32 diff -u -r1.32 changes.html --- changes.html28 Sep 2015 00:43:48 - 1.32 +++ changes.html2 Oct 20

Re: [PATCH] New attribute to create target clones

PING. On Fri, Sep 25, 2015 at 1:28 AM, Evgeny Stupachenko wrote: > I've fixed ICE and review issues. > x86 make check and bootstrap passed. > > Thanks, > Evgeny > > ChangeLog > > 2015-09-25 Evgeny Stupachenko > > gcc/ > * Makefile.in (OBJS): Add multiple_target.o. > * multiple_

Re: [Patch AArch64] Use default_elf_asm_named_section instead of special cased hook

On 2 October 2015 at 14:01, Ramana Radhakrishnan wrote: >>> #undef TARGET_ASM_NAMED_SECTION >>> -#define TARGET_ASM_NAMED_SECTION aarch64_elf_asm_named_section >>> +#define TARGET_ASM_NAMED_SECTION default_elf_asm_named_section >> >> Isn't it sufficient to simply remove the #define completely

Re: [Patch AArch64] Use default_elf_asm_named_section instead of special cased hook

On 02/10/15 13:46, Marcus Shawcroft wrote: > On 2 October 2015 at 11:08, Ramana Radhakrishnan > wrote: > >> * config/aarch64/aarch64-elf.h (TARGET_ASM_NAMED_SECTION): Use >> default_elf_asm_named_section. >> * config/aarch64/aarch64.c (aarch64_elf_asm_named_section): Del

Re: [GCC, ARM] armv8 linux toolchain asan testcase fail due to stl missing conditional code

On 01/10/15 21:21, Christophe Lyon wrote: On 1 October 2015 at 11:10, Kyrill Tkachov wrote: On 30/09/15 17:39, Kyrill Tkachov wrote: On 09/06/15 09:17, Kyrill Tkachov wrote: On 05/06/15 14:14, Kyrill Tkachov wrote: On 05/06/15 14:11, Richard Earnshaw wrote: On 05/06/15 14:08, Kyrill Tkacho

Re: [patch] libstdc++/67747 Allocate space for dirent::d_name

On 10/02/2015 02:37 PM, Sebastian Huber wrote: > > > On 02/10/15 14:16, Florian Weimer wrote: >> On 09/29/2015 01:37 PM, Jonathan Wakely wrote: >>> >POSIX says that dirent::d_name has an unspecified length, so calls to >>> >readdir_r must pass a buffer with enough trailing space for >>> >{NAME_MA

Re: [PATCH] x86 interrupt attribute

Hi, Here is a new patch. Added HJ's changes and review changes. Implement x86 interrupt attribute The interrupt and exception handlers are called by x86 processors. X86 hardware pushes information onto stack and calls the handler. The requirements are 1. Both interrupt and exception handlers m

Re: [Patch AArch64] Use default_elf_asm_named_section instead of special cased hook

On 2 October 2015 at 11:08, Ramana Radhakrishnan wrote: > * config/aarch64/aarch64-elf.h (TARGET_ASM_NAMED_SECTION): Use > default_elf_asm_named_section. > * config/aarch64/aarch64.c (aarch64_elf_asm_named_section): Delete. > --- > gcc/config/aarch64/aarch64-elf.h | 2 +-

Re: [patch] libstdc++/67747 Allocate space for dirent::d_name

On 10/02/2015 02:34 PM, Jonathan Wakely wrote: > On 02/10/15 14:16 +0200, Florian Weimer wrote: >> On 09/29/2015 01:37 PM, Jonathan Wakely wrote: >>> POSIX says that dirent::d_name has an unspecified length, so calls to >>> readdir_r must pass a buffer with enough trailing space for >>> {NAME_MAX}+

Re: [patch] libstdc++/67747 Allocate space for dirent::d_name

On 02/10/15 14:16, Florian Weimer wrote: On 09/29/2015 01:37 PM, Jonathan Wakely wrote: >POSIX says that dirent::d_name has an unspecified length, so calls to >readdir_r must pass a buffer with enough trailing space for >{NAME_MAX}+1 characters. I wasn't doing that, which works OK on >GNU/Linu

[SH][committed]

Hi, The attached SH patch converts sequences such as: movtr2 movtr13 into: movtr2 mov r2,r13 This shortens the live range of the T bit register and is better for parallel execution. It doesn't happen often, but it's easy to avoid it. Unfortunately

Re: [patch] libstdc++/67747 Allocate space for dirent::d_name

On 02/10/15 14:16 +0200, Florian Weimer wrote: On 09/29/2015 01:37 PM, Jonathan Wakely wrote: POSIX says that dirent::d_name has an unspecified length, so calls to readdir_r must pass a buffer with enough trailing space for {NAME_MAX}+1 characters. I wasn't doing that, which works OK on GNU/Linu

Re: [patch] libstdc++/67747 Allocate space for dirent::d_name

On 09/29/2015 01:37 PM, Jonathan Wakely wrote: > POSIX says that dirent::d_name has an unspecified length, so calls to > readdir_r must pass a buffer with enough trailing space for > {NAME_MAX}+1 characters. I wasn't doing that, which works OK on > GNU/Linux and BSD where d_name is a large array, b

[gomp4.1] Unsigned long long doacross implementation

Hi! The default set of routines use long as the iterator type, if some loops need either unsigned long, or long long/unsigned long long, they need to use another implementation (__int128 iterators are not supported). This patch adds those entry points and fixes some issues on the compiler side. 2

Re: Do not use TYPE_CANONICAL in useless_type_conversion

Hi, actually I do not quite understand why we need a TYPE_ALIGN_OK flag that is only used in Ada. Somehow other languages seem to have no problem of that kind. You remember, when I removed the TYPE_ALIGN_OK handing (initially it wasn't clear to me that it's entire use is only to make Ada happy)

Re: [wwwdocs] Buildstat update for 5.x

On Sun, 13 Sep 2015, Tom G. Christensen wrote: > Testresults for 5.2.0: > i686-unknown-linux-gnu > powerpc-unknown-linux-gnu > sparc-sun-solaris2.10 > sparc64-sun-solaris2.10 > x86_64-unknown-linux-gnu (4) > x86_64-w64-mingw32 > > Testresults for 5.1.0: > hppa64-hp-hpux11.00 Thanks,

  1   2   >