When we remove an RTL call, we wouldn't clean up references to the
return value of the call in debug insns. Make it so that we do.
Regstrapped on x86_64- and i686-linux-gnu. Ok to install?
for gcc/ChangeLog
PR debug/89528
* valtrack.c (dead_debug_insert_temp): Reset debug ref
On Wed, 3 Apr 2019, Christophe Lyon wrote:
> On Wed, 3 Apr 2019 at 15:19, Christophe Lyon
> wrote:
> >
> > On Wed, 3 Apr 2019 at 10:24, Richard Biener wrote:
> > >
> > > On Wed, 3 Apr 2019, Christophe Lyon wrote:
> > >
> > > > Hi!
> > > >
> > > > On Fri, 29 Mar 2019 at 20:02, Jeff Law wrote:
>
On Fri, 5 Apr 2019 at 09:38, Richard Biener wrote:
>
> On Wed, 3 Apr 2019, Christophe Lyon wrote:
>
> > On Wed, 3 Apr 2019 at 15:19, Christophe Lyon
> > wrote:
> > >
> > > On Wed, 3 Apr 2019 at 10:24, Richard Biener wrote:
> > > >
> > > > On Wed, 3 Apr 2019, Christophe Lyon wrote:
> > > >
> > >
On Fri, 5 Apr 2019, Christophe Lyon wrote:
> On Fri, 5 Apr 2019 at 09:38, Richard Biener wrote:
> >
> > On Wed, 3 Apr 2019, Christophe Lyon wrote:
> >
> > > On Wed, 3 Apr 2019 at 15:19, Christophe Lyon
> > > wrote:
> > > >
> > > > On Wed, 3 Apr 2019 at 10:24, Richard Biener wrote:
> > > > >
>
Joern Rennecke writes:
> sched_macro_fuse_insns uses the value in condreg1 without
> checking the return value of targetm.fixed_condition_code_regs. As
> this variables
> is not initialized anywhere, this leads to constructing cc_reg_1 with
> an undefined value,
> and then using that in reg_refer
On Fri, Apr 5, 2019 at 9:05 AM Alexandre Oliva wrote:
>
> When we remove an RTL call, we wouldn't clean up references to the
> return value of the call in debug insns. Make it so that we do.
>
> Regstrapped on x86_64- and i686-linux-gnu. Ok to install?
Looks OK but can you adjust the testcase t
Alexandre Oliva writes:
> When we remove an RTL call, we wouldn't clean up references to the
> return value of the call in debug insns. Make it so that we do.
>
> Regstrapped on x86_64- and i686-linux-gnu. Ok to install?
>
>
> for gcc/ChangeLog
>
> PR debug/89528
> * valtrack.c (dea
Hi.
The patch adds a new config that makes LTO+PGO bootstrap faster by
using LTO only in stage4. In stage3, generators are build with LTO
in order to collect a reasonable profile for LTO FE.
Ready for trunk?
Thanks,
Martin
ChangeLog:
2019-04-05 Martin Liska
* Makefile.in: Regenerate
On Apr 4, 2019, Richard Biener wrote:
>> If there's any instruction or view that would be reached by the earlier
>> bind (the one that precedes the one we'd drop or reset), it would get
>> wrong debug information if we were to drop the bind rather than reset
>> it. If there isn't, whatever happ
On Thu, Apr 4, 2019 at 4:05 PM Vladislav Ivanishin wrote:
>
> Richard Biener writes:
>
> > On Mon, Apr 1, 2019 at 5:36 PM Vladislav Ivanishin wrote:
> >>
> >> Hi!
> >>
> >> This is a fairly trivial change fixing a false negative in
> >> -Wmaybe-uninitialized. I am pretty sure this is simply an
Hi there,
The "*neon_mov" patterns for 128 bit sized quantities uses the "Dn"
constraint to match vmov.f32 and vmov.i patterns.
This constraint boils down to using the `neon_immediate_valid` function.
Once the constraint has matched, the output C statement asserts that function
passes.
The output
Hi,
this error-recovery regressions is mostly another case - similar to
c++/89488 - where we want to check that maybe_instantiate_noexcept
actually succeeds before calling a function that asserts
!DEFERRED_NOEXCEPT_SPEC_P, nothrow_spec_p this time. I think we want to
do that consistently in b
On Fri, 5 Apr 2019, Alexandre Oliva wrote:
> On Apr 4, 2019, Richard Biener wrote:
>
> >> If there's any instruction or view that would be reached by the earlier
> >> bind (the one that precedes the one we'd drop or reset), it would get
> >> wrong debug information if we were to drop the bind r
Richard Biener writes:
> On Thu, Apr 4, 2019 at 4:05 PM Vladislav Ivanishin wrote:
>>
>> Richard Biener writes:
>>
>> > On Mon, Apr 1, 2019 at 5:36 PM Vladislav Ivanishin wrote:
>> >>
>> >> Hi!
>> >>
>> >> This is a fairly trivial change fixing a false negative in
>> >> -Wmaybe-uninitialized.
Hi.
The patch adds support for profile for GIMPLE FE. That can be useful
in the future.
Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
Ready to be installed after stage1 opens?
Thanks,
Martin
gcc/ChangeLog:
2019-04-05 Martin Liska
* gimple-pretty-print.c (du
On Fri, 5 Apr 2019 at 11:07, Richard Sandiford
wrote:
> > 2019-04-04 Joern Rennecke
> >
> > * sched-deps.c (sched_macro_fuse_insns): Check return value of
> > targetm.fixed_condition_code_regs.
>
> OK, thanks.
Thanks for the review.
Is that OK restricted to delayed applying once
Joern Rennecke writes:
> On Fri, 5 Apr 2019 at 11:07, Richard Sandiford
> wrote:
>
>
>> > 2019-04-04 Joern Rennecke
>> >
>> > * sched-deps.c (sched_macro_fuse_insns): Check return value of
>> > targetm.fixed_condition_code_regs.
>>
>> OK, thanks.
>
> Thanks for the review.
>
> Is t
This patch fixes stray notes emitted by -Waddress-of-packed-member when
disabled via -w.
Successfully bootstrapped & regrtested on x86_64-pc-linux-gnu.
Committed to trunk as r270169.
gcc/c-family/ChangeLog:
PR c/89985
* c-warn.c (check_address_or_pointer_of_packed_member): Add
When not using -fdata-sections or when there is no containing name,
GCC falls back to a default section name. Because AIX uses XCOFF, it
always has generated it's own section (CSECT) names (based on the
filename) and not using the GCC defaults.
unicode.org icu code has elicited behavior from the
Richard Earnshaw (lists) writes:
> On 29/03/2019 11:01, Andrea Corallo wrote:
>> Hi all,
>> simple patch addressing minor style issue into
>> gcc/config/aarch64/cortex-a57-fma-steering.c.
>>
>> make BOOT_CFLAGS='-mcpu=cortex-a57' bootstrap
>>
>> Okay for trunk?
>>
>> Bests
>> Andrea
>>
>>
>> 2
The change committed yesterday to handle the C++ 2a string literals
as non-type template arguments treats empty string initializers for
trailing pointer array elements as zero. That's of course wrong
because addresses of literals are non-zero.
The attached tweak constrains the trimming of traili
The standard says "If *this and p are the same object, has no effect."
Previously we ended up clearing the path.
* include/bits/fs_path.h (path::operator=(path&&)): Check for self
assignment.
* src/c++17/fs_path.cc (path::operator=(const path&)): Likewise.
* testsu
We need to handle DT_UNKNOWN earlier, not only during directory
recursion, so that the cached file_type value in the directory_entry
won't be used.
* src/c++17/fs_dir.cc (_Dir::advance(bool, error_code&)): Handle
d_type == DT_UNKNOWN immediately.
(_Dir::should_recurse(bool
Implement the proposed resolution of LWG 2708 by moving the _M_options
and _M_pending members out of the recursive_directory_iterator into the
shared _Dir_stack object. Because _Dir_stack is an opaque type, the
member functions that access the _M_options and _M_pending variables
cannot be inline.
The equality operators for directory iterators are not explicitly
specified in the standard, they're only required to meet the iterator
requirements. This means we don't need to declare them at namespace
scope and can implement them as hidden friends.
Also add 'noexcept' to directory_iterator's d
PR libstdc++/89986
* config/abi/pre/gnu.ver: Add missing exports.
* testsuite/27_io/filesystem/iterators/directory_iterator.cc: Test
increment member.
Tested powerpc64le-linux, committed to trunk.
commit 0043393da0a00cfd04867ec63873ed42d0db23a7
Author: Jonathan Wa
On 03/04/19 17:25 +0100, Jonathan Wakely wrote:
Avoid creating arbitrarily large objects on the stack when creating
temporaries in order to provide the strong exception-safety guarantee.
Also implement Antony Polukhin's suggestion to whitelist specific types
that can be efficiently move-assigned
* doc/xml/manual/status_cxx2020.xml: Update status.
* include/std/variant (visit): Define for C++2a (P0655R1).
* testsuite/20_util/variant/visit_r.cc: New test.
Tested powerpc64le-linux, committed to trunk.
commit 60c01065cfdda4f93bd140b0ea37be16f537877c
Author: Jonathan
We still need to document PSTL in the libstdc++ manual, but this adds
it to the release notes.
Committed to CVS.
Index: htdocs/gcc-9/changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-9/changes.html,v
retrieving revision 1.57
di
On Fri, Apr 05, 2019 at 10:45:59AM -0600, Martin Sebor wrote:
> The change committed yesterday to handle the C++ 2a string literals
> as non-type template arguments treats empty string initializers for
> trailing pointer array elements as zero. That's of course wrong
> because addresses of literal
On 4/5/19 10:45 AM, Martin Sebor wrote:
The change committed yesterday to handle the C++ 2a string literals
as non-type template arguments treats empty string initializers for
trailing pointer array elements as zero. That's of course wrong
because addresses of literals are non-zero.
The attache
On 4/5/19 2:25 PM, Martin Sebor wrote:
On 4/5/19 10:45 AM, Martin Sebor wrote:
The change committed yesterday to handle the C++ 2a string literals
as non-type template arguments treats empty string initializers for
trailing pointer array elements as zero. That's of course wrong
because addresse
On 4/5/19 7:11 AM, Paolo Carlini wrote:
Hi,
this error-recovery regressions is mostly another case - similar to
c++/89488 - where we want to check that maybe_instantiate_noexcept
actually succeeds before calling a function that asserts
!DEFERRED_NOEXCEPT_SPEC_P, nothrow_spec_p this time. I th
On 05/04/19 19:06 +0100, Jonathan Wakely wrote:
* doc/xml/manual/status_cxx2020.xml: Update status.
* include/std/variant (visit): Define for C++2a (P0655R1).
* testsuite/20_util/variant/visit_r.cc: New test.
This implementation is wrong, the conversions to R need to hap
On Fri, Apr 05, 2019 at 12:12:11AM -0400, Jason Merrill wrote:
> On 4/4/19 5:18 PM, Marek Polacek wrote:
> > On Thu, Mar 28, 2019 at 03:54:30PM -0400, Jason Merrill wrote:
> > > On 3/20/19 4:12 PM, Marek Polacek wrote:
> > > > The fix for 77656 caused us to call convert_nontype_argument even for
>
On 4/5/19 4:50 PM, Marek Polacek wrote:
On Fri, Apr 05, 2019 at 12:12:11AM -0400, Jason Merrill wrote:
On 4/4/19 5:18 PM, Marek Polacek wrote:
On Thu, Mar 28, 2019 at 03:54:30PM -0400, Jason Merrill wrote:
On 3/20/19 4:12 PM, Marek Polacek wrote:
The fix for 77656 caused us to call convert_no
>From d95934a0f325e0934ada829378c3c0dfd6b3628c Mon Sep 17 00:00:00 2001
From: Thomas Rodgers
Date: Fri, 5 Apr 2019 15:27:35 -0700
Subject: [PATCH] Replace direct PSTL uses of assert() with a macro
This also replaces calls to __TBB_ASSERT so that there are two macro
definitions provided by c++conf
On Fri, 5 Apr 2019 at 23:29, Jonathan Wakely wrote:
>
> On 05/04/19 19:06 +0100, Jonathan Wakely wrote:
> > * doc/xml/manual/status_cxx2020.xml: Update status.
> > * include/std/variant (visit): Define for C++2a (P0655R1).
> > * testsuite/20_util/variant/visit_r.cc: New test.
>
>
On 05/04/2019 16:53, Andrea Corallo wrote:
>
> Richard Earnshaw (lists) writes:
>
>> On 29/03/2019 11:01, Andrea Corallo wrote:
>>> Hi all,
>>> simple patch addressing minor style issue into
>>> gcc/config/aarch64/cortex-a57-fma-steering.c.
>>>
>>> make BOOT_CFLAGS='-mcpu=cortex-a57' bootstrap
>
On Sat, 6 Apr 2019 at 01:55, Ville Voutilainen
wrote:
>
> On Fri, 5 Apr 2019 at 23:29, Jonathan Wakely wrote:
> >
> > On 05/04/19 19:06 +0100, Jonathan Wakely wrote:
> > > * doc/xml/manual/status_cxx2020.xml: Update status.
> > > * include/std/variant (visit): Define for C++2a (P0655R
On 4/5/19 12:21 PM, Marek Polacek wrote:
On Fri, Apr 05, 2019 at 10:45:59AM -0600, Martin Sebor wrote:
The change committed yesterday to handle the C++ 2a string literals
as non-type template arguments treats empty string initializers for
trailing pointer array elements as zero. That's of cours
On Sat, 6 Apr 2019 at 02:19, Ville Voutilainen
wrote:
> And this patch fixes the existing visitation so that we don't
> over-eagerly cast to void. The main gist of it is
>
> -else if constexpr (is_same_v<_Result_type, void>)
> +else if constexpr (!__same_return_types &&
> +
On Sat, 6 Apr 2019 at 02:48, Ville Voutilainen
wrote:
> and
>
> - return __do_visit(std::forward<_Visitor>(__visitor),
> - std::forward<_Variants>(__variants)...);
> + return (_Res)
> +__do_visit(std::forward<_Visitor>(__visitor),
> + std::forward
On Sat, 6 Apr 2019 at 02:55, Ville Voutilainen
wrote:
> Just in case that cast looks scary: the implicit conversion is also
> deep down in __visit_invoke, so
> we do actually require implicit convertibility as we are supposed to.
> If that's still too scary,
> we can just do
>
> - return (_R
44 matches
Mail list logo
