rtl-tests.cc and simplify-rtx.cc used partial specialisation
to try to restrict the NUM_POLY_INT_COEFFS>1 tests without
resorting to preprocessor tests. That now triggers an error
in some configurations, since the NUM_POLY_INT_COEFFS>1 tests
used the global poly_int64, whose definition does not de
Hi,
Profiledbootstrap fails with ICE in update_loop_exit_probability_scale_dom_bbs
called from loop unroling.
The reason is that under relatively rare situations, we may run into case where
loop has multiple exits and all are considered as likely but then we scale down
the profile and one of the ex
On Fri, 10 Nov 2017, Jakub Jelinek wrote:
> On Fri, Nov 10, 2017 at 08:52:16AM +0100, Richard Biener wrote:
> > > @@ -958,8 +959,10 @@ pass_store_merging::terminate_all_aliasi
> > >unsigned int i;
> > >FOR_EACH_VEC_ELT (cur->m_store_info, i, info)
> > > {
> > > - if (ref_maybe_
On Fri, Nov 10, 2017 at 08:52:16AM +0100, Richard Biener wrote:
> > @@ -958,8 +959,10 @@ pass_store_merging::terminate_all_aliasi
> >unsigned int i;
> >FOR_EACH_VEC_ELT (cur->m_store_info, i, info)
> > {
> > - if (ref_maybe_used_by_stmt_p (stmt, gimple_assign_lhs (info->stmt
On Thu, 9 Nov 2017, Jakub Jelinek wrote:
> Hi!
>
> We want to terminate a chain if a chain with different base (or insn
> outside of any chain) has a store that the current stmt might use, or
> overwrite. The functions it used didn't cover the store after store
> case which in the middle-end ali
Hi!
We want to terminate a chain if a chain with different base (or insn
outside of any chain) has a store that the current stmt might use, or
overwrite. The functions it used didn't cover the store after store
case which in the middle-end aliasing model needs to avoid tbaa, because
the latter st
in
Date: Fri, 27 Oct 2017 12:21:02 +0200
Subject: [PATCH] Fix profiledbootstrap.
ChangeLog:
2017-10-27 Martin Liska
* Makefile.tpl: Use proper name of folder as it was renamed
during transition to 4 stages.
* Makefile.in: Regenerate.
---
Makefile.in | 4 ++--
Makefile.tpl | 4 ++--
2 files
Hi,
this patch fixes ICE during profiledbootstrap about hot BB being dominated by
cold. This is verified by RTL verify_flow_info and was added by Theresa
along with patches to undo mistakes in in sane profile.
The implementation is odd because this is not about dominance but reachability.
It is a
> eric@polaris:~/build/gcc/native/gcc> rm ada/sem_util.o
> eric@polaris:~/build/gcc/native/gcc> make ADAFLAGS="-gnatpgn"
> /home/eric/build/gcc/native/./prev-gcc/xgcc -
> B/home/eric/build/gcc/native/./prev-gcc/
> -B/home/eric/install/gcc/x86_64-suse- linux/bin/
> -B/home/eric/install/gcc/x86_64-su
> We have local modifications in the Ada front-end so I cannot reproduce it
> with the pristine tree either. :-(
I apparently screwed up yesterday, this can be reproduced as such in the
directory of a boostrapped compiler:
eric@polaris:~/build/gcc/native/gcc> rm ada/sem_util.o
eric@polaris:~/bu
> The following C testcase shows how profiledbootstrap fails with checking
> compiler. We have a (nested) FUNCTION_DECL inside of BLOCK_VARS of an
> inline function, when it gets inlined, it is moved into
> BLOCK_NONLOCALIZED_VARS. And, decls_for_scope calls process_scope_var
> with NULL decl and
On Fri, Mar 24, 2017 at 09:07:54AM -0400, Jason Merrill wrote:
> >> And when it's cloned.
> >>
> >> But does it make sense for gen_decl_die to call
> >> dwarf2out_abstract_function when decl is null? That seems wrong.
> >
> > Before r144529 we had just:
> > if (DECL_ORIGIN (decl) != decl)
> >
On Fri, Mar 24, 2017 at 3:46 AM, Jakub Jelinek wrote:
> On Thu, Mar 23, 2017 at 05:24:31PM -0400, Jason Merrill wrote:
>> On Thu, Mar 23, 2017 at 4:44 PM, Jakub Jelinek wrote:
>> > The following C testcase shows how profiledbootstrap fails with checking
>> > compiler. We have a (nested) FUNCTION
On Fri, Mar 24, 2017 at 12:45:28PM +0100, Richard Biener wrote:
> On Fri, Mar 24, 2017 at 9:43 AM, Jakub Jelinek wrote:
> > On Fri, Mar 24, 2017 at 09:29:00AM +0100, Richard Biener wrote:
> >> Yeah, the thing BLOCK_NONLOCALIZED_VARS wants to do is optimize generated
> >> dwarf by adding a DW_AT_ab
On Fri, Mar 24, 2017 at 9:43 AM, Jakub Jelinek wrote:
> On Fri, Mar 24, 2017 at 09:29:00AM +0100, Richard Biener wrote:
>> Yeah, the thing BLOCK_NONLOCALIZED_VARS wants to do is optimize generated
>> dwarf by adding a DW_AT_abstract_origin (just to refer to the
>> subprogram DIE) but
>
> Well, for
On Fri, Mar 24, 2017 at 09:29:00AM +0100, Richard Biener wrote:
> Yeah, the thing BLOCK_NONLOCALIZED_VARS wants to do is optimize generated
> dwarf by adding a DW_AT_abstract_origin (just to refer to the
> subprogram DIE) but
Well, for FUNCTION_DECLs in BLOCK_VARS/BLOCK_NONLOCALIZED_VARS we actual
On Fri, Mar 24, 2017 at 8:46 AM, Jakub Jelinek wrote:
> On Thu, Mar 23, 2017 at 05:24:31PM -0400, Jason Merrill wrote:
>> On Thu, Mar 23, 2017 at 4:44 PM, Jakub Jelinek wrote:
>> > The following C testcase shows how profiledbootstrap fails with checking
>> > compiler. We have a (nested) FUNCTION
On Thu, Mar 23, 2017 at 05:24:31PM -0400, Jason Merrill wrote:
> On Thu, Mar 23, 2017 at 4:44 PM, Jakub Jelinek wrote:
> > The following C testcase shows how profiledbootstrap fails with checking
> > compiler. We have a (nested) FUNCTION_DECL inside of BLOCK_VARS of an
> > inline function, when i
On Thu, Mar 23, 2017 at 4:44 PM, Jakub Jelinek wrote:
> The following C testcase shows how profiledbootstrap fails with checking
> compiler. We have a (nested) FUNCTION_DECL inside of BLOCK_VARS of an
> inline function, when it gets inlined, it is moved into
> BLOCK_NONLOCALIZED_VARS. And, decls
Hi!
The following C testcase shows how profiledbootstrap fails with checking
compiler. We have a (nested) FUNCTION_DECL inside of BLOCK_VARS of an
inline function, when it gets inlined, it is moved into
BLOCK_NONLOCALIZED_VARS. And, decls_for_scope calls process_scope_var
with NULL decl and non-
On 09/27/2016 10:38 PM, Jeff Law wrote:
On 09/27/2016 01:30 PM, Martin Sebor wrote:
The attached one line patch increases a local buffer size to
avoid an apparently justified (though in reality likely a false
positive) -Wformat-length warning in varasm.c. The warning has
been reported to break
On 09/27/2016 01:30 PM, Martin Sebor wrote:
The attached one line patch increases a local buffer size to
avoid an apparently justified (though in reality likely a false
positive) -Wformat-length warning in varasm.c. The warning has
been reported to break profiledbootstrap on powerp64le (though
n
The attached one line patch increases a local buffer size to
avoid an apparently justified (though in reality likely a false
positive) -Wformat-length warning in varasm.c. The warning has
been reported to break profiledbootstrap on powerp64le (though
not ordinary bootstrap).
An arguably better s
On 2015.01.16 at 12:03 +0100, Richard Biener wrote:
> On Fri, Jan 16, 2015 at 9:34 AM, Markus Trippelsdorf
> wrote:
> > On 2015.01.16 at 00:15 +0100, Jan Hubicka wrote:
> >> Hi,
> >> can_remove_node_now_p assumes that the node in question has no direct
> >> calls, this
> >> however is not checked
On Fri, Jan 16, 2015 at 9:34 AM, Markus Trippelsdorf
wrote:
> On 2015.01.16 at 00:15 +0100, Jan Hubicka wrote:
>> Hi,
>> can_remove_node_now_p assumes that the node in question has no direct calls,
>> this
>> however is not checked in inline_call that leads to alias to be removed from
>> comdat g
On 2015.01.16 at 00:15 +0100, Jan Hubicka wrote:
> Hi,
> can_remove_node_now_p assumes that the node in question has no direct calls,
> this
> however is not checked in inline_call that leads to alias to be removed from
> comdat group while it should not.
>
> Bootstrapped/regtested x86_64-linux,
Hi,
can_remove_node_now_p assumes that the node in question has no direct calls,
this
however is not checked in inline_call that leads to alias to be removed from
comdat group while it should not.
Bootstrapped/regtested x86_64-linux, comitted.
Honza
PR ipa/64612
* ipa-inline-tra
ok.
David
On Tue, Feb 4, 2014 at 11:28 AM, Rong Xu wrote:
> Hi,
>
> The attached patch fixes the duplicated definition error in
> gcov-tool.c in profiledbootstrap.
>
> Google branch only and tested with profiledbootstrap.
>
> Ok for checking in?
>
> -Rong
Hi,
The attached patch fixes the duplicated definition error in
gcov-tool.c in profiledbootstrap.
Google branch only and tested with profiledbootstrap.
Ok for checking in?
-Rong
Index: gcov-tool.c
===
--- gcov-tool.c (revision 2074
On Sat, Sep 7, 2013 at 10:15 AM, Uros Bizjak wrote:
> Hello!
>
> It looks that it is too hard for the compiler to track deref_align
> initialization through dependent deref_base boolean. The patch bellow
> fixes "may be used uninitialized" warning that breaks
> profiledbootstrap.
>
> 2013-09-07 U
Hello!
It looks that it is too hard for the compiler to track deref_align
initialization through dependent deref_base boolean. The patch bellow
fixes "may be used uninitialized" warning that breaks
profiledbootstrap.
2013-09-07 Uros Bizjak
* ipa-prop.c (ipa_modify_call_arguments): Initial
Sorry for the delay. The patch is ok and I have committed it to the
google branch.
thanks,
David
On Thu, Aug 1, 2013 at 4:51 PM, Dinar Temirbulatov wrote:
> Ping?
> Hi,
> Here is the patch, Tested by profiledbootstrap. Ok for google gcc-4.8?
> thanks, Dinar.
Ping?
Hi,
Here is the patch, Tested by profiledbootstrap. Ok for google gcc-4.8?
thanks, Dinar.
profiledbootstrap-fix1.patch
Description: Binary data
Hi,
Here is the patch, Tested by profiledbootstrap. Ok for google gcc-4.8?
thanks, Dinar.
On Wed, Jul 31, 2013 at 12:01 AM, Rong Xu wrote:
> Will do.
>
> The patch was in gcc-4_7 by Dehao.
>
> r194713 | dehao | 2012-12-24 16:49:06 -0800 (Mon, 24 Dec 2012) | 5 lines
>
> Fix the pro
Will do.
The patch was in gcc-4_7 by Dehao.
r194713 | dehao | 2012-12-24 16:49:06 -0800 (Mon, 24 Dec 2012) | 5 lines
Fix the profiled bootstrap:
1. Set the default value of gcov-debug to be 0.
2. Merge profile summaries from different instrumented binaries.
On Tue, Jul 30, 2013 at 12:58 PM, Xi
Ok. Rong, can you help commit the parameter default setting patch?
thanks,
David
On Tue, Jul 30, 2013 at 12:48 PM, Rong Xu wrote:
> We have seen the issue before. It does fail the profile boostrap as it
> reads a wrong gcda file.
> I thought it had been fixed. (The fix was as David mentioned, s
>I need to understand why this affects profile bootstrap -- is this due
>to file name conflict?
Yes, It is simple. During the profiledbootstrap on x86_64 platform for
libiberty the compiler picks an incorrect profile from the current
directory(non-pic version), while compiling pic version, and
prof
We have seen the issue before. It does fail the profile boostrap as it
reads a wrong gcda file.
I thought it had been fixed. (The fix was as David mentioned, setting
the default value of the parameter to 0).
-Rong
On Tue, Jul 30, 2013 at 12:02 PM, Xinliang David Li wrote:
> I need to understand
I need to understand why this affects profile bootstrap -- is this due
to file name conflict?
The fix is wrong -- please do not remove the parameter. If it is a
problem, a better fix is to change the default parameter value to 0.
David
On Tue, Jul 30, 2013 at 11:56 AM, Teresa Johnson wrote:
>
cc'ing Rong and David since this came from LIPO support.
The patch as-is removes the one use of PARAM_GCOV_DEBUG (which is on
by default) without removing the parameter itself. What is the failure
mode you see from this code?
Thanks,
Teresa
On Tue, Jul 30, 2013 at 11:50 AM, Dinar Temirbulatov
w
Hello
This change allows to complete profiledbootstrap on the google gcc-4.8
branch, tested with make bootstrap with no new regressions. OK for
google 4.8?
thanks, Dinar.
profiledbootstrap-fix.patch
Description: Binary data
On Thu, Jan 5, 2012 at 1:13 AM, Jakub Jelinek wrote:
> Hi!
>
> profiledbootstrap with --disable-poststage-build-with-cxx and -fexceptions
> currently fails on x86_64-linux. The problem is that no EDGE_FAKE edge is
> added from noreturn fatal_error call to the exit block, eventhough
> fatal_error
Hi!
profiledbootstrap with --disable-poststage-build-with-cxx and -fexceptions
currently fails on x86_64-linux. The problem is that no EDGE_FAKE edge is
added from noreturn fatal_error call to the exit block, eventhough
fatal_error calls exit.
The problem is that we have several places which add
On Wed, Dec 21, 2011 at 3:21 PM, Jakub Jelinek wrote:
> Hi!
>
> My profiledbootstrap --enable-checking=release (and lots of other configury
> options) failed this morning, I got warnings (promoted to errors due to
> -Werror*) because our uninitialized warning code couldn't figure these out.
>
> Fi
Hi!
My profiledbootstrap --enable-checking=release (and lots of other configury
options) failed this morning, I got warnings (promoted to errors due to
-Werror*) because our uninitialized warning code couldn't figure these out.
Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux,
Hi!
I've bootstrapped/regtested and profiledbootstrapped/regtested
the following patch from Zdenek on x86_64-linux and i686-linux
and with richi's approval from IRC committed to trunk.
2011-03-07 Zdenek Dvorak
PR bootstrap/48000
* cfgloopmanip.c (fix_bb_placements): Return imm
46 matches
Mail list logo
