On 2 October 2016 at 23:05, Doug Gilmore wrote:
> Hi Christophe,
>
>> From: Christophe Lyon [christophe.l...@linaro.org]
>> Sent: Saturday, October 01, 2016 7:57 AM
>> To: Doug Gilmore
>> Cc: gcc-patches@gcc.gnu.org
>> Subject: Re: Fix PR tree-optimization/77808, ICE in
>> duplicate_ssa_name_ptr_
On Fri, Sep 30, 2016 at 05:57:45PM +, Joseph Myers wrote:
> On Fri, 30 Sep 2016, Jeff Law wrote:
>
> > On 09/30/2016 11:34 AM, Joseph Myers wrote:
> > > On Fri, 30 Sep 2016, James Greenhalgh wrote:
> > >
> > > > + case EXCESS_PRECISION_TYPE_STANDARD:
> > > > + case EXCESS_PRECISION_
On 08/18/2016 03:36 PM, Jakub Jelinek wrote:
> On Thu, May 12, 2016 at 04:12:21PM +0200, Martin Liška wrote:
>> --- a/gcc/asan.c
>> +++ b/gcc/asan.c
>> @@ -243,6 +243,11 @@ static unsigned HOST_WIDE_INT asan_shadow_offset_value;
>> static bool asan_shadow_offset_computed;
>> static vec sanitized_
Following patch adjusts expected test dumps and also introduces various
new tests.
Martin
>From 4ddafab1e533a1d3580d2f883955d61fe23aa353 Mon Sep 17 00:00:00 2001
From: marxin
Date: Mon, 19 Sep 2016 17:39:29 +0200
Subject: [PATCH 3/3] Introduce tests for -fsanitize-address-use-after-scope
gcc/tes
On 02/10/16 20:03, Andrew Pinski wrote:
On Sun, Oct 2, 2016 at 7:50 AM, Jeff Law wrote:
On 10/02/2016 04:48 AM, Andreas Schwab wrote:
This miscompiles the stage2 ada compiler.
No target identified.
He reported it in a bug report, aarch64-linux-gnu.
As I mentioned in PR 77816 I can't repr
On Mon, Oct 03, 2016 at 11:27:38AM +0200, Martin Liška wrote:
> > Plus, as I've mentioned before, it would be nice to optimize - for ASAN_MARK
> > unpoison appearing strictly before (i.e. dominating) the first (non-shadow)
> > memory read
> > or write in the function (explicit or possible through
On Fri, Sep 30, 2016 at 12:01:47PM +0200, Jose E. Marchesi wrote:
>
> In case anybody missed it, the Linux kernel side to make use
> of this has also been finished meanwhile. Of course it can not
> be accepted without compiler support; and this feature patch
> is much more versatil
Hello!
I plan to commit the attached patch later today.
2016-10-03 Uros Bizjak
* configure.ac (strict_warn): Merge -Wmissing-format-attribute and
-Woverloaded-virtual checks for warning options.
* configure: Regenerate.
Bootstrapped on x86_64-linux-gnu.
Uros.
diff --git a/gcc/c
On 01/10/16 00:12 +0300, Ville Voutilainen wrote:
I do this with a rather heavy heart, but since gcc6 compiles boost 1.62,
I'll rather have gcc7 do so as well, and I'll throw the tuple fix for lwg2729
to the wolves not because I want to, but because I have to.
Tested on Linux-x64.
2016-10-01 V
On 28/09/16 21:34 +0200, François Dumont wrote:
Hi
Here is the patch to share a mutex pool between debug mode and
shared_ptr implementation. It saves 392 bytes on generated .so and
will make sure that fixing false sharing will impact both usages.
I preferred to leave implementation in
PR libstdc++/77814
* include/bits/std_abs.h: Use "C++" language linkage.
* testsuite/17_intro/headers/c++2011/linkage.cc: Move to
the end. Add .
I'll commit to trunk when testing finishes.
commit 2dc6b0497b7d0ec0cb298f749419d70a43c2ab70
Author: Jonathan Wakely
D
Hi,
This patch adds -ffat-lto-objects option to an avr target testcase.
The compiler defaults to thin LTO objects if built with linker plugin
support, and the error expected by the testcase appears only at link
time, if at all. Forcing fat LTO object file creation generates the
error co
On 08/18/2016 05:53 PM, Jeff Law wrote:
> On 08/18/2016 09:51 AM, Andi Kleen wrote:
>>> I'd prefer to make updates atomic in multi-threaded applications.
>>> The best proxy we have for that is -pthread.
>>>
>>> Is it slower, most definitely, but odds are we're giving folks
>>> garbage data otherwis
On Thu, 29 Sep 2016, Louis Krupp wrote:
> My target was gfortran.
>
> In any case, someone else fixed this problem.
Good.
Note that by target we are referring to the platform (processor plus
operating system). You can see this by looking for a line started
with "Target:" in the output of `gcc -
On 10/03/16 08:13, Martin Liška wrote:
On 08/18/2016 05:53 PM, Jeff Law wrote:
On 08/18/2016 09:51 AM, Andi Kleen wrote:
I'd prefer to make updates atomic in multi-threaded applications.
The best proxy we have for that is -pthread.
Is it slower, most definitely, but odds are we're giving folks
On Sun, 2016-10-02 at 07:04 -0500, Segher Boessenkool wrote:
> On Thu, Sep 29, 2016 at 11:36:29AM -0600, Jeff Law wrote:
> > On 09/29/2016 11:25 AM, Bernd Schmidt wrote:
> > > On 09/29/2016 07:47 PM, David Malcolm wrote:
> > > > This patch adds a new function, print_rtx_function, intended
> > > > f
Andrew Haley writes:
> On 30/09/16 23:16, Rainer Orth wrote:
>> me too, though mostly to have maximum test coverage (primarily on
>> Solaris). As expected, a x86_64-apple-darwin16 bootstrap with
>> --enable-objc-gc just failed for me. I'm testing the following patch
>> (on top of Jakub's).
>>
Hi Richard,
another question as I'm working through your comments...
On 29/09/16 11:45, Richard Biener wrote:
+ /* The region from the byte array that we're inserting into. */
+ tree ptr_wide_int
+ = native_interpret_expr (dest_int_type, ptr + first_byte,
+
Hi Martin,
> On 09/30/2016 02:31 PM, Rainer Orth wrote:
>> this would be i386-pc-solaris2.12. I'm not sure if the constructor
>> priority detection works in a cross scenario.
>>
>> I'm attaching the resulting assembly (although for Solaris as, the gas
>> build is still running).
>
> Hi. Sorry, I
On Sun, 2016-10-02 at 13:07 +, Bernd Edlinger wrote:
> Hi Dave,
>
> here is the new version of the input.c patch:
>
> I have updated the comments, and revised the test case as requested.
> I have additionally done a bootstrap with build config=bootstrap
> -asan.
Thanks. A couple of nits in
This implements the hooks for separate shrink-wrapping for rs6000.
It handles GPRs and LR. The GPRs get a component number corresponding
to their register number; LR gets component number 0.
2016-06-07 Segher Boessenkool
* config/rs6000/rs6000.c (machine_function): Add new fields
This is the main substance of this patch series.
Instead of doing all of the prologue and epilogue in one spot, it often
is better to do components of it at different places, so that they are
executed less frequently.
What exactly is a component is completely up to the target; this code
treats it
A restore is supposed to restore some certain register. Restoring it
into some other register will not work. Don't.
2016-06-07 Segher Boessenkool
* regrename.c (build_def_use): Invalidate chains that have a
REG_CFA_RESTORE on some instruction.
---
gcc/regrename.c | 7 +
A few testcases for separate shrink-wrapping: test whether it works in a
trivial case; whether it creates more than one prologue where that is
useful; whether it puts prologues inside a loop if that is cheaper.
2016-10-03 Segher Boessenkool
gcc/testsuite/
* gcc.target/powerpc/shrink-w
I updated according to Jeff's latest comments (importantly, we cannot
move a *logue in front of a move in general), and added some testcases.
Bootstrapping is in progress on today's trunk, powerpc64-linux and
powerpc64le-linux.
Is this okay to commit now?
Segher
Segher Boessenkool (6):
sepa
If there is a separately wrapped register restore on some path that
is dead (say, control goes into an endless loop after it), then we
cannot delete that restore because that would confuse the DWARF CFI
(if there is another path joining).
This happens with gcc.dg/torture/pr53168.c, for example.
2
This patch adds a new command-line flag "-fshrink-wrap-separate", a status
flag "shrink_wrapped_separate", hooks for abstracting the target components,
and documentation for all those.
2016-06-07 Segher Boessenkool
* common.opt (-fshrink-wrap-separate): New flag.
* doc/invoke.
This patch updates the fortran FE to generate errors, rather than
warnings, for non-positive integer tile clause arguments. I noticed this
problem when I ported over the C/C++ compile time test cases to fortran.
In addition to the two new test files, a couple of other existing tests
needed to be up
> "Eric" == Eric Botcazou writes:
Eric> So, because of its excessive pickiness, the warning ends up making the
user
Eric> butcher informative comments. How is that helpful?
Those comments are not informative. In most cases I kept the original
text just to forestall complaints. But reall
On 10/03/16 10:07, Cesar Philippidis wrote:
Nathan, I haven't looked too deeply into your tile changes yet. Do you
know of the fortran FE is doing anything wrong? I haven't checked if
it's lowering the tile clause in the proper format yet.
thanks for working on this. The problems I noticed (&
On 10/03/2016 03:03 PM, Rainer Orth wrote:
> Hi Martin,
>
>> On 09/30/2016 02:31 PM, Rainer Orth wrote:
>>> this would be i386-pc-solaris2.12. I'm not sure if the constructor
>>> priority detection works in a cross scenario.
>>>
>>> I'm attaching the resulting assembly (although for Solaris as, t
The versioned namespace build has been broken on all branches for some
time. It's due to new code that doesn't use the namespace macros in
the right places. This fixes all issues.
Rather than declaring the std::experimental::* namespaces in
I've added a new file that declares them and is
only in
On Sun, Oct 02, 2016 at 06:15:18PM +0800, Chung-Lin Tang wrote:
> This patch fixes the two ICEs listed on PR77371.
> One is due to the Fortran omp_privatize_by_reference hook returning true
> for types like 'character(kind=1)[1:XX] *', causing them to be processed
> by the path intended for C++ ref
As usual when removing target macros they should be poisoned in system.h.
--
Joseph S. Myers
jos...@codesourcery.com
Hi,
this patch removes PCOMMIT instruction since it was deprecated,
please visit
https://software.intel.com/en-us/blogs/2016/09/12/deprecate-pcommit-instruction
for details.
Regtested on x86_64. Is it Ok for trunk?
2016-10-03 Andrew Senkevich
gcc/
* common/config/i386/i386-co
Ping: https://gcc.gnu.org/ml/gcc-patches/2016-09/msg01781.html
> 2016-09-26 Eric Botcazou
>
> * expmed.c (expand_shift_1): Add MAY_FAIL parameter and do not assert
> that the result is non-zero if it is true.
> (maybe_expand_shift): New wrapper around expand_shift_1.
>
>From: Christophe Lyon [christophe.l...@linaro.org]
>Sent: Monday, October 03, 2016 12:05 AM
>To: Doug Gilmore
>Cc: gcc-patches@gcc.gnu.org
>Subject: Re: Fix PR tree-optimization/77808, ICE in
>duplicate_ssa_name_ptr_info, at tree-ssanames.c:630 starting with r240439
>
>On 2 October 2016 at 23:05,
Ping?
Best regards,
Thomas
On 22/09/16 14:41, Thomas Preudhomme wrote:
Hi,
This patch is part of a patch series to add support for atomic operations on
ARMv8-M Baseline targets in GCC. This specific patch adapts atomic and exclusive
load and store patterns to the constraints of ARMv8-M Baseli
On October 3, 2016 3:02:04 PM GMT+02:00, Kyrill Tkachov
wrote:
>Hi Richard,
>another question as I'm working through your comments...
>
>On 29/09/16 11:45, Richard Biener wrote:
>>
>>> + /* The region from the byte array that we're inserting into.
>*/
>>> + tree ptr_wide_int
>>> + =
Ping?
Best regards,
Thomas
On 22/09/16 14:44, Thomas Preudhomme wrote:
Hi,
This patch is part of a patch series to add support for atomic operations on
ARMv8-M Baseline targets in GCC. This specific patch refactors the expander and
splitter for atomics to make the logic work with ARMv8-M Base
Ping?
Best regards,
Thomas
On 22/09/16 14:46, Thomas Preudhomme wrote:
Hi,
This patch is part of a patch series to add support for atomic operations on
ARMv8-M Baseline targets in GCC. This specific patch makes the necessary change
for compare and swap to work for ARMv8-M Baseline, doubleword
Ping?
Best regards,
Thomas
On 22/09/16 14:47, Thomas Preudhomme wrote:
Hi,
This patch is part of a patch series to add support for atomic operations on
ARMv8-M Baseline targets in GCC. This specific patch adds support for remaining
atomic operations (exchange, addition, substraction, bitwise
On 22/09/16 17:15, Thomas Preudhomme wrote:
On 22/09/16 16:47, Richard Earnshaw (lists) wrote:
On 22/09/16 15:51, Thomas Preudhomme wrote:
Sorry, noticed an error in the patch. It was not caught during testing
because GCC was built with --with-mode=thumb. Correct patch attached.
Best regards,
On 10/03/2016 06:26 AM, Nathan Sidwell wrote:
On 10/03/16 08:13, Martin Liška wrote:
On 08/18/2016 05:53 PM, Jeff Law wrote:
On 08/18/2016 09:51 AM, Andi Kleen wrote:
I'd prefer to make updates atomic in multi-threaded applications.
The best proxy we have for that is -pthread.
Is it slower, m
Ping?
Best regards,
Thomas
On 22/09/16 14:50, Thomas Preudhomme wrote:
Hi,
This patch is part of a patch series to add support for atomic operations on
ARMv8-M Baseline targets in GCC. This specific patch enables atomic and
synchronization support added in previous patches of the series and a
This shares the code for std::gcd and std::experimental::gcd, and
similarly for lcm.
I realised I'd mixed up the gcd.cc and lcm.cc tests so this patch also
swaps them around.
* doc/xml/manual/status_cxx2017.xml: Update gcd/lcm status.
* doc/html/*: Regenerate.
* include/e
Hi!
This is a next step in extending the -Wint-in-bool-context warning
to cover the case when a conditional expression has only
one arm which evaluates to a non-boolean integer value.
With a previous version of this warning, we found PR 77574,
among with several more or less false positives, but
On Mon, Oct 03, 2016 at 03:26:10PM +, Joseph Myers wrote:
> As usual when removing target macros they should be poisoned in system.h.
Here is the patch with that poisoning. Bootstrapped/regtested on
x86_64-linux and i686-linux again, ok for trunk?
2016-10-03 Jakub Jelinek
gcc/
*
Hi!
libsanitizer isn't right now prepared to handle vector types, and we don't
instrument vector additions/multiplications etc. for overflow etc. either,
so this patch just turns the single case that slipped through.
As I wrote in the PR, in the future we should probably change libubsan to
handle
Hi!
In param_list some entries could be error_mark_node, we should just ignore
those. ALso, this patch optimizes by testing cxx_dialect < cxx14 just once.
Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
2016-10-03 Jakub Jelinek
PR c++/77791
* parser.c (c
> >>I would to somehow resolve the discussion related to default value
> >>selection.
> >>Is the prevailing consensus that we should set -fprofile-update=atomic
> >>when
> >>-pthread is set? If so, I'll prepare a patch. I tend to do it this way.
> >
> >This is my preference.
> Likewise.
I still th
+FAIL: gcc.dg/tree-ssa/builtin-sprintf.c execution test
FAIL: test_a_double:364: "%a" expected result for "0x0.0p+0"
doesn't match function call return value: 20 != 6
FAIL: test_a_double:365: "%a" expected result for "0x1.0p+0"
doesn't match function call return value: 20
On 3 October 2016 at 18:07, Doug Gilmore wrote:
>>From: Christophe Lyon [christophe.l...@linaro.org]
>>Sent: Monday, October 03, 2016 12:05 AM
>>To: Doug Gilmore
>>Cc: gcc-patches@gcc.gnu.org
>>Subject: Re: Fix PR tree-optimization/77808, ICE in
>>duplicate_ssa_name_ptr_info, at tree-ssanames.c:6
>From: Christophe Lyon [christophe.l...@linaro.org]
>Sent: Monday, October 03, 2016 11:23 AM
>To: Doug Gilmore
>Cc: gcc-patches@gcc.gnu.org
>Subject: Re: Fix PR tree-optimization/77808, ICE in
>duplicate_ssa_name_ptr_info, at tree-ssanames.c:630 starting with r240439
>
>On 3 October 2016 at 18:07,
The Go runtime package in libgo is picking up C macros from
runtime_sysinfo.go and then re-exporting them to runtime.inc. This
can cause name conflicts. Change the Makefile so that we only put the
macros we need into runtime.inc. These are the constants that are
actually defined by Go code, not
Committed this port from trunk to gomp4
nathan
Index: ChangeLog.gomp
===
--- ChangeLog.gomp (revision 240724)
+++ ChangeLog.gomp (working copy)
@@ -1,3 +1,8 @@
+2016-10-03 Nathan Sidwell
+
+ * gimple-pretty-print.c (dump_gimple_ca
On Sun, Oct 2, 2016 at 6:27 PM, Fritz Reese wrote:
> All,
>
> The attached fixes an[other] ICE in the comparison between UNIONs.
> This time the ICE is due to a BT_UNION component comparing itself to a
> BT_DERIVED component, thus considering their FL_STRUCT and FL_UNION
> typenodes to be equal. T
On 22 September 2016 at 17:26, Jan Hubicka wrote:
>> Hi,
>> The attached patch tries to extend ipa bits propagation to handle
>> pointer alignment propagation.
>> The patch just disables ipa-cp-alignment pass, I suppose we want to
>> eventually remove it ?
>
> Yes, can you please verify that align
I've committed this to gomp4. It gets a few tile-related things out of the way.
1) we were asserting we never saw tile clauses in a few places. That'll change
soon, and the processing required of them is nothing, so just accept them. We
don't need to gimplify the operands, as they have to be
Hi,
This patch improves Early VRP by folding stmts using vrp_fold_stmt as it
is done in ssa_propagate for VRP.
I have also changed EVRP to handle POINTER_TYPE_P. I will send follow up
patches to use this in IPA-VRP.
Bootstrapped and regression testd on x86_64-linux-gnu with no new
regressi
As noted in BZs 71550 and 71403 (and possibly others, I'm going to have
to do some searching). Jump threading can sometimes fuse two loops, in
the process creating an irreducible loop and invalidating the cached
iteration information.
The no longer valid cached iteration information can resul
Committed as obvious (r240730).
Thanks,
Prathamesh
2016-10-03 Prathamesh Kulkarni
* ipa-cp.c (propagate_bits_accross_jump_function): Introduce space
between callee name and param in dump message in call to fprintf.
diff --git a/gcc/ipa-cp.c b/gcc/ipa-cp.c
index 88baf69..1dc5cb
Bootstrap+test in progress on x86_64-unknown-linux-gnu.
OK to commit if passes ?
Thanks,
Prathamesh
diff --git a/gcc/ipa-prop.c b/gcc/ipa-prop.c
index 5ed9bbf..d71ffcf 100644
--- a/gcc/ipa-prop.c
+++ b/gcc/ipa-prop.c
@@ -5667,6 +5667,9 @@ ipcp_transform_function (struct cgraph_node *node)
fbi.b
The attached patch removes an assumption from the implementation
of the -Wplacement-new warning that the size of the array type
enclosed in parentheses and accepted by G++ as an extension is
constant. The assumption causes an ICE in 6.2.0 and 7.0.
Is the patch good to commit to both 7.0 and the
On 03/10/16 15:41 +0100, Jonathan Wakely wrote:
The versioned namespace build has been broken on all branches for some
time. It's due to new code that doesn't use the namespace macros in
the right places. This fixes all issues.
Rather than declaring the std::experimental::* namespaces in
I've a
As the subject states, this patch maps the '*' tile clause arguments
onto integer_zero_node. Before the fortran FE was using mapping it onto
-1. This patch should make the clause parsing in fortran on par with the
C and C++ FEs.
This patch has been applied to gomp-4_0-branch.
Cesar
2016-10-03 Ce
On 10/03/2016 07:59 AM, Jakub Jelinek wrote:
> with -fopenmp. The var is actually properly allocatable in the latter case,
> while it is not with your patch on the first testcase, you just copy over the
> host pointer, that
> is definitely not going to work on non-shared memory offloading.
I th
On 05.09.2016 17:13, Andrew Haley wrote:
> As discussed. I think I should ask a Global reviewer to approve this
> one. For obvious reasons I haven't included the diffs to the deleted
> gcc/java and libjava directories. The whole tree, post GCJ-deletion,
> is at svn+ssh://gcc.gnu.org/svn/gcc/bran
The attached patch adds definitions of TARGET_PRINTF_POINTER_FORMAT
to the rs6000 pair of linux.h and linux64.h headers, analogous to
the config/linux.h header. This appears to be necessary since
unlike most other back ends, the rs6000 back end doesn't include
the latter linux.h.
The patch fixes
On Mon, Oct 03, 2016 at 05:30:35PM -0600, Martin Sebor wrote:
> The attached patch adds definitions of TARGET_PRINTF_POINTER_FORMAT
> to the rs6000 pair of linux.h and linux64.h headers, analogous to
> the config/linux.h header. This appears to be necessary since
> unlike most other back ends, the
On 10/03/2016 07:10 PM, Segher Boessenkool wrote:
On Mon, Oct 03, 2016 at 05:30:35PM -0600, Martin Sebor wrote:
The attached patch adds definitions of TARGET_PRINTF_POINTER_FORMAT
to the rs6000 pair of linux.h and linux64.h headers, analogous to
the config/linux.h header. This appears to be nec
On 3 October 2016 at 20:36, Doug Gilmore wrote:
>>From: Christophe Lyon [christophe.l...@linaro.org]
>>Sent: Monday, October 03, 2016 11:23 AM
>>To: Doug Gilmore
>>Cc: gcc-patches@gcc.gnu.org
>>Subject: Re: Fix PR tree-optimization/77808, ICE in
>>duplicate_ssa_name_ptr_info, at tree-ssanames.c:6
72 matches
Mail list logo
