Re: [Patch, avr] Fix PR65210

Senthil Kumar Selvaraj schrieb: This (rather trivial) patch fixes PR65210. The ICE happens because code wasn't handling io_low attribute where it is supposed to. Hi Senthil, could you line out for what these new attributes are good for? The Compiler just maps the argument to a

Re: [PATCH][AArch64] Improve code generation for float16 vector code

On Tue, Sep 08, 2015 at 09:21:08AM +0100, James Greenhalgh wrote: > On Mon, Sep 07, 2015 at 02:09:01PM +0100, Alan Lawrence wrote: > > On 04/09/15 13:32, James Greenhalgh wrote: > > > In that case, these should be implemented as inline assembly blocks. As it > > > stands, the code generation for

Re: [PATCH 11/15][AArch64] vreinterpret(q?), vget_(low|high), vld1(q?)_dup

On Tue, Jul 28, 2015 at 12:26:22PM +0100, Alan Lawrence wrote: > gcc/ChangeLog: > > * config/aarch64/arm_neon.h (vreinterpret_p8_f16, vreinterpret_p16_f16, > vreinterpret_f16_f64, vreinterpret_f16_s8, vreinterpret_f16_s16, > vreinterpret_f16_s32, vreinterpret_f16_s64,

Re: [PATCH][RTL-ifcvt] PR rtl-optimization/67465: Do not ifcvt complex blocks if the else block is empty

On 07/09/15 20:14, H.J. Lu wrote: On Mon, Sep 7, 2015 at 9:29 AM, Kyrill Tkachov wrote: Hi all, This patch fixes the PRs in the ChangeLog that have been reported against my if-conversion patch. The problem occurs when the 'then' block is complex but the else block is

Re: [PATCH] PR target/67480: AVX512 bitwise logic insns pattern is incorrect

Hi, On 07 Sep 19:07, Alexander Fomin wrote: > (define_insn "3" > - [(set (match_operand:VI 0 "register_operand" "=x,v") > - (any_logic:VI > - (match_operand:VI 1 "nonimmediate_operand" "%0,v") > - (match_operand:VI 2 "nonimmediate_operand" "xm,vm")))] > + [(set

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

Hi! This patch does two things: 1) removes fatal error from #pragma omp target update if object is not mapped (at all, partial mapping is still a fatal error); the 4.1 draft spec says that nothing is copied if the object is not mapped (first hunk) 2) implements nowait support for #pragma omp

Re: [Patch, avr] Fix PR65210

On Tue, Sep 08, 2015 at 09:16:35AM +0200, Georg-Johann Lay wrote: > Senthil Kumar Selvaraj schrieb: > > This (rather trivial) patch fixes PR65210. The ICE happens because code > > wasn't handling io_low attribute where it is supposed to. > > Hi Senthil, could you line out for what these new

Re: [0/7] Type promotion pass and elimination of zext/sext

Hi Andrew, Previously, there is a discussion thread in binutils mailing list: https://sourceware.org/ml/binutils/2015-04/msg00032.html Nick proposed a way to fix, Richard Henderson hold similar opinion as you. Regards, Renlin On 07/09/15 12:45, pins...@gmail.com wrote: On Sep 7, 2015,

Re: [PATCH][AArch64][1/3] Expand signed mod by power of 2 using CSNEG

On Wed, Sep 02, 2015 at 02:00:23PM +0100, Kyrill Tkachov wrote: > > On 01/09/15 11:40, Kyrill Tkachov wrote: > > Hi James, > > > > On 01/09/15 10:25, James Greenhalgh wrote: > >> On Thu, Aug 13, 2015 at 01:36:50PM +0100, Kyrill Tkachov wrote: > >> > >> Some comments below. > > Thanks, I'll

Re: [PATCH][AArch64] Improve code generation for float16 vector code

On Mon, Sep 07, 2015 at 02:09:01PM +0100, Alan Lawrence wrote: > On 04/09/15 13:32, James Greenhalgh wrote: > > In that case, these should be implemented as inline assembly blocks. As it > > stands, the code generation for these intrinsics will be very poor with this > > patch applied. > > > > I'm

Re: [PATCH][RTL-ifcvt] PR rtl-optimization/67465: Do not ifcvt complex blocks if the else block is empty

Hi Kyrill, > PR rtl-optimization/67481 is a testsuite regression on sparc-solaris that > Rainer reported. I haven't tested > that this patch fixes that, but I suspect that the root cause is the > same. Rainer, could you please > check that this fixes the regression for you? I've now checked that

Re: [PATCH][ARM][3/3] Expand mod by power of 2

On 07/09/15 10:27, Ramana Radhakrishnan wrote: On 24/07/15 11:55, Kyrill Tkachov wrote: commit d562629e36ba013b8f77956a74139330d191bc30 Author: Kyrylo Tkachov Date: Fri Jul 17 16:30:01 2015 +0100 [ARM][3/3] Expand mod by power of 2 diff --git

[RFC] Prevent unnecessary recompilation for trivial params.def changes

Hi, this patch adds generation of params.list, a file containing a list of names of all the parameters in params.def. By including params.list in params.h, rather than params.def itself, we prevent recompiling the 118 c files that include params.h for trivial changes in params.def, such as

Re: [PATCH 13/15][ARM/AArch64 Testsuite] Add float16 tests to advsimd-intrinsics testsuite

On 08/09/15 11:55, Kyrill Tkachov wrote: Hi all, On 08/09/15 11:52, Alan Lawrence wrote: Ping. (Thanks, Christophe!). Original message: https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02366.html On 25/08/15 14:28, Alan Lawrence wrote: Christophe Lyon wrote: On 28 July 2015 at 13:26, Alan

Re: [PATCH 14/15][ARM/AArch64 Testsuite]Add test of vcvt{,_high}_i{f32_f16,f16_f32}

On 08/09/15 11:56, Alan Lawrence wrote: Ping. (Thanks, Christophe!) Correct version here: https://gcc.gnu.org/ml/gcc-patches/2015-08/msg01501.html Cheers, Alan On 25/08/15 15:21, Christophe Lyon wrote: On 25 August 2015 at 15:57, Alan Lawrence wrote: Sorry - wrong

Re: [RFC] Prevent unnecessary recompilation for trivial params.def changes

On 08/09/15 13:00, Andreas Schwab wrote: Tom de Vries writes: @@ -3236,6 +3236,10 @@ installdirs: $(mkinstalldirs) $(DESTDIR)$(man1dir) $(mkinstalldirs) $(DESTDIR)$(man7dir) +params.list: $(srcdir)/params-list.h $(srcdir)/params.def + $(CPP)

[PATCH] Fix seq_cost prototype to use signed int

All other cost helper functions are using signed int to hold cost while seq_cost is using unsigned int. This fix this. bootstrap OK on x86. OK for trunk? 2015-09-08 Jiong Wang gcc/ * rtl.h (seq_cost): Change return type from "unsigned" to "int". * rtlanal.c

Re: [PATCH 2/5] completely_scalarize arrays as well as records.

Hi, On Mon, Sep 07, 2015 at 02:15:45PM +0100, Alan Lawrence wrote: > In-Reply-To: <55e0697d.2010...@arm.com> > > On 28/08/15 16:08, Alan Lawrence wrote: > > Alan Lawrence wrote: > >> > >> Right. I think VLA's are the problem with pr64312.C also. I'm testing a fix > >> (that declares arrays with

Re: [PATCH PR66388]Add sizetype cand for BIV of smaller type if it's used as index of memory ref

On Wed, Sep 2, 2015 at 10:12 PM, Richard Biener wrote: > On Wed, Sep 2, 2015 at 5:26 AM, Bin Cheng wrote: >> Hi, >> This patch is a new approach to fix PR66388. IVO today computes iv_use with >> iv_cand which has at least same type precision as the

[patch match.pd c c++]: Ignore results of 'shorten_compare' and move missing patterns in match.pd

Hi, This patch is the first part of obsoleting 'shorten_compare' function for folding. It adjusts the uses of 'shorten_compare' to ignore folding returned by it, and adds missing pattterns to match.pd to allow full bootstrap of C/C++ without regressions. Due we are using 'shorten_compare' for

Re: [RFC] Prevent unnecessary recompilation for trivial params.def changes

Tom de Vries writes: > After a subsequent rebuild I don't see anything being rebuild. So I don't > observe 'continuous rebuilding'. What happens when you just touch params-list.h or params.def? move-if-change will leave the target untouched when unchanged (that's the

[PATCH, fortran] PR 53379 Backtrace on error termination

libbacktrace is not supported on AIX. This patch breaks bootstrap on AIX. It's okay if Fortran backtrace does not work on AIX, but not all targets support libbacktrace. Thanks, David

Re: [RS6000] Fix PowerPC ICE due to secondary_reload ignoring reload replacements

On Mon, Sep 7, 2015 at 11:47 PM, Alan Modra wrote: > In https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67378 analysis I show > the reason for this PR is that insns emitted by secondary reload > patterns are being generated without taking into account other reloads > that may have

Re: [RFC] Try vector as a new representation for vector masks

2015-09-04 23:42 GMT+03:00 Jeff Law : > On 09/01/2015 07:08 AM, Ilya Enkovich wrote: >> >> On 27 Aug 09:55, Richard Biener wrote: >>> >>> I suggest you try modifying those parts first according to this >>> scheme that will most likely uncover issues we missed. >>> >>> Thanks,

Re: [RS6000] Fix PowerPC ICE due to secondary_reload ignoring reload replacements

David Edelsohn wrote: > On Mon, Sep 7, 2015 at 11:47 PM, Alan Modra wrote: > > In https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67378 analysis I show > > the reason for this PR is that insns emitted by secondary reload > > patterns are being generated without taking into account

Re: [RFC] Prevent unnecessary recompilation for trivial params.def changes

Tom de Vries writes: > @@ -3236,6 +3236,10 @@ installdirs: > $(mkinstalldirs) $(DESTDIR)$(man1dir) > $(mkinstalldirs) $(DESTDIR)$(man7dir) > > +params.list: $(srcdir)/params-list.h $(srcdir)/params.def > + $(CPP) $(srcdir)/params-list.h | sed

Re: [PATCH 14/15][ARM/AArch64 Testsuite]Add test of vcvt{,_high}_i{f32_f16,f16_f32}

Ping. (Thanks, Christophe!) Correct version here: https://gcc.gnu.org/ml/gcc-patches/2015-08/msg01501.html Cheers, Alan On 25/08/15 15:21, Christophe Lyon wrote: On 25 August 2015 at 15:57, Alan Lawrence wrote: Sorry - wrong version posted. The hunk for

Re: [PATCH 13/15][ARM/AArch64 Testsuite] Add float16 tests to advsimd-intrinsics testsuite

Hi all, On 08/09/15 11:52, Alan Lawrence wrote: Ping. (Thanks, Christophe!). Original message: https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02366.html On 25/08/15 14:28, Alan Lawrence wrote: Christophe Lyon wrote: On 28 July 2015 at 13:26, Alan Lawrence wrote: This

Re: [PATCH] New attribute to create target clones

Ping. On Thu, Aug 27, 2015 at 2:18 PM, Evgeny Stupachenko wrote: > Hi All, > > Based on RFC: > https://gcc.gnu.org/ml/gcc-patches/2015-08/msg01322.html > > The patch implement an extension to Function Multiversioning that > allows to clone a function for multiple targets. >

RE: [0/7] Type promotion pass and elimination of zext/sext

> Renlin Li wrote: > Hi Andrew, > > Previously, there is a discussion thread in binutils mailing list: > > https://sourceware.org/ml/binutils/2015-04/msg00032.html > > Nick proposed a way to fix, Richard Henderson hold similar opinion as you. Both Nick and Richard H seem to think it is an

Re: [PATCH PR66388]Add sizetype cand for BIV of smaller type if it's used as index of memory ref

On Tue, Sep 8, 2015 at 6:06 PM, Bin.Cheng wrote: > On Wed, Sep 2, 2015 at 10:12 PM, Richard Biener > wrote: >> On Wed, Sep 2, 2015 at 5:26 AM, Bin Cheng wrote: >>> Hi, >>> This patch is a new approach to fix PR66388. IVO

Re: [PATCH 13/15][ARM/AArch64 Testsuite] Add float16 tests to advsimd-intrinsics testsuite

Ping. (Thanks, Christophe!). Original message: https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02366.html On 25/08/15 14:28, Alan Lawrence wrote: Christophe Lyon wrote: On 28 July 2015 at 13:26, Alan Lawrence wrote: This is a respin of

Re: [PATCH][AArch64] Improve code generation for float16 vector code

On 08/09/15 09:26, James Greenhalgh wrote: On Tue, Sep 08, 2015 at 09:21:08AM +0100, James Greenhalgh wrote: On Mon, Sep 07, 2015 at 02:09:01PM +0100, Alan Lawrence wrote: On 04/09/15 13:32, James Greenhalgh wrote: In that case, these should be implemented as inline assembly blocks. As it

[PATCH] Implement GOMP_OFFLOAD_unload_image in intelmic plugin

Hi! This patch supports unloading of target images from the device. Unfortunately __offload_unregister_image requires the whole descriptor for unloading, which must contain target code inside, for this reason the plugin keeps descriptors for all offloaded images in memory. Also the patch removes

[hsa] HSA: reuse get_integer_type_by_bytes

Hello. Following patch is a small clean-up which reuses already existing function. Martin >From e592465b575c477f9311eba2e66e9bf1ec4c54fc Mon Sep 17 00:00:00 2001 From: mliska Date: Fri, 4 Sep 2015 14:38:13 +0200 Subject: [PATCH 3/7] HSA: reuse get_integer_type_by_bytes.

[hsa] Use newly added hsa_op_immed ctor.

Hello. Following patch uses new hsa_op_immed ctor, which is mainly used in HSAIL emission of instructions that generate kernel from kernel dispatching. Martin >From 7cf50d2831ec8b161858ecdbfb19b6287230384d Mon Sep 17 00:00:00 2001 From: mliska Date: Fri, 4 Sep 2015 14:19:25

[PATCH, committed] Trivial typo fix in pretty-print.h

"pinter" -> "printer" Committed to trunk as r227562, under the "obvious" rule. gcc/ChangeLog: * pretty-print.h (printer_fn): Fix typo in comment. --- gcc/pretty-print.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/pretty-print.h b/gcc/pretty-print.h index

[gomp4] force global locks for nvptx targets

This patch forces GOACC_LOCK to use locks in global memory regardless if the lock us for a worker or a gang. We were using a shared memory for worker locks, but we ran into an issue with that would sporadically involve deadlocks in worker reductions. We're still investigating that issue, but for

Re: [PATCH] Fix PR64078

On 09/07/2015 07:46 AM, Bernd Edlinger wrote: Hi, On Mon, 7 Sep 2015 12:07:00, Marek Polacek wrote: On Sun, Sep 06, 2015 at 07:21:13PM +0200, Bernd Edlinger wrote: Hi, we observed sporadic failures of the following two test cases (see PR64078): c-c++-common/ubsan/object-size-9.c and

[hsa] Introduce a new ctor for hsa_op_immed.

Hello. Following patch adds a new ctor for hsa_op_immed class. Martin >From dc72ba0001f5b98c97da9ea5c03fccf2340e893d Mon Sep 17 00:00:00 2001 From: mliska Date: Fri, 4 Sep 2015 13:33:08 +0200 Subject: [PATCH 1/7] HSA: introduce a new ctor for hsa_op_immed. gcc/ChangeLog:

[Ping] [C++ Patch] PR 53184 ("Unnecessary anonymous namespace warnings")

Hi, On 08/24/2015 02:55 PM, Paolo Carlini wrote: Hi, today I spent more time on this issue (which is getting duplicates). In fact it boils down to two separate issues: 1- Give a name to the warning, to make possible disabling it. 2- Don't talk about "anonymous namespace" when no anonymous

C++ PATCH for c++/67041 (auto variable template and lambda)

If the lambda mangling scope is a variable, we don't want to call tsubst_copy on it, as that will call mark_used. Use plain tsubst instead. Tested x86_64-pc-linux-gnu, applying to trunk. commit 60cf545572ad4214d70e99808f53312db592e341 Author: Jason Merrill Date: Tue Sep 8

Re: [patch] Enable lightweight checks with _GLIBCXX_ASSERTIONS.

On 07/09/2015 20:27, Jonathan Wakely wrote: > This patch adds the "debug mode lite" we've been talking about, by > changing __glibcxx_assert to be activated by _GLIBCXX_ASSERTIONS > instead of _GLIBCXX_DEBUG (and making the latter imply the former). > > _GLIBCXX_ASSERTIONS is already used in

[hsa] Add support for __builtin_mem{set,cpy}.

Hello. Following patch adds support for HSAIL emission on the pair of builtins. Martin >From 4bb6a85f805dd7cab4f4a2ce7f148803214e5a9e Mon Sep 17 00:00:00 2001 From: mliska Date: Fri, 4 Sep 2015 16:29:21 +0200 Subject: [PATCH 4/7] HSA: add support for __builtin_mem{set,cpy}.

Re: debug mode symbols cleanup

On 07/09/2015 13:03, Jonathan Wakely wrote: > On 05/09/15 22:53 +0200, François Dumont wrote: >>I remember Paolo saying once that we were not guarantiing any abi >> compatibility for debug mode. I haven't found any section for >> unversioned symbols in gnu.ver so I simply uncomment the global

Re: [PATCH] Prevent unnecessary recompilation for trivial params.def changes

On 09/08/2015 07:21 AM, Tom de Vries wrote: [ was: Re: [RFC] Prevent unnecessary recompilation for trivial params.def changes ] On 08/09/15 14:03, Andreas Schwab wrote: Tom de Vries writes: After a subsequent rebuild I don't see anything being rebuild. So I don't

[PATCH] Make tsan tests less picky about ansi escape codes in diagnostics.

In a similar vein to [1], this does the same for the tsan tests. These tests also suffer from being overly picky w.r.t. ansi escape codes in the runtime diagnostics (which can't be suppressed as explained in [2]). Tested on a remote x86_64-linux-gnu target. 2015-09-08 Jonathan Roelofs

[PATCH] [graphite] Remove limit_scops

This patch removes graphite-scop-detection.c:limit_scops function and fix related issues arising because of that. The functionality limit_scop was added as an intermediate step to discard the loops which graphite could not handle. Removing limit_scop required handling of different cases of loops

Re: [Patch, fortran] PR66681 - Wrong result in assigning this_image() to a complex coarray

> This is something of a corner case, where gfc_conv_expr comes back > with a SAVE_EXPR, in the case of complex, scalar, coarray lvalues. The > first field of the SAVE_EXPR is a perfectly viable expression to > assign to, so I have taken that. If anybody out there has a better > solution, please

Re: [4/7] Use correct promoted mode sign for result of GIMPLE_CALL

On 09/08/2015 08:39 AM, Jeff Law wrote: > Is this another instance of the PROMOTE_MODE issue that was raised by > Jim Wilson a couple months ago? It looks like a closely related problem. The one I am looking at has confusion with a function arg and a local variable as they have different sign

Re: [PATCH] Make ubsan tests less picky about ansi escape codes in diagnostics.

On 9/4/15 12:20 AM, Yury Gribov wrote: On 09/03/2015 07:45 PM, Jonathan Roelofs wrote: On 9/3/15 10:17 AM, Jakub Jelinek wrote: On Thu, Sep 03, 2015 at 10:15:02AM -0600, Jonathan Roelofs wrote: +kcc, mrs Ping On 8/27/15 4:44 PM, Jonathan Roelofs wrote: The attached patch makes the

Re: [PATCH] Make tsan tests less picky about ansi escape codes in diagnostics.

On 9/8/15 4:09 PM, Jonathan Roelofs wrote: In a similar vein to [1], this does the same for the tsan tests. These tests also suffer from being overly picky w.r.t. ansi escape codes in the runtime diagnostics (which can't be suppressed as explained in [2]). Tested on a remote

Re: [patch] Enable lightweight checks with _GLIBCXX_ASSERTIONS.

Hi, On Mon, 7 Sep 2015, Jonathan Wakely wrote: > > Interesting. Is this mode ABI-compatible with the default mode? > > Yes, that's the main reason I want to make this change. > > > Should _FORTIFY_SOURCE imply _GLIBCXX_ASSERTIONS? > > Yes, I think it should. Then at least those assertions

Re: [PATCH, fortran] PR 53379 Backtrace on error termination

On Tue, Sep 8, 2015 at 9:51 AM, FX wrote: >> #define _FCLOEXEC 0x0010L >> #define O_CLOEXEC _FCLOEXEC /* sets FD_CLOEXEC on open */ > > That’s weird, and definitely an AIX bug: >

[C++ Patch] PR 67369 ("[5/6 Regression] ICE (in tsubst_decl, at cp/pt.c:11302) with -std=c++14")

Hi, in this regression, an ICE is triggered in tsubst_decl, [case FUNCTION_DECL] at: /* Nobody should be tsubst'ing into non-template functions. */ gcc_assert (DECL_TEMPLATE_INFO (t) != NULL_TREE); indeed, 't' is just the 'main' function. A simple way to avoid it is tweaking the

Re: [PATCH] Import liboffloadmic from upstream

On Mon, Aug 31, 2015 at 20:07:49 +0200, Jakub Jelinek wrote: > On Mon, Aug 31, 2015 at 08:56:58PM +0300, Ilya Verbin wrote: > > On Mon, Aug 31, 2015 at 16:49:53 +0200, Jakub Jelinek wrote: > > > 1) Is the library backwards ABI compatible? Can you run e.g. > > >libabigail abidiff in between

Re: [PATCH, fortran] PR 53379 Backtrace on error termination

On Tue, Sep 8, 2015 at 9:40 AM, David Edelsohn wrote: > On Tue, Sep 8, 2015 at 9:15 AM, FX wrote: >>> libbacktrace is not supported on AIX. This patch breaks bootstrap on AIX. >>> It's okay if Fortran backtrace does not work on AIX, but not all >>>

libbacktrace patch committed: fix test for mmap failure

PR 67457 points out a crash in libbacktrace when there is no memory available. This is because the code testing the mmap result for failure is broken. This patch fixes it. Bootstrapped and ran libbacktrace tests. Committed to mainline. Ian 2015-09-08 Ian Lance Taylor PR

Re: [PATCH, fortran] PR 53379 Backtrace on error termination

> /home/dje/src/src/libbacktrace/posix.c: In function 'backtrace_open': > /home/dje/src/src/libbacktrace/posix.c:67:32: error: overflow in > implicit constant conversion [-Werror=overflow] > descriptor = open (filename, O_RDONLY | O_BINARY | O_CLOEXEC); ?? I have a hard time understanding how

Re: [PATCH, libiberty] Fix PR63758 by using the _NSGetEnviron() API on Darwin.

On Tue, Sep 8, 2015 at 7:20 AM, Iain Sandoe wrote: > >> This seems likely to break cross-compilers to Darwin that do not have >> the system libraries available. I guess I don't care about that if >> you don't. > > I do care about it, but I'm not visualising the case... > >

[PATCH] Prevent unnecessary recompilation for trivial params.def changes

[ was: Re: [RFC] Prevent unnecessary recompilation for trivial params.def changes ] On 08/09/15 14:03, Andreas Schwab wrote: Tom de Vries writes: After a subsequent rebuild I don't see anything being rebuild. So I don't observe 'continuous rebuilding'. What happens

Re: [PATCH] Import liboffloadmic from upstream

On Tue, Sep 08, 2015 at 04:40:22PM +0300, Ilya Verbin wrote: > Looks like this is the only incompatible change. Given that the library is > used > only by libgomp plugin, this isn't a big problem. I will add a comment into > plugin prohibiting the use of the return value. In fact, if something

Re: [PATCH, libiberty] Fix PR63758 by using the _NSGetEnviron() API on Darwin.

On Mon, Sep 7, 2015 at 8:23 AM, Iain Sandoe wrote: > > include/ > > Roland McGrath > > PR other/63758 > * environ.h: New file. > > libiberty/ > > Roland McGrath > Iain Sandoe

Re: [PATCH 15/15][ARM] Update sourcebuild.texi with testsuite/effective-target hooks

Original message here: https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02363.html On 28/07/15 12:27, Alan Lawrence wrote: > This documents the change to arm_neon_fp16_ok in the first patch; the addition > of arm_neon_fp16_hw_ok in the last patch; and corrects a cross-reference. > > (I tried using

Re: [PATCH, fortran] PR 53379 Backtrace on error termination

> libbacktrace is not supported on AIX. This patch breaks bootstrap on AIX. > It's okay if Fortran backtrace does not work on AIX, but not all > targets support libbacktrace. libbacktrace is designed to be compiled on all targets. Some targets offer full support, some offer nothing, but

Re: [PATCH, fortran] PR 53379 Backtrace on error termination

On Tue, Sep 8, 2015 at 9:15 AM, FX wrote: >> libbacktrace is not supported on AIX. This patch breaks bootstrap on AIX. >> It's okay if Fortran backtrace does not work on AIX, but not all >> targets support libbacktrace. > > libbacktrace is designed to be compiled on all

Re: libbacktrace patch committed: Graceful fallback if out of memory

> From: Ian Lance Taylor > Date: Tue, 8 Sep 2015 18:46:21 +0200 > PR other/67457 > * backtrace.c: #include "internal.h". > (struct backtrace_data): Add can_alloc field. > (unwind): If can_alloc is false, don't try to get file/line > information. > (backtrace_full): Set

[AArch64] Delete aarch64_symbol_context which is not used

The concept of aarch64_symbol_context is not used in AArch64, this patch remove it and all relevant code. ok for trunk? 2015-09-08 Jiong. Wang gcc/ * config/aarch64/aarch64-protos.h (aarch64_symbol_context): Delete. * config/aarch64/aarch64.c

Re: [PATCH] Fix seq_cost prototype to use signed int

Jeff Law writes: > On 09/08/2015 06:17 AM, Jiong Wang wrote: >> >> All other cost helper functions are using signed int to hold cost >> while seq_cost is using unsigned int. >> >> This fix this. bootstrap OK on x86. >> >> OK for trunk? >> >> 2015-09-08 Jiong Wang >> >>

Re: [RFC, PR target/65105] Use vector instructions for scalar 64bit computations on 32bit target

On 21 Aug 10:38, Jeff Law wrote: > On 08/21/2015 07:44 AM, Ilya Enkovich wrote: > >>Our of curiosity, what does LLVM do here in terms of costing > >>models? > > > >Unfortunately I have no idea where and how LLVM does this > >optimization. Will try to find out. For now I just try to follow a >

Re: [PATCH, fortran] PR 53379 Backtrace on error termination

> Not a big issue, but slightly better if (O_CLOEXEC>>32) != 0 is also true. > See, if AIX should ever define this to a sensible value, the above would > disappear the feature. However, if they did, then this expression should > then be false. Sounds good. This being a libbacktrace patch,

[Patch] Teach RTL ifcvt to handle multiple simple set instructions

Hi, RTL "noce" ifcvt will currently give up if the branches it is trying to make conditional are too complicated. One of the conditions for "too complicated" is that the branch sets more than one value. One common idiom that this misses is something like: int d = a[i]; int e = b[i]; if

Re: [patch] Enable lightweight checks with _GLIBCXX_ASSERTIONS.

On 09/07/2015 09:59 PM, Jonathan Wakely wrote: > On 07/09/15 21:31 +0200, Florian Weimer wrote: >> * Jonathan Wakely: >> >>> This patch adds the "debug mode lite" we've been talking about, by >>> changing __glibcxx_assert to be activated by _GLIBCXX_ASSERTIONS >>> instead of _GLIBCXX_DEBUG (and

Re: [PATCH] Fix seq_cost prototype to use signed int

On 09/08/2015 06:17 AM, Jiong Wang wrote: All other cost helper functions are using signed int to hold cost while seq_cost is using unsigned int. This fix this. bootstrap OK on x86. OK for trunk? 2015-09-08 Jiong Wang gcc/ * rtl.h (seq_cost): Change return type

Re: [patch] Enable lightweight checks with _GLIBCXX_ASSERTIONS.

On 08/09/15 17:00 +0200, Florian Weimer wrote: On 09/07/2015 09:59 PM, Jonathan Wakely wrote: On 07/09/15 21:31 +0200, Florian Weimer wrote: * Jonathan Wakely: This patch adds the "debug mode lite" we've been talking about, by changing __glibcxx_assert to be activated by _GLIBCXX_ASSERTIONS

Re: [4/7] Use correct promoted mode sign for result of GIMPLE_CALL

On 09/07/2015 03:27 PM, Kugan wrote: On 07/09/15 23:10, Michael Matz wrote: Hi, On Mon, 7 Sep 2015, Kugan wrote: For the following testcase (compiling with -O1; -O2 works fine), we have a stmt with stm_code SSA_NAME (_7 = _ 6) and for which _6 is defined by a GIMPLE_CALL. In this case, we

Re: libbacktrace patch committed: Graceful fallback if out of memory

On Tue, Sep 8, 2015 at 5:00 PM, Hans-Peter Nilsson wrote: > > I've committed the following as obvious, following the pattern > of the other files including internal.h, after observing > all-libbacktrace (i.e. built for the host) complete. > > libbacktrace: > *

Re: [PATCH, fortran] PR 53379 Backtrace on error termination

Mike Stump writes: > Not a big issue, but slightly better if (O_CLOEXEC>>32) != 0 is also > true. See, if AIX should ever define this to a sensible value, the > above would disappear the feature. However, if they did, then this > expression should then be false. Yes, I

Re: [PATCH][RTL-ifcvt] PR rtl-optimization/67465: Do not ifcvt complex blocks if the else block is empty

On 08/09/15 10:26, Rainer Orth wrote: Hi Kyrill, PR rtl-optimization/67481 is a testsuite regression on sparc-solaris that Rainer reported. I haven't tested that this patch fixes that, but I suspect that the root cause is the same. Rainer, could you please check that this fixes the regression

Re: [PATCH] Fix seq_cost prototype to use signed int

On 09/08/2015 09:17 AM, Jiong Wang wrote: Jeff Law writes: On 09/08/2015 06:17 AM, Jiong Wang wrote: All other cost helper functions are using signed int to hold cost while seq_cost is using unsigned int. This fix this. bootstrap OK on x86. OK for trunk? 2015-09-08 Jiong Wang

Re: [PATCH, fortran] PR 53379 Backtrace on error termination

On Sep 8, 2015, at 6:53 AM, David Edelsohn wrote: > On Tue, Sep 8, 2015 at 9:51 AM, FX wrote: >>> #define _FCLOEXEC 0x0010L >>> #define O_CLOEXEC _FCLOEXEC /* sets FD_CLOEXEC on open */ >> >> That’s weird, and definitely

Re: [patch] Enable lightweight checks with _GLIBCXX_ASSERTIONS.

On 08/09/15 15:14 +0200, Michael Matz wrote: Hi, On Mon, 7 Sep 2015, Jonathan Wakely wrote: > Interesting. Is this mode ABI-compatible with the default mode? Yes, that's the main reason I want to make this change. > Should _FORTIFY_SOURCE imply _GLIBCXX_ASSERTIONS? Yes, I think it should.

Re: [PATCH, rs6000] Add memory barriers to tbegin, tend, etc.

On Thu, Sep 3, 2015 at 5:58 PM, Peter Bergner wrote: > While debugging a transaction lock elision issue, we noticed that the > compiler was moving some loads/stores outside of the transaction body, > because the HTM instructions were not marked as memory barriers, which > is

[AArch64] Handle const address in aarch64_print_operand

I came across this issue when I was cleaning up some releveant TLS code. Currently, unlike other TLS modes, TLS local executable addresses haven't been through the same aarch64 address split logic which force the address into base + offset. This actually is good. The address assigned to TLS

Re: [PATCH, Darwin] Some driver TLC (improve support for the '-arch' flag).

On Sep 7, 2015, at 11:24 AM, Iain Sandoe wrote: > For some Darwin compilers, "-arch " can be used (a) in place of, but to > indicate the same as, a multilib flag like "-m32" and (b) multiple times to > indicate that the User wants a FAT object with multiple arch

Re: [C++ Patch] PR 67369 ("[5/6 Regression] ICE (in tsubst_decl, at cp/pt.c:11302) with -std=c++14")

OK. Jason

Re: [PATCH, fortran] PR 53379 Backtrace on error termination

> #define _FCLOEXEC 0x0010L > #define O_CLOEXEC _FCLOEXEC /* sets FD_CLOEXEC on open */ That’s weird, and definitely an AIX bug: http://pubs.opengroup.org/onlinepubs/9699919799/functions/open.html How does that even work? open() takes int as second arg. FX

Re: [PATCH 15/15][ARM] Update sourcebuild.texi with testsuite/effective-target hooks

On 08/09/15 15:00, Alan Lawrence wrote: Original message here: https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02363.html On 28/07/15 12:27, Alan Lawrence wrote: This documents the change to arm_neon_fp16_ok in the first patch; the addition of arm_neon_fp16_hw_ok in the last patch; and corrects

Re: [PATCH, libiberty] Fix PR63758 by using the _NSGetEnviron() API on Darwin.

On 8 Sep 2015, at 15:00, Ian Lance Taylor wrote: > On Mon, Sep 7, 2015 at 8:23 AM, Iain Sandoe wrote: >> >> include/ >> >>Roland McGrath >> >>PR other/63758 >>* environ.h: New file. >> >> libiberty/ >> >>Roland

Re: [wwwdocs] Document some gcc-6 changes

On Tue, 8 Sep 2015, Manuel López-Ibáñez wrote: > I also took the liberty of rewriting the list of new command-line > options to be less repetitive. Nice! (Aka "Thanks for doing this, please go ahead and commit.") There is one question, but that is not introduced by your change: Does the

libbacktrace patch committed: Graceful fallback if out of memory

I've committed this libbacktrace patch to mainline to do a graceful fallback if no memory can be allocated. In that case we print out the PC addresses without trying to resolve file/line information. This is imperfect but better than the earlier behaviour of producing a series of error messages.

[wwwdocs] Document some gcc-6 changes

I also took the liberty of rewriting the list of new command-line options to be less repetitive. OK? Index: htdocs/gcc-6/changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-6/changes.html,v retrieving revision 1.26 diff -u

Re: [wwwdocs] Document some gcc-6 changes

On 09/08/2015 11:14 AM, Manuel López-Ibáñez wrote: I also took the liberty of rewriting the list of new command-line options to be less repetitive. ... a negative value. -A new command-line option -Wshift-overflow has been - added for the C and C++ compilers, which warns

Re: [wwwdocs] Document some gcc-6 changes

On 8 September 2015 at 19:26, Martin Sebor wrote: > On 09/08/2015 11:14 AM, Manuel López-Ibáñez wrote: >> a negative value. >> -A new command-line option -Wshift-overflow has been >> - added for the C and C++ compilers, which warns about left shift >> +

[PATCH] Convert SPARC to LRA

The following patch converts the sparc backend over to LRA. The three major obstacles to overcome were: 1) The funky "U" constraint. It was a register constraint, but did not evaluate to a register class, and was used to help handling unaligned integer register pairs. It turns out to