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 compile-tim
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 attr
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, at
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
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 th
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, vreinterpre
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 a/gcc/config/arm/arm.c b/gcc/config/arm/a
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 incorpo
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 (match_operand:VI4
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
empty.
In this case the c
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 targ
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
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 use. On 64bit
>> platforms like AArch64, this
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 today computes iv_use with
>>> iv_cand which has at least same type precis
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 f
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
https://gcc.gnu.org/ml/gcc-patches/2015-07/msg00488.html,
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 is a respin of
https://
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 add_options_for_arm_neon_fp16 has
move
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 's/^#.*//;/^$$/d' > tmp-params.list
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 Law
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 version posted. The hunk
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 some
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) $(srcdir)/params-list.h | sed
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.
> __attribute__((target_
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
stan
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 whole point of it), so it wi
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 (seq_cost): Likewise.
--
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 occurred. We run i
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
> 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 issue
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, Richard.
>>>
>>
>> I t
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 a
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 other reloads
> > t
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 cod
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 th
> 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 libbackt
[ 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 when you just touch para
OK.
Jason
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 the
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 targets. Some targets offe
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
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
>>> targets support libbacktrace.
>>
>> libbacktrac
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 other/67457
* mma
> /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 th
> #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
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:
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/open.html
Welcome to AIX
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
>
> PR other/63758
> * pex-unix.c: Obtain the environment int
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
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 an
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 McGrath
>>Iain Sandoe
>>
>>
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...
>
> AFAICS, when built as a
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.
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 bad. Looking deeper,
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 (d
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 mak
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 from "unsigned" to "int"
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 (aarch64_expand_mov_immediate): Likewise.
(
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 vari
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 slices.
> OK for trunk?
Ok.
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
>>
>> gcc/
>>* rtl.h (seq_
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
gcc/
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 an AIX bug:
>> http://pubs.opengroup.or
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
i
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 a
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
> >comm
> 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, you
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
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.
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 -r1.26
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 language
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 abo
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
>> +-Wshift-overflow warns
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 c-c++-co
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 09:47:18 2015 -0400
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 u
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 Parall
"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 6e8a3
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 th
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:
2015-09-04 Martin Lis
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 +0200
Subject: [PATC
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.
gcc/ChangeLog:
2015-09-
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}.
gcc/ChangeLog:
2015
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 nam
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 e
> 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 sp
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
observe 'continuous rebuildi
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 a
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 ext
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
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 ubsan
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 x86_64-linux-gnu
> 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 can_alloc field in bdata.
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:
> * backtrace.c: #include .
Than
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 think this might be even
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
95 matches
Mail list logo
