Hello,
this patch fixes some remaining issues with pointer-sizes for llp64
abi in libstdc++.
ChangeLog
2012-12-21 Kai Tietz
* config/os/mingw32/os_defines.h (_GLIBCXX_LLP64): Define if llp64
abi is used.
* config/os/mingw32-w64/os_defines.h: Likewise.
*
We checked also spec2000 and eembc_2_0 on Atom - no visible regressions and
gains
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On
Behalf Of Xinliang David Li
Sent: Friday, December 21, 2012 11:26 AM
To: Jan Hubicka
Cc: GCC Patches; Ahmad
So far we see a regression on one of eembc_1_1 test because of following change:
/* X86_TUNE_USE_VECTOR_FP_CONVERTS: Prefer vector packed SSE conversion
from FP to FP. */
- m_CORE2I7 | m_AMDFAM10 | m_GENERIC,
+ m_AMDFAM10 | m_GENERIC,
Probably we should keep it as is while there is
On Fri, 21 Dec 2012, Kai Tietz wrote:
2012-12-21 Kai Tietz
* config/os/mingw32/os_defines.h (_GLIBCXX_LLP64): Define if llp64
abi is used.
* config/os/mingw32-w64/os_defines.h: Likewise.
* libsupc++/cxxabi.h (__base_class_type_info): Change
type
Hi,
On 12/21/2012 08:59 AM, Kai Tietz wrote:
Index: libsupc++/cxxabi.h
===
--- libsupc++/cxxabi.h (Revision 194655)
+++ libsupc++/cxxabi.h (Arbeitskopie)
@@ -356,7 +356,7 @@ namespace __cxxabiv1
{
public:
const
On Thu, Dec 20, 2012 at 8:20 AM, Jan Hubicka hubi...@ucw.cz wrote:
On Wed, Dec 19, 2012 at 4:29 PM, Andrew Pinski pins...@gmail.com wrote:
On Wed, Dec 19, 2012 at 12:08 PM, Rong Xu x...@google.com wrote:
Hi,
This patch adds the supprot of atomic update the profile counters.
2012/12/21 Paolo Carlini paolo.carl...@oracle.com:
Hi,
On 12/21/2012 08:59 AM, Kai Tietz wrote:
Index: libsupc++/cxxabi.h
===
--- libsupc++/cxxabi.h (Revision 194655)
+++ libsupc++/cxxabi.h (Arbeitskopie)
@@ -356,7 +356,7
Hi,
This patch adds support of atomic update of profiles counters. The goal is to
improve
the poor counter values for highly thread programs.
The atomic update is under a new option -fprofile-gen-atomic=N
N=0: default, no atomic update
N=1: atomic update edge counters.
N=2: atomic
Hi,
On 12/21/2012 10:16 AM, Kai Tietz wrote:
Well, I thought it is always present for gcc due gstdint.h header,
As far as I know, that project isn't finished yet, there are still
targets which neither provide the header, neither GCC synthetizes it.
See also dg-require-effective-target
2012/12/21 Paolo Carlini paolo.carl...@oracle.com:
Hi,
On 12/21/2012 10:16 AM, Kai Tietz wrote:
Well, I thought it is always present for gcc due gstdint.h header,
As far as I know, that project isn't finished yet, there are still targets
which neither provide the header, neither GCC
On 20/12/12 18:06, Greta Yorsh wrote:
Currently, GCC uses generic ARMv7-A tuning for Cortex-A7.
This patch adds an initial pipeline description for Cortex-A7. Details:
* integer/vfp is based on the pipeline description for Cortex-A5,
* models dual issue in limited circumstances using
On 12/21/2012 10:36 AM, Kai Tietz wrote:
well, issue isn't that 'long' is always 'ptrdiff_t'.
But then, if we just change the type without paying attention to size
(and alignment) aren't we looking for BIG ABI trouble?!?
Paolo.
2012/12/21 Paolo Carlini paolo.carl...@oracle.com:
On 12/21/2012 10:36 AM, Kai Tietz wrote:
well, issue isn't that 'long' is always 'ptrdiff_t'.
But then, if we just change the type without paying attention to size (and
alignment) aren't we looking for BIG ABI trouble?!?
Huh? We have
On Fri, Dec 21, 2012 at 6:33 AM, Alexandre Oliva aol...@redhat.com wrote:
libmudflap emits a global initializer that registers memory ranges for
global data symbols. However, even if IPA decides not to emit a symbol
because it's unused, we'd still emit registration sequences for them in
some
On 12/21/2012 04:02 AM, Gerald Pfeifer wrote:
PING.
On Fri, 2 Nov 2012, Gerald Pfeifer wrote:
Rainer (or others),
the FAQ entry below seems obsolete to me (dates back more than a
decade). Shall we remove it, or is there something else we still
should document (in addition to
On Fri, Dec 21, 2012 at 10:50:58AM +0100, Richard Biener wrote:
On Fri, Dec 21, 2012 at 6:33 AM, Alexandre Oliva aol...@redhat.com wrote:
libmudflap emits a global initializer that registers memory ranges for
global data symbols. However, even if IPA decides not to emit a symbol
because
On Fri, Dec 21, 2012 at 10:13 AM, Jan Hubicka hubi...@ucw.cz wrote:
On Thu, Dec 20, 2012 at 8:20 AM, Jan Hubicka hubi...@ucw.cz wrote:
On Wed, Dec 19, 2012 at 4:29 PM, Andrew Pinski pins...@gmail.com wrote:
On Wed, Dec 19, 2012 at 12:08 PM, Rong Xu x...@google.com wrote:
Hi,
Hello,
I changed type in __base_class_type_info to wide-enough type only for
llp64 target.
By this we have no (new) side-effects to other targets.
ChangeLog
2012-12-21 Kai Tietz
* config/os/mingw32/os_defines.h (_GLIBCXX_LLP64): Define if llp64
abi is used.
*
On 12/21/2012 10:59 AM, Kai Tietz wrote:
Hello,
I changed type in __base_class_type_info to wide-enough type only for
llp64 target.
By this we have no (new) side-effects to other targets.
Thanks. This is something I can approve at once.
Paolo.
On 17 December 2012 14:28, Richard Earnshaw rearn...@arm.com wrote:
On 21/11/12 11:48, Matthew Gretton-Dann wrote:
On 21 November 2012 00:05, Matthias Klose d...@ubuntu.com wrote:
looks fine, except one missing chunk from my original patch. maybe left
out
intentionally.
Matthias
Hello!
2012-12-21 Kai Tietz
* config/os/mingw32/os_defines.h (_GLIBCXX_LLP64): Define if llp64
abi is used.
* config/os/mingw32-w64/os_defines.h: Likewise.
* libsupc++/cxxabi.h (__base_class_type_info): Adjust
type of __offset_flags for llp64.
*
2012/12/21 Uros Bizjak ubiz...@gmail.com:
Hello!
2012-12-21 Kai Tietz
* config/os/mingw32/os_defines.h (_GLIBCXX_LLP64): Define if llp64
abi is used.
* config/os/mingw32-w64/os_defines.h: Likewise.
* libsupc++/cxxabi.h (__base_class_type_info): Adjust
type
The testcase in the PR no longer reproduces its issue.
Tested on x86_64-unknown-linux-gnu, applied.
Richard.
2012-12-21 Richard Biener rguent...@suse.de
PR tree-optimization/52996
* gcc.dg/torture/pr52996.c: New testcase.
Index: testsuite/gcc.dg/torture/pr52996.c
Alexandre Oliva wrote:
Revision 193063 brought in calls to fraiseexcept() into libquadmath,
which caused a build regression on Fedora 16 (BLAG 160k actually) x86_64
while building an i686-linux-gnu native toolchain.
The problem is that glibc has an extern inline definition of
fraiseexcept that
On Fri, Dec 21, 2012 at 10:13 AM, Jan Hubicka hubi...@ucw.cz wrote:
On Thu, Dec 20, 2012 at 8:20 AM, Jan Hubicka hubi...@ucw.cz wrote:
On Wed, Dec 19, 2012 at 4:29 PM, Andrew Pinski pins...@gmail.com
wrote:
On Wed, Dec 19, 2012 at 12:08 PM, Rong Xu x...@google.com wrote:
Hi,
The following patch includes gmp.h from system.h (where it belongs,
as a system header). This makes sure the libintl kludge we put in
place with --disable-nls doesn't wreck the gmp header (well, hopefully,
earlier fix to be reverted?).
This runs into issues with Ada which basically does (ugh)
This runs into issues with Ada which basically does (ugh)
#ifdef __cplusplus
extern C {
#endif
#include system.h
#endif __cplusplus
}
#endif
which of course wrecks any system header doing __cplusplus
conditional stuff (such as gmp.h including iosfwd). The patch
un-kludges this
On Fri, 21 Dec 2012, Eric Botcazou wrote:
This runs into issues with Ada which basically does (ugh)
#ifdef __cplusplus
extern C {
#endif
#include system.h
#endif __cplusplus
}
#endif
which of course wrecks any system header doing __cplusplus
conditional stuff (such
The patch looks fine to me (though it's not 100% obvious to me as not only
system headers are now no longer in extern C - which is why I simply
tried the un-wrapping). So, if you think that's ok please commit it.
Done.
--
Eric Botcazou
Hello,
In the example accompanying this patch, check_instantiated_arg tries
to ensure that a non-type template argument should be a constant if it
has integral or enumeration type.
The problem is that an alias template which type-id is, e.g, an
integer, looks like an argument that has
On Fri, 21 Dec 2012, Eric Botcazou wrote:
The patch looks fine to me (though it's not 100% obvious to me as not only
system headers are now no longer in extern C - which is why I simply
tried the un-wrapping). So, if you think that's ok please commit it.
Done.
Thanks.
Richard.
Hello,
this patch fixes the layout of the info_type-s for llp64 target. We
used here 'long' type instead
of pointer-scalar-witdth type. That's an issue for llp64 targets.
I introduced for that the LONGPTR_T helper-macro, which has either
scalar-size of 'long' type, or
of 'long long' type's
This patch widens the scope of the PR54838 fix (and reverts one
of my earlier kludges). Like on the GIMPLE level threading
an edge through a loop header is seldomly a good idea as it may
create multiple-entry loops (not handled at all), create
irreducible regions (not handled at all).
In theory
After the Ada fix including gmp.h from system.h is trivial
(by means of double-int.h it was included from almost everywhere
already)
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Ok for trunk? (Go frontend unfixed)
Thanks,
Richard.
2012-12-21 Richard Biener
On Fri, Dec 21, 2012 at 02:14:37PM +0100, Richard Biener wrote:
After the Ada fix including gmp.h from system.h is trivial
(by means of double-int.h it was included from almost everywhere
already)
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Ok for trunk? (Go frontend
Hi!
Peter reported on IRC that many asan tests are failing on ppc, apparently
lots of that is related to -fsection-anchors decls and STRING_CSTs, which
weren't considered before.
This patch attempts to fix all the issues I saw on the few testcases.
Bootstrapped/regtested on x86_64-linux and
On Fri, Dec 21, 2012 at 1:59 AM, Kai Tietz ktiet...@googlemail.com wrote:
Hello,
this patch fixes some remaining issues with pointer-sizes for llp64
abi in libstdc++.
See comments below.
ChangeLog
2012-12-21 Kai Tietz
* config/os/mingw32/os_defines.h (_GLIBCXX_LLP64): Define if
On Fri, Dec 21, 2012 at 3:48 AM, Kai Tietz ktiet...@googlemail.com wrote:
2012/12/21 Paolo Carlini paolo.carl...@oracle.com:
On 12/21/2012 10:36 AM, Kai Tietz wrote:
well, issue isn't that 'long' is always 'ptrdiff_t'.
But then, if we just change the type without paying attention to size
The example is valid, but I am not sure I understand your explanation...
-- Gaby
This change introduced a large number of failures on AIX of the form:
cannot specify -o with -c, -S or -E with multiple files
during the execution of collect2-ld
Note that all of the tests are run with the gcc driver, not the
gfortran driver. If I use gfortran, every thing is okay.
I still am
2012/12/21 Gabriel Dos Reis g...@integrable-solutions.net:
On Fri, Dec 21, 2012 at 3:48 AM, Kai Tietz ktiet...@googlemail.com wrote:
2012/12/21 Paolo Carlini paolo.carl...@oracle.com:
On 12/21/2012 10:36 AM, Kai Tietz wrote:
well, issue isn't that 'long' is always 'ptrdiff_t'.
But then, if
On Fri, Dec 21, 2012 at 5:14 AM, Richard Biener rguent...@suse.de wrote:
After the Ada fix including gmp.h from system.h is trivial
(by means of double-int.h it was included from almost everywhere
already)
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Ok for trunk? (Go
On Fri, Dec 21, 2012 at 10:50:26AM -0500, David Edelsohn wrote:
This change introduced a large number of failures on AIX of the form:
cannot specify -o with -c, -S or -E with multiple files
during the execution of collect2-ld
Note that all of the tests are run with the gcc driver, not
Gabriel Dos Reis g...@integrable-solutions.net writes:
The example is valid, but I am not sure I understand your
explanation...
Ah, sorry. I realize just now that I haven't mentioned the initial
erratic behaviour. Maybe that could have made my message easier to
understand.
So consider the
I think revision 194665 breaks bootstrap on at least x86_64-apple-darwin10:
g++ -c -g -DIN_GCC -fno-exceptions -fno-rtti -fasynchronous-unwind-tables
-W -Wall -Wno-narrowing -Wwrite-strings -Wcast-qual -Wmissing-format-attribute
-pedantic -Wno-long-long -Wno-variadic-macros
On Wed, Dec 19, 2012 at 12:11 PM, Rong Xu x...@google.com wrote:
Hi,
This patch updates the support for FDO build in linux kernel for gcc 4.7.
Tested with 2.6.34 kernel and google internal benchmarks.
Thanks,
-Rong
2012-12-19 Rong Xu x...@google.com
* libgcc/libgcov.c
Jakub,
I deleted the directory that failed and commented out that option for
last nights bootstrap.
I can't reproduce the failure today when I restore the option. I
wonder if there is a race condition with parallel check. I will
continue to monitor it.
Thanks, David
On Fri, Dec 21, 2012 at
General background: Private module variables and module procedures can
be marked as TREE_PUBLIC()= 0, unless they are used in the specification
expression of the dummy argument or result variable of public module
procedures (or private module procedures in public generic interfaces).
That
Kernel build does not link in libgcc, which defines the function.
David
On Fri, Dec 21, 2012 at 8:31 AM, Teresa Johnson tejohn...@google.com wrote:
On Wed, Dec 19, 2012 at 12:11 PM, Rong Xu x...@google.com wrote:
Hi,
This patch updates the support for FDO build in linux kernel for gcc 4.7.
On Fri, Dec 21, 2012 at 9:52 AM, Xinliang David Li davi...@google.com wrote:
Kernel build does not link in libgcc, which defines the function.
Then will the same issue occur with the reference to __builtin_clzll
in gcov_histo_index in gcov-io.c?
Teresa
David
On Fri, Dec 21, 2012 at 8:31
On Fri, 2012-12-21 at 15:15 +0100, Jakub Jelinek wrote:
Hi!
Peter reported on IRC that many asan tests are failing on ppc, apparently
lots of that is related to -fsection-anchors decls and STRING_CSTs, which
weren't considered before.
This patch attempts to fix all the issues I saw on the
On Fri, Dec 21, 2012 at 1:25 AM, Jan Hubicka hubi...@ucw.cz wrote:
Hi,
This patch adds support of atomic update of profiles counters. The goal is
to improve
the poor counter values for highly thread programs.
The atomic update is under a new option -fprofile-gen-atomic=N
N=0: default, no
Fix one of the remaining issues of PR 55763: MOVE_ALLOC with CLASS(*)
either for both arguments or only for TO=.
Build and regtested on x86-64-gnu-linux.
OK for the trunk?
Tobias
2012-12-21 Tobias Burnus bur...@net-b.de
PR fortran/55763
* check.c (gfc_check_move_alloc): Handle unlimited
On Fri, 2012-12-21 at 08:46 +0100, Jakub Jelinek wrote:
Given Alex' comments in the PR, the second hunk is definitely ok for trunk,
the first one can be applied too (but you can skip it too if you want, it
shouldn't make a difference).
Thanks, I checked in both chunks.
Steve Ellcey
On Fri, Dec 21, 2012 at 8:26 AM, Dominique Dhumieres domi...@lps.ens.fr wrote:
I think revision 194665 breaks bootstrap on at least x86_64-apple-darwin10:
g++ -c -g -DIN_GCC -fno-exceptions -fno-rtti -fasynchronous-unwind-tables
-W -Wall -Wno-narrowing -Wwrite-strings -Wcast-qual
The gcc.target/arm/ftest-*.c tests check various macros that are set for
ARM targets by setting flags based on preprocessor directives that check
those macros. The tests are skipped if the current test platform
doesn't support executing programs for the architecture for which flags
are being
that's in the histrogram merge. we disable the merge in kernel fdo
(instead, we do offline merge).
-Rong
On Fri, Dec 21, 2012 at 9:55 AM, Teresa Johnson tejohn...@google.com wrote:
clzll
That doesn't make much sense. What do the lines around this look like?
I am currently bootstrapping r194675 with revision 194665 reverted.
The diff between gcc/auto-host.h with(-)/without(+) r194665 reverted
and --enable-checking=release for (+) looks like:
--- ../build_w/gcc/auto-host.h
On 12/20/2012 03:23 AM, Thomas Koenig wrote:
Ping?
Thomas
Hi Janus,
Oops, right. Here is the correct one.
Yes, OK
Regards,
Jerry
I think I understand at least part of the problem:
(1) I configure gcc with
../work/configure --prefix=/opt/gcc/gcc4.8w
--enable-languages=c,c++,fortran,objc,obj-c++,java,ada,lto --with-gmp=/opt/mp
--with-system-zlib --with-isl=/opt/mp --enable-lto --enable-plugin
--enable-build-with-cxx
i.e.,
The following patch fixes
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55775
The bug is related to PR55330.
The patch was successfully bootstrapped and tested on x86/x86-64.
Committed as rev. 194680.
2012-12-21 Vladimir Makarov vmaka...@redhat.com
PR middle-end/55775
*
From: Vladimir Makarov vmaka...@redhat.com
Date: Fri, 21 Dec 2012 16:22:15 -0500
2012-12-21 Vladimir Makarov vmaka...@redhat.com
PR middle-end/55775
* lra-assigns.c (improve_inheritance): Do nothing after
LRA_MAX_INHERITANCE_PASSES pass.
*
The more you dig into the UNSPEC_VOLATILE/asm volatile business, the more you
find that the manual, the code and the comments entirely disagree about them.
At least something is consistent: as documented in rtl.def, UNSPEC_VOLATILE
can trap. That's why it was wrong for the unspec_may_trap_p
Another two fixes for CLASS(*). (We really should audit all calls to
gfc_find_derived_vtab for possible issues with CLASS(*).)
If I haven't miscounted, there is still one other failure in the PR.
Build and regtested on x86-64-gnu-linux.
OK for the trunk?
Tobias
2012-12-21 Tobias Burnus
On 12-12-21 4:29 PM, David Miller wrote:
From: Vladimir Makarov vmaka...@redhat.com
Date: Fri, 21 Dec 2012 16:22:15 -0500
2012-12-21 Vladimir Makarov vmaka...@redhat.com
PR middle-end/55775
* lra-assigns.c (improve_inheritance): Do nothing after
In Go it is an error if the same name is defined in both the package
block (names defined anywhere in the package) and the file block (names
defined only in a single file, which can only happen via import
statements). The gccgo frontend was not diagnosing this error. This
patch fixes the
This is a regression present on mainline and 4.7 branch, in the form of an
assertion failure. The check that an RACW type is acceptable an actual for a
formal type to which a pragma Remote_Access applies need not be performed on a
defaulted formal type appearing in the internal instantiation
On Fri, Dec 21, 2012 at 10:25 AM, Dodji Seketeli do...@redhat.com wrote:
Gabriel Dos Reis g...@integrable-solutions.net writes:
The example is valid, but I am not sure I understand your
explanation...
Ah, sorry. I realize just now that I haven't mentioned the initial
erratic behaviour.
On Fri, 21 Dec 2012, Dominique Dhumieres wrote:
I think revision 194665 breaks bootstrap on at least x86_64-apple-darwin10:
Same on everything FreeBSD.
PR 55784 - [4.8 regression] declaration of C function 'const char*
strsignal(int)' conflicts with /usr/include/string.h:112: error: previous
From: Vladimir Makarov vmaka...@redhat.com
Date: Fri, 21 Dec 2012 17:04:34 -0500
If I go this way, it will be another reload which is trying to do
everything at once. Also after 2 passes the inheritance improve code
(as inheritance code itself) usually does nothing for big majority of
70 matches
Mail list logo
