On 10/18/19 11:27 AM, Tobias Burnus wrote:
Currently, one has for
!$omp target exit data map(delete:x)
in the original dump:
#pragma omp target exit data map(delete:*x) map(alloc:x [pointer
assign, bias: 0])
The "alloc:" not only does not make sense but also gives run-time
messages like
Now that vectorizable_operation vectorises most loop stmts involved
in a reduction, it needs to be aware of reductions in fully-masked loops.
The LOOP_VINFO_CAN_FULLY_MASK_P parts of vectorizable_reduction now only
apply to cases that use vect_transform_reduction.
This new way of doing things is d
Bernhard Reutner-Fischer writes:
> On 23 October 2019 13:16:19 CEST, Richard Sandiford
> wrote:
>
>>+++ gcc/config/gcn/gcn.c 2019-10-23 12:13:54.091122156 +0100
>>@@ -3786,8 +3786,7 @@ gcn_expand_builtin (tree exp, rtx target
>>a vector. */
>>
>> opt_machine_mode
>>-gcn_vectorize_get_
"H.J. Lu" writes:
> On Wed, Oct 23, 2019 at 4:51 AM Richard Sandiford
> wrote:
>>
>> Richard Biener writes:
>> > On Wed, Oct 23, 2019 at 1:00 PM Richard Sandiford
>> > wrote:
>> >>
>> >> This patch is the first of a series that tries to remove two
>> >> assumptions:
>> >>
>> >> (1) that all vec
On 10/23/19 10:02 PM, Jan Hubicka wrote:
>> Hi,
>> this patch orders symbols where we copy sections to match the order
>> of files in the command line. This optimizes streaming process since we
>> are not opening and closing files randomly and also we read them more
>> sequentially. This saves so
Hello.
There are 2 more places that need to be handled similarly
to not do out of bounds access.
Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
Ready to be installed?
Thanks,
Martin
>From b6698f326fa4625aca6b2fa65824f5aed8b331df Mon Sep 17 00:00:00 2001
From: Martin Liska
Hi Gerald.
I noticed that https://gcc.gnu.org does not have a news item related
to this contribution. Would you mind adding one? (Our web pages are
now in GIT, cf. https://gcc.gnu.org/about.html - let me know if you need
help.)
Done! :)
OK. (I assume that for the culprit input, an error is show at some point.)
Thanks for ubsan testing!
Tobias
On 10/24/19 10:38 AM, Martin Liška wrote:
Hello.
There are 2 more places that need to be handled similarly
to not do out of bounds access.
Patch can bootstrap on x86_64-linux-gnu and s
ping
On Montag, 14. Oktober 2019 12:27:11 CEST Matthias Kretz wrote:
> This time with testcase. Is the subdir for the test ok?
>
> gcc/ChangeLog:
>
> 2019-10-11 Matthias Kretz
>
> * gcc/cp/parser.c (cp_parser_operator): Parse operator?: as an
> attempt to overload the conditional
Hi!
Jonathan has showed me a testcase with std::allocator::{,de}allocate
and std::construct_at which FAILs with the current constexpr new
implementation.
There are two problems that make the testcase rejected, and further issues
(not addressed by this patch) where supposedly invalid C++20 code is
Hi Joseph.
I've done the move of GCC wwwdocs to git (using the previously posted and
discussed scripts), including setting up the post-receive hook to do the
same things previously covered by the old CVS hooks, and minimal updates
to the web pages dealing with the CVS setup fo
* include/bits/random.h (uniform_random_bit_generator): Define for
C++20.
* testsuite/26_numerics/random/concept.cc: New test.
* testsuite/26_numerics/random/pr60037-neg.cc: Adjust dg-error line.
This is the last piece of P0898R3, "C++20 concepts library", although
On 24/10/19 10:34 +0100, Jonathan Wakely wrote:
* include/bits/random.h (uniform_random_bit_generator): Define for
C++20.
* testsuite/26_numerics/random/concept.cc: New test.
* testsuite/26_numerics/random/pr60037-neg.cc: Adjust dg-error line.
This is the last pie
> On 10/23/19 10:02 PM, Jan Hubicka wrote:
> >> Hi,
> >> this patch orders symbols where we copy sections to match the order
> >> of files in the command line. This optimizes streaming process since we
> >> are not opening and closing files randomly and also we read them more
> >> sequentially. T
The no longer xfailed testcases lacked appropriate target restriction.
The following should fix the sparc fallout.
Applied to trunk.
Richard.
2019-10-24 Richard Biener
PR tree-optimization/65930
* gcc.dg/vect/vect-reduc-2char-big-array.c: Adjust again.
* gcc.dg/vect
On 10/23/19 1:01 PM, Matthew Malcomson wrote:
> Hi Martin,
Hello.
>
> I'm getting close to putting up a patch series that I believe could go
> in before stage1 close.
>
> I currently have to do testing on sanitizing the kernel, and track down
> a bootstrap comparison diff in the code handling
Bootstrapped and regtested on x86_64-redhat-linux, s390x-redhat-linux
and ppc64le-redhat-linux.
v1 -> v2: Improved commit message.
r266734 has introduced a new instance of jump threading pass in order to
take advantage of opportunities that combine opens up. It was perceived
back then that it w
On 23/10/2019 15:21, Richard Earnshaw (lists) wrote:
On 23/10/2019 09:28, Christophe Lyon wrote:
On 21/10/2019 14:24, Richard Earnshaw (lists) wrote:
On 21/10/2019 12:51, Christophe Lyon wrote:
On 18/10/2019 21:48, Richard Earnshaw wrote:
Each patch should produce a working compiler (it did w
Committed.
Richard.
2019-10-24 Richard Biener
PR tree-optimization/65930
* gcc.dg/vect/vect-reduc-2short.c: Fix typo.
Index: gcc/testsuite/gcc.dg/vect/vect-reduc-2short.c
===
--- gcc/testsuite/gcc.dg/vect/vect-
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2019-10-24 Richard Biener
PR tree-optimization/92203
* treee-ssa-sccvn.c (eliminate_dom_walker::eliminate_stmt):
Skip eliminating conversion stmts inserted by insertion.
* gcc.dg/torture/
On 02.10.19 17:06, Martin Jambor wrote:
> Hi,
>
> I seem to remember I minimized gcc.dg/ipa/ipa-sra-19.c on power but
> perhaps I am wrong because the testcase fails there with a
> power-specific error:
>
> gcc.dg/ipa/ipa-sra-19.c:19:3: error: AltiVec argument passed to unprototyped
> function
>
On Tue, Oct 22, 2019 at 10:57:42AM -0400, Jason Merrill wrote:
> > So, do you prefer to do it the other way during build_cxx_call?
>
> It seems more straightforward.
I've tried this approach, but am running into issues:
1) the normal constructors aren't an issue, all I was missing is passing
t
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2019-10-24 Richard Biener
PR tree-optimization/92205
* tree-vect-loop.c (vectorizable_reduction): Restrict
search for alternate vectype_in to lane-reducing patterns
we support.
*
The clause (new in OpenACC 2.6) makes any device code use the local
memory address for each of the variables specified unless the given
variable is already present on the current device. – Or in words of
OpenACC 2.7 (in Sect. 2.7.9 no_create clause):
"The no_create clause may appear on structu
Hi,
On Thu, Oct 24 2019, Andreas Krebbel wrote:
> On 02.10.19 17:06, Martin Jambor wrote:
>> Hi,
>>
>> I seem to remember I minimized gcc.dg/ipa/ipa-sra-19.c on power but
>> perhaps I am wrong because the testcase fails there with a
>> power-specific error:
>>
>> gcc.dg/ipa/ipa-sra-19.c:19:3: er
Hi,
this patch removes the code to reverse read/written bitmaps into
not_read/no_written bitmaps and make sure we compute/stream only useful
summaries. For cc1 ipa-reference now consumes about 17MB while previously
we needed about 64MB after consdensing the bitmaps and over 200MB before.
It also
This patch makes a couple of simple tweaks to improve code generation for the
MSP430 430X ISA.
"Address-word instructions" support 20-bit operands without the extension word
required by regular 430X instructions. Using them where possible reduces code
size and improves performance.
We use the "Ya"
There exist implementations HARD_REGNO_NREGS_HAS_PADDING and
HARD_REGNO_NREGS_WITH_PADDING functions in the msp430 back end, but they have
never been tied to their respective target macros.
Defining the target macros so these functions are used has no effect on GCC
testresults or on code size. So
Hi,
some additional straightforward bits in typeck.c, which I noticed when I
went through the cp_build_binary_op callers. Tested x86_64-linux.
Thanks, Paolo.
//
/cp
2019-10-24 Paolo Carlini
* typeck.c (cp_build_modify_expr): Prefer error + inform to
er
Hi!
On 2017-04-26T13:08:11-0700, Cesar Philippidis wrote:
> This patches updates the c and c++ FEs to report consistent error
> messages for invalid reductions involving array elements, struct
> members, and class members. Most of those variables were already
> rejected by the generic OpenMP code
Hi,
this patch adds heap memory use report which with -Q is now output
during WPA stream in and after IPA passes. This is useful to catch
inordinary large memory use of a given pass.
Bootstrapped/regtested x86_64-linux, OK?
* config.in: Regenerate.
* configure: Regenerate.
Hi Frederik and Jakub!
On 2019-10-21T09:08:28+0200, "Harwath, Frederik"
wrote:
> OpenACC requires that, if a variable is used in reduction clauses on two
> nested loops, then there
> must be reduction clauses for that variable on all loops that are nested in
> between the two loops
> and all t
Hello.
For the symbol/call summary we don't expect that it will be streamed
for PCH purpose. So that, I would like to mark all gt_pch_nx functions
with gcc_unreachable.
Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
Ready to be installed?
Thanks,
Martin
gcc/ChangeLog:
2
Ping: https://gcc.gnu.org/ml/gcc-patches/2019-09/msg01690.html
On 10/17/2019 10:28 AM, Martin Sebor wrote:
Ping: https://gcc.gnu.org/ml/gcc-patches/2019-09/msg01690.html
Other than the suggestions I got for optimization (for GCC 11)
and additional buffer overflow detection for [static] arrays),
Other than the comments from Joseph any feedback on the patch
itself and my questions?
Ping: https://gcc.gnu.org/ml/gcc-patches/2019-10/msg01015.html
On 10/14/2019 02:41 PM, Martin Sebor wrote:
Attached is a more fleshed out version of the built-in implemented
to run in a pass of its own. I di
Ping: https://gcc.gnu.org/ml/gcc-patches/2019-10/msg00860.html
Should I add something like the -Wzero-length-array-bounds option
to allow some of the questionable idioms seen in the kernel?
On 10/11/2019 10:34 AM, Martin Sebor wrote:
On 9/10/19 4:35 PM, Jeff Law wrote:
On 9/6/19 1:27 PM, Marti
Ping: https://gcc.gnu.org/ml/gcc-patches/2019-10/msg00570.html
On 10/14/2019 08:34 PM, Martin Sebor wrote:
Ping: https://gcc.gnu.org/ml/gcc-patches/2019-10/msg00570.html
On 10/8/19 5:51 PM, Martin Sebor wrote:
Attached is a resubmission of the -Wrestrict implementation for
the sprintf family o
> Hello.
>
> For the symbol/call summary we don't expect that it will be streamed
> for PCH purpose. So that, I would like to mark all gt_pch_nx functions
> with gcc_unreachable.
>
> Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
>
> Ready to be installed?
OK,
thanks!
Hon
On 23/10/19 20:27 +0100, Jonathan Wakely wrote:
--- a/libstdc++-v3/include/bits/allocator.h
+++ b/libstdc++-v3/include/bits/allocator.h
@@ -154,13 +154,42 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
_GLIBCXX20_CONSTEXPR
allocator(const allocator<_Tp1>&) _GLIBCXX_NOTHROW { }
+#if __cplusp
On Thu, 24 Oct 2019, Jan Hubicka wrote:
> Hi,
> this patch adds heap memory use report which with -Q is now output
> during WPA stream in and after IPA passes. This is useful to catch
> inordinary large memory use of a given pass.
Looks reasonable but please let others some time to comment on
po
There is no need to store and pass around the allocator object when it's
an instance of std::allocator. Define a partial specialization of
std::use_future_t and the corresponding completion token so that no
allocator is stored. Overload the completion handler constructor to not
expect an allocator
Instead of
t.c:4:3: note: node 0x3751bf0 (max_nunits=1)
t.c:4:3: note: stmt 0 sum_24 = _5 + sum_30;
t.c:4:3: note: stmt 1 sum_25 = _10 + sum_24;
t.c:4:3: note: stmt 2 sum_26 = _14 + sum_25;
t.c:4:3: note: stmt 3 sum_27 = _18 + sum_26;
t.c:4:3: note:
On Thu, Oct 24, 2019 at 12:56 AM Richard Sandiford
wrote:
>
> "H.J. Lu" writes:
> > On Wed, Oct 23, 2019 at 4:51 AM Richard Sandiford
> > wrote:
> >>
> >> Richard Biener writes:
> >> > On Wed, Oct 23, 2019 at 1:00 PM Richard Sandiford
> >> > wrote:
> >> >>
> >> >> This patch is the first of a
On 24.10.19 15:26, Martin Jambor wrote:
> Hi,
>
> On Thu, Oct 24 2019, Andreas Krebbel wrote:
>> On 02.10.19 17:06, Martin Jambor wrote:
>>> Hi,
>>>
>>> I seem to remember I minimized gcc.dg/ipa/ipa-sra-19.c on power but
>>> perhaps I am wrong because the testcase fails there with a
>>> power-spec
Power and IBM Z require a function prototype if a vector argument is
passed. Complete the prototype of k to prevent errors from being
triggered on these platforms
Committed after the discussion here:
https://gcc.gnu.org/ml/gcc-patches/2019-10/msg01737.html
gcc/testsuite/ChangeLog:
2019-10-24 A
On 24/10/19 15:31 +0100, Jonathan Wakely wrote:
On 23/10/19 20:27 +0100, Jonathan Wakely wrote:
--- a/libstdc++-v3/include/bits/allocator.h
+++ b/libstdc++-v3/include/bits/allocator.h
@@ -154,13 +154,42 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
_GLIBCXX20_CONSTEXPR
allocator(const allo
On 23/10/19 12:50 -0700, Thomas Rodgers wrote:
Thomas Rodgers writes:
Let's try this again.
That's better, thanks :-)
+ * include/Makefile.am: Add header.
+* include/Makefile.in: Regenerate.
+ * include/std/stop_token: New file.
+ * include/std/version (__cpp_lib_
due to my own stupidity I found myself wandering into dump_begin, but
then got confused due to an uninitialized pointer not pointing to
somewhere totally insane.
Applying this cleanup patch to initialize the decls at creation time.
nathan
--
Nathan Sidwell
2019-10-24 Nathan Sidwell
* dump
I discovered that while regular TPIs are pointed to by their owning
decl, reduced TPIs were not. This caused me problems on the modules
branch and seems like a needless inconsistency. While there I did a
small amount of cleanup.
I also noticed that convert_generic_types_to_packs was passing
On 24/10/2019 11:16, Christophe Lyon wrote:
On 23/10/2019 15:21, Richard Earnshaw (lists) wrote:
On 23/10/2019 09:28, Christophe Lyon wrote:
On 21/10/2019 14:24, Richard Earnshaw (lists) wrote:
On 21/10/2019 12:51, Christophe Lyon wrote:
On 18/10/2019 21:48, Richard Earnshaw wrote:
Each patc
Hi,
We had a chat offline with Kyrill in which he approved committing the
patch now. Unfortunately he isn't able to access his email until
tomorrow to confirm his approval, but given a time sensitive deadline
and the fact that this patch was previously reviewed and accepted on
trunk I am com
Since r269045 we're missing a space in a diagnostic.
Bootstrapped/regtested on x86_64-linux, applying to trunk and 9.
2019-10-24 Marek Polacek
* decl.c (reshape_init_r): Add missing space.
--- gcc/cp/decl.c
+++ gcc/cp/decl.c
@@ -6239,7 +6239,7 @@ reshape_init_r (tree type, reshape_it
Hi!
On Thu, Oct 24, 2019 at 01:20:26PM -0400, Marek Polacek wrote:
> Since r269045 we're missing a space in a diagnostic.
>
> Bootstrapped/regtested on x86_64-linux, applying to trunk and 9.
>
> 2019-10-24 Marek Polacek
>
> * decl.c (reshape_init_r): Add missing space.
This change rem
Our application is embedded. And in addition to cold boot (reload
everything; start over from scratch), we support warm boot. As part of
supporting warm boot, read-write data that needs to be initialized, is
initialized by code. And we ensure at link time that the traditional
initialized read-wr
Hi!
My script to check for PR79475-like issues discovered what looks like a bug
in the Ada FE. I actually have no idea how it can work properly for any
entries after the last US entry or "United-States", because it will only
match United-StatesUY to US or UZ to Uruguay etc., rather than
United-St
We already had those in fact, just under other names. Use the standard
names so that the vectorizer can use it.
Committing to trunk; will backport to 9 and 8 later.
Segher
2019-10-24 Segher Boessenkool
* config/rs6000/altivec.md (altivec_vavgu): Rename to...
(uavg3_ceil):
On Wed, Oct 23, 2019 at 7:48 AM Hongtao Liu wrote:
>
> Update patch:
> Add m constraint to define_insn (sse_1_round *sse_1_round when under sse4 but not avx512f.
It looks to me that the original insn is incompletely defined. It
should use nonimmediate_operand, "m" constraint and pointer
size mod
This patch to the libstdc++ docs should get the remaining status entries for
C++20 lib.
Ed
Index: doc/xml/manual/status_cxx2020.xml
===
--- doc/xml/manual/status_cxx2020.xml (revision 277405)
+++ doc/xml/manual/status_cxx2020.xml (wor
When fixing c++/91889 (r276251) I was assuming that we couldn't have a ck_qual
under a ck_ref_bind, and I was introducing it in the patch and so this
+ if (next_conversion (convs)->kind == ck_qual)
+ {
+ gcc_assert (same_type_p (TREE_TYPE (expr),
+next_conversion (co
On 24/10/19 19:12 +, Smith-Rowland, Edward M wrote:
This patch to the libstdc++ docs should get the remaining status entries for
C++20 lib.
I need to add some missing macros, like __cpp_lib_bitops.
jthread isn't committed yet, but it will be, so OK for trunk, thanks.
The patch moves the matching of the nonstandard type-spec
BYTE to its own matching function. During this move, a
check for invalid matching in free-form source code it
detected (see byte_4.f90). OK to commit?
2019-10-24 Steven G. Kargl
* decl.c (match_byte_typespec): New function. M
I added support for reduced code size printf and puts functions to Newlib for
MSP430 a while ago [1]. By removing support for reentrancy, streams and
buffering we can greatly reduce code size, which is always often a limitation
when using printf on microcontrollers.
This patch adds an interface to
I noticed that for code like
struct S {
int *foo : 3;
};
we generate nonsensical
r.C:2:8: error: function definition does not declare parameters
2 | int *foo : 3;
It talks about a function because after parsing the declspecs of 'foo' we don't
see either ':' or "name :", so we
Hi,
most of IPA summaries we maintain actually needs to be kept for
offline functions. This patch releases fnsummary and call summary
for inline clones. This needs bit of refactoring since we need to keep
size info for clones and for lto partitioning, so I split it out into
separate size summary.
I wasn't properly setting LOOKUP_CONSTINIT in grokfield and so we didn't
detect a non-const initializer.
Bootstrapped/regtested on x86_64-linux, ok for trunk?
2019-10-24 Marek Polacek
PR c++/92134 - constinit malfunction in static data member.
* decl2.c (grokfield): Set LOOKUP
Hi!
The following patch extends the omp declare variant handling to perform
another check during gimplification, where it already can redirect various
calls to base functions to their corresponding variants.
The scoring is still unimplemented, so right now it does the redirection
only if there is
Hi!
When looking into the constexpr new issues and adding is_std_construct_at
function, I've noticed that with --enable-symvers=gnu-versioned-namespace
all of that fails, because construct_at (but for other things
forward or move etc.) aren't directly in std namespace, but in inline
namespace insi
Hi,
we never free m_vector in summaries. Fixed thus.
Honza
* symbols-summary.h (fast_function_summary::release,
fast_call_summary::release): Free m_vector.
Index: symbol-summary.h
===
--- symbol-summary.h(revisio
On 23/10/19 20:28 +0100, Jonathan Wakely wrote:
* include/std/functional (invoke): Add constexpr for C++20.
* include/std/version (__cpp_lib_constexpr_invoke): Define.
* testsuite/20_util/function_objects/invoke/constexpr.cc: New test.
This is an easy one, because I alrea
On 10/11/2019 04:41 AM, Jakub Jelinek wrote:
On Fri, Oct 11, 2019 at 01:23:12PM +0200, Richard Biener wrote:
(coreutils-0.22)
.debug_info(D1) | .debug_abbrev(D2) | .debug_str(D4) | .ctf
(uncompressed) | ratio (.ctf/(D1+D2+0.5*D4))
ls 30616 |1136 |21098
Richard,
Thanks for your comments.
>+ /* For PHI node that is not in loop header, its source operands should
>+be defined inside the loop, which are seen as loop variant. */
>+ if (def_bb != loop->header || !skip_head)
>+ return false;
> so if we have
>
> for (;;)
>
On Fri, Oct 25, 2019 at 2:39 AM Uros Bizjak wrote:
>
> On Wed, Oct 23, 2019 at 7:48 AM Hongtao Liu wrote:
> >
> > Update patch:
> > Add m constraint to define_insn (sse_1_round > *sse_1_round > when under sse4 but not avx512f.
>
> It looks to me that the original insn is incompletely defined. It
Committed.
Gerald
2019-10-25 Gerald Pfeifer
* doc/xml/gnu/gpl-3.0.xml: Switch www.gnu.org to https.
Index: libstdc++-v3/doc/xml/gnu/gpl-3.0.xml
===
--- libstdc++-v3/doc/xml/gnu/gpl-3.0.xml(revision 277213)
+++ li
Hello Ma’am/Sir,
I represent a company which has *1**7**+* years of experience in
providing *accurate
Translation, Interpretation, App/Web Localization, and Content Moderation
Services within fastest TAT.*
Our team has expertise in all types of languages including national and
international (*Ara
On Tue, 10 Sep 2019, Arnaud Charlet wrote:
> Allright, there are already similar kludges elsewhere, so I've applied the
> following patch which fixes it:
>
> 2019-09-10 Arnaud Charlet
>
> * doc/install.texi: Fix syntax for html generation.
>
> Index: doc/install.texi
> =
Committed.
2019-10-25 Gerald Pfeifer
* doc/xml/manual/policy_data_structures_biblio.xml: Switch
pubs.opengroup.org to https.
Index: doc/xml/manual/policy_data_structures_biblio.xml
===
--- doc/xml/manual/policy_d
On Fri, Oct 25, 2019 at 1:23 PM Hongtao Liu wrote:
>
> On Fri, Oct 25, 2019 at 2:39 AM Uros Bizjak wrote:
> >
> > On Wed, Oct 23, 2019 at 7:48 AM Hongtao Liu wrote:
> > >
> > > Update patch:
> > > Add m constraint to define_insn (sse_1_round > > *sse_1_round > > when under sse4 but not avx512f.
I looked for a replacement, and there does not appear to be one, so I
remove the link.
Committed.
Gerald
- Log -
commit 61592c09663a83809c5115cb7dfddeb3bd606418
Author: Gerald Pfeifer
Date: Fri Oct 25 07:55:49 2019 +0200
htt
78 matches
Mail list logo