Re: [PATCH] Fix num_imm_uses (PR tree-optimization/70405)

On Wed, Mar 30, 2016 at 07:45:23AM +0200, Richard Biener wrote: > >Indeed, GCC 3.4 warns about this: > >pr70405-3.c:7: warning: suggest explicit braces to avoid ambiguous > >`else' > >That warning is still in there under -Wparentheses, but doesn't trigger > >anymore. > > Sounds like poor

Re: [PATCH] Fix num_imm_uses (PR tree-optimization/70405)

On March 29, 2016 7:54:16 PM GMT+02:00, Jakub Jelinek wrote: >On Tue, Mar 29, 2016 at 07:46:32PM +0200, Bernd Schmidt wrote: >> On 03/29/2016 07:28 PM, Jeff Law wrote: >> >On 03/29/2016 11:23 AM, Jakub Jelinek wrote: >> >>Hi! >> >> >> >>The recent change to num_imm_uses (to add

Re: [PATCH] c++/67376 Comparison with pointer to past-the-end, of array fails inside constant expression

On 03/29/2016 12:54 PM, Jason Merrill wrote: On 03/28/2016 06:04 PM, Martin Sebor wrote: + && compare_tree_int (arg1, 0) == 0) This can be integer_zerop. Sure. +case GE_EXPR: +case EQ_EXPR: +case LE_EXPR: + return boolean_false_node; +

[PATCH][PR target/63890] Turn on ACCUMULATE_OUTGOING_ARGS when profiling on darwin

As discussed in the BZ, 32bit Darwin will create a mis-aligned stack when profiling is enabled. As noted in c#9, the 32bit x86 port was enabling A_O_A when producing unwind info, this is why darwin was functional. After Jan's patch from 2013, we stopped turning on A_O_A in that situation

Re: [PATCH] Fix PR tree-optimization/59124 (bogus -Warray-bounds warning)

On Tue, 29 Mar 2016, Richard Biener wrote: > On Tue, Mar 29, 2016 at 1:23 PM, Patrick Palka wrote: > > On Tue, 29 Mar 2016, Richard Biener wrote: > > > >> On Sun, Mar 27, 2016 at 11:37 PM, Patrick Palka > >> wrote: > >> > On Sun, 27 Mar 2016, Patrick

Re: [RFA][PATCH][tree-optimization/64058] Add new coalescing tie breaker heuristic V2

On 03/29/2016 11:34 AM, Peter Bergner wrote: On Wed, 2016-03-23 at 01:49 -0600, Jeff Law wrote: +/* This represents a conflict graph. Implemented as an array of bitmaps. + A full matrix is used for conflicts rather than just upper triangular form. + this make sit much simpler and faster

Re: C PATCH for c/70297 (crash with duplicate typedefs and -g)

On 03/23/2016 07:52 AM, Marek Polacek wrote: On Mon, Mar 21, 2016 at 09:57:54PM +0100, Richard Biener wrote: On March 21, 2016 6:55:28 PM GMT+01:00, Marek Polacek wrote: This PR points out to a GC problem: when we freed a duplicate typedef, we were leaving its type in the

Fix overflow in loop peeling code

Hi, this patch fixes stupid overflow in tree-ssa-loop-ivcanon.c. If the estimated number of execution of loop is INT_MAX+1 it will get peeled incorrectly. Bootstrapped/regtested x86_64-linux and committed (it is regression WRT the RTL implementation) Honza * tree-ssa-loop-ivcanon.c

Re: [PATCH v2] sanitize paths used in regular expression

On Feb 8, 2016, at 2:26 PM, Zach Welch wrote: > > Ping. From what I see, my patch has not yet been committed. Can I talk > someone into taking care of that for me? I had hoped that someone would commit it for you. Committed revision 234533.

Re: [PATCH] hurd: align -p and -pg behavior on Linux

Hello, Thomas Schwinge, on Tue 29 Mar 2016 23:19:09 +0200, wrote: > On Wed, 24 Feb 2016 23:46:36 +0100, I wrote: > > On Sat, 19 Sep 2015 14:00:23 +0200, Samuel Thibault > > wrote: > > > On Linux, -p and -pg do not make gcc link against libc_p.a, only > > > -profile does

Re: [PATCH] hurd: align -p and -pg behavior on Linux

Hi! On Wed, 24 Feb 2016 23:46:36 +0100, I wrote: > On Sat, 19 Sep 2015 14:00:23 +0200, Samuel Thibault > wrote: > > On Linux, -p and -pg do not make gcc link against libc_p.a, only > > -profile does (as documented in r11246), and thus people expect -p > > (Yo, 20 years

Re: [PATCH,boehm-gc] Use mmap instead of brk on kfreebsd & hurd too

Hi! On Sun, 31 Aug 2014 17:20:04 +0200, Samuel Thibault wrote: > Please use mmap instead of brk on kfreebsd and hurd too. > Also, using anonymous memory is faster on the Hurd. > [patch] Thanks; finally committed in r234534: commit

[PATCH] PR testsuite/70364: Properly align stack in gcc.target/i386/cleanup-[12].c

Tested on x86-64. OK for trunk? H.J. --- PR testsuite/70364 * gcc.target/i386/cleanup-1.c: Include . (check): New function. (bar): Call check. (foo): Align stack to 16 bytes when calling bar. * gcc.target/i386/cleanup-2.c: Likewise. ---

Re: [PATCH] Disable guality tests for powerpc*-linux*

On Tue, Mar 29, 2016 at 11:42 AM, Jakub Jelinek wrote: > On Tue, Mar 29, 2016 at 11:34:17AM -0700, Mike Stump wrote: >> On Mar 29, 2016, at 7:45 AM, David Edelsohn wrote: >> > We have no plans to make code generation a slave to the testsuite. >> > The

[PATCH] PR target/70439: Properly check conflict between DRAP register and __builtin_eh_return

Since %ecx can't be used for both DRAP register and __builtin_eh_return, we need to check if crtl->drap_reg uses %ecx before using %ecx for __builtin_eh_return. Testing on x86-64. OK for trunk if there are no regressions? H.J. --- PR target/70439 * config/i386/i386.c

Re: [PATCH] c++/67376 Comparison with pointer to past-the-end, of array fails inside constant expression

On 03/28/2016 06:04 PM, Martin Sebor wrote: + && compare_tree_int (arg1, 0) == 0) This can be integer_zerop. + case GE_EXPR: + case EQ_EXPR: + case LE_EXPR: + return boolean_false_node; + case GT_EXPR: + case LT_EXPR:

Re: [PATCH][ARM][4.9 Backport] PR target/69875 Fix atomic_loaddi expansion

On 16 March 2016 at 16:54, Ramana Radhakrishnan wrote: > On Wed, Feb 24, 2016 at 11:23 AM, Kyrill Tkachov > wrote: >> Hi all, >> >> This is the GCC 4.9 backport of >> https://gcc.gnu.org/ml/gcc-patches/2016-02/msg01338.html. >> The

Re: RFA: PATCH to tree-inline.c:remap_decls for c++/70353 (ICE with __func__ and constexpr)

On 03/29/2016 06:37 AM, Jan Hubicka wrote: On Mon, Mar 28, 2016 at 11:26 PM, Jason Merrill wrote: The constexpr evaluation code uses the inlining code to remap the constexpr function body for evaluation so that recursion works properly. In this testcase __func__ is declared

Re: [PATCH] Disable guality tests for powerpc*-linux*

On Tue, Mar 29, 2016 at 11:34:17AM -0700, Mike Stump wrote: > On Mar 29, 2016, at 7:45 AM, David Edelsohn wrote: > > We have no plans to make code generation a slave to the testsuite. > > The testsuite is a tool, successful results from the testsuite is not > > a goal unto

Re: [PATCH] Disable guality tests for powerpc*-linux*

On Mar 29, 2016, at 7:45 AM, David Edelsohn wrote: > We have no plans to make code generation a slave to the testsuite. > The testsuite is a tool, successful results from the testsuite is not > a goal unto itself. > > This patch is okay. We look forward to the day when

Re: [PATCH] Fix simplify_shift_const_1 once more (PR rtl-optimization/70429)

On Tue, Mar 29, 2016 at 07:21:29PM +0200, Jakub Jelinek wrote: > This is a case similar to the LSHIFTRT I've fixed recently. > But, unlike LSHIFTRT, which can be handled by masking at the outer level, > ASHIFTRT would need outer sign extension, so most of the time 2 outer > operations in addition

Re: [PATCH] Fix simplify_shift_const_1 once more (PR rtl-optimization/70429)

On Tue, Mar 29, 2016 at 11:47:57AM -0600, Jeff Law wrote: > >>>2016-03-29 Jakub Jelinek > >>> > >>> PR rtl-optimization/70429 > >>> * combine.c (simplify_shift_const_1): For ASHIFTRT don't optimize > >>> (cst1 >> count) >> cst2 into (cst1 >> cst2) >> count if > >>>

Re: [PATCH] Fix num_imm_uses (PR tree-optimization/70405)

On Tue, Mar 29, 2016 at 07:46:32PM +0200, Bernd Schmidt wrote: > On 03/29/2016 07:28 PM, Jeff Law wrote: > >On 03/29/2016 11:23 AM, Jakub Jelinek wrote: > >>Hi! > >> > >>The recent change to num_imm_uses (to add support for NULL USE_STMT) > >>broke it totally, fortunately we have just one user of

Re: [PATCH] Fix ix86_expand_vector_set (PR target/70421)

On Tue, Mar 29, 2016 at 11:44:15AM -0600, Jeff Law wrote: > On 03/29/2016 11:05 AM, Jakub Jelinek wrote: > >Hi! > > > >The various blendm expanders look like: > >(define_insn "_blendm" > > [(set (match_operand:V48_AVX512VL 0 "register_operand" "=v") > > (vec_merge:V48_AVX512VL > >

Re: [PATCH] Fix simplify_shift_const_1 once more (PR rtl-optimization/70429)

On 03/29/2016 11:43 AM, Jakub Jelinek wrote: On Tue, Mar 29, 2016 at 11:34:29AM -0600, Jeff Law wrote: This is a case similar to the LSHIFTRT I've fixed recently. But, unlike LSHIFTRT, which can be handled by masking at the outer level, ASHIFTRT would need outer sign extension, so most of the

Re: [PATCH] Fix num_imm_uses (PR tree-optimization/70405)

On 03/29/2016 07:28 PM, Jeff Law wrote: On 03/29/2016 11:23 AM, Jakub Jelinek wrote: Hi! The recent change to num_imm_uses (to add support for NULL USE_STMT) broke it totally, fortunately we have just one user of this function right now. I've filed a PR for GCC 7 so that we get a warning on

Re: [PATCH] Fix ix86_expand_vector_set (PR target/70421)

On 03/29/2016 11:05 AM, Jakub Jelinek wrote: Hi! The various blendm expanders look like: (define_insn "_blendm" [(set (match_operand:V48_AVX512VL 0 "register_operand" "=v") (vec_merge:V48_AVX512VL (match_operand:V48_AVX512VL 2 "nonimmediate_operand" "vm")

Re: [PATCH] Fix simplify_shift_const_1 once more (PR rtl-optimization/70429)

On Tue, Mar 29, 2016 at 11:34:29AM -0600, Jeff Law wrote: > >This is a case similar to the LSHIFTRT I've fixed recently. > >But, unlike LSHIFTRT, which can be handled by masking at the outer level, > >ASHIFTRT would need outer sign extension, so most of the time 2 outer > >operations in addition

Re: [RFA][PATCH][tree-optimization/64058] Add new coalescing tie breaker heuristic V2

On Wed, 2016-03-23 at 01:49 -0600, Jeff Law wrote: > > +/* This represents a conflict graph. Implemented as an array of bitmaps. > + A full matrix is used for conflicts rather than just upper triangular > form. > + this make sit much simpler and faster to perform conflict merges. */

Re: [PATCH] Fix num_imm_uses (PR tree-optimization/70405)

On Tue, Mar 29, 2016 at 11:28:20AM -0600, Jeff Law wrote: > On 03/29/2016 11:23 AM, Jakub Jelinek wrote: > >The recent change to num_imm_uses (to add support for NULL USE_STMT) > >broke it totally, fortunately we have just one user of this function > >right now. I've filed a PR for GCC 7 so that

[PATCH] Fix simplify_shift_const_1 once more (PR rtl-optimization/70429)

Hi! This is a case similar to the LSHIFTRT I've fixed recently. But, unlike LSHIFTRT, which can be handled by masking at the outer level, ASHIFTRT would need outer sign extension, so most of the time 2 outer operations in addition to the kept two inner shifts, which is IMHO very unlikely to ever

Re: [PATCH] Fix num_imm_uses (PR tree-optimization/70405)

On 03/29/2016 11:23 AM, Jakub Jelinek wrote: Hi! The recent change to num_imm_uses (to add support for NULL USE_STMT) broke it totally, fortunately we have just one user of this function right now. I've filed a PR for GCC 7 so that we get a warning on this. Fixed thusly,

[PATCH] Fix num_imm_uses (PR tree-optimization/70405)

Hi! The recent change to num_imm_uses (to add support for NULL USE_STMT) broke it totally, fortunately we have just one user of this function right now. I've filed a PR for GCC 7 so that we get a warning on this. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?

Re: [PATCH] Disable guality tests for powerpc*-linux*

On Tue, Mar 29, 2016 at 12:01:20PM -0500, Bill Schmidt wrote: > Again, this is good information to know about. But the "stuff" we were > talking about was the failures on powerpc*, and I took what you said to > mean that nobody was working on those. It sounds like you're saying > that the

Re: [PATCH] Disable guality tests for powerpc*-linux*

Hi Jakub, Thanks for the information; I really do appreciate it! On Tue, 2016-03-29 at 17:33 +0200, Jakub Jelinek wrote: > On Tue, Mar 29, 2016 at 08:19:39AM -0500, Bill Schmidt wrote: > > When you say that "the debug info quality is already pretty bad on > > powerpc*," do you mean that it is

Re: [PATCH] Disable guality tests for powerpc*-linux*

On March 29, 2016 4:45:44 PM GMT+02:00, David Edelsohn wrote: >On Mon, Mar 28, 2016 at 8:38 PM, Bill Schmidt > wrote: >> Hi, >> >> For a long time we've had hundreds of failing guality tests. These >> failures don't seem to have any correlation

[PATCH] Fix ix86_expand_vector_set (PR target/70421)

Hi! The various blendm expanders look like: (define_insn "_blendm" [(set (match_operand:V48_AVX512VL 0 "register_operand" "=v") (vec_merge:V48_AVX512VL (match_operand:V48_AVX512VL 2 "nonimmediate_operand" "vm") (match_operand:V48_AVX512VL 1 "register_operand" "v")

Re: [Patch, Fortran, pr70397, v1] [5/6 Regression] ice while allocating ultimate polymorphic

Hi Paul, hi Dominique thanks for the fast review and error check, respectively. Committed as r234528. Regards, Andre On Tue, 29 Mar 2016 15:34:13 +0200 Paul Richard Thomas wrote: > Hi Andre, > > Yes, it is better to play safe :-) OK for trunk. > >

Re: [RFC][PATCH, ARM 0/8] ARMv8-M Security Extensions

On 26/12/15 01:39, Thomas Preud'homme wrote: > [Sending on behalf of Andre Vieira] > > Hello, > > This patch series aims at implementing an alpha status support for ARMv8-M's > Security Extensions. It is only posted as RFC at this stage. You can find the > specification of ARMV8-M Security

Re: [RFC][PATCH v2, ARM 5/8] ARMv8-M Security Extension's cmse_nonsecure_entry: clear registers

On 29/01/16 17:07, Andre Vieira (lists) wrote: > On 26/12/15 01:54, Thomas Preud'homme wrote: >> [Sending on behalf of Andre Vieira] >> >> Hello, >> >> This patch extends support for the ARMv8-M Security Extensions >> 'cmse_nonsecure_entry' attribute to safeguard against leak of >> information

a patch for PR68695

The following patch improves the code in 2 out of 3 cases in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68695 The patch uses more accurate costs for the RA cost improvement optimization after colouring. The patch was tested and bootstrapped on x86-64. It is hard to create a test

[COMMITTED] Add myself as GCC maintainer

I've added myself to the "Write After Approval" maintainers (Committed revision 234526): 2016-03-29 Kelvin Nilsen * MAINTAINERS (Write After Approval): Add myself. -- Kelvin Nilsen, Ph.D. kdnil...@linux.vnet.ibm.com home office: 801-756-4821, cell:

Fix bogus vtable mismatch warnings

Hi, this patch fixes bogus warning While building libreoffice we get: /aux/hubicka/libreoffice2/core/sw/source/core/attr/calbck.cxx:27:1: note: virtual method �_ZN2sw16LegacyModifyHintD2Ev.localalias.7� sw::LegacyModifyHint::~LegacyModifyHint() {} ^

Re: FW: [PATCH] Cilk Keywords (_Cilk_spawn and _Cilk_sync) for C

On Tue, Mar 29, 2016 at 17:15:11 +0200, Thomas Schwinge wrote: > On Mon, 28 Mar 2016 19:40:22 +0300, Ilya Verbin wrote: > > Do you plan to commit this patch? :) > > Well, I'm also still waiting for you guys to merge (via the upstream > Intel sources repository) my GNU Hurd

Re: FW: [PATCH] Cilk Keywords (_Cilk_spawn and _Cilk_sync) for C

Hi! On Mon, 28 Mar 2016 19:40:22 +0300, Ilya Verbin wrote: > Do you plan to commit this patch? :) Well, I'm also still waiting for you guys to merge (via the upstream Intel sources repository) my GNU Hurd portability patches; submitted to GCC in

[PATCH] New flag in order to dump information about template instantiations.

Hi, the attached patch adds a new compilation flag 'ftemplate-instantiations' in order to allow dumping debug information for template instantiations. This flag has 2 possible values: none(by default) and hreadable, that prints witch templates instantiations had been made in a human readable way.

Re: [PATCH] Disable guality tests for powerpc*-linux*

On Mon, Mar 28, 2016 at 8:38 PM, Bill Schmidt wrote: > Hi, > > For a long time we've had hundreds of failing guality tests. These > failures don't seem to have any correlation with gdb functionality for > POWER, which is working fine. At this point the value of

Re: [PATCH] Disable guality tests for powerpc*-linux*

On Tue, Mar 29, 2016 at 3:19 PM, Bill Schmidt wrote: > Hi Jakub, > > On Tue, 2016-03-29 at 08:53 +0200, Jakub Jelinek wrote: >> On Mon, Mar 28, 2016 at 07:38:46PM -0500, Bill Schmidt wrote: >> > For a long time we've had hundreds of failing guality tests. These >> >

Re: [PATCH 1/2] Do not verify CFG if a function is discarded (PR

On Tue, Mar 29, 2016 at 3:07 PM, Martin Liška wrote: > On 03/29/2016 02:10 PM, Richard Biener wrote: >> On Tue, Mar 29, 2016 at 1:42 PM, Martin Liška wrote: >>> Hello. >>> >>> The problem with the original patch is that I'm forced to produce >>> an empty BB to

Re: [RS6000, PATCH] PR70052, ICE compiling _Decimal128 test case

On Tue, Mar 29, 2016 at 6:14 AM, Alan Modra wrote: > On Fri, Mar 25, 2016 at 07:36:34PM +1030, Alan Modra wrote: >> +2016-03-25 Alan Modra >> + >> + PR target/70052 >> + * config/rs6000/constraints.md (j): Simplify. >> + *

Re: [PATCH 3/4, libgomp] Resolve deadlock on plugin exit, HSA plugin parts

Hi, On Sun, Mar 27, 2016 at 06:26:29PM +0800, Chung-Lin Tang wrote: > On 2016/3/25 上午 02:40, Martin Jambor wrote: > > On the whole, I am fine with the patch but there are two issues: > > > > First, and generally, when you change the return type of a function, > > you must document what return

Re: [Patch, Fortran, pr70397, v1] [5/6 Regression] ice while allocating ultimate polymorphic

Hi Andre, Yes, it is better to play safe :-) OK for trunk. Thanks Paul On 29 March 2016 at 14:55, Andre Vehreschild wrote: > Hi all, > > here is the trunk version of the patch for the regression reported in > pr70397. Applying the gcc-5 patch to trunk lead to a regression, which

Re: Also test -O0 for OpenACC C, C++ offloading test cases

Hi! On Thu, 24 Mar 2016 22:31:29 +0100, I wrote: > --- libgomp/testsuite/libgomp.oacc-c++/c++.exp > +++ libgomp/testsuite/libgomp.oacc-c++/c++.exp > # Initialize dg. > dg-init > +torture-init > > # Turn on OpenACC. > lappend ALWAYS_CFLAGS "additional_flags=-fopenacc" > @@ -104,7 +101,26 @@

Re: [PATCH] Disable guality tests for powerpc*-linux*

Hi Jakub, On Tue, 2016-03-29 at 08:53 +0200, Jakub Jelinek wrote: > On Mon, Mar 28, 2016 at 07:38:46PM -0500, Bill Schmidt wrote: > > For a long time we've had hundreds of failing guality tests. These > > failures don't seem to have any correlation with gdb functionality for > > POWER, which is

Re: [PATCH 1/2] Do not verify CFG if a function is discarded (PR

On 03/29/2016 02:10 PM, Richard Biener wrote: > On Tue, Mar 29, 2016 at 1:42 PM, Martin Liška wrote: >> Hello. >> >> The problem with the original patch is that I'm forced to produce >> an empty BB to produce true/false edge needed for the 'index' check: >> >>

Re: [PATCH ARM v3] PR69770 -mlong-calls does not affect calls to __gnu_mcount_nc generated by -pg

On 29 March 2016 at 13:41, Charles Baylis wrote: > On 29 March 2016 at 02:16, Kugan wrote: >> >> Hi Charles, >> >> +static void >> +arm_emit_long_call_profile_insn () >> +{ >> + rtx sym_ref = gen_rtx_SYMBOL_REF (Pmode,

[Patch, Fortran, pr70397, v1] [5/6 Regression] ice while allocating ultimate polymorphic

Hi all, here is the trunk version of the patch for the regression reported in pr70397. Applying the gcc-5 patch to trunk lead to a regression, which the modified patch resolves now. The technique to solve the ice is the same as for gcc-5: > The routine gfc_copy_class_to_class() assumed that both

[C++/70393] constexpr constructor

This patch fixes 70393 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70393 'ab's construction used to be dynamic but becomes static with C++11 constexpr constructors (I'm not sure whether we're doing more than the std requires, but that's not important). However, 'AB's bases are not laid out

[PATCH] PR70424 Relax alignment conservativeness

Since 4.9 we're quite "strict" about "less alignment" that we might possibly know about when seeing dereferences. But that implementation is quite inconsistent as treating 1-byte alignment as never "possibly known" (unless we see a decl). So the following patch makes us only consider the case

Re: Proposed Patch for Bug 69687

On 03/03/2016 03:55 PM, Marcel Böhme wrote: @@ -4254,7 +4255,9 @@ Please use "diff -p" so that we get information about which function is being patched. Are all the places being patched really problematic ones where an input file could realistically cause an overflow, or just the string

Re: [PATCH] Fix in-tree gmp/mpfr/mpc generation (PR 67728)

On Tue, 29 Mar 2016, Bernd Edlinger wrote: > On 29.03.2016 at 10:32, Richard Biener wrote: > > On Mon, Mar 28, 2016 at 2:44 PM, Bernd Edlinger > > wrote: > >> > >> Hi, > >> > >> as described in the tracker we have bootstrap problems with in-tree > >> gmp-6.1.0 > >> on

Re: [PATCH 1/2] Do not verify CFG if a function is discarded (PR

On Tue, Mar 29, 2016 at 1:42 PM, Martin Liška wrote: > Hello. > > The problem with the original patch is that I'm forced to produce > an empty BB to produce true/false edge needed for the 'index' check: > > /home/marxin/Programming/testhsa/run_tests/012-switch/switch-5.c:28:9:

Re: [PATCH] Fix PR tree-optimization/59124 (bogus -Warray-bounds warning)

On Tue, Mar 29, 2016 at 1:23 PM, Patrick Palka wrote: > On Tue, 29 Mar 2016, Richard Biener wrote: > >> On Sun, Mar 27, 2016 at 11:37 PM, Patrick Palka wrote: >> > On Sun, 27 Mar 2016, Patrick Palka wrote: >> > >> >> In unrolling of the inner loop in

Re: [DOC Patch] Add sample for @cc constraint

On 03/28/2016 12:03 AM, David Wohlferd wrote: On 3/24/2016 8:00 AM, Bernd Schmidt wrote: > More problematic than a lack of documentation is that I haven't been able to find an executable testcase. If you could adapt your example for use in gcc.target/i386, that would be even more important. It

[PATCH 2/2] Fix PR hsa/70402

Second part of the patch set which omits one split_block (compared to the original patch). Acceptable just in case the first part will be accepted. Thanks Martin >From 2a9a8f11ea1ecd04c3b9915ff77fc791c55632da Mon Sep 17 00:00:00 2001 From: marxin Date: Tue, 29 Mar 2016 12:06:20

[PATCH 1/2] Do not verify CFG if a function is discarded (PR

Hello. The problem with the original patch is that I'm forced to produce an empty BB to produce true/false edge needed for the 'index' check: /home/marxin/Programming/testhsa/run_tests/012-switch/switch-5.c:28:9: error: true/false edge after a non-GIMPLE_COND in bb 4

[PATCH ARM v3] PR69770 -mlong-calls does not affect calls to __gnu_mcount_nc generated by -pg

On 29 March 2016 at 02:16, Kugan wrote: > > Hi Charles, > > +static void > +arm_emit_long_call_profile_insn () > +{ > + rtx sym_ref = gen_rtx_SYMBOL_REF (Pmode, "__gnu_mcount_nc"); > + /* if movt/movw are not available, use a constant pool */ > + if

Re: Goodbye REG_LIVE_LENGTH

On 03/25/2016 11:00 PM, Alan Modra wrote: I'll also prepare a patch to delete REG_LIVE_LENGTH everywhere. Like this. Bootstrapped and regression tested x86_64-linux. OK for stage1? Oh wow that's a lot of stuff removed. Ok for this and the FREQ_CALLS_CROSSED patch. Bernd

Re: [PATCH] Fix PR tree-optimization/59124 (bogus -Warray-bounds warning)

On Tue, 29 Mar 2016, Richard Biener wrote: > On Sun, Mar 27, 2016 at 11:37 PM, Patrick Palka wrote: > > On Sun, 27 Mar 2016, Patrick Palka wrote: > > > >> In unrolling of the inner loop in the test case below we introduce > >> unreachable code that otherwise contains

[PATCH] Fix PR hsa/70402

Hello. As reported here: https://github.com/HSAFoundation/gccbrig/issues/3#issuecomment-199887172, our current HSA back-end emits a SBR instruction that is followed by a jump instruction for cases where index is outside of array of labels. However, as mentioned here:

Re: RFA: PATCH to tree-inline.c:remap_decls for c++/70353 (ICE with __func__ and constexpr)

> On Mon, Mar 28, 2016 at 11:26 PM, Jason Merrill wrote: > > The constexpr evaluation code uses the inlining code to remap the constexpr > > function body for evaluation so that recursion works properly. In this > > testcase __func__ is declared as a static local variable, so

Re: [RS6000, PATCH] PR70052, ICE compiling _Decimal128 test case

On Fri, Mar 25, 2016 at 07:36:34PM +1030, Alan Modra wrote: > +2016-03-25 Alan Modra > + > + PR target/70052 > + * config/rs6000/constraints.md (j): Simplify. > + * config/rs6000/predicates.md (easy_fp_constant): Exclude > + decimal float 0.D. > + *

Re: [PATCH] Fix in-tree gmp/mpfr/mpc generation (PR 67728)

On 29.03.2016 at 10:32, Richard Biener wrote: > On Mon, Mar 28, 2016 at 2:44 PM, Bernd Edlinger > wrote: >> >> Hi, >> >> as described in the tracker we have bootstrap problems with in-tree gmp-6.1.0 >> on certain targets, and also a linker issue with check-mpc due to

Re: [PATCH, libgomp] Rewire OpenACC async

I've updated this patch for trunk (as attached), and re-tested without regressions. This patch is still a fix for libgomp.oacc-c-c++-common/asyncwait-1.c, which FAILs right now. ChangeLog is still as before. Is this okay for trunk? Thanks, Chung-Lin On 2015/12/22 4:58 PM, Chung-Lin Tang wrote:

Re: [genmatch] reject duplicate captures used as arguments in user-defined predicates

On Tue, 29 Mar 2016, Prathamesh Kulkarni wrote: > Hi, > I suppose we should reject duplicate captures used as "arguments" in user > defined predicates ? > eg: > (match (foo @0 @0) > match-template) > The attached patch prints error "duplicate capture id" for above pattern. > Bootstrapped+tested

Re: RFA: PATCH to tree-inline.c:remap_decls for c++/70353 (ICE with __func__ and constexpr)

On Mon, Mar 28, 2016 at 11:26 PM, Jason Merrill wrote: > The constexpr evaluation code uses the inlining code to remap the constexpr > function body for evaluation so that recursion works properly. In this > testcase __func__ is declared as a static local variable, so rather

[genmatch] reject duplicate captures used as arguments in user-defined predicates

Hi, I suppose we should reject duplicate captures used as "arguments" in user defined predicates ? eg: (match (foo @0 @0) match-template) The attached patch prints error "duplicate capture id" for above pattern. Bootstrapped+tested on x86_64-pc-linux-gnu. Ok for trunk ? Thanks, Prathamesh diff

Re: [PATCH PR69489/01]Improve tree ifcvt by storing/tracking DR against its innermost loop bahavior if possible

On Mon, Mar 28, 2016 at 9:57 PM, Bin.Cheng wrote: > Sorry, Should have replied to gcc-patches list. > > Thanks, > bin > > -- Forwarded message -- > From: "Bin.Cheng" > Date: Tue, 29 Mar 2016 03:55:04 +0800 > Subject: Re: [PATCH

Re: [PATCH] Fix in-tree gmp/mpfr/mpc generation (PR 67728)

On Mon, Mar 28, 2016 at 2:44 PM, Bernd Edlinger wrote: > > Hi, > > as described in the tracker we have bootstrap problems with in-tree gmp-6.1.0 > on certain targets, and also a linker issue with check-mpc due to the changed > mpfr library path. Hum, in-tree gmp 6.1.0

Re: [PATCH] Fix PR tree-optimization/59124 (bogus -Warray-bounds warning)

On Sun, Mar 27, 2016 at 11:37 PM, Patrick Palka wrote: > On Sun, 27 Mar 2016, Patrick Palka wrote: > >> In unrolling of the inner loop in the test case below we introduce >> unreachable code that otherwise contains out-of-bounds array accesses. >> This is because the

Re: [PATCH] Disable guality tests for powerpc*-linux*

Jakub Jelinek writes: > For guality, the most effective test for regressions is simply always > running contrib/test_summary after all your bootstraps and then just > diffing up that against the same from earlier bootstrap. Or use contrib/compare_tests. Andreas. -- Andreas

Re: [PATCH] Disable guality tests for powerpc*-linux*

On Mon, Mar 28, 2016 at 07:38:46PM -0500, Bill Schmidt wrote: > For a long time we've had hundreds of failing guality tests. These > failures don't seem to have any correlation with gdb functionality for > POWER, which is working fine. At this point the value of these tests to > us seems

Re: [PATCH, lower-subreg] Fix 70355

On Mon, Mar 28, 2016 at 07:57:54PM -0700, Richard Henderson wrote: > The ICE comes fixing up a debug_insn, and the debug info incoming to the > pass seems reasonable. Just recognizing that this situation is possible and > not asserting appears to work. > > Ok? > > > r~ > *