Hi,
On Fri, May 22, 2020 at 09:42:28AM -0400, David Malcolm wrote:
> On Fri, 2020-05-22 at 01:30 +0200, Mark Wielaard wrote:
> > This is on top of the stdbool.h and stdint.h patches.
>
> Sorry, I didn't see those patches; I've replied to them now.
No worries, there was no hurry and I didn't CC y
On 5/22/20 9:18 PM, Patrick Palka wrote:
On Fri, 22 May 2020, Jason Merrill wrote:
On 5/20/20 10:08 PM, Patrick Palka wrote:
On Wed, 20 May 2020, Patrick Palka wrote:
On Tue, 19 May 2020, Jason Merrill wrote:
On 5/8/20 11:42 AM, Patrick Palka wrote:
On Wed, 6 May 2020, Patrick Palka wrote:
Hi, Doug,
Sorry about the delay, I'd somehow missed your email.
On May 20, 2020, Douglas B Rupp wrote:
> 2020-05-20 Douglas B Rupp
> libstdc++v3/
> * crossconfig.m4 (GLIBCXX_CHECK_MATH_DECL): More reject macros.
> * configure: Rebuild.
The change is ok, but the ChangeLog entry
On Fri, 22 May 2020, Jason Merrill wrote:
> On 5/20/20 10:08 PM, Patrick Palka wrote:
> > On Wed, 20 May 2020, Patrick Palka wrote:
> > > On Tue, 19 May 2020, Jason Merrill wrote:
> > >
> > > > On 5/8/20 11:42 AM, Patrick Palka wrote:
> > > > > On Wed, 6 May 2020, Patrick Palka wrote:
> > > > >
>
On 5/19/20 7:34 PM, Marek Polacek wrote:
On Mon, May 18, 2020 at 03:51:36PM -0400, Jason Merrill wrote:
On 5/16/20 6:34 PM, Marek Polacek wrote:
Since GCC 9, C++17 support is no longer experimental. It was too late
to change the default C++ dialect to C++17 in GCC 10, but I think now
it's time
In a function call expression in C++17 evaluation of the function pointer is
sequenced before evaluation of the function arguments, but that doesn't
apply to function calls that were written using operator syntax. In
particular, for operators with right-to-left ordering like assignment, we
must no
TL;DR: This patch updates configure.ac and Makefile.in in libcpp and
libdecnumber to substitute AR archiver.
AIX supports "FAT" libraries containing 32 bit and 64 bit objects
(similar to Darwin), but commands for manipulating libraries do not
default to accept both 32 bit and 64 bit object files.
This patch provides support for distinguishing target_mem_descs introduced
via structured data lifetimes from those arising from dynamic data
lifetimes. This is a prerequisite for the following reference-count
self-checking patch.
This patch (and those following it) are not vital for this patch s
This is a new version of the reference count self-checking code, adjusted
to work with the new (old) dynamic_refcount counting scheme. The key
observation is that a target_mem_desc that was created from a dynamic
data lifetime should not contribute to the structured refcount for splay
tree keys in
Using the ability to distinguish structural from dynamic mappings'
target_mem_descs, we can adjust how the assertions in goacc_exit_datum
and goacc_exit_data_internal work. This is possibly a slightly stronger
test than the one introduced earlier in this patch series -- though
actually I haven't q
This patch adjusts the semantics of dynamic reference counts, as described
in the parent email. There are also two new test cases derived from
Thomas's test in the email:
https://gcc.gnu.org/pipermail/gcc-patches/2020-May/546166.html
that work now.
OK?
Julian
ChangeLog
libgomp/
This patch provides a bug fix (on top of previous patches in this
series) that allows the PR92843 test case to pass. Data mapped in with
"acc_map_data" is not copied out by an "exit data" directive.
OK?
Julian
ChangeLog
PR libgomp/92843
libgomp/
* oacc-mem.c (goacc_exit
Since goacc_enter_datum only maps a single data item now, there is no
need to pass "kinds" as an array. Passing as a scalar allows for some
simplification in the function's callers.
OK?
Julian
ChangeLog
libgomp/
* oacc-mem.c (goacc_enter_datum): Use scalar kind argument instead
This patch adds some missing unlocking from error paths in the OpenACC
attach/detach code, noticed during development of other patches in
this series.
OK?
Julian
ChangeLog
libgomp/
* oacc-mem.c (acc_attach_async): Add missing gomp_mutex_unlock on
error path.
(goa
After questions from Thomas:
https://gcc.gnu.org/pipermail/gcc-patches/2020-May/545399.html
here is a patch series that adjusts how reference counting works
(again) for dynamic data lifetimes in OpenACC. Since the "overhaul"
patch was applied,
https://gcc.gnu.org/legacy-ml/gcc-patches/2019-12/m
On Fri, May 22, 2020 at 12:48 PM Jakub Jelinek wrote:
>
> On Fri, May 22, 2020 at 12:37:29PM -0700, Ian Lance Taylor wrote:
> > Thanks for looking into this.
> >
> > Unfortunately, my push is still failing. I'm not sure why.
> >
> > remote: *** ChangeLog format failed:
> > remote: ERR: cannot fin
With -fstrong-eval-order=all we evaluate the function address before the
arguments. But this caused trouble with virtual functions and
-fsanitize=vptr; we would do vptr sanitization as part of calculating the
'this' argument, and separately look at the vptr in order to find the
function address.
On Thu, May 21, 2020 at 6:03 PM Jason Merrill wrote:
>
> On Fri, May 15, 2020 at 11:39 AM Martin Liška wrote:
> >
> > On 5/15/20 3:22 PM, Marek Polacek wrote:
> > > On Fri, May 15, 2020 at 03:12:27PM +0200, Martin Liška wrote:
> > >> On 5/15/20 2:42 PM, Marek Polacek wrote:
> > >>> I actually use
On 21/05/20 2:17 pm, Jonathan Wakely wrote:
Why is the optimization not done for C++03 mode?
I did it this way because the new std::copy overload rely on std::copy_n
implementation details which is a C++11 algo.
It looks like the uses of 'auto' can be reaplced easily, and
__enable_if_t<> c
Hi,
streaming code assumes that INTEGER_CST never appears in non-trivial component.
This is not true and we sometimes stream such components which sort of silently
works but breaks our IL invariant about tree sharing. This patch fixes one
instance of this problem where ENUMERAL_TYPE lists all its
GCC maintainers:
The following patch adds support for builtins vec_genbm(), vec_genhm(),
vec_genwm(), vec_gendm(), vec_genqm(), vec_cntm(), vec_expandm(),
vec_extractm(). Support for instructions mtvsrbm, mtvsrhm, mtvsrwm,
mtvsrdm, mtvsrqm, cntm, vexpandm, vextractm.
The test has been tested on
On 5/20/20 10:08 PM, Patrick Palka wrote:
On Wed, 20 May 2020, Patrick Palka wrote:
On Tue, 19 May 2020, Jason Merrill wrote:
On 5/8/20 11:42 AM, Patrick Palka wrote:
On Wed, 6 May 2020, Patrick Palka wrote:
On Wed, 6 May 2020, Patrick Palka wrote:
On Tue, 5 May 2020, Patrick Palka wrote:
On Fri, May 22, 2020 at 12:37:29PM -0700, Ian Lance Taylor wrote:
> Thanks for looking into this.
>
> Unfortunately, my push is still failing. I'm not sure why.
>
> remote: *** ChangeLog format failed:
> remote: ERR: cannot find a ChangeLog location in message
> remote:
> remote: Please see: htt
On Fri, May 22, 2020 at 4:11 AM Jakub Jelinek wrote:
>
> On Fri, May 22, 2020 at 12:04:10PM +0100, Richard Earnshaw wrote:
> > >> The directories in question are
> > >>
> > >> gcc/go/gofrontend
> > >> libgo
> > >> gcc/testsuite/go.test/test
> > >
> > > The script has:
> > > ignored_prefixes = [
>
On Thu, May 21, 2020 at 09:05:09AM -0400, David Malcolm wrote:
> So the above should read something like:
>
> + exit(1); /* { dg-warning "call to 'exit' from within signal handler"
> "warning" } */
> + /* { dg-message "note: '_exit' is a possible signal-safe alternative for
> 'exit'" "replacem
On Fri, 22 May 2020 at 19:15, Thomas Koenig via Gcc wrote:
>
> Hi,
>
> what's currently in trunk (as of a few hours ago) fails for me with
>
>File "contrib/mklog.py", line 36, in
> from unidiff import PatchSet
> ModuleNotFoundError: No module named 'unidiff'
>
> I think this is an error
The PR is originally about a bogus warning, which turned out to originate
from truncation of mangled names. Fix: simply extend temporaries to hold
all possible allowed values (which would enable 2^31 equivalences).
Regtested on x86_64-pc-linux-gnu.
The testcase checks the presence of properly ge
Hi,
what's currently in trunk (as of a few hours ago) fails for me with
File "contrib/mklog.py", line 36, in
from unidiff import PatchSet
ModuleNotFoundError: No module named 'unidiff'
I think this is an error which would have to be taken into account
one way or another - maybe include i
On Fri, May 22, 2020 at 01:22:10PM +0200, Richard Biener wrote:
> On Wed, May 20, 2020 at 10:37 PM Segher Boessenkool
> wrote:
> >
> > On Wed, May 20, 2020 at 12:30:30PM +0200, Richard Biener wrote:
> > > I think this is the wrong way to approach this. You're doing too many
> > > things at once.
On 5/21/20 2:16 AM, Martin Liška wrote:
Hello Martin.
Can you please compare the current mklog.py. Is there anything
you miss compared to your current script?
Nope, it matches the format I get with my script and even works
better and runs faster. Very nice! I'll be happy to switch to
using i
On 22/05/20 09:49 +0200, Matthias Kretz wrote:
On Donnerstag, 21. Mai 2020 17:46:01 CEST Marc Glisse wrote:
On Thu, 21 May 2020, Jonathan Wakely wrote:
> On 27/04/20 17:09 +0200, Matthias Kretz wrote:
>> From: Matthias Kretz
>>
>>PR libstdc++/84949
>>* include/std/limits: Let is
> >
>
> I got
>
> ../../src-master/gcc/lto-streamer-out.c: In constructor
> ??DFS::DFS(output_block*, tree, bool, bool, bool)??:
> ../../src-master/gcc/lto-streamer-out.c:807:10: error: too many
> arguments for format [-Werror=format-extra-args]
> 807 | " Streaming single tree\n",
On Fri, May 22, 2020 at 7:15 AM Jan Hubicka wrote:
>
> Hi,
> this patch cleans up dumping of streaming so it is clear how dump is organized
> and how much space individual components needs.
>
> Compiling:
>
> int a=1;
> main()
> {
> return a;
> }
>
> The output is now:
>
> Creating output block
2020-05-22 Uroš Bizjak
gcc/ChangeLog:
PR target/95255
* config/i386/i386.md (2): Do not try to
expand non-sse4 ROUND_ROUNDEVEN rounding via SSE support routines.
gcc/testsuite/ChangeLog:
PR target/95255
* gcc.target/i386/pr95255.c: New test.
Bootstrapped and regression tes
ping?
On Tue, May 19, 2020 at 5:32 PM kamlesh kumar
wrote:
> can someone look at the patch, please?
>
>
> On Wed, Apr 8, 2020 at 9:29 PM Jeff Law wrote:
>
>> On Mon, 2020-04-06 at 14:58 +0530, kamlesh kumar via Gcc-patches wrote:
>> > Hi Richard,
>> > Here is a discussion we did some time ago
>
On Fri, 22 May 2020, Patrick Palka wrote:
> When comparing two special member function templates to see if one hides
> the other (as per P0848R3), we need to check satisfaction which we can't
> do on templates. So this patch makes add_method skip the eligibility
> test on member function template
On Thu, May 21, 2020 at 10:37 AM H.J. Lu wrote:
>
> On Wed, May 20, 2020 at 4:21 AM H.J. Lu wrote:
> >
> > On Tue, May 19, 2020 at 11:10 PM Uros Bizjak wrote:
> > >
> > > On Tue, May 19, 2020 at 11:40 PM H.J. Lu wrote:
> > >
> > > > > > > > > > > I will take a look to see if we share the same C
On Fri, 22 May 2020, Patrick Palka wrote:
> When comparing two special member function templates to see if one hides
> the other (as per P0848R3), we need to check satisfaction which we can't
> do on templates. So this patch makes add_method skip the eligibility
> test on member function template
When comparing two special member function templates to see if one hides
the other (as per P0848R3), we need to check satisfaction which we can't
do on templates. So this patch makes add_method skip the eligibility
test on member function templates and just lets them coexist.
Passes 'make check-c
Hi,
this patch avoids stremaing completely useless stray references to gobal decl
stream. I am re-testing the patch (rebased to current tree) on x86_64-linux
and intend to commit once testing finishes.
Honza
gcc/ChangeLog:
2020-05-22 Jan Hubicka
* lto-streamer-out.c (lto_output_tree
On Fri, 2020-05-22 at 01:30 +0200, Mark Wielaard wrote:
Hi Mark
> This is on top of the stdbool.h and stdint.h patches.
Sorry, I didn't see those patches; I've replied to them now.
> This adds a flag to c_parser so we know when we were trying to
> constract a string literal. If there is a parse
Hi,
this patch cleans up dumping of streaming so it is clear how dump is organized
and how much space individual components needs.
Compiling:
int a=1;
main()
{
return a;
}
The output is now:
Creating output block for function_body
Streaming tree
Start of LTO_trees of size 1
E
On Thu, 2020-05-21 at 00:27 +0200, Mark Wielaard wrote:
> Plus [u]intptr_t and associated constants.
>
> Refactor the bool, true, false, code so it fits into the
> new table based design.
>
> gcc/c-family/ChangeLog:
>
> * known-headers.cc (get_stdlib_header_for_name): Add a new
>
> gcc/t
On Tue, 2020-05-19 at 23:36 +0200, Mark Wielaard wrote:
> Currently gcc suggests to use _Bool instead of bool and doesn't give
> any suggestions when true or false are used, but undefined. This
> patch
> makes it so that (for C99 or higher) a fixit hint is emitted to
> include
> .
[...snip...]
Lo
Hi,
We didn’t quite have time to push this through before the 10.1
deadline, but (since it’s a correctness issue) it should be applied
to the branch too.
tested on x86_64-darwin, linux
OK for master?
10.2 after some bake time?
thanks
Iain
===
It was agreed amongst the implementors that the
On Sat, May 2, 2020 at 4:55 AM H.J. Lu wrote:
>
> Currently patchable area is at the wrong place. It is placed immediately
> after function label, before both .cfi_startproc and ENDBR. This patch
> adds UNSPECV_PATCHABLE_AREA for pseudo patchable area instruction and
> changes ENDBR insertion pa
On Wed, May 20, 2020 at 10:37 PM Segher Boessenkool
wrote:
>
> On Wed, May 20, 2020 at 12:30:30PM +0200, Richard Biener wrote:
> > I think this is the wrong way to approach this. You're doing too many
> > things at once. Try to fix the powerpc regression with the extra
> > flag_rtl_unroll_loops,
Hi
This is almost obvious - except perhaps I’m missing some more
efficient way of doing it; it seems less than ideal to have to build the
ctor call twice with exactly the same inputs.
If there’s no better way ….
tested on x86_64-darwin, linux
OK for master, 10.2?
thanks
iain
Part of the PR
On Thu, May 21, 2020 at 10:17 PM Segher Boessenkool
wrote:
>
> Hi!
>
> On Thu, May 21, 2020 at 03:29:49PM +0200, Martin Liška wrote:
> > Adding Segher to CC, he can help us.
>
> Oh dear. Are you sure?
>
> > On 5/21/20 2:51 PM, Martin Liška wrote:
> > >Back to this I noticed that ppc64le target bu
On Fri, May 22, 2020 at 12:04:10PM +0100, Richard Earnshaw wrote:
> >> The directories in question are
> >>
> >> gcc/go/gofrontend
> >> libgo
> >> gcc/testsuite/go.test/test
> >
> > The script has:
> > ignored_prefixes = [
> > 'gcc/d/dmd/',
> > 'gcc/go/frontend/',
>
> The directory is gcc
On Fri, May 22, 2020 at 11:52 AM Hongtao Liu wrote:
> > On a related note, it looks that pmov stores are modelled in a wrong
> > way. For example, this pattern;
> >
> > (define_insn "*avx512f_v8div16qi2_store"
> > [(set (match_operand:V16QI 0 "memory_operand" "=m")
> > (vec_concat:V16QI
> >
This fixes handling of clobbers when commoning stores.
Bootstrapped / tested on x86-64_unknown-linux-gnu.
2020-05-22 Richard Biener
PR tree-optimization/95268
* tree-ssa-sink.c (sink_common_stores_to_bb): Handle clobbers
properly.
* g++.dg/torture/pr95268.C:
On 22/05/2020 05:57, Jakub Jelinek wrote:
> On Thu, May 21, 2020 at 03:12:21PM -0700, Ian Lance Taylor via Gcc wrote:
>> Hi, this unfortunately breaks gccgo development. Significant parts of
>> the gccgo sources are simply copied from other repositories. Those
>> other repositories do not use Cha
On Thu, May 21, 2020 at 12:09 PM Martin Liška wrote:
>
> On 5/18/20 1:49 PM, Richard Biener wrote:
> > On Mon, May 18, 2020 at 1:10 PM Jakub Jelinek via Gcc-patches
> > wrote:
> >>
> >> On Mon, May 18, 2020 at 01:03:40PM +0200, Jakub Jelinek wrote:
> The optimize attribute is used to specify
This is the patch I pushed. There's two places that are a bit ugly,
COND_EXPR handling and reduction handling. Both uglinesses should go
away once we go SLP-only so I did not spend more than 10 minutes looking
for a nicer solution (eh). I'm going to push through with moderately
clean solutions
On Fri, May 22, 2020 at 2:41 PM Uros Bizjak wrote:
>
> On Fri, May 22, 2020 at 6:55 AM Hongtao Liu wrote:
> >
> > On Thu, May 21, 2020 at 7:18 PM Uros Bizjak wrote:
> > >
> > > On Thu, May 21, 2020 at 7:35 AM Hongtao Liu wrote:
> > > >
> > > > On Wed, May 20, 2020 at 11:43 PM Uros Bizjak wrote
This fixes a leftover early out in determining the sequence of stores
to materialize.
Bootstrapped / tested on x86_64-unknown-linux-gnu.
Richard.
2020-05-22 Richard Biener
PR tree-optimization/95248
* tree-ssa-loop-im.c (sm_seq_valid_bb): Remove bogus early out.
* gc
On Donnerstag, 21. Mai 2020 17:46:01 CEST Marc Glisse wrote:
> On Thu, 21 May 2020, Jonathan Wakely wrote:
> > On 27/04/20 17:09 +0200, Matthias Kretz wrote:
> >> From: Matthias Kretz
> >>
> >>PR libstdc++/84949
> >>* include/std/limits: Let is_iec559 reflect whether
> >>_
On Thu, May 21, 2020 at 2:54 PM H.J. Lu wrote:
>
> Add -mavx512vpopcntdq for -march=native if AVX512VPOPCNTDQ is available.
>
> PR target/95258
> * config/i386/driver-i386.c (host_detect_local_cpu): Detect
> AVX512VPOPCNTDQ.
OK.
Thanks,
Uros.
> ---
> gcc/config/i386/dri
Hi Richard,
Thanks for the suggestions.
> -Original Message-
> From: Richard Sandiford [mailto:richard.sandif...@arm.com]
> Sent: Thursday, May 21, 2020 5:22 PM
> To: Yangfei (Felix)
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH PR95254] aarch64: gcc generate inefficient code w
60 matches
Mail list logo
