Martin Liška mli...@suse.cz writes:
diff --git a/gcc/testsuite/g++.dg/ipa/ipa-icf-4.C
b/gcc/testsuite/g++.dg/ipa/ipa-icf-4.C
index 9d17889..9434289 100644
--- a/gcc/testsuite/g++.dg/ipa/ipa-icf-4.C
+++ b/gcc/testsuite/g++.dg/ipa/ipa-icf-4.C
@@ -44,5 +44,5 @@ int main()
}
/* {
;
+}
+
+/* { dg-final { scan-ipa-dump Semantic equality hit:bar-foo icf } } */
+/* { dg-final { scan-ipa-dump Equal symbols: 1 icf } } */
+/* { dg-final { cleanup-ipa-dump icf } } */
FAIL: gcc.dg/ipa/ipa-icf-21.c (test for excess errors)
Excess errors:
/usr/local/gcc/gcc-20141019/gcc/testsuite
I am surprised you hit the size limits with 4.9 only - for quite some time
we keep all virtual functions in callgarph until inlining. In fact 4.9 is
first
that works harder to drop them early (because I hit the problem with LTO
where they artifically bloat the size of LTO object
Hi Honza,
As David says, we will do some more experiments with the change you
suggest and speculative devirtualization, but we needed to make this
change in part to get an internal release out. One of the issues was a
recent change to cp/decl2.c to make virtual function decls needed
under
On 18 October 2014 14:43, Anthony Brandon anthony.bran...@gmail.com wrote:
Never mind about functions.texi. I figured out how to do it.
Here is the new diff and changelog.
libiberty/ChangeLog:
2014-10-18 Anthony Brandon anthony.bran...@gmail.com
* filename_cmp.c (filename_eq): No
+/* Member functions for string_table. */
+
+string_table *
+string_table::create ()
Why this is not a constructor?
We use static initializer because it's not suggested to put too much
logic in constructor.
Why not? :)
+}
The two hunks probably can be unified.
Why
Honza, not sure if this patch is idea, but this will unblock mingw
build problems. Can this one get in?
Hmm, the patch is somewhat ugly and I do not know why MingW32 defines mkdir
macro and how. If Kai Tietz or other MingW32 maintainer is OK about it, the
patch is OK.
Honza
thanks,
David
On 17-10-14 21:24, Eric Botcazou wrote:
Let's look at the effect of the option (after the recent fix for PR61605) on
gcc.target/i386/fuse-calller-save.c:
...
foo:
.LFB1:
.cfi_startproc
- pushq %rbx
- .cfi_def_cfa_offset 16
- .cfi_offset 3, -16
- movl
On Sat, Oct 18, 2014 at 12:04:36PM -0400, John David Anglin wrote:
The attached change removes support for the hppa -mjump-in-delay
option.
You can set it to Ignored in pa.opt now? And remove the mask?
Segher
On 19-Oct-14, at 10:13 AM, Segher Boessenkool wrote:
You can set it to Ignored in pa.opt now? And remove the mask?
Yes, will update.
Dave
--
John David Anglin dave.ang...@bell.net
It turned out that there is another spot where we need to allow
initializing objects with static storage duration with compound
literals even in C99 -- when the compound literal is inside the
initializer. Fixed in the same way as previously.
Bootstrapped/regtested on x86_64-linux, ok for trunk?
On Sun, Oct 19, 2014 at 1:19 AM, Jan Hubicka hubi...@ucw.cz wrote:
I am surprised you hit the size limits with 4.9 only - for quite some time
we keep all virtual functions in callgarph until inlining. In fact 4.9 is
first
that works harder to drop them early (because I hit the problem
On Sat, 18 Oct 2014, Manuel López-Ibáñez wrote:
What about this version?
Give an error whenever the @dfn{base standard} (see @option{-Wpedantic})
requires a diagnostic, in cases where there is undefined behavior at
compile-time
Only in *some* such cases of compile-time undefined behavior.
On Sat, 18 Oct 2014, Krzesimir Nowak wrote:
+ pp_verbatim (context-printer,
+%s from %r%s:%d%R, prefix, locus,
+ diagnostic_report_from (context, map, In file included);
We don't want to split up diagnostic text like that, because for
translation it may be necessary to
On Sun, 19 Oct 2014, Marek Polacek wrote:
It turned out that there is another spot where we need to allow
initializing objects with static storage duration with compound
literals even in C99 -- when the compound literal is inside the
initializer. Fixed in the same way as previously.
In opts.c I have:
case OPT_fprofile_use:
if (!opts_set-x_flag_branch_probabilities)
...
/* Indirect call profiling should do all useful transformations
speculative devirtualization does. */
if (!opts_set-x_flag_devirtualize_speculatively
It was in upstream 4.9 branch, but got reverted in r215061 to fix
PR/61214 and PR/62224.
David
On Sun, Oct 19, 2014 at 1:21 AM, Jan Hubicka hubi...@ucw.cz wrote:
Hi Honza,
As David says, we will do some more experiments with the change you
suggest and speculative devirtualization, but we
Panu-Kristian Poiksalo poiks...@gmail.com writes:
Chapter 16.4, RTL Template in GCC Internals Manual, found at
https://gcc.gnu.org/onlinedocs/gccint/RTL-Template.html#RTL-Template
has this to say about match_scratch:
(match_scratch:m n constraint)
This expression is also a placeholder
Hello,
looking though the patterns on the branch (not specifically the ones
attached here), I am surprised to see so few calls to has_single_use. In
RTL-land, we don't even valueize if there are several uses, so the
question doesn't occur. In generic, we assume everything is single use
(CSE
Hello!
The fix that fixed scheduler issues with AND addresses (the fix
prevented early exit for MEM_READONLY_P addresses when AND alignment
addresses were involved) caused some fall-out for libgo testsuite.
These tests triggered an assert in mems_in_disjoint_alias_sets_p,
which checks for zero
Hello,
This patch sets SECTION_EXCLUDE flag for LTO sections in lhd_begin_section and
fixes the corresponding gcc_GAS_CHECK_FEATURE.
Approved here: https://gcc.gnu.org/ml/gcc-patches/2014-10/msg01535.html
Bootstrapped and regtested on i686-linux and x86_64-linux, committed to trunk.
-- Ilya
It was in upstream 4.9 branch, but got reverted in r215061 to fix
PR/61214 and PR/62224.
Was your tests done with this change or without?
Honza
On Friday 2014-10-17 13:34, Jonathan Wakely wrote:
Index: htdocs/gcc-5/changes.html
===
@@ -128,12 +164,13 @@
ul
li Class codestd::experimental::any/code; /li
li Function template
On Friday 2014-10-10 11:53, Kyrill Tkachov wrote:
This adds a new configure-time option --enable-fix-cortex-a53-835769 that
will enable the Cortex-A53 erratum fix by default
so you don't have to specify -mfix-cortex-a53-835769 every time.
Documentation in install.texi is added.
Thank you.
Hello.
I added missing gimple_asm_string comparison for a function with an asm
statement.
Bootstrap and regression tests still running, ready for trunk after it finishes?
Thank you,
Martin
gcc/ChangeLog:
2014-10-19 Martin Liska mli...@suse.cz
* ipa-icf-gimple.c
Hello.
I added missing gimple_asm_string comparison for a function with an asm
statement.
Bootstrap and regression tests still running, ready for trunk after it
finishes?
OK.
(I remember pointing this out at review :))
Honza
Thank you,
Martin
gcc/ChangeLog:
2014-10-19 Martin
Looks good to me. Thanks for taking care of F2003's IEEE support.
Committed as rev. 216443, thanks for the review.
PS: You might want to browse through the current (F2008 + corrigenda
+ first F2015 additions) draft at
http://j3-fortran.org/doc/year/14/14-007r2.pdf
See especially the
On Sat, Oct 18, 2014 at 4:19 PM, Xinliang David Li davi...@google.com wrote:
On Sat, Oct 18, 2014 at 3:27 PM, Jan Hubicka hubi...@ucw.cz wrote:
The difference in instrumentation runtime is huge -- as topn profiler
is pretty expensive to run.
With FDO, it is probably better to make early
No, Google branch is not yet synced to 4.9 head.
David
On Sun, Oct 19, 2014 at 1:11 PM, Jan Hubicka hubi...@ucw.cz wrote:
It was in upstream 4.9 branch, but got reverted in r215061 to fix
PR/61214 and PR/62224.
Was your tests done with this change or without?
Honza
Hi Jakub,
Thanks for your feedback! See below.
On Thu, Oct 16, 2014 at 4:52 PM, Jakub Jelinek ja...@redhat.com wrote:
On Mon, Oct 13, 2014 at 10:16:19PM +0100, Nathaniel Smith wrote:
Got total silence the last 4 times I posted this, and users have been
bugging me about it offline, so trying
30 matches
Mail list logo