[gomp4,committed] Remove release_dangling_ssa_names

[ was: Re: [PATCH] Don't create superfluous parm in expand_omp_taskreg ] On 24/09/15 11:02, Thomas Schwinge wrote: Hi Tom! On Thu, 24 Sep 2015 08:36:27 +0200, Tom de Vries wrote: >On 24/09/15 08:23, Thomas Schwinge wrote: > >On Tue, 11 Aug 2015 20:53:39 +0200, Tom de Vries wrote: > >>Don't

[RS6000] Make -msingle-pic-base remove the ELFv2 global entry code

For other ABIs, -msingle-pic-base makes gcc omit loading of the PIC register in function prologues. This patch makes the option affect ELFv2 too. I wrote a patch like this during the initial ELFv2 effort, but there were many more important patches to push and this one somehow got dropped. Dusted

Re: [PATCH, PR target/67761] Fix i686-*-* bootstrap comparison failure

Hello! > My recenttly introduced STV pass doesn't skip debug instructions and it > causes transformation > (mistly cost computation) depending on debug info. It causes bootstrap > comparison failure. This > patch fixes. Bootstrapped for i686-linux. Testing for > x86_64-unknown-linux-gnu{,m3

Re: [PATCH] Convert SPARC to LRA

On 30/09/15 04:07, Jeff Law wrote: If the port does get occasional fixes (primarily driven by BZs), but not getting updated on a regular basis (such as conversion to LRA, conversion to RTL prologue/epilogue, etc), may be only getting occasional testing, etc. Then it's probably fair to call it in

Re: [PATCH] add static-linked PIE support

On Tue, Sep 29, 2015 at 09:34:07PM -0400, Rich Felker wrote: > This is the gcc side support of the static-linked PIE functionality > added to binutils in commit 9b8b325a1f4cdaf235e7d803849dde6ededec865: And unfortunately I wasn't aware of this: https://gcc.gnu.org/git/?p=gcc.git;a=commitdiff;h=e9

Re: [PATCH GCC]Improve rtl loop inv cost by checking if the inv can be propagated to address uses

On Tue, Sep 29, 2015 at 1:21 AM, Jeff Law wrote: > On 09/28/2015 05:28 AM, Bernd Schmidt wrote: >> >> On 09/28/2015 11:43 AM, Bin Cheng wrote: >>> >>> Bootstrap and test on x86_64 and x86_32. Will test it on aarch64. So >>> any >>> comments? >>> >>> Thanks, >>> bin >>> >>> 2015-09-28 Bin Cheng

Re: [PATCH] Convert SPARC to LRA

On 09/29/2015 09:43 AM, Jeff Law wrote: FWIW, I tried to build a simple cc0 target with LRA (v850-elf), but it fell over pretty early. Essentially LRA doesn't seem to be cc0-aware in split_reg as ultimately inserted something between a cc0-setter and cc0-user. Oops. Yes, that is true. When

Re: [PATCH] x86 interrupt attribute

On Tue, Sep 29, 2015 at 5:02 PM, H.J. Lu wrote: > On Tue, Sep 29, 2015 at 4:53 PM, Mike Stump wrote: >> On Sep 29, 2015, at 3:10 PM, H.J. Lu wrote: >>> On Tue, Sep 29, 2015 at 2:23 PM, Mike Stump wrote: On Sep 29, 2015, at 1:59 PM, H.J. Lu wrote: > commit f3a6675a8d69d810d2cad0c090a76

Re: [PATCH] x86 interrupt attribute

On Tue, Sep 29, 2015 at 5:23 PM, Mike Stump wrote: > On Sep 29, 2015, at 3:40 PM, H.J. Lu wrote: >> How about adding a "no_caller_saved_registers" attribute? > > You can save all call clobbered registers with 3 instructions? Really? I’m > skeptical. Anyway, if you do this by turning off great

Re: [PATCH] Convert SPARC to LRA

On 09/29/2015 08:00 AM, Richard Biener wrote: On Tue, Sep 29, 2015 at 3:39 PM, Jeff Law wrote: On 09/29/2015 07:19 AM, Oleg Endo wrote: On Mon, 2015-09-28 at 15:28 -0500, Segher Boessenkool wrote: We can at least change the default to LRA, so new ports get it unless they like to hurt themse

[PATCH] add static-linked PIE support

This is the gcc side support of the static-linked PIE functionality added to binutils in commit 9b8b325a1f4cdaf235e7d803849dde6ededec865: https://sourceware.org/git/?p=binutils-gdb.git;a=commitdiff;h=9b8b325a1f4cdaf235e7d803849dde6ededec865 I've moved the linking spec change from SUBTARGET_LINK_S

Re: [PATCH][AArch64] Add separate insn sched class for vector LDP & STP

On Tue, Sep 29, 2015 at 12:52 AM, Evandro Menezes wrote: > In some micro-architectures the insns to load or store pairs of vector > registers are implemented rather differently from those affecting lanes in > vector registers. Then, it's important that such insns be described > likewise different

Re: [PATCH] x86 interrupt attribute

On Sep 29, 2015, at 3:40 PM, H.J. Lu wrote: > How about adding a "no_caller_saved_registers" attribute? You can save all call clobbered registers with 3 instructions? Really? I’m skeptical. Anyway, if you do this by turning off great swaths of registers, then, I guess that doesn’t surprise m

[committed, PATCH] gcc.dg/debug/pr65771.c: Use tls_runtime instead of tls.

The attached patch fixes a few failures on bare-metal arm that happen because `dg-require-effective-target tls` only checks for compile-time support, whereas `dg-require-effective-target tls_runtime` checks for runtime support too. The latter is needed due to the missing support for __aeabi_rea

Re: [PATCH] x86 interrupt attribute

On Tue, Sep 29, 2015 at 4:53 PM, Mike Stump wrote: > On Sep 29, 2015, at 3:10 PM, H.J. Lu wrote: >> On Tue, Sep 29, 2015 at 2:23 PM, Mike Stump wrote: >>> On Sep 29, 2015, at 1:59 PM, H.J. Lu wrote: commit f3a6675a8d69d810d2cad0c090a762094a0a8622 Author: H.J. Lu Date: Tue Sep

Re: [PATCH] x86 interrupt attribute

On Sep 29, 2015, at 3:10 PM, H.J. Lu wrote: > On Tue, Sep 29, 2015 at 2:23 PM, Mike Stump wrote: >> On Sep 29, 2015, at 1:59 PM, H.J. Lu wrote: >>> commit f3a6675a8d69d810d2cad0c090a762094a0a8622 >>> Author: H.J. Lu >>> Date: Tue Sep 29 13:47:18 2015 -0700 >>> >>> Define EPILOGUE_USES in i

[patch committed FT32] Limit MEMSET, MEMCPY to <512 bytes

The attached patch limits the MEMSET (setmemsi pattern) and MEMCPY (movmemsi pattern) instructions to 0-511 bytes. There is a hardware limitation on large MEMSET, MEMCPY operations that the library versions of memset() and memcpy() deal with. [gcc] 2015-09-29 James Bowman * config/ft3

Re: [google][gcc-4_9] Remove unused key field in gcov_fn_info

ok. David On Tue, Sep 29, 2015 at 4:08 PM, Rong Xu wrote: > You are right. I attached the updated patch to this email. > > On Tue, Sep 29, 2015 at 3:10 PM, Xinliang David Li wrote: >>else >> { >>gfi_ptr = gi_ptr->functions[f_ix]; >> - if (gfi_ptr && gfi_ptr

Re: [google][gcc-4_9] Remove unused key field in gcov_fn_info

You are right. I attached the updated patch to this email. On Tue, Sep 29, 2015 at 3:10 PM, Xinliang David Li wrote: >else > { >gfi_ptr = gi_ptr->functions[f_ix]; > - if (gfi_ptr && gfi_ptr->key == gi_ptr) > + if (gfi_ptr) > length = GCO

Re: [PATCH] x86 interrupt attribute

On Tue, Sep 29, 2015 at 3:19 PM, H.J. Lu wrote: > On Tue, Sep 29, 2015 at 2:12 PM, Mike Stump wrote: >> On Sep 29, 2015, at 1:16 PM, H.J. Lu wrote: >>> On Tue, Sep 29, 2015 at 11:49 AM, Mike Stump wrote: To be feature complete, it would be nice to have two styles of interrupt functio

Go patch committed: Accept untyped integer values as indexes

This patch by Chris Manghane fixes the Go frontend to accept any untyped integer value as an index, even if the default type of the value is not "int". This fixes https://golang.org/issue/11545 . Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian Index: gcc/

RE: [PATCH] use MIN fusion for ISL-14

The AST looked very bad: see https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67754#c2 There were a lot of testcases failing with MAX fusion. Sebastian -Original Message- From: Tobias Grosser [mailto:tob...@grosser.es] Sent: Tuesday, September 29, 2015 5:14 PM To: Sebastian Pop; gcc-patches@

Re: [PATCH] x86 interrupt attribute

On Tue, Sep 29, 2015 at 2:12 PM, Mike Stump wrote: > On Sep 29, 2015, at 1:16 PM, H.J. Lu wrote: >> On Tue, Sep 29, 2015 at 11:49 AM, Mike Stump wrote: >>> To be feature complete, it would be nice to have two styles of interrupt >>> functions, one that returns with iret, and one that returns wi

Re: [PATCH] use MIN fusion for ISL-14

On 09/30/2015 12:10 AM, Sebastian Pop wrote: This patch fixes PR66754 by reverting an earlier unintended change. We now generate a much simpler AST for interchange-1.c: ISL AST generated by ISL: { for (int c1 = 0; c1 <= 1334; c1 += 1) { S_7(c1); for (int c3 = 0; c3 <= 1334; c3 += 1)

Re: [PATCH] remove dead code of commutative_reductions

Tobias Grosser wrote: > On 09/29/2015 06:26 PM, Sebastian Pop wrote: > >This code is not used anymore after we removed the previous loop optimizer > >(not > >based on the ISL scheduler.) We will add back the detection of commutative > >reductions after we improve the code generation of scalar dep

Re: [PATCH] x86 interrupt attribute

On Tue, Sep 29, 2015 at 2:23 PM, Mike Stump wrote: > On Sep 29, 2015, at 1:59 PM, H.J. Lu wrote: >> commit f3a6675a8d69d810d2cad0c090a762094a0a8622 >> Author: H.J. Lu >> Date: Tue Sep 29 13:47:18 2015 -0700 >> >>Define EPILOGUE_USES in i386 so that all preserved registers are used >>by

Re: [google][gcc-4_9] Remove unused key field in gcov_fn_info

else { gfi_ptr = gi_ptr->functions[f_ix]; - if (gfi_ptr && gfi_ptr->key == gi_ptr) + if (gfi_ptr) length = GCOV_TAG_FUNCTION_LENGTH; - else -length = 0; } The removal of 'else' path seems wrong. David On Tue

[PATCH] use MIN fusion for ISL-14

This patch fixes PR66754 by reverting an earlier unintended change. We now generate a much simpler AST for interchange-1.c: ISL AST generated by ISL: { for (int c1 = 0; c1 <= 1334; c1 += 1) { S_7(c1); for (int c3 = 0; c3 <= 1334; c3 += 1) S_4(c1, c3); S_5(c1); } for (int c1

[RFA][PATCH] Fix building cr16-elf with trunk compiler

This code from builtins.c: /* If we don't need too much alignment, we'll have been guaranteed proper alignment by get_trampoline_type. */ if (TRAMPOLINE_ALIGNMENT <= STACK_BOUNDARY) return tramp; It's entirely conceivable that TRAMPOLINE_ALIGNMENT will be the same as STACK_BOUND

Re: [PATCH] x86 interrupt attribute

On Sep 29, 2015, at 2:23 PM, Mike Stump wrote: > On Sep 29, 2015, at 1:59 PM, H.J. Lu wrote: >> commit f3a6675a8d69d810d2cad0c090a762094a0a8622 >> Author: H.J. Lu >> Date: Tue Sep 29 13:47:18 2015 -0700 >> >> Define EPILOGUE_USES in i386 >> Please take a look. Oh, and with that, I don’t t

Re: [PATCH] x86 interrupt attribute

On Sep 29, 2015, at 1:59 PM, H.J. Lu wrote: > commit f3a6675a8d69d810d2cad0c090a762094a0a8622 > Author: H.J. Lu > Date: Tue Sep 29 13:47:18 2015 -0700 > >Define EPILOGUE_USES in i386 so that all preserved registers are used >by the epilogue of interrupt handler. Don't explicitly mark

Re: [PATCH] x86 interrupt attribute

On Sep 29, 2015, at 1:16 PM, H.J. Lu wrote: > On Tue, Sep 29, 2015 at 11:49 AM, Mike Stump wrote: >> To be feature complete, it would be nice to have two styles of interrupt >> functions, one that returns with iret, and one that returns with ret. The >> point is that the user might want to cal

Re: [PATCH] x86 interrupt attribute

On Tue, Sep 29, 2015 at 1:16 PM, H.J. Lu wrote: > On Tue, Sep 29, 2015 at 11:49 AM, Mike Stump wrote: >> To be feature complete, it would be nice to have two styles of interrupt >> functions, one that returns with iret, and one that returns with ret. The >> point is that the user might want to

Re: [AArch64_be] Fix vtbl[34] and vtbx4

Ping? On 15 September 2015 at 18:25, Christophe Lyon wrote: > This patch re-implements vtbl[34] and vtbx4 AdvSIMD intrinsics using > existing builtins, and fixes the behaviour on aarch64_be. > > Tested on aarch64_be-none-elf and aarch64-none-elf using the Foundation Model. > > OK? > > Christophe

Re: [testsuite] Fix order of dg-do and dg-require-effective-target directives

On Sep 29, 2015, at 1:29 PM, Christophe Lyon wrote: > The attached patch fixes the order on the few testcases where I > noticed it was wrong. > OK? Ok.

Re: [PATCH] Make compute_deps, extend_schedule static

On 09/29/2015 10:19 PM, Aditya Kumar wrote: From: hiraditya No functional changes intended. Passes make check and bootstrap. LGTM. Tobias

[google][gcc-4_9] Remove unused key field in gcov_fn_info

Hi, This patch is for google/gcc-4_9 branch. The 'key' field in gcov_fn_info is designed to allow gcov function data to be COMDATTed, but the comdat elimination never works. This patch removes this field to reduce the instrumented object size. Thanks, -Rong Removed the unused 'key' field in gco

Re: [PATCH] Clear variables with stale SSA_NAME_RANGE_INFO (PR tree-optimization/67690)

On September 29, 2015 4:21:16 PM GMT+02:00, Marek Polacek wrote: >On Fri, Sep 25, 2015 at 06:22:44PM +0200, Richard Biener wrote: >> On September 25, 2015 3:49:34 PM GMT+02:00, Marek Polacek > wrote: >> >On Fri, Sep 25, 2015 at 09:29:30AM +0200, Richard Biener wrote: >> >> On Thu, 24 Sep 2015, Ma

RE: [PATCH][AArch64] Add separate insn sched class for vector LDP & STP

It's been committed as 228253. Thank y'all for playing. Cheers, -- Evandro Menezes Austin, TX > -Original Message- > From: Kyrill Tkachov [mailto:kyrylo.tkac...@arm.com] > Sent: Tuesday, September 29, 2015 4:01 > To: Marcus Shawcroft; Evandro Menezes; gcc-

[testsuite] Fix order of dg-do and dg-require-effective-target directives

I have noticed that both dg-do and dg-require-effective-target modify the value of dg-do-what, which means that dg-do directives must appear before dg-require-effective-target. Indeed if the effective-target property is false, but dg-do is executed later, the test would fail instead of being unsup

[Patch] Add OPT_Wattributes to ignored attributes on template args

Hi, In canonicalize_type_argument attributes are being discarded with a warning. Should it be added to OPT_Wattributes? 2015-09-29 Ryan Mansfield * pt.c (canonicalize_type_argument): Use OPT_Wattributes in warning. Index: cp/pt.c

[PATCH] Make compute_deps, extend_schedule static

From: hiraditya No functional changes intended. Passes make check and bootstrap. gcc/ChangeLog: 2015-09-29 Aditya Kumar * graphite-dependences.c (scop_get_dependences): Moved in down in order to be visible to its caller. * graphite-poly.h: Removed compute_deps, and ex

Re: [PATCH] x86 interrupt attribute

On Tue, Sep 29, 2015 at 11:49 AM, Mike Stump wrote: > To be feature complete, it would be nice to have two styles of interrupt > functions, one that returns with iret, and one that returns with ret. The > point is that the user might want to call functions from a interrupt handler > and not sa

Re: Fold acc_on_device

On 09/29/15 15:52, Bernd Schmidt wrote: Ok, although I really don't quite see the need to drop the expander. Unnecessary code duplication. It's better to say something once in one place, than try and say it twice in two different places. nathan

Re: Fold acc_on_device

On 09/29/2015 08:21 PM, Nathan Sidwell wrote: This patch folds acc_on_device as a regular builtin, but postponed until we know which compiler we're in. As suggested by Bernd, we use the existing builtin folding machinery. Trunk is still using the older PTX runtime scheme (Thomas is working on

[committed, PATCH] Fix typos in comments in config/i386/i386.c

Index: gcc/ChangeLog === --- gcc/ChangeLog (revision 228264) +++ gcc/ChangeLog (working copy) @@ -1,3 +1,8 @@ +2015-09-29 H.J. Lu + + * config/i386/i386.c (ix86_function_arg): Fix typo in comments. + (ix86_ns

Re: Elimitate duplication of get_catalogs in different abi

On 25/09/2015 17:58, Jonathan Wakely wrote: > On 25/09/15 16:10 +0100, Jonathan Wakely wrote: >> On 25/09/15 16:08 +0100, Jonathan Wakely wrote: >>> On 23/09/15 21:28 +0200, François Dumont wrote: On 05/09/2015 23:02, François Dumont wrote: > On 22/08/2015 14:24, Daniel Krügler wrote:

[openacc] use cuda error routine

The cuda library has provided cuGetErrorString since at least 5.5, along with documentation of same. What's been missing until cuda 7.0 is a declaration in the cuda header file. I've merged this patch from the gomp4 branch to the nvptx libgomp plugin. nathan 2015-09-29 Nathan Sidwell *

Re: [Patch ifcvt costs 0/3] Introduce a new target hook for ifcvt costs.

On Sep 29, 2015, at 7:31 AM, James Greenhalgh wrote: > On Tue, Sep 29, 2015 at 11:16:37AM +0100, Richard Biener wrote: >> On Fri, Sep 25, 2015 at 5:04 PM, James Greenhalgh >> wrote: >>> >>> In relation to the patch I put up for review a few weeks ago to teach >>> RTL if-convert to handle multipl

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

On 09/29/2015 05:37 AM, 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, but fai

Re: [PATCH] x86 interrupt attribute

To be feature complete, it would be nice to have two styles of interrupt functions, one that returns with iret, and one that returns with ret. The point is that the user might want to call functions from a interrupt handler and not save and restore all call clobbered registers. By allowing a r

New OpenACC pass and Target Hook

This patch implements an openacc device-specific lowering pass, and an openacc target hook for validating compute dimensions. The pass 'oaccdevlow' is inserted early after LTO readback. It is active for offloaded openacc functions, and openacc routines. Currently its only action is to valida

Re: [gomp4] error on acc loops not associated with offloaded acc regions

On 09/29/2015 02:48 AM, Thomas Schwinge wrote: > On Mon, 28 Sep 2015 10:08:34 -0700, Cesar Philippidis > wrote: >> I've applied this patch to gomp-4_0-branch which teaches omplower how to >> error when it detects acc loops which aren't nested inside an acc >> parallel or kernels region or locate

Re: [PATCH] Fix warnings building pdp11 port

On 09/29/2015 12:11 PM, Trevor Saunders wrote: On Tue, Sep 29, 2015 at 10:55:46AM -0600, Jeff Law wrote: The pdp11 port fails to build with the trunk because of a warning. Essentially VRP determines that the result of using BRANCH_COST is a constant with the range [0..1]. That's always less tha

Fold acc_on_device

This patch folds acc_on_device as a regular builtin, but postponed until we know which compiler we're in. As suggested by Bernd, we use the existing builtin folding machinery. Trunk is still using the older PTX runtime scheme (Thomas is working on that), so the only change there is in the h

[PATCH,committed] xfail Fortran tests on i386-*freebsd*

Neither test mention below has a chance to ever pass on i386-*-freebsd* without a rewrite of the testecases. So, I've xfailed both. 2015-09-29 Steven G. Kargl gfortran.dg/ieee/ieee_4.f90: xfail on i386-*-freebsd* gfortran.dg/round_4.f90: ditto. Index: gfortran.dg/ieee/ieee_4.

Re: [PATCH] Fix warnings building pdp11 port

On Tue, Sep 29, 2015 at 10:55:46AM -0600, Jeff Law wrote: > The pdp11 port fails to build with the trunk because of a warning. > Essentially VRP determines that the result of using BRANCH_COST is a > constant with the range [0..1]. That's always less than 4, 3 and the > various other magic constan

[gomp4.1] Fixup handling of doacross loops with noreturn body

On Thu, Sep 24, 2015 at 08:32:10PM +0200, Jakub Jelinek wrote: > then there is a bug with ordered loops that have noreturn body (need to add > some edge for that case and condition checking), This patch fixes the above issue, if we have any of the ordered > collapse loops that might have zero iter

Re: [PATCH] Fix building microblaze targets with trunk

On 09/29/2015 10:01 AM, Jeff Law wrote: The microblaze port as a "*p++" statement which computes a result that is never used (the memory result). This removes the spurious memory dereference and the unused value warning. Tested by building the microblaze targets in config-all.mk. Installed o

[PATCH] Fix building interix targets

I'm resisting the temptation to declare interix dead (it's been tried before). I'm guessing it hasn't built since early 2012. But the fix is trivial enough and it's not like interix needs lots of care and maintenance. Tested by building the interix targets in config-list.mk. Installed on t

[PATCH] Fix building microblaze targets with trunk

The microblaze port as a "*p++" statement which computes a result that is never used (the memory result). This removes the spurious memory dereference and the unused value warning. Tested by building the microblaze targets in config-all.mk. Installed on the trunk. Jeff commit b2e58a1a53a3bbb

[PATCH] Fix warnings building pdp11 port

The pdp11 port fails to build with the trunk because of a warning. Essentially VRP determines that the result of using BRANCH_COST is a constant with the range [0..1]. That's always less than 4, 3 and the various other magic constants used with BRANCH_COST and VRP issues a warning about that c

Re: [patch] Leave errno unchanged by successful std::stoi etc

On 09/29/2015 10:15 AM, Jakub Jelinek wrote: On Tue, Sep 29, 2015 at 05:10:20PM +0100, Jonathan Wakely wrote: That looks wrong to me, you only restore errno if you don't throw :(. If you throw, then errno might remain 0, which is IMHO undesirable. My thinking was that a failed conversion that

Patch for PR 66424 has been backported to GCC-5 branch

The following patch has been committed to gcc 5 branch as rev. 228256. The patch was bootstrapped and tested on x86/x86-64. Index: ChangeLog === --- ChangeLog (revision 228250) +++ ChangeLog (working copy) @@ -1,3 +1,12 @@ +201

Re: patch to fix PR66424

On 09/29/2015 10:23 AM, Matthias Klose wrote: This was marked as a regression in 5 and 6, but never backported to the gcc-5-branch. Is it time to backport? Thanks for the remainder. I've just committed the patch to gcc 5 branch.

[PATCH] Fix undefined behaviour in SH port

More left shifts of negative signed values to fix in the SH port. I'm not sure how these were missed last week or if they were introduced between the point when I tested last week and yesterday. Regardless, they're fixed in the obvious way. Tested by building all the sh targets form config-a

[PATCH, PR target/67761] Fix i686-*-* bootstrap comparison failure

Hi, My recenttly introduced STV pass doesn't skip debug instructions and it causes transformation (mistly cost computation) depending on debug info. It causes bootstrap comparison failure. This patch fixes. Bootstrapped for i686-linux. Testing for x86_64-unknown-linux-gnu{,m32} is in progre

Re: [PATCH] remove dead code of commutative_reductions

On 09/29/2015 06:26 PM, Sebastian Pop wrote: This code is not used anymore after we removed the previous loop optimizer (not based on the ISL scheduler.) We will add back the detection of commutative reductions after we improve the code generation of scalar dependences (by not going out of SSA f

[PATCH] Fix undefined behaviour in rx port

And the rx port. Tested by building the rx targets in config-all.mk. Installed on the trunk. Jeff commit 67dd8bdfba4072f24ea1a2bd07ffacc91185ee89 Author: Jeff Law Date: Mon Sep 28 19:25:14 2015 -0400 [PATCH] Fix undefined behaviour in rx port * config/rx/constraints.md (Int0

Re: [C PATCH] Fix missing warning (PR c/67730)

On Tue, 29 Sep 2015, Marek Polacek wrote: This fixes missing warning for the attached testcase. In such a case, we must use the expansion point location. I didn't simply add loc = expansion_point_location_if_in_system_header (loc); as might be seen elsewhere in the codebase because we pass LO

Re: [C PATCH] Fix missing warning (PR c/67730)

On Tue, Sep 29, 2015 at 06:04:55PM +0200, Marc Glisse wrote: > On Tue, 29 Sep 2015, Marek Polacek wrote: > > >This fixes missing warning for the attached testcase. In such a case, > >we must use the expansion point location. I didn't simply add > > loc = expansion_point_location_if_in_system_hea

Re: [C PATCH] Fix missing warning (PR c/67730)

On Tue, 29 Sep 2015, Marek Polacek wrote: > This fixes missing warning for the attached testcase. In such a case, > we must use the expansion point location. I didn't simply add > loc = expansion_point_location_if_in_system_header (loc); > as might be seen elsewhere in the codebase because we

[PATCH] Fix undefined behaviour in rl78 port

And in the rl78 port. Tested by building the rl78 targets in config-all.mk. Installed on the trunk. Jeff commit 6d8cde85a30e36e5b5842b8d66837a8b4815d197 Author: Jeff Law Date: Mon Sep 28 19:25:04 2015 -0400 [PATCH] Fix undefined behaviour in rl78 port * config/rl78/rl78-expa

Re: [patch] Leave errno unchanged by successful std::stoi etc

On Tue, Sep 29, 2015 at 05:10:20PM +0100, Jonathan Wakely wrote: > >That looks wrong to me, you only restore errno if you don't throw :(. > >If you throw, then errno might remain 0, which is IMHO undesirable. > > My thinking was that a failed conversion that throws an exception > should be allowed

[PATCH] remove dead code of commutative_reductions

This code is not used anymore after we removed the previous loop optimizer (not based on the ISL scheduler.) We will add back the detection of commutative reductions after we improve the code generation of scalar dependences (by not going out of SSA for scalar dependences just to expose them to th

[PATCH] Fix undefined behaviour in msp430 port

Similar to the fixes from the weekend. Avoiding left shifts of negative signed values in the obvious way. Tested by building msp430 targets from config-all.mk. Installed on the trunk. Jeff commit 679cec5bd2f9ca9c6dabff89d0103790d560c0cb Author: Jeff Law Date: Mon Sep 28 19:24:56 2015 -04

Re: [patch] Leave errno unchanged by successful std::stoi etc

On 29/09/15 17:25 +0200, Jakub Jelinek wrote: On Tue, Sep 29, 2015 at 04:15:41PM +0100, Jonathan Wakely wrote: We set errno=0 in __gnu_cxx::__stoa in order to reliably detect when it gets set to ERANGE. This restores the previous value when the conversion is successful. Tested powerpc64le-linux

Re: [Patch,optimization]: Optimized changes in the estimate register pressure cost.

On 09/25/2015 11:51 PM, Ajit Kumar Agarwal wrote: I have made the following changes in the estimate_reg_pressure_cost function used by the loop invariant and IVOPTS. Earlier the estimate_reg_pressure cost uses the cost of n_new variables that are generated by the Loop Invariant and IVOPTS. T

[C PATCH] Fix missing warning (PR c/67730)

This fixes missing warning for the attached testcase. In such a case, we must use the expansion point location. I didn't simply add loc = expansion_point_location_if_in_system_header (loc); as might be seen elsewhere in the codebase because we pass LOC down to convert_for_assignment where many

Re: [patch] Leave errno unchanged by successful std::stoi etc

On Tue, Sep 29, 2015 at 04:15:41PM +0100, Jonathan Wakely wrote: > We set errno=0 in __gnu_cxx::__stoa in order to reliably detect when > it gets set to ERANGE. This restores the previous value when the > conversion is successful. > > Tested powerpc64le-linux, committed to trunk. > commit 412f75d

[gomp4] Rename oacc_transform pass

I've committed this to gomp4 branch. It renames the oacc_transform pass to oacc_device_lower, in line with the (now withdrawn) patch for mainline. I'm preparing a version of the pass for mainline with a different initial use than acc_on_device folding. nathan 2015-09-29 Nathan Sidwell

Re: [PATCH] Clarify __atomic_compare_exchange_n docs

On 09/29/2015 06:00 AM, Jonathan Wakely wrote: Someone on IRC incorrectly parsed the docs at https://gcc.gnu.org/onlinedocs/gcc-5.2.0/gcc/_005f_005fatomic-Builtins.html#index-g_t_005f_005fatomic_005fcompare_005fexchange_005fn-3536 as: IF ( desired is written into *ptr AND the execu

[patch] Leave errno unchanged by successful std::stoi etc

We set errno=0 in __gnu_cxx::__stoa in order to reliably detect when it gets set to ERANGE. This restores the previous value when the conversion is successful. Tested powerpc64le-linux, committed to trunk. commit 412f75dc37b1048e14996c9caafa46c00db8eb30 Author: Jonathan Wakely Date: Tue Sep 29

Re: [Patch ifcvt costs 0/3] Introduce a new target hook for ifcvt costs.

On Tue, Sep 29, 2015 at 11:16:37AM +0100, Richard Biener wrote: > On Fri, Sep 25, 2015 at 5:04 PM, James Greenhalgh > wrote: > > Hi, > > > > In relation to the patch I put up for review a few weeks ago to teach > > RTL if-convert to handle multiple sets in a basic block [1], I was > > asking about

Re: [PATCH] Convert SPARC to LRA

On Mon, 2015-09-28 at 15:28 -0500, Segher Boessenkool wrote: > On Mon, Sep 28, 2015 at 03:23:37PM -0400, Vladimir Makarov wrote: > > There are more ports using reload than LRA now. Even some major ports > > (e.g. ppc64) did not switch to LRA. > > There still are some failures in the testsuite (I

Re: patch to fix PR66424

This was marked as a regression in 5 and 6, but never backported to the gcc-5-branch. Is it time to backport? Matthias On 21.07.2015 21:54, Vladimir Makarov wrote: The following patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66424 The patch was tested and bootstrapped on x86/

Re: [PATCH] Clear variables with stale SSA_NAME_RANGE_INFO (PR tree-optimization/67690)

On Fri, Sep 25, 2015 at 06:22:44PM +0200, Richard Biener wrote: > On September 25, 2015 3:49:34 PM GMT+02:00, Marek Polacek > wrote: > >On Fri, Sep 25, 2015 at 09:29:30AM +0200, Richard Biener wrote: > >> On Thu, 24 Sep 2015, Marek Polacek wrote: > >> > >> > As Richi said in > >

Re: [PATCH] Convert SPARC to LRA

On Tue, Sep 29, 2015 at 3:39 PM, Jeff Law wrote: > On 09/29/2015 07:19 AM, Oleg Endo wrote: >> >> On Mon, 2015-09-28 at 15:28 -0500, Segher Boessenkool wrote: >> >>> We can at least change the default to LRA, so new ports get it unless >>> they like to hurt themselves. >>> >>> I don't think it mak

Re: [Patch, Fortran, 66927, v2] [6 Regression] ICE in gfc_conf_procedure_call

Hi Mikael, hi all, sorry for the late reply, but I was a bit busy lately and the patch was not as easy as expected. Mikael, I addressed your question about clarifying the comment and while doing so the question arose "what happens when the function returns a class object?" You have one guess; co

Re: OpenACC subarray data alignment in fortran

Ping. In the meantime, I'll apply this patch to gomp-4_0-branch. Cesar On 09/22/2015 08:24 AM, Cesar Philippidis wrote: > In both OpenACC and OpenMP, each subarray has at least two data mappings > associated with them, one for the pointer and another for the data in > the array section (fortan a

[PATCH] x86 interrupt attribute

Hi, The patch below implements interrupt attribute for x86 processors. 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 must use the 'IRET' ins

Re: [gomp4] Remove erroneous test and unreachable situation.

Hi, The original patch still missed some situations (thanks Cesar!) and the attached patch addresses those. It also adds some new tests. Jim Index: libgomp/ChangeLog.gomp === --- libgomp/ChangeLog.gomp (revision 228245) +++ libgomp

Re: [PATCH] Convert SPARC to LRA

On 09/28/2015 02:28 PM, Segher Boessenkool wrote: On Mon, Sep 28, 2015 at 03:23:37PM -0400, Vladimir Makarov wrote: There are more ports using reload than LRA now. Even some major ports (e.g. ppc64) did not switch to LRA. There still are some failures in the testsuite (ICEs even) so we're not

Re: [PATCH] Convert SPARC to LRA

On 09/29/2015 07:19 AM, Oleg Endo wrote: On Mon, 2015-09-28 at 15:28 -0500, Segher Boessenkool wrote: We can at least change the default to LRA, so new ports get it unless they like to hurt themselves. I don't think it makes sense to keep reload around *just* for the ports that are in "mainten

[patch] libstdc++/67583 Fix invalid sputn calls in tests

As the PR says, we're calling sputn with a string that is shorter than the length we specify. I'm not sure if the length was significant (I don't think so), but rather than change that I extended the strings to that length. Tested powerpc64le-linux, committed to trunk. commit 4f835652c412fbd030

Re: [PATCH] Convert SPARC to LRA

On Mon, 2015-09-28 at 15:28 -0500, Segher Boessenkool wrote: > We can at least change the default to LRA, so new ports get it unless > they like to hurt themselves. > > I don't think it makes sense to keep reload around *just* for the ports > that are in "maintenance mode": by the time we are dow

[PATCH] Fix PR67741

Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-09-29 Richard Biener PR tree-optimization/67741 * tree-ssa-math-opts.c (pass_cse_sincos::execute): Only recognize builtin calls with correct signature. * gcc.dg/torture/pr67741.c: New

Re: [patch] Reduce space and time overhead of std::thread

On 23/09/15 17:18 +0100, Jonathan Wakely wrote: For PR 65393 I avoided some unnecessary shared_ptr copies while launching a std::thread. This goes further and avoids shared_ptr entirely, using unique_ptr instead. This reduces the memory overhead of a std::thread by 32 bytes (on 64-bit) and avoids

Re: [PATCH 2/4 v2] bb-reorder: Add the "simple" algorithm

On 09/25/2015 04:16 PM, Segher Boessenkool wrote: v2 changes: - Add a file header comment; - Use "for" loop initial declarations; - Handle asm goto. Testing this on x86_64-linux; okay if it succeeds? No objections from me. Let's give Steven another day or so to comment. Bernd

Re: [PATCH 1/3, libgomp] Adjust offload plugin interface for avoiding deadlock on exit

On 2015/9/25 上午 04:27, Ilya Verbin wrote: > On Thu, Aug 27, 2015 at 21:44:50 +0800, Chung-Lin Tang wrote: >> We've discovered that, for several of the libgomp plugin interface routines, >> if the target specific routine calls exit() (usually upon a fatal condition), >> deadlock ensues. We found thi

  1   2   >