On April 24, 2019 11:26:28 PM GMT+02:00, Jeff Law wrote:
>On 4/24/19 4:44 AM, Richard Biener wrote:
>>> Given that we can use the lattice copy propagator by just adding the
>>> pass to passes.def whereas using the RPN VN actually requires a
>little
>>> bit of real code (to set up the entry/exits f
On April 24, 2019 8:31:44 PM GMT+02:00, Jeff Law wrote:
>On 4/24/19 4:44 AM, Richard Biener wrote:
>> On Tue, Apr 23, 2019 at 4:29 PM Jeff Law wrote:
>>>
>>>
>>> As discussed in the BZ, this patch addresses the false positive
>warning
>>> by cleaning up the const/copy propagations left in the IL
From: David Edelsohn
> I have committed this patch.
>
> Thanks, David
Thanks, I'll warn you if I ever understand what's wrong with it.
Clément
This patch to the Go frontend by Than McIntosh fixes a problem with
Function_declaration::import_function relating to how no-name or
"sink" parameters are handled. In Gogo::start_function (for the
non-inline case) when parameter bindings are being added, parameters
with empty/sink names are rename
On 24/04/19 16:01 -0700, Thomas Rodgers wrote:
should be remove_cvref_t
* include/pstl/execution_defs.h (__enable_if_execution_policy):
Use std::__remove_cv_ref_t when building with GCC
From cb7bd9a39acacbf81df0d03da8714fa463057cc5 Mon Sep 17 00:00:00 2001
From: Thomas
should be remove_cvref_t
* include/pstl/execution_defs.h (__enable_if_execution_policy):
Use std::__remove_cv_ref_t when building with GCC
>From cb7bd9a39acacbf81df0d03da8714fa463057cc5 Mon Sep 17 00:00:00 2001
From: Thomas Rodgers
Date: Wed, 24 Apr 2019 15:53:45 -0700
Su
Hi,
This patch removes the monolithic and cumbersome to maintain
core/threadasm.S, and splitted it into multiple parts, one for each
intended target cpu/os.
Added .type and .size directives for all asm implementations of
fiber_switchContent and callWithStackShell where they were missing,
fixing t
On 23/04/19 14:05 -0700, Thomas Rodgers wrote:
Jonathan Wakely writes:
On 23/04/19 09:54 -0700, Thomas Rodgers wrote:
* doc/xml/manual/using.xml: Add PSTL linker flags to table 3.1.
OK - thanks.
Committed to trunk
I've also committed this update for the C++17 status docs.
com
On 24/04/19 15:00 -0700, Thomas Rodgers wrote:
Missed in prior uglification passes.
* include/pstl/execution_defs.h:
Uglfiy inline namespace.
s/Uglfiy/Uglify
Uglify ExecPolicy and T template parameters.
The ChangeLog entry should be formatted without that fir
Missed in prior uglification passes.
* include/pstl/execution_defs.h:
Uglfiy inline namespace.
Uglify ExecPolicy and T template parameters.
>From 07d15a71f60d9dec58882f9cfbf80e4b1ed043f3 Mon Sep 17 00:00:00 2001
From: Thomas Rodgers
Date: Wed, 24 Apr 2019 14:57:26
This change revealed two testsuite bugs where some string comparisons
only compiled by converting the strings to filesystem::path objects.
* include/bits/fs_path.h (operator<, operator<=, operator>)
(operator>=, operator==, operator!=): Make hidden friends, as per
LWG 3065
This valid test started to be accepted with r265789, which removed the
following checks. I wonder if we want to remove them in gcc 8, or
just leave them in place. For trunk I'm going to add the new test.
@@ -6985,27 +7071,10 @@ convert_nontype_argument (tree type, tree expr,
tsubst_flags_t comp
On 4/24/19 4:44 AM, Richard Biener wrote:
>> Given that we can use the lattice copy propagator by just adding the
>> pass to passes.def whereas using the RPN VN actually requires a little
>> bit of real code (to set up the entry/exits for the relevant SEME
>> regions), I went with the lattice copy
If require_all_args, we aren't waiting for more args to be deduced later.
Tested x86_64-pc-linux-gnu, applying to trunk.
* pt.c (coerce_template_parms): Do add empty pack when
require_all_args.
---
gcc/cp/pt.c | 2 +-
gcc/testsuite/g++.dg/cpp1y/var-t
On 24/04/19 11:40 -0700, Thomas Rodgers wrote:
* testsuite/lib/libstdc++.exp (check_effective_target_tbb-backend):
Add check for Thread Building Blocks 2018 or later.
From bd3de7b67f184ed1387b63dc3bda1d12f7ebee04 Mon Sep 17 00:00:00 2001
From: Thomas Rodgers
Date: Wed, 24 Ap
Hi,
This patch adds -fversion=Linux_Pre_2639 to the druntime testsuite
compile flags if the linux OS version is older than 2.6.39. The
second part is from upstream druntime e03164b5, fixing PR d/89432.
Bootstrapped and regression tested on x86_64-linux-gnu.
Committed to trunk as r270554.
--
I
* testsuite/lib/libstdc++.exp (check_effective_target_tbb-backend):
Add check for Thread Building Blocks 2018 or later.
>From bd3de7b67f184ed1387b63dc3bda1d12f7ebee04 Mon Sep 17 00:00:00 2001
From: Thomas Rodgers
Date: Wed, 24 Apr 2019 11:34:14 -0700
Subject: [PATCH] Check TBB ve
On 4/24/19 4:44 AM, Richard Biener wrote:
> On Tue, Apr 23, 2019 at 4:29 PM Jeff Law wrote:
>>
>>
>> As discussed in the BZ, this patch addresses the false positive warning
>> by cleaning up the const/copy propagations left in the IL between DOM's
>> jump threading and erroneous path isolation.
>>
On 4/24/19 7:05 AM, Segher Boessenkool wrote:
> Hi Jeff,
>
> On Mon, Apr 22, 2019 at 09:09:12AM -0600, Jeff Law wrote:
>> First, it re-uses combine's make_field_assignment in the aarch64
>> backend.
>
> That's not really acceptable. make_field_assignment depends intimately
> on make_extraction,
On Tue, Apr 23, 2019 at 4:27 AM CHIGOT, CLEMENT wrote:
>
> Description:
> * This patch removes -mvsx and -maltivec for go aix/ppc.
> These options don't seem compatible with Go stack layout.
>
> Tests:
> * AIX 7.2: Configure/Build: SUCCESS
>
> Changelog:
> * config/rs6000/aix71.h (SUBT
Some minor tweaks to C++17 and C++2a components
* include/std/any (any::any(ValueType&&)): Use __and_v.
* include/std/numeric (midpoint(T, T, T), midpoint(T*, T*, T*)):
Likewise.
Tested powerpc64le-linux, committed to trunk.
commit b31d88f0da87ad6d07f012deb51756a5ade7883
Some more recent-ish changes to the C++ working draft
* include/std/tuple (apply): Use remove_reference_t instead of decay_t
as per P0777R1.
* include/std/type_traits (__result_of_memfun): Use remove_reference
instead of __remove_cvref_t and remove redundant is_sam
The basic_string_view::pointer and basic_string_view::reference typedefs
are supposed to refer to the non-const value type.
In previous standards having traits_type::char_type different to
value_type was simply undefined, but in the C++2a draft it's ill-formed,
as changed by P1148R0. For std::bas
> 2019-04-24 Jakub Jelinek
>
> PR target/90193
> * rtl.c (classify_insn): Return JUMP_INSN for asm goto.
> * emit-rtl.c (try_split): Copy over REG_LABEL_TARGET.
>
> * gcc.target/i386/pr90193.c: New test.
OK, thanks.
--
Eric Botcazou
Hi,
This patch adds necessary functionality in the libphobos testsuite to
mark druntime and phobos unit-tests to be skipped on matched targets.
Added entries to skip curl tests if libcurl is not installed.
Regression tested on x86_64-linux-gnu on multilibs -m32, -mx32, and
-m64 where curl was not
On Wed, Apr 24, 2019 at 9:31 AM Ian Lance Taylor wrote:
>
> On Tue, Apr 23, 2019 at 1:27 AM CHIGOT, CLEMENT
> wrote:
> >
> > Description:
> > * This patch removes -mvsx and -maltivec for go aix/ppc.
> > These options don't seem compatible with Go stack layout.
> >
> > Tests:
> > * AIX 7
On Tue, Apr 23, 2019 at 1:27 AM CHIGOT, CLEMENT wrote:
>
> Description:
> * This patch removes -mvsx and -maltivec for go aix/ppc.
> These options don't seem compatible with Go stack layout.
>
> Tests:
> * AIX 7.2: Configure/Build: SUCCESS
>
> Changelog:
> * config/rs6000/aix71.h (SUBT
On Mon, Apr 22, 2019 at 11:51 PM Michael Forney wrote:
>
> This matches how the the rest of the libc functions are declared and
> prevents a potential conflict with the system declaration when libiberty.h
> is included from a directory that did not check for vasprintf, but it is
> available. It al
On 24.04.2019 13:25, Iain Buclaw wrote:
> On Wed, 24 Apr 2019 at 13:03, Kamil Rytarowski wrote:
>>
>> On 24.04.2019 03:30, Iain Buclaw wrote:
>>> On Wed, 24 Apr 2019 at 01:56, Kamil Rytarowski wrote:
On 24.04.2019 01:13, Iain Buclaw wrote:
>>> https://explore.dgnu.org/z/U29cni
>>>
>>> I
On Wed, Apr 24, 2019 at 03:11:27PM +0200, Richard Biener wrote:
>
> While the PR is about an ICE during debug stmt expansion
> it shows a missed optimization ultimatively leading to a
> debug stmt refering to VIEW_CONVERT_EXPR({}) which isn't
> handled.
>
> The following fixes the missed optimiza
While the PR is about an ICE during debug stmt expansion
it shows a missed optimization ultimatively leading to a
debug stmt refering to VIEW_CONVERT_EXPR({}) which isn't
handled.
The following fixes the missed optimization and thus also
the ICE by instead producing a debug stmt refering to 0.
Hi Jeff,
On Mon, Apr 22, 2019 at 09:09:12AM -0600, Jeff Law wrote:
> First, it re-uses combine's make_field_assignment in the aarch64
> backend.
That's not really acceptable. make_field_assignment depends intimately
on make_extraction, which is very much combine-specific (and wrong in so
many wa
Bootstrap and regtest running on x86_64-unknown-linux-gnu.
Richard.
2019-04-24 Richard Biener
PR middle-end/90213
* gimple-fold.c (fold_const_aggregate_ref_1): Do multiplication
by size and BITS_PER_UNIT on poly-wide-ints.
Index: gcc/gimple-fold.c
==
On Mon, Apr 22, 2019 at 2:14 AM Jakub Jelinek wrote:
>
> As detailed in the PR, unlike most other targets, on ARM EABI the floating
> point registers are saved lazily, when EH personality routine calls
> __gnu_unwind_frame (usually in the CONTINUE_UNWINDING macro).
> That means the unwinder itself
On 24/04/19 11:21 +0100, Nina Dinka Ranns wrote:
New diff attached.
Thanks, this looks great. I think we can apply this as soon as stage 1
begins (which should be Real Soon Now).
On 24/04/19 10:46 +0100, Jonathan Wakely wrote:
PR libstdc++/90220 (partial)
* include/std/any (any_cast(any*), any_cast(const any*)): Do
not attempt ill-formed static_cast to pointers to non-object types.
* testsuite/20_util/any/misc/any_cast.cc: Test std::any_cas
On Wed, 24 Apr 2019 at 13:03, Kamil Rytarowski wrote:
>
> On 24.04.2019 03:30, Iain Buclaw wrote:
> > On Wed, 24 Apr 2019 at 01:56, Kamil Rytarowski wrote:
> >>
> >> On 24.04.2019 01:13, Iain Buclaw wrote:
> >>> Hi,
> >>>
> >>> This patch adds missing implementation of TARGET_D_CRITSEC_SIZE, whic
0524.
>
> That breaks -mabi=ilp32:
>
> /opt/gcc/gcc-20190424/libphobos/libdruntime/core/sys/posix/sys/stat.d:713:13:
> error: static assert (104u == 128u) is false
> 713 | static assert(stat_t.sizeof == 128);
> | ^
> make[8]: *** [Makefile:
On 24.04.2019 03:30, Iain Buclaw wrote:
> On Wed, 24 Apr 2019 at 01:56, Kamil Rytarowski wrote:
>>
>> On 24.04.2019 01:13, Iain Buclaw wrote:
>>> Hi,
>>>
>>> This patch adds missing implementation of TARGET_D_CRITSEC_SIZE, which
>>> would be noticed when using any bare synchronized statements.
>>>
On Tue, Apr 23, 2019 at 4:29 PM Jeff Law wrote:
>
>
> As discussed in the BZ, this patch addresses the false positive warning
> by cleaning up the const/copy propagations left in the IL between DOM's
> jump threading and erroneous path isolation.
>
> In the past we'd been handling this stuff with
On Tue, Apr 23, 2019 at 10:25 AM Richard Sandiford
wrote:
>
> This patch explains the encoding used for the precision field in
> TYPE_VECTOR_SUBPARTS when NUM_POLY_INT_COEFFS == 2. OK to install?
OK.
Richard.
> Richard
>
>
> 2019-04-19 Richard Sandiford
>
> gcc/
> * tree.h (TYPE_VEC
On Tue, 23 Apr 2019 at 21:28, Jonathan Wakely wrote:
>
> On 23/04/19 18:43 +0100, Nina Dinka Ranns wrote:
> >On Thu, 18 Apr 2019 at 21:35, Jonathan Wakely wrote:
> >>
> >> On 16/04/19 17:59 +0100, Nina Dinka Ranns wrote:
> >> >On Tue, 16 Apr 2019 at 15:18, Jonathan Wakely wrote:
> >> >>
> >> >>
From: claziss
gcc/
-xx-xx Claudiu Zissulescu
* config/arc/arc-options.def: Fix typos and spelling mistakes.
* config/arc/arc.c (arc_init): Cleanup warning message.
(arc_override_options): Likewise.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@270540
138bc75d-0
PR libstdc++/90220 (partial)
* include/std/any (any_cast(any*), any_cast(const any*)): Do
not attempt ill-formed static_cast to pointers to non-object types.
* testsuite/20_util/any/misc/any_cast.cc: Test std::any_cast with
function types.
Tested powerpc64l
Hi!
On Tue, Apr 16, 2019 at 08:32:50PM +0200, Jakub Jelinek wrote:
> 2019-04-16 Jakub Jelinek
>
> PR target/89093
> * config/aarch64/aarch64.c (aarch64_process_one_target_attr): Don't skip
> whitespace at the start of target attribute string.
>
> * gcc.target/aarch64/p
On Wed, Apr 24, 2019 at 9:36 AM Jakub Jelinek wrote:
> Hi!
>
> ix86_expand_sse_fp_minmax bypasses the expanders and so might end up with
> both if_true and if_false being a MEM, which violates the condition of the
> insn it wants to match.
>
> The following patch makes sure at most one of the ope
Thanks for catching, Jakub!
The update listed below, will integrate it with next revision patch.
gcc/ChangeLog
2019-04-24 Kewen Lin
PR middle-end/80791
* target.def (predict_doloop_p): New hook.
* targhooks.h (default_predict_doloop_p): New declaration.
* tar
On Wed, 24 Apr 2019 at 10:05, Robin Dapp wrote:
>
> Hi,
>
> the attached patch is against the current HEAD, fixing one additional
> test case.
>
Thanks, this has been committed.
> Parallel testing does not seem to work anymore. Adding the following
> define for ${PWD_COMMAND} makes it work agai
The DOC/DOC0/DOC1 are like keywords to be placed into the warning
message. They shouldn't be translated as they can referenced directly
in the specific processor architectural options. So, I will use %qs
for them, and fix the other signalized problems.
Thank you
On Wed, Apr 17, 2019 at 2:25 PM Jak
On Wed, Apr 24, 2019 at 04:41:01PM +0800, Kewen.Lin wrote:
> gcc/ChangeLog
Not a review, just ChangeLog nits.
> 2019-04-24 Kewen Lin
>
> PR middle-end/80791
> * target.def (predict_doloop_p): New.
> * targhooks.h (default_predict_doloop_p): New.
> * targhooks.c (defaul
Hi all,
As PR80791 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80791, on
some targets which support low-overhead loop, the related compare
type ivs use is possible to become dead and removed eventually.
However, the current ivopt cost modeling doesn't consider this
possible elimination, it leads
Hi,
the attached patch is against the current HEAD, fixing one additional
test case.
Parallel testing does not seem to work anymore. Adding the following
define for ${PWD_COMMAND} makes it work again.
diff --git a/libphobos/testsuite/Makefile.in
b/libphobos/testsuite/Makefile.in
index 26ed875d
On April 24, 2019 10:02:00 AM GMT+02:00, Jakub Jelinek wrote:
>Hi!
>
>The GIMPLE bb removal code doesn't really remove GIMPLE_LABEL
>stmts referencing FORCED_LABELs (and likewise non-local labels,
>except it turns them into FORCED_LABELs), but instead moves them
>at the start of some other bb.
>
>
On April 24, 2019 9:57:05 AM GMT+02:00, Jakub Jelinek wrote:
>Hi!
>
>The parloops code when walking over PHI arguments from the loop exit
>edge assumes that the arguments must be SSA_NAMEs, without checking.
>As can be seen on the following testcase, in some cases it can be a
>constant
>as well.
Hi!
The GIMPLE bb removal code doesn't really remove GIMPLE_LABEL
stmts referencing FORCED_LABELs (and likewise non-local labels,
except it turns them into FORCED_LABELs), but instead moves them
at the start of some other bb.
As can be seen on the testcases, that breaks
if (prev_stmt &&
Hi!
The parloops code when walking over PHI arguments from the loop exit
edge assumes that the arguments must be SSA_NAMEs, without checking.
As can be seen on the following testcase, in some cases it can be a constant
as well. And we don't really need to do anything special in those cases,
the c
Hi!
ix86_expand_sse_fp_minmax bypasses the expanders and so might end up with
both if_true and if_false being a MEM, which violates the condition of the
insn it wants to match.
The following patch makes sure at most one of the operands is a MEM.
Bootstrapped/regtested on x86_64-linux and i686-li
Hi!
The i386 backend has a splitter:
(define_split
[(match_operand 0 "tls_address_pattern")]
"TARGET_TLS_DIRECT_SEG_REFS"
[(match_dup 0)]
"operands[0] = ix86_rewrite_tls_address (operands[0]);")
which just copies the pattern of an insn and adjusts tls references in
there. Unfortunately it
On Apr 24 2019, Iain Buclaw wrote:
> This patch adds arch64*-*-linux* as a supported libphobos target,
> something that has been passing the testsuite for a while now.
>
> Committed to trunk as r270524.
That breaks -mabi=ilp32:
/opt/gcc/gcc-20190424/libphobos/libdruntime/core/s
> The Ada changes need those guards because the file is compiled by both
> the system compiler and by the newly built compilers; when compiled by
> system compiler, as the FE is built with -fno-exceptions I'd hope the EH
> stuff isn't really used there and at least until GCC 9.1 is released we have
60 matches
Mail list logo
