Andrew Stubbs writes:
>
> +AMD GCN
> +
> +
> +A new back end targeting AMD GCN GPUs has been contributed to GCC. The
> +following devices are supported (GCC identifiers in parentheses):
Can you add a few words on the current limitations?
-Andi
When r265789 changed instantiation_dependent_r in such a way that
TEMPLATE_PARM_INDEXes are instantiation-dependent only when their
types are dependent, it broke this test. The test uses a combination
of SFINAE and narrow conversions to detect if an expression is constexpr.
"p" is a template
This patch ought to fix the rest of 78244, a missing narrowing warning in
decltype.
As I explained in Bugzilla, there can be three scenarios:
1) decltype is in a template and it has no dependent expressions, which
is the problematical case. finish_compound_literal just returns the
compound
On 1/17/19 4:59 AM, Paolo Carlini wrote:
Hi,
just use in more places the carefully constructed typespec_loc. Note for
another time: the 'type name' we print for tests like cpp0x/auto52.C
doesn't seem particularly clear to me. Tested x86_64-linux, as usual.
OK.
Jason
On 17/01/2019 17:39, Andi Kleen wrote:
Can you add a few words on the current limitations?
How's this?
Andrew
diff --git a/htdocs/backends.html b/htdocs/backends.html
index bb70aa6..eecd09a 100644
--- a/htdocs/backends.html
+++ b/htdocs/backends.html
@@ -81,6 +81,7 @@ csky |
On Thu, 2019-01-17 at 09:10 +, Richard Sandiford wrote:
>
> > +static bool supported_simd_type (tree t)
>
> Missing line break after "static bool".
Fixed.
> > +static bool currently_supported_simd_type (tree t, tree b)
>
> Same here.
Fixed.
> > + return 0;
>
> The return
On Thu, Jan 17, 2019 at 05:51:39PM +, Andrew Stubbs wrote:
> On 17/01/2019 17:39, Andi Kleen wrote:
> > Can you add a few words on the current limitations?
>
> How's this?
Looks good.
-Andi
> -Original Message-
> From: Richard Biener
> Sent: 17 January 2019 07:53
> To: Alejandro Martinez Vicente
> Cc: GCC Patches ; nd ; Richard
> Sandiford
> Subject: Re: [Vectorizer] Add SLP support for masked loads
>
> On Wed, Jan 16, 2019 at 2:37 PM Alejandro Martinez Vicente
> wrote:
> > > PR rtl-optimization/88870
> > > * dce.c (deletable_insn_p): Never delete const/pure calls that can
> > > throw if we can't alter the cfg or delete dead exceptions.
> > > (mark_insn): Don't call find_call_stack_args for such calls.
> > >
> > > * gcc.dg/pr88870.c: New test.
> > OK.
The following reportedly fixes PR86736, a dwarf2out ICE with
-gpubnames (default on darwin) when compiling with -g0
(default on darwin) but linking with -g. pubnames (and types)
are really part of early debug - there is no point in emitting
those late (but the early part also fails, see PR88878
Hi James,
On 16/01/19 18:27, James Greenhalgh wrote:
On Tue, Jan 15, 2019 at 09:29:46AM -0600, Kyrill Tkachov wrote:
Hi all,
This patch adds a tuning struct for the Arm Ares CPU and uses it for
-m{cpu,tune}=ares.
The tunings are an initial attempt and may be improved upon in the future, but
Steve Ellcey writes:
> +/* Return true for types that could be supported as SIMD return or
> + argument types. */
> +
> +static bool supported_simd_type (tree t)
Missing line break after "static bool".
> +{
> + if (SCALAR_FLOAT_TYPE_P (t) || INTEGRAL_TYPE_P (t) || POINTER_TYPE_P (t))
> +
On Thu, Jan 17, 2019 at 7:36 AM Jakub Jelinek wrote:
>
> On Wed, Jan 16, 2019 at 08:57:25PM -0800, H.J. Lu wrote:
> > Check unaligned pointer conversion and strip NOPS.
>
> > -check_address_of_packed_member (tree type, tree rhs)
> > +check_address_or_pointer_of_packed_member (tree type, tree rhs)
On Thu, 17 Jan 2019, Jason Merrill wrote:
> This updated patch removes {short ,}unsigned int from the expected
> diagnostics. And also improves error_init to accept additional arguments,
> like pedwarn_init already does.
This version is OK.
--
Joseph S. Myers
jos...@codesourcery.com
When we partially instantiate the constexpr if, we walk through its body to
see what it uses from the enclosing local_specializations. That walk was
overlooking the use of 'count' in the captures of the innermost lambda,
because we weren't walking into the capture list.
Tested
On 1/17/19 2:09 PM, Marek Polacek wrote:
This patch ought to fix the rest of 78244, a missing narrowing warning in
decltype.
As I explained in Bugzilla, there can be three scenarios:
1) decltype is in a template and it has no dependent expressions, which
is the problematical case.
The code that detects out-of-bounds pointers in calls to built-ins
like memcpy with POINTER_PLUS_EXPR arguments tried a little too
hard to detect out-of-bounds offsets and didn't handle anti-ranges
correctly. I don't see how it could have ever worked and there
were no tests for it in the test
On Mon, Jan 07, 2019 at 09:07:35AM -0600, Christophe Lyon wrote:
> Hi,
>
> This small patch adds a short description of --with-multilib-list for aarch64.
> OK?
OK.
Thanks,
James
>
> Thanks,
>
> Christophe
> 2019-01-07 Christophe Lyon
>
> PR target/85596
> * doc/install.texi
On Mon, Jan 14, 2019 at 08:02:45AM -0600, wuyuan (E) wrote:
> Hi Kyrill:
> The gcc 7.3.0 does not discard the store1 and load1 command; I did
> not expect the community's latest gcc changes so large .
> now I downloaded the latest GCC code, put the patch into GCC source
>
On 1/15/19 12:59 PM, Joseph Myers wrote:
On Tue, 15 Jan 2019, Jason Merrill wrote:
I actually incorporated the C++ part of these changes into yesterday's commit,
using Martin's first suggestion. Here's the adjusted C patch, which I'd like
a C maintainer to approve.
The front-end changes are
On Thu, Jan 17, 2019 at 07:47:32AM -0600, Jakub Jelinek wrote:
> Hi!
>
> arm_neon.h on both targets contained a couple of spots with invalid
> #pragma GCC target syntax. This doesn't result in errors, just warnings and
> those warnings are surpressed in system headers, so are visible with
>
On Thu, Jan 17, 2019 at 12:27 PM H.J. Lu wrote:
>
> On Thu, Jan 17, 2019 at 7:36 AM Jakub Jelinek wrote:
> >
> > On Wed, Jan 16, 2019 at 08:57:25PM -0800, H.J. Lu wrote:
> > > Check unaligned pointer conversion and strip NOPS.
> >
> > > -check_address_of_packed_member (tree type, tree rhs)
> > >
Hi,
This patch fixes build failure found when testing on OSX.
Bootstrapped and tested on x86_64-linux-gnu, though have verified is
correct on OSX.
Committed to trunk as r268050.
--
Iain
---
diff --git a/libphobos/libdruntime/core/sys/posix/aio.d b/libphobos/libdruntime/core/sys/posix/aio.d
On 1/16/19 1:06 PM, Richard Biener wrote:
> On Wed, Jan 16, 2019 at 10:20 AM Martin Liška wrote:
>>
>> Hi.
>>
>> The patch is about resetting TYPE_MODE of vector types. This is problematic
>> when an inlining among different ISAs happen. Then we end up with a different
>> mode than when it's
Hi,
just use in more places the carefully constructed typespec_loc. Note for
another time: the 'type name' we print for tests like cpp0x/auto52.C
doesn't seem particularly clear to me. Tested x86_64-linux, as usual.
Thanks, Paolo.
/cp
2019-01-17 Paolo Carlini
> AFAIK Ada and Go use -fnon-call-exceptions by default and heavily rely on
> it.
Right, the Ada compiler uses -fnon-call-exceptions -fdelete-dead-exceptions
and ACATS could not be passed with optimization enabled without the former.
Btw, there are a couple of improper direct uses of
On Thu, Jan 17, 2019 at 12:27:31PM +0100, Eric Botcazou wrote:
> > AFAIK Ada and Go use -fnon-call-exceptions by default and heavily rely on
> > it.
>
> Right, the Ada compiler uses -fnon-call-exceptions -fdelete-dead-exceptions
> and ACATS could not be passed with optimization enabled without
Hi!
As the following testcases show, we are unable to share stack slots
for (large) variables from inline functions if something in those inline
functions can throw externally.
The issue is that the clobbers we have even in the EH paths
are usually removed by ehcleanup1 which attempts to make
On Thu, Jan 17, 2019 at 12:21 PM Martin Liška wrote:
>
> On 1/16/19 1:06 PM, Richard Biener wrote:
> > On Wed, Jan 16, 2019 at 10:20 AM Martin Liška wrote:
> >>
> >> Hi.
> >>
> >> The patch is about resetting TYPE_MODE of vector types. This is problematic
> >> when an inlining among different
On 14/01/2019 13:55, Andrew Stubbs wrote:
I will now rebase, retest, change all the dates to 2019, and get it
committed.
This is now done! :-)
THe Newlib port is also committed, so all the pieces needed for testing
GCN should be available to everybody now.
To be clear, the libgomp port
On Thu, Jan 17, 2019 at 1:51 PM Andrew Stubbs wrote:
>
> On 14/01/2019 13:55, Andrew Stubbs wrote:
> > I will now rebase, retest, change all the dates to 2019, and get it
> > committed.
>
> This is now done! :-)
>
> THe Newlib port is also committed, so all the pieces needed for testing
> GCN
On Wed, Jan 16, 2019 at 07:12:56PM +0800, Wei Xiao wrote:
> > > Yes, but please test the compiler after the revert. Please also create
> > > a runtime testcase out of the testcase in the PR.
> Yes, we have tested it but current runtime testcase can't cover the corner
> case to expose the
On 1/16/19 6:44 PM, David Malcolm wrote:
PR c++/88699 reports an ICE within this assertion in add_method:
gcc_assert (!current_fns || !DECL_DESTRUCTOR_P (method));
when adding an overloaded operator to a class via a using_decl, due to
DECL_DESTRUCTOR_P requiring a FUNCTION_DECL, but
On Wed, Jan 16, 2019 at 8:57 PM H.J. Lu wrote:
>
> On Wed, Jan 16, 2019 at 01:28:26PM +0100, Jakub Jelinek wrote:
> > On Wed, Jan 16, 2019 at 04:11:44AM -0800, H.J. Lu wrote:
> > > > Why? What is so special about C and (implicit?) casts where the rhs
> > > > isn't
> > > > ADDR_EXPR? Aren't all
On Mon, 14 Jan 2019 at 17:40, Jan Hubicka wrote:
>
> Hello,
> > > Index: params.def
> > > ===
> > > --- params.def (revision 267882)
> > > +++ params.def (working copy)
> > > @@ -227,7 +227,7 @@ DEFPARAM(PARAM_LARGE_UNIT_INSNS,
> >
Hi All,
This test was added back when builtins were being used instead of ACLE
intrinsics. The test as far as I can tell is really testing vcombine,
however some of these builtins no longer exist and causes an ICE.
This fixes the testcase by changing it to use neon intrinsics.
Regtested on
Hi Steve,
I currently do not, but I will contact appropriate personnel & try to get
one submitted soon.
--
Nathan
On Wed, Jan 16, 2019 at 8:24 PM Steve Kargl <
s...@troutmask.apl.washington.edu> wrote:
> Nathan,
>
> Thanks for taking an interesting in improving gfortran. A
> scan of the bug
On 17/01/2019 15:02, Tamar Christina wrote:
> Hi All,
>
> This test was added back when builtins were being used instead of ACLE
> intrinsics. The test as far as I can tell is really testing vcombine,
> however some of these builtins no longer exist and causes an ICE.
>
> This fixes the
Hi!
arm_neon.h on both targets contained a couple of spots with invalid
#pragma GCC target syntax. This doesn't result in errors, just warnings and
those warnings are surpressed in system headers, so are visible with
-Wsystem-headers only. Anyway, the end result was that these pragmas were
Hi,
now that the rest of the patch series has been committed, here's an
updated version of this patch that applies to trunk.
Thanks,
- Tom
[libbacktrace] Add printdwarftest_dwz_cmp.sh test-case
Add test-case that verifies that libbacktrace can find the same debug
information with and without
On Thu, 17 Jan 2019, Jakub Jelinek wrote:
> Hi!
>
> As the following testcases show, we are unable to share stack slots
> for (large) variables from inline functions if something in those inline
> functions can throw externally.
>
> The issue is that the clobbers we have even in the EH paths
>
On 17-01-19 01:35, Ian Lance Taylor wrote:
> On Wed, Jan 16, 2019 at 4:17 PM Tom de Vries wrote:
>>
>> this handles DW_FORM_GNU_ref_alt which references the .debug_info
>> section in the .gnu_debugaltlink file.
>>
>> OK for trunk?
>>
>> Thanks,
>> - Tom
>>
>> On 11-12-18 11:14, Tom de Vries
On 17-01-19 15:14, Tom de Vries wrote:
> On 17-01-19 01:35, Ian Lance Taylor wrote:
>> On Wed, Jan 16, 2019 at 4:17 PM Tom de Vries wrote:
>>>
>>> this handles DW_FORM_GNU_ref_alt which references the .debug_info
>>> section in the .gnu_debugaltlink file.
>>>
>>> OK for trunk?
>>>
>>> Thanks,
>>>
On Wed, Jan 16, 2019 at 3:09 PM Jakub Jelinek wrote:
>
> On Mon, Jan 14, 2019 at 03:23:07PM -0800, H.J. Lu wrote:
> > There are no regressions with this patch:
> >
> > https://gcc.gnu.org/ml/gcc-patches/2019-01/msg00792.html
>
> As the patch seems to be a step forward and fixes an important
On Fri, 11 Jan 2019 at 23:59, Jeff Law wrote:
>
> On 1/8/19 5:03 AM, Richard Sandiford wrote:
> > Bernd Edlinger writes:
> >> On 1/7/19 10:23 AM, Jakub Jelinek wrote:
> >>> On Sun, Dec 16, 2018 at 06:13:57PM +0200, Dimitar Dimitrov wrote:
> - /* Clobbering the STACK POINTER register is an
Ping? I think that kind of patch is OK for stage4?
On Mon, 7 Jan 2019 at 16:07, Christophe Lyon wrote:
>
> Hi,
>
> This small patch adds a short description of --with-multilib-list for aarch64.
> OK?
>
> Thanks,
>
> Christophe
On Thu, Jan 17, 2019 at 03:06:57PM +0100, Richard Biener wrote:
> > + tree new_var = *t;
> > + if (VAR_P (new_var)
> > + && !DECL_HARD_REGISTER (new_var)
> > + && !TREE_THIS_VOLATILE (new_var)
> > + && !DECL_HAS_VALUE_EXPR_P (new_var)
> > + && !is_gimple_reg (new_var)
>
On Windows stat("foo/bar/../.") will resolve to "foo" even if that is a
non-directory and "foo/bar" does not exist. This is the expected
behaviour and consistent with boost::filesystem, so don't try to correct
it. The only unwanted behaviour is that stat("baz/") fails due to a
mingw bug (fixed in
* src/c++17/fs_ops.cc
(equivalent(const path&, const path&, error_code&))
[_GLIBCXX_FILESYSTEM_IS_WINDOWS]: Use GetFileInformationByHandle to
compare files instead of relying on incomplete info returned by stat.
Tested x86_64-linux and x86_64-w64-mingw32,
PR libstdc++/4
* src/c++17/fs_ops.cc (absolute(const path&, error_code&)): Do nothing
if the path is already absolute.
[_GLIBCXX_FILESYSTEM_IS_WINDOWS]: Normalize root-directory.
[!_GLIBCXX_FILESYSTEM_IS_WINDOWS]: Pass error_code to current_path.
On Wed, Jan 16, 2019 at 08:57:25PM -0800, H.J. Lu wrote:
> Check unaligned pointer conversion and strip NOPS.
> -check_address_of_packed_member (tree type, tree rhs)
> +check_address_or_pointer_of_packed_member (tree type, tree rhs)
> {
>if (INDIRECT_REF_P (rhs))
> rhs = TREE_OPERAND
AMD GCN has now been committed to the trunk.
Is the attached OK for the website? Most of the wording has been
modelled on the existing C-SKY announcements.
Thanks
Andrew
diff --git a/htdocs/backends.html b/htdocs/backends.html
index bb70aa6..eecd09a 100644
--- a/htdocs/backends.html
+++
My patch for 64372 removed a bogus lvalue-rvalue conversion for one arm of a
?: expression where the other arm is a throw. But we still need to require
any overload to be resolved, even though we aren't getting that from
decay_conversion anymore.
Tested x86_64-pc-linux-gnu, applying to trunk.
I committed the following patch which implements svmulh to
aarch64/sve-acle-branch. branch
Thanks,
Kugan
From 33b76de8ef5f370dfacba0addef2fe0b1f2a61db Mon Sep 17 00:00:00 2001
From: Kugan Vivekanandarajah
Date: Fri, 18 Jan 2019 07:33:26 +1100
Subject: [PATCH] [SVE ACLE] Implements svmulh
Hi,
this adds myself to MAINTAINERS in the Write After Approval section.
Committed to trunk as 268059.
---
Index: ChangeLog
===
--- ChangeLog (revision 268058)
+++ ChangeLog (working copy)
@@ -1,3 +1,7 @@
+2019-01-18 Li
I committed the following patch which implements svdot to
aarch64/sve-acle-branch. branch
Thanks,
Kugan
From b75cd8ba8f911c137380677b85882c22a6467bf6 Mon Sep 17 00:00:00 2001
From: Kugan Vivekanandarajah
Date: Fri, 18 Jan 2019 09:07:10 +1100
Subject: [PATCH] [SVE ACLE] Implements svdot
Hi,
This patch adds one part of libphobos support for hppa-linux-gnu, as
posted by John. Apologies for it taking a while to get round to
downstreaming this.
Preliminary testsuite runs have been done which show some promising
results, but still a little more to do, particularly with handling the
Hi,
This patch fixes getStackBottom in core.thread to support targets
where the stack grows up, notably hppa configurations. After other
fixes in the compiler, this allows core.thread unittests to pass, as
well as the garbage collector to work correctly on hppa-linux-gnu.
Bootstrapped and
> > > For r267160, I'd expect you want to revert just the config/i386/ part and
> > > keep the testcases, they should work even with the changes reverted,
> > > right?
> > >
> > The testcase part also need to be reverted since we have changed them
> > according to the incorrect intrinsic list in
I am going to check this patch in as obvious. My earlier patch for Aarch64
SIMD support added some warning tests to three Fortran gomp tests as they were
needed for aarch64. Unfortunately, I forgot to add '{ target aarch64-*-* }'
to the warning checks so the tests failed on x86 where the warning
This patch to libgo change the runtime function sigprof to skip to
sigtrampgo if we don't find sigtramp. This fixes GCC PR 88202.
Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu.
Bootstrapped and ran runtime/pprof test on aarch64-linux-gnu.
Committed to mainline.
Ian
Index:
On Fri, Jan 18, 2019 at 3:42 AM Wei Xiao wrote:
>
> > > > For r267160, I'd expect you want to revert just the config/i386/ part
> > > > and
> > > > keep the testcases, they should work even with the changes reverted,
> > > > right?
> > > >
> > > The testcase part also need to be reverted since
When instantiating a partial specialization of a template member
function for a full specialization of a class template, we test
whether the context of variables local to the partial specialization,
i.e., the partial specialization itself, is dependent, and this ICEs
in
63 matches
Mail list logo
