From: Vladimir Makarov
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
> programs. It has no s
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
On Fri, Dec 21, 2012 at 10:25 AM, Dodji Seketeli wrote:
> Gabriel Dos Reis 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
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 of
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 problem.
On 12-12-21 4:29 PM, David Miller wrote:
From: Vladimir Makarov
Date: Fri, 21 Dec 2012 16:22:15 -0500
2012-12-21 Vladimir Makarov
PR middle-end/55775
* lra-assigns.c (improve_inheritance): Do nothing after
LRA_MAX_INHERITANCE_PASSES pass.
* lra-constrain
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
PR
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 hook
From: Vladimir Makarov
Date: Fri, 21 Dec 2012 16:22:15 -0500
> 2012-12-21 Vladimir Makarov
>
> PR middle-end/55775
> * lra-assigns.c (improve_inheritance): Do nothing after
> LRA_MAX_INHERITANCE_PASSES pass.
> * lra-constraints.c (MAX_CONSTRAINT_ITERATION_NUMBE
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
PR middle-end/55775
* lra-assigns.c (improve_i
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.,
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
> 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 2012
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 wrote:
> clzll
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 check
On Fri, Dec 21, 2012 at 8:26 AM, Dominique Dhumieres 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
> -Wmissing-format-att
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
sell..
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
PR fortran/55763
* check.c (gfc_check_move_alloc): Handle unlimited polymorphic.
* t
On Fri, Dec 21, 2012 at 1:25 AM, Jan Hubicka 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=0: default, no
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 Fri, Dec 21, 2012 at 9:52 AM, Xinliang David Li 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 AM, Teresa Johns
Kernel build does not link in libgcc, which defines the function.
David
On Fri, Dec 21, 2012 at 8:31 AM, Teresa Johnson wrote:
> On Wed, Dec 19, 2012 at 12:11 PM, Rong Xu wrote:
>> Hi,
>>
>> This patch updates the support for FDO build in linux kernel for gcc 4.7.
>> Tested with 2.6.34 kernel a
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 give
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 11:
On Wed, Dec 19, 2012 at 12:11 PM, Rong Xu 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
>
> * libgcc/libgcov.c
> (gcov_counter_ac
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 -Wno-overlength-s
Gabriel Dos Reis 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 test case of the message:
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
On Fri, Dec 21, 2012 at 5:14 AM, 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 "un
2012/12/21 Gabriel Dos Reis :
> On Fri, Dec 21, 2012 at 3:48 AM, Kai Tietz wrote:
>> 2012/12/21 Paolo Carlini :
>>> 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
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
The example is valid, but I am not sure I understand your explanation...
-- Gaby
On Fri, Dec 21, 2012 at 3:48 AM, Kai Tietz wrote:
> 2012/12/21 Paolo Carlini :
>> 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 f
On Fri, Dec 21, 2012 at 1:59 AM, Kai Tietz 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 llp64
>
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 i686-
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 fron
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
PR b
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
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 size,
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,
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 integral/en
> 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
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 __cplusp
> 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
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)
#
Hi,
On 12/07/2012 06:05 AM, Jason Merrill wrote:
It's perfectly OK to initialize a base class of abstract type; it's
only an error to create a full object of such a type. So this patch
moves the check from more generic initialization code out into a
function that's definitely creating a new o
> On Fri, Dec 21, 2012 at 10:50:58AM +0100, Richard Biener wrote:
> > On Fri, Dec 21, 2012 at 6:33 AM, Alexandre Oliva 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'
> On Fri, Dec 21, 2012 at 10:13 AM, Jan Hubicka wrote:
> >> On Thu, Dec 20, 2012 at 8:20 AM, Jan Hubicka wrote:
> >> >> On Wed, Dec 19, 2012 at 4:29 PM, Andrew Pinski
> >> >> wrote:
> >> >> >
> >> >> > On Wed, Dec 19, 2012 at 12:08 PM, Rong Xu wrote:
> >> >> > > Hi,
> >> >> > >
> >> >> > > Thi
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 i
The testcase in the PR no longer reproduces its issue.
Tested on x86_64-unknown-linux-gnu, applied.
Richard.
2012-12-21 Richard Biener
PR tree-optimization/52996
* gcc.dg/torture/pr52996.c: New testcase.
Index: testsuite/gcc.dg/torture/pr52996.c
2012/12/21 Uros Bizjak :
> 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
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.
> *
On 17 December 2012 14:28, Richard Earnshaw wrote:
> On 21/11/12 11:48, Matthew Gretton-Dann wrote:
>>
>> On 21 November 2012 00:05, Matthias Klose wrote:
>>>
>>>
>>> looks fine, except one missing chunk from my original patch. maybe left
>>> out
>>> intentionally.
>>>
>>>Matthias
>>>
>>> Ind
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.
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.
* config/os/
On Fri, Dec 21, 2012 at 10:13 AM, Jan Hubicka wrote:
>> On Thu, Dec 20, 2012 at 8:20 AM, Jan Hubicka wrote:
>> >> On Wed, Dec 19, 2012 at 4:29 PM, Andrew Pinski wrote:
>> >> >
>> >> > On Wed, Dec 19, 2012 at 12:08 PM, Rong Xu wrote:
>> >> > > Hi,
>> >> > >
>> >> > > This patch adds the supprot
On Fri, Dec 21, 2012 at 10:50:58AM +0100, Richard Biener wrote:
> On Fri, Dec 21, 2012 at 6:33 AM, Alexandre Oliva 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,
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 6:33 AM, Alexandre Oliva 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 cases, which,
2012/12/21 Paolo Carlini :
> 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 ABI-trouble due long i
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.
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 simple_alu_i
2012/12/21 Paolo Carlini :
> 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
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 stdint_
> 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=0: default, no atomic update
> N=1: atomic update edge counters.
> N=2: a
2012/12/21 Paolo Carlini :
> 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 @@ namesp
> On Thu, Dec 20, 2012 at 8:20 AM, Jan Hubicka wrote:
> >> On Wed, Dec 19, 2012 at 4:29 PM, Andrew Pinski wrote:
> >> >
> >> > On Wed, Dec 19, 2012 at 12:08 PM, Rong Xu wrote:
> >> > > Hi,
> >> > >
> >> > > This patch adds the supprot of atomic update the profile counters.
> >> > > Tested with g
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 __cla
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 __offset
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 not
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 S
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.
* libsupc+
72 matches
Mail list logo
