On Tue, 4 Apr 2017, Marc Glisse wrote:
> On Tue, 4 Apr 2017, Markus Trippelsdorf wrote:
>
> > On 2017.04.03 at 11:16 +0200, Richard Biener wrote:
> > >
> > > The following extends split_address_to_core_and_offset to handle
> > > POINTER_PLUS_EXPR to be able to simplify
> > > (unsigned long) &MEM
On Mon, Apr 3, 2017 at 10:34 PM, Jakub Jelinek wrote:
> Hi!
>
> This patch deals just with correctness of vector shifts by scalar
> non-immediate. The manuals say the shift count is bits [0:63] of
> the corresponding source operand (XMM reg or memory in some cases),
> and if the count is bigger t
On Tue, 4 Apr 2017, Markus Trippelsdorf wrote:
On 2017.04.03 at 11:16 +0200, Richard Biener wrote:
The following extends split_address_to_core_and_offset to handle
POINTER_PLUS_EXPR to be able to simplify
(unsigned long) &MEM[(void *)&D.15512 + 12B] - (unsigned long) ((const int
*) &D.15512 +
On 2017.04.03 at 11:16 +0200, Richard Biener wrote:
>
> The following extends split_address_to_core_and_offset to handle
> POINTER_PLUS_EXPR to be able to simplify
> (unsigned long) &MEM[(void *)&D.15512 + 12B] - (unsigned long) ((const int
> *) &D.15512 + 4) which appears during niter analysis.
This fixes a crash with -fconcepts on a few testcases.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit cee8cb05c77df3dd3c7d93dd70fb1061823f0e68
Author: Jason Merrill
Date: Thu Mar 30 22:04:05 2017 -0400
* semantics.c (finish_template_type): Check CLASSTYPE_TEMPLATE_INFO.
dif
I forgot to include PATCH and the PR in the subject line, sorry about
that. Also, I have run a full bootstrap and testsuite to verify that I
haven't missed any references to the extraneous copy of
target-libpath.exp in libffi.
We currently have two copies of target-libpath.exp in the tree under
gcc/testsuite/lib and libffi/testsuite/lib. It was originally pulled
into the libffi project from downstream gcc in 2009
(https://github.com/libffi/libffi/commit/5cbe2058c128e848446ae79fe15ee54260a90559).
Then in 2012, Anthony Gr
On 03/04/17 20:06 +0100, Jonathan Wakely wrote:
On 03/04/17 11:48 -0700, Mike Stump wrote:
[ dup, I sent the wrong from the wrong email address, you might not see it
because of that ]
On Apr 3, 2017, at 11:47 AM, Mike Stump wrote:
On Apr 3, 2017, at 10:53 AM, Jonathan Wakely wrote:
This
In this testcase we were giving a STRING_CST a VLA type, which is
nonsensical and leads to the asan crash. After fixing that the
initialization semantics are still wrong, so this patch corrects that
as well.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 0ada9f9492a4184e638bd8796db5fb68532
The following patch fixes a broken diagnostic:
#'unary_plus_expr' not supported by expression#
The code to handle UNARY_PLUS_EXPR is already in place in
cxx_pretty_printer::unary_expression. However, UNARY_PLUS_EXPR
is not checked in cxx_pretty_printer::expression, so that we
don't call cxx_pretty
> The following C testcase shows how profiledbootstrap fails with checking
> compiler. We have a (nested) FUNCTION_DECL inside of BLOCK_VARS of an
> inline function, when it gets inlined, it is moved into
> BLOCK_NONLOCALIZED_VARS. And, decls_for_scope calls process_scope_var
> with NULL decl and
Hi!
This patch deals just with correctness of vector shifts by scalar
non-immediate. The manuals say the shift count is bits [0:63] of
the corresponding source operand (XMM reg or memory in some cases),
and if the count is bigger than number of bits - 1 in the vector element,
it is treated as num
That builtin actually expects a single pointer argument.
Andreas.
* builtins.def (BUILT_IN_UPDATE_SETJMP_BUF): Fix type.
diff --git a/gcc/builtins.def b/gcc/builtins.def
index e383c80145..58d78dbbde 100644
--- a/gcc/builtins.def
+++ b/gcc/builtins.def
@@ -884,7 +884,7 @@ DEF_LIB_BUILTIN
On 03/04/17 11:48 -0700, Mike Stump wrote:
[ dup, I sent the wrong from the wrong email address, you might not see it
because of that ]
On Apr 3, 2017, at 11:47 AM, Mike Stump wrote:
On Apr 3, 2017, at 10:53 AM, Jonathan Wakely wrote:
This fixes lots of typos throughout the tree. Can this
Bill,
can you give this patch a spin please? I've smoke tested it on a ppc64le
x-compiler, but don't have one to run executables. regression testing
on an x86_64-linux system is ok.
The DEPENDENT_TYPE_VALID_P thing is a red herring.
It is the canonical type table's equal function considering
On 03/04/17 18:19 +0100, Jonathan Wakely wrote:
* doc/xml/manual/status_cxx2017.xml: Update C++17 status table.
* doc/xml/manual/appendix_contributing.xml (contrib.organization): Add
directories for debug, parallel and profile headers.
* doc/html/*: Regenerate.
Co
On Apr 3, 2017, at 10:53 AM, Jonathan Wakely wrote:
>
> This fixes lots of typos throughout the tree. Can this be committed to
> trunk as obvious?
Yes, it can. However, I like it when even this patch is reviewed. I reviewed
it, looks good.
You missed initialisers -> initializers.
Not sure I
On 4/3/17 12:01 PM, Peter Bergner wrote:
> On 4/3/17 11:04 AM, Peter Bergner wrote:
>> On 4/2/17 1:53 PM, Segher Boessenkool wrote:
>>> I also have a fix for the dfp-builtin-1.c problem.
>>
>> Something like the following maybe? It seems to work for me.
>> I think the hard_dfp predicate was added
This fixes lots of typos throughout the tree. Can this be committed to
trunk as obvious?
I've bootstrapped the C and C++ compilers, but not Ada, and of course
not compiled all the target changes in gcc/config/*
Is one big commit OK, or one per-ChangeLog? Or something else?
(Currently I have one
On Mon, Apr 03, 2017 at 10:56:50AM -0500, Pat Haugen wrote:
> On 03/31/2017 06:28 PM, Kelvin Nilsen wrote:
> > +! { dg-do compile { target { powerpc*-*-* } } }
> > +! { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } {
> > "-mcpu=power9" } }
>
> Shouldn't it be -mcpu=405 in the
I think this started working when I made std::reference_wrapper use
std::__invoke, so we can uncomment the tests.
* testsuite/20_util/reference_wrapper/invoke.cc: Uncomment tests
that no longer fail.
Tested powerpc64le-linux, committed to trunk.
commit fef7c523742f474bb3322bd58
On Mon, Apr 03, 2017 at 11:01:23AM -0600, Jeff Law wrote:
> > PR rtl-optimization/60818
> > * simplify-rtx.c (simplify_binary_operation_1): Do not replace
> > a compare of comparisons with the thing compared if this results
> > in a different machine mode.
> >
> >gcc/testsuite/
> >
* include/bits/ios_base.h: Correct comment.
* testsuite/util/testsuite_hooks.h: Likewise.
Tested powerpc64le-linux, committed to trunk.
commit 9622234eee9d9c61dcfcc8eefa7d5aac0ee44eb4
Author: Jonathan Wakely
Date: Mon Apr 3 16:54:40 2017 +0100
Fix comments in libstdc++ he
* doc/xml/manual/status_cxx2017.xml: Update C++17 status table.
* doc/xml/manual/appendix_contributing.xml (contrib.organization): Add
directories for debug, parallel and profile headers.
* doc/html/*: Regenerate.
Committed to trunk.
commit 2263ee270e922b504e7d5c1
We can't fully implement this without compiler support (see PR80265)
but this adds _GLIBCXX17_CONSTEXPR where possible.
I've left /* _GLIBCXX17_CONSTEXPR */ comments where we need to add it
once the compiler allows it.
* include/bits/char_traits.h (__gnu_cxx::char_traits): Add
_G
On 04/03/2017 11:00 AM, Jakub Jelinek wrote:
On Mon, Apr 03, 2017 at 10:56:13AM -0600, Jeff Law wrote:
On 04/01/2017 06:24 AM, Jakub Jelinek wrote:
Apparently Darwin has insane default stack size for pthread_create
unless overridden through pthread_attr_setstacksize - 512kB, compared e.g.
to Li
On 4/3/17 11:04 AM, Peter Bergner wrote:
> On 4/2/17 1:53 PM, Segher Boessenkool wrote:
>> I also have a fix for the dfp-builtin-1.c problem.
>
> Something like the following maybe? It seems to work for me.
> I think the hard_dfp predicate was added after the dfp-builtin-[12].c
> test cases were
On 03/31/2017 03:40 PM, Segher Boessenkool wrote:
The function simplify_binary_operation_1 has code that does
/* Convert (compare (gt (flags) 0) (lt (flags) 0)) to (flags). */
but this transformation is only valid if "flags" has the same machine
mode as the outer compare. This fixes it.
Bootst
On Mon, Apr 03, 2017 at 10:56:13AM -0600, Jeff Law wrote:
> On 04/01/2017 06:24 AM, Jakub Jelinek wrote:
> > Apparently Darwin has insane default stack size for pthread_create
> > unless overridden through pthread_attr_setstacksize - 512kB, compared e.g.
> > to Linux usual default of around 8MB. F
On 04/01/2017 06:24 AM, Jakub Jelinek wrote:
Hi!
Apparently Darwin has insane default stack size for pthread_create
unless overridden through pthread_attr_setstacksize - 512kB, compared e.g.
to Linux usual default of around 8MB. For typical OpenMP uses that is way
too low, so the following patc
On 03/04/17 16:43 +0100, Jonathan Wakely wrote:
On 02/04/17 02:45 +0300, Ville Voutilainen wrote:
PR libstdc++/79141
* include/bits/stl_pair.h (__wrap_nonesuch): New.
(operator=(typename conditional<
__and_,
is_copy_assignable<_T2>>::value,
const pair&, const __wrap_nonesuch&>::type)
> On S/390 UNITS_PER_WORD is:
> 8 with -m64
> 4 with -m31
> 8 with -m31 -mzarch
>
> This has been chosen to support use of 64 bit registers also in 32 bit
> code. Code compiled with -m31 -mzarch is supposed to adhere to the 32
> bit ABI. In order to make that work it was required to prevent
> UN
On 4/2/17 1:53 PM, Segher Boessenkool wrote:
> I also have a fix for the dfp-builtin-1.c problem.
Something like the following maybe? It seems to work for me.
I think the hard_dfp predicate was added after the dfp-builtin-[12].c
test cases were added, which is maybe why Mike used powerpc_vsx_ok?
On 03/31/2017 06:28 PM, Kelvin Nilsen wrote:
> +! { dg-do compile { target { powerpc*-*-* } } }
> +! { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } {
> "-mcpu=power9" } }
Shouldn't it be -mcpu=405 in the exclude string since that's what's
specified below?
> +! { dg-require-e
On 02/04/17 02:45 +0300, Ville Voutilainen wrote:
PR libstdc++/79141
* include/bits/stl_pair.h (__wrap_nonesuch): New.
(operator=(typename conditional<
__and_,
is_copy_assignable<_T2>>::value,
const pair&, const __wrap_nonesuch&>::type)): Change __nonesuch
to __wrap_nonesuch.
On 02/04/17 15:08 +0300, Ville Voutilainen wrote:
Implement std::is_aggregate.
* include/std/type_traits (is_aggregate, is_aggregate_v): New.
* testsuite/20_util/is_aggregate/requirements/explicit_instantiation.cc:
New.
* testsuite/20_util/is_aggregate/requirements/typedefs.cc: Lik
On 01/04/17 17:48 +0100, Andrew Jenner wrote:
Index: libstdc++-v3/testsuite/25_algorithms/nth_element/58800.cc
===
--- libstdc++-v3/testsuite/25_algorithms/nth_element/58800.cc (revision
475331)
+++ libstdc++-v3/testsuite/25_algor
On 04/01/2017 11:21 AM, Gerald Pfeifer wrote:
On Sat, 1 Apr 2017, Andrew Jenner wrote:
In the course of working with ia16, I found a case where the
sorted_allocnos
array in ira-color.c requires more than ira_allocnos_num entries. The
following patch allows this array to expand when this happens.
On 04/01/2017 10:47 AM, Andrew Jenner wrote:
About 10 years ago, Rask Ingemann Lambertsen sent a patch series to add
a 16-bit x86 (i.e. 8088, 8086, 80186 and 80286 CPUs) back end. This work
was never committed. Recently I've been doing some work on this back
end, and today we released a Sourcery
On 01/04/17 17:48 +0100, Andrew Jenner wrote:
* src/c++11/cow-stdexcept.cc (_ITM_RU2): Declare.
(txnal_read_ptr): Allow 16-bit pointers.
* src/c++11/codecvt.cc (read_utf8_code_point): Handle 16-bit int.
OK for Stage 1, thanks.
On 4/3/17 9:41 AM, Peter Bergner wrote:
> On 4/2/17 1:53 PM, Segher Boessenkool wrote:
>> I also have a fix for the dfp-builtin-1.c problem.
>
> You mean you have a patch to the regex to match both std/stw and ld/lwz?
I think we should also add:
/* { dg-require-effective-target hard_dfp } */
On 4/2/17 1:53 PM, Segher Boessenkool wrote:
> On Sun, Apr 02, 2017 at 09:48:36AM -0500, Peter Bergner wrote:
>> On 4/2/17 2:29 AM, Andreas Schwab wrote:
+/* { dg-require-effective-target dfp } */
>> [snip]
>>> FAIL: gcc.target/powerpc/pr80246.c (test for excess errors)
>>> Excess errors:
>>>
Hi Eric,
I ran into a similiar problem as PR80117 on S/390 starting with your
patch merging the system*.ads files for some biarch plattforms.
Starting with that change GCC does not bootstrap on S/390 when
configured with --with-mode=zarch.
On S/390 UNITS_PER_WORD is:
8 with -m64
4 with -m31
8 wit
On Mon, 3 Apr 2017, Marc Glisse wrote:
> On Mon, 3 Apr 2017, Richard Biener wrote:
>
> > --- gcc/match.pd(revision 246642)
> > +++ gcc/match.pd(working copy)
> > @@ -1153,7 +1153,14 @@ DEFINE_INT_AND_FLOAT_ROUND_FN (RINT)
> > (if (tree_nop_conversion_p (type, TREE_TYPE (@0))
> >
On Mon, 3 Apr 2017, Richard Biener wrote:
--- gcc/match.pd(revision 246642)
+++ gcc/match.pd(working copy)
@@ -1153,7 +1153,14 @@ DEFINE_INT_AND_FLOAT_ROUND_FN (RINT)
(if (tree_nop_conversion_p (type, TREE_TYPE (@0))
&& tree_nop_conversion_p (type, TREE_TYPE (@1))
Ping. Any ideas how to move this forward?
On Fri, Mar 24, 2017 at 05:22:00PM +0100, Marek Polacek wrote:
> On Thu, Mar 23, 2017 at 05:09:58PM -0400, Jason Merrill wrote:
> > On Thu, Mar 23, 2017 at 4:34 PM, Marek Polacek wrote:
> > > On Tue, Mar 14, 2017 at 02:34:30PM -0400, Jason Merrill wrote:
The following fixes PR80281.
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Richard.
2017-04-03 Richard Biener
PR middle-end/80281
* match.pd (A + (-B) -> A - B): Make sure to preserve unsigned
arithmetic done for the negate or the plus.
* fol
Allow targets to define the default for the short enums option.
gcc/
* config/arm/arm.c: (ARM_DEFAULT_SHORT_ENUMS): Provide default
definition.
* config/arm/rtems.h (ARM_DEFAULT_SHORT_ENUMS) Define.
---
gcc/config/arm/arm.c | 6 +-
gcc/config/arm/rtems.h | 2 ++
2 f
On 04/03/2017 12:11 PM, Pedro Alves wrote:
> Hi Jakub,
>
> On 04/01/2017 01:27 PM, Jakub Jelinek wrote:
>
>> 2017-04-01 Jakub Jelinek
>>
>> PR debug/80263
>> * dwarf2out.c (modified_type_die): Try harder not to emit internal
>> sizetype type into debug info.
>>
>> * gcc.dg/
Hi Jakub,
On 04/01/2017 01:27 PM, Jakub Jelinek wrote:
> 2017-04-01 Jakub Jelinek
>
> PR debug/80263
> * dwarf2out.c (modified_type_die): Try harder not to emit internal
> sizetype type into debug info.
>
> * gcc.dg/debug/dwarf2/pr80263.c: New test.
Thanks much for t
> Thanks, that at least fixes tasking.
Great, here's what I have installed on the mainline (it only affects the Ada
compiler) after testing on x86-64/Linux, Aarch64/Linux and SPARC/Solaris.
2017-04-03 Eric Botcazou
* tree-nested.c (get_descriptor_type): Make sure that the alignment
On 03/29/2017 12:23 PM, Dominik Vogt wrote:
> The attached patch removes the XFAIL in attr-alloc_size-11.c on
> s390. (PR 79356).
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79356
Applied. Thanks!
-Andreas-
On 03/27/2017 05:03 PM, Robin Dapp wrote:
> this patch fixes the vcond shift testcase that failed since setting
> PARAM_MIN_VECT_LOOP_BOUND in the s390 backend.
Applied. Thanks!
-Andreas-
On Fri, 31 Mar 2017, Sebastian Pop wrote:
> On Fri, Mar 31, 2017 at 12:06 PM, Richard Biener wrote:
> > That's not a default definition but bogus SSA form. You have to fix that,
> > not this symptom.
> >
>
> Ok.
> It also crashes when adding a call to verifty_ssa
>
> diff --git a/gcc/auto-pro
On Apr 03 2017, Eric Botcazou wrote:
>> In which way are the bits reserved?
>
> I don't know, but that's what I was told by the ARM folks.
>
>> This does not work for ILP32, because the descriptor address starts off
>> at address 4 modulo 8, and adding 4 clears the bit.
>
> I see, can you try the
On Fri, 31 Mar 2017, Sebastian Pop wrote:
> On Fri, Mar 31, 2017 at 1:50 PM, Richard Biener wrote:
> > That looks like a workaround rather than the correct fix. Who sets .MEM?
> > It should assign the proper def.
> >
>
> Moving the auto_profile pass above several other passes fixes the problem
> In which way are the bits reserved?
I don't know, but that's what I was told by the ARM folks.
> This does not work for ILP32, because the descriptor address starts off
> at address 4 modulo 8, and adding 4 clears the bit.
I see, can you try the attached patchlet?
--
Eric BotcazouIndex: tree
The following extends split_address_to_core_and_offset to handle
POINTER_PLUS_EXPR to be able to simplify
(unsigned long) &MEM[(void *)&D.15512 + 12B] - (unsigned long) ((const int
*) &D.15512 + 4) which appears during niter analysis.
We seem to have various copies of similar code but refactorin
On Fri, 31 Mar 2017, Bin.Cheng wrote:
> On Fri, Mar 31, 2017 at 2:57 PM, Bin.Cheng wrote:
> > On Fri, Mar 31, 2017 at 11:37 AM, Richard Biener wrote:
> >> On Fri, 31 Mar 2017, Markus Trippelsdorf wrote:
> >>
> >>> On 2017.03.31 at 11:16 +0200, Richard Biener wrote:
> >>> > On Fri, 31 Mar 2017, R
On Nov 13 2016, Eric Botcazou wrote:
> Index: config/aarch64/aarch64.c
> ===
> --- config/aarch64/aarch64.c (revision 242334)
> +++ config/aarch64/aarch64.c (working copy)
> @@ -14502,6 +14502,10 @@ aarch64_optab_supported_p (int o
Hi,
this fixes the ICE that has been introduced by the new implementation of stack
checking in ILP32 mode, hence a regression present on mainline and 6 branch.
I'd also like to mention that the patch only restores the implementation that
was originally submitted, i.e. it was probably working out
On Fri, Mar 31, 2017 at 03:37:33PM -0600, Martin Sebor wrote:
> On 03/29/2017 04:23 AM, Dominik Vogt wrote:
> >The attached patch removes the XFAIL in attr-alloc_size-11.c on
> >s390. (PR 79356).
> >https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79356
> >
> >Untested.
>
> It looks like the only tar
Thanks Kyrill for review. I'm pinging the patch.
Martin
On 03/21/2017 10:24 AM, Kyrill Tkachov wrote:
>
>
> Hi Martin,
>
> On 13/03/17 08:25, marxin wrote:
>> gcc/testsuite/ChangeLog:
>>
>> 2017-03-13 Martin Liska
>>
>> * g++.dg/ext/mv8.C: Add aarch64* targets.
>>
>> gcc/ChangeLog:
>>
>
On Sun, Apr 02, 2017 at 08:44:03PM +0200, Uros Bizjak wrote:
> x86 part LGTM.
>
> Hopefully, this infrastructure will allow us to fix (or it already
> fixes) PR 56309 [1].
I think only allows to. The target hook has access to the if_info
structure which contains the original basic blocks, their
64 matches
Mail list logo
