Rainer Orth writes:
> The patch allowed the bootstrap to complete successfully, but since
> libgo.so now has undefined references to log2 and trunc, all Go
> execution still fail.
I went ahead and removed the uses of the libc log2 and trunc functions,
on the basis that although they are in C99 t
Hi!
On the following testcase we ICE, because now that clone_tree_hash
inserts elements into decl_table hash table, if the hash table is expanded
during that, storing into *slot might either store it at a wrong spot, or
in completely unrelated memory.
Fixed by storing into the hash table immediat
Hi!
As mentioned in the PR, if SET_SRC MEM (with REG_EQUIV note) isn't a MEM
that can be used in general operations, but requires some specialized
instruction to load it, decreasing its mem_cost results sometimes in
worse code, the MEM is loaded with a specialized insn, then stored into
a stack sl
Hi!
On this testcase we ICE, because slpeel_tree_peel_loop_to_edge is first
called with a loop that has a virtual PHI and no virtual PHI in the loop
exit bb and this function doesn't update the vop properly after inserting
second loop and adding all the conditional guards, then vect_loop_versionin
The gccgo frontend was crashing on code like
type T1 struct { F1 *[1]T2 }
type T2 struct { X int }
It needed to know the size of T2 before it was computed. This patch
fixes the problem. I added a testcase as bug417.go in the master Go
testsuite, which I will merged into the gccgo testsuite late
Hi!
Bootstrapped/regtested on x86_64-linux and i686-linux,
committed to trunk as obvious:
2012-02-16 Jakub Jelinek
PR translation/52264
* cgraphunit.c (verify_cgraph_node): Fix a typo.
--- gcc/cgraphunit.c.jj 2012-02-12 15:46:32.0 +0100
+++ gcc/cgraphunit.c2012-02
On 02/15/2012 08:24 PM, Jack Howarth wrote:
On Wed, Feb 15, 2012 at 11:48:29PM +, Iain Sandoe wrote:
On 15 Feb 2012, at 23:39, Patrick Marlier wrote:
PR libitm/52220
* alloc_cpp.cc: No dummy definitions for darwin.
* eh_cpp.cc: Likewise.
libgcc/
2012-02-15 Iain S
On Wed, Feb 15, 2012 at 11:48:29PM +, Iain Sandoe wrote:
>
> On 15 Feb 2012, at 23:39, Patrick Marlier wrote:
>
>> PR libitm/52220
>> * alloc_cpp.cc: No dummy definitions for darwin.
>> * eh_cpp.cc: Likewise.
>>
>> libgcc/
>> 2012-02-15 Iain Sandoe
>> Patrick Marlier
On 15 February 2012 17:44, Rainer Orth wrote:
>
> Ok for mainline?
Yes, although I would prefer the config option to be
--enable-libstdcxx-gthreads or --enable-libstdcxx-threads, since I
prefer it to be clear that a config option relates just to libstdc++,
and this only seems to affect libstdc++ n
On 15 Feb 2012, at 23:39, Patrick Marlier wrote:
PR libitm/52220
* alloc_cpp.cc: No dummy definitions for darwin.
* eh_cpp.cc: Likewise.
libgcc/
2012-02-15 Iain Sandoe
Patrick Marlier
PR libitm/52220
* config/darwin-crt-tm.c: Generate d
Hi,
In PR/52220, eh-1.C fails because the dummy function
__cxa_allocate_exception is used instead of the one from libstdc++.
Indeed, the function is defined in libitm and thus is used.
The fix consists to move the dummy function definitions to darwin-crt-tm.c.
This fix was mainly contributed
This patch is another installment in my endless(?) series of copy-edits
to the GCC manual. Here, I've fixed up a number of places to uniformly
use e.g. "32-bit int" instead of sometimes that and sometimes "32 bit"
and sometimes "32bit". And, as a noun phrase, it's properly "32 bits".
Likewis
This patch merges inline functions where a plain declaration is
read from a pph file after an earlier pph file provides the full
definition.
Making this merge happen required refactoring the several pph
routines. Ideally, we should have merge versions of the common tree
streamer. However, we liv
On Feb 15, 2012, at 11:18 AM, Patrick Marlier wrote:
> PS: this is my first commit so I hope I get it right. Otherwise do not
> hesitate to yell at me.
Thanks. I know of two people that will test and yell, if anything goes wrong.
I'll spin up a build...
Rainer Orth writes:
> 2012-02-13 Rainer Orth
>
> * configure.ac (libgo_cv_lib_makecontext_stack_top): Set on
> sparc*-*-solaris2.[89]*.
> (MAKECONTEXT_STACK_TOP): Define if appropriate.
> (libgo_cv_lib_setcontext_clobbers_tls): Use it.
> * configure: Regenerate.
>
Hello Everyone,
This patch is for the Cilkplus branch affecting the C and C++ compilers.
This patch will fix a problem in metadata dwarf code generation part when we
are trying to catch the register number when a spill occurs.
Thanks,
Balaji V. Iyer.
diff --git a/gcc/ChangeLog.cilk b
On Feb 15, 2012, at 11:09 AM, Steven Bosscher wrote:
> On Wed, Feb 15, 2012 at 7:59 PM, DJ Delorie wrote:
>>
>> I would classify these as "obvious" but please make sure they go in
>> before the poison patch ;-)
>>
>>> And this will be needed to make those ports do what was intended:
>>>
>>>
> > The typo is obvious, but whether or not the wrapped code still works
> > isn't (to me).
> >
> > * src/c++98/locale.cc (locale::facet::_S_get_c_locale): Fix
> > typo.
>
> Please check this in.
Done. Thanks!
This patch introduces a new file builtins.def that is used as central registry
to hold built-ins' information.
The file is used by defining DEF_BUILTIN macre and then including the file as
described in the head comment of builtins.def.
Up to here it's all code clean-up and no functional change.
> The typo is obvious, but whether or not the wrapped code still works
> isn't (to me).
>
> * src/c++98/locale.cc (locale::facet::_S_get_c_locale): Fix
> typo.
Please check this in.
-benjamin
The typo is obvious, but whether or not the wrapped code still works
isn't (to me).
* src/c++98/locale.cc (locale::facet::_S_get_c_locale): Fix typo.
Index: libstdc++-v3/src/c++98/locale.cc
===
--- libstdc++-v3/src/c++98/loc
As per request, syncs libstdc++ with libjava for installation of python
modules using top-level --with-python-dir configure argument.
I think we are going to need GLIBCXX_ENABLE_PYTHON anyway for the other
python bug report.
-benjamin
tested x86/linux
tested x86/linux --with-python-dir=/share/p
On Wed, Feb 15, 2012 at 1:32 PM, Michael Meissner
wrote:
> This patch fixes PR 52199, which shows up on power7 systems when the compiler
> is vectorizing DImode operations, and the vector duplicate code was not
> prepared to accept CONST and SYMBOL_REF's.
>
> I have done a bootstrap and make compa
Here the committed patch approved off-list by Mike Stump which fixed
PR/52042.
Tested on darwin10/11 with x86_64.
PS: this is my first commit so I hope I get it right. Otherwise do not
hesitate to yell at me.
Thanks.
--
Patrick.
2012-02-15 Iain Sandoe
Patrick Marlier
On Feb 15, 2012, at 9:38 AM, Iain Sandoe wrote:
> So, this just brings us into line with the current system compiler.
Ah, the patch is ok. Perfect, thanks.
On Wed, Feb 15, 2012 at 7:59 PM, DJ Delorie wrote:
>
> I would classify these as "obvious" but please make sure they go in
> before the poison patch ;-)
>
>> And this will be needed to make those ports do what was intended:
>>
>> * config/rl78/rl78.h: Replace SMALL_REGISTER_CLASSES with hook
Alexandre Oliva writes:
> On Feb 13, 2012, Richard Sandiford wrote:
>> does this avoid the kind of memrefs_conflict_p cycle I was seeing in:
>
> I don't know that it does, I'd missed that bit.
>
> If you still have a preprocessed testcase, I'd be glad to give it a
> quick try. Failing that, I ca
I would classify these as "obvious" but please make sure they go in
before the poison patch ;-)
> And this will be needed to make those ports do what was intended:
>
> * config/rl78/rl78.h: Replace SMALL_REGISTER_CLASSES with hook.
> * config/rx/rx.h: Remove SMALL_REGISTER_CLASSES.
>
On Wed, Feb 15, 2012 at 7:36 PM, Steven Bosscher wrote:
> Hello,
>
> While going through some old patches of mine, I noticed this:
>
> config/rl78/rl78.h:#define SMALL_REGISTER_CLASSES 1
> config/rx/rx.h:#define SMALL_REGISTER_CLASSES 0
>
> Can't possibly work, that target
Hello,
While going through some old patches of mine, I noticed this:
config/rl78/rl78.h:#define SMALL_REGISTER_CLASSES 1
config/rx/rx.h:#define SMALL_REGISTER_CLASSES 0
Can't possibly work, that target macro was replaced with a hook.
Attached patch OK for trunk?
Ciao!
S
This patch fixes PR 52199, which shows up on power7 systems when the compiler
is vectorizing DImode operations, and the vector duplicate code was not
prepared to accept CONST and SYMBOL_REF's.
I have done a bootstrap and make compare, and there were no regressions. Is
this ok to install? I will
On Fri, 2012-02-10 at 13:02 -0800, Andrew Pinski wrote:
> On Fri, Feb 10, 2012 at 12:46 PM, Michael Meissner
> wrote:
> > I was looking at the routelookup EEMBC benchmark and it has code of the
> > form:
> >
> > while ( this_node->cmpbit > next_node->cmpbit )
> >{
> > this_node = next_
On Wed, Feb 15, 2012 at 11:59:15AM -0600, Aldy Hernandez wrote:
> Hmmm, isn't %K for trees? We're talking gimple, and FUNCTION_DECLs,
> which don't have a TREE_BLOCK, here.
Gimple stmts have gimple_block, I guess you'd need to create some tree
and set its TREE_BLOCK and EXPR_LOCATION from gimple_
The libgo setcontext TLS configure test was failing on Solaris 8 and
9/SPARC, breaking bootstrap due to the unhandled case in
runtime/proc.c. This turned out to be a known issue, documented in the
makecontext(3C) manpage:
NOTES
The semantics of the uc_stack member of the ucontext_t
Like other MIPS ELF targets, the mips-sgi-irix6.5 configuration doesn't
include dbxelf.h, causing objc.dg/stabs-1.m to FAIL. Same cure as in
Richard's patch.
Tested with the appropriate runtest invocations, installed on mainline.
Rainer
2012-02-11 Rainer Orth
PR target/5215
On 02/15/12 11:32, Jakub Jelinek wrote:
On Wed, Feb 15, 2012 at 09:29:18AM -0800, Richard Henderson wrote:
On 02/15/2012 01:48 AM, Richard Guenther wrote:
Hmm. I think you rather want to teach local_pure_const about TM
properties you want to know and have them propagated properly
(of course un
gcc.dg/pr51867.c currently FAILs on Solaris 8 and 9:
FAIL: gcc.dg/pr51867.c (test for excess errors)
Excess errors:
/vol/gcc/src/hg/trunk/local/gcc/testsuite/gcc.dg/pr51867.c:7:12: warning: incomp
atible implicit declaration of built-in function 'sqrtf' [enabled by default]
/vol/gcc/src/hg/trunk/l
Vladimir Makarov wrote:
> On 02/14/2012 12:00 PM, Ulrich Weigand wrote:
> > This is fixed by the following patch. However, I'm not quite sure if this
> > is
> > the best way to fix the problem, given that the REG_N_REFS use occurs only
> > for
> > debug output purposes -- should we actually do t
On 02/14/2012 12:00 PM, Ulrich Weigand wrote:
Hello,
compiling anything, even just an empty function, with
-fschedule-insns -fsched-pressure -fdump-rtl-sched1
causes the compiler to crash:
Program received signal SIGSEGV, Segmentation fault.
0x0840d9e0 in REG_N_REFS (regno=58) at ../../gcc-h
As discussed the improved/relaxed test for gthreads support breaks
symbol versioning on Solaris 8 and 9. The best solution seems to
disable it by default on those targets, allowing users to reenable it if
they don't care about compatibility.
The following patch does just that, disabling only if s
On 15 Feb 2012, at 17:16, Mike Stump wrote:
On Feb 15, 2012, at 2:21 AM, Iain Sandoe wrote:
I noticed when working on GAS for darwin that we fail to prepend
the user label prefix for extended identifiers.
Does this match the native compiler on the platform? This is abi
changing, and we c
The following patch fixes wrong code generation for SPEC2000 mesa when
we use tuning for bulldozer.
The patch was successfully bootstrapped on x86/x86-64.
Committed as rev. 184278.
2012-02-15 Vladimir Makarov
* lra-coalesce.c (lra_coalesce): Check the split pseudos too.
Index: lra-
Prompted by the suggestions in the PR what to do about the new exports
of
_ZNSt19istreambuf_iteratorIcSt11char_traitsIcEEppEv
_ZNSt19istreambuf_iteratorIwSt11char_traitsIwEEppEv
at GLIBCXX_3.4.5 on Solaris, I've followed the preprocess *.ver files
route Jakub suggested to just omit those two on
On Wed, Feb 15, 2012 at 09:29:18AM -0800, Richard Henderson wrote:
> On 02/15/2012 01:48 AM, Richard Guenther wrote:
> > Hmm. I think you rather want to teach local_pure_const about TM
> > properties you want to know and have them propagated properly
> > (of course unless pure/const which is about
On 02/15/2012 01:48 AM, Richard Guenther wrote:
> Hmm. I think you rather want to teach local_pure_const about TM
> properties you want to know and have them propagated properly
> (of course unless pure/const which is about optimization and has an
> easy fallback default yours wouldn't have that -
On Feb 15, 2012, at 2:21 AM, Iain Sandoe wrote:
> I noticed when working on GAS for darwin that we fail to prepend the user
> label prefix for extended identifiers.
Does this match the native compiler on the platform? This is abi changing, and
we can't do it, unless it mismatches the native com
On 02/15/2012 06:09 PM, Rainer Orth wrote:
> Paolo Bonzini writes:
>
>> This must be a separate macro in acinclude.m4 that is AC_REQUIREd from
>> gcc_AC_INITFINI_ARRAY.
>
> Here's an updated patch that does so. During testing, I found one
> additional complication, though. With the Sun as/GNU
Paolo Bonzini writes:
> This must be a separate macro in acinclude.m4 that is AC_REQUIREd from
> gcc_AC_INITFINI_ARRAY.
Here's an updated patch that does so. During testing, I found one
additional complication, though. With the Sun as/GNU ld combination on
Solaris 11/x86, after the bootstrap h
Georg-Johann Lay wrote:
> http://gcc.gnu.org/gcc-4.7/changes.html
>
> But there is a problem with the link to built-ins in onlinedocs.
>
> The HTML reads
>
> Support for AVR-specific href="http://gcc.gnu.org/onlinedocs/gcc/AVR-Built_002din-Functions.html";>built-in
> functions has been added.
On Wed, 15 Feb 2012, Christian Bruel wrote:
> Removal of this NOP_EXPR if !CAN_HAVE_LOCATION_P fixes the problem.
> looks safe from my testing, because the loc is inserted using
> 'protected_set_expr_location', whereas no loc for a constant case
> doesn't seem to introduce new problems with the de
[rth, please correct me if I have misrepresented the TM semantics and
rules].
On 02/15/12 09:04, Richard Guenther wrote:
Ok, I had a look at the bugreport and am curios why
__attribute__((always_inline))
static void asmfunc(void)
{
__asm__ ("");
}
__attribute__((transaction_safe))
static
Gerald Pfeifer wrote:
> Looks good! Please fix the one typo, and consider the other comments
> as well, and go ahead and commit this I'd say. We can always make
> further changes later.
Committed with the following changes:
* Support for XMEGA cores
* Mini-example for address space code.
http
Some parts outside the backend use ACCUMULATE_OUTGOING_ARGS without including
tm_p.h which lead to a build warning because avr.h does
#define ACCUMULATE_OUTGOING_ARGS avr_accumulate_outgoing_args()
This patcs moved the prototype from avr-protos.h to avr.h:
* config/avr/avr-protos.h (avr_
On Wed, Feb 15, 2012 at 3:30 PM, Aldy Hernandez wrote:
> On 02/15/12 03:48, Richard Guenther wrote:
>>
>> On Tue, Feb 14, 2012 at 6:47 PM, Richard Henderson wrote:
>>>
>>> On 02/14/2012 08:46 AM, Aldy Hernandez wrote:
The call to ipa_tm_diagnose_tm_safe() does nothing because there are
Georg-Johann Lay wrote:
> Jakub Jelinek wrote:
>> On Wed, Feb 01, 2012 at 02:00:53PM +0100, Georg-Johann Lay wrote:
* gcc.dg/lto/20091013-1_1.c: xfail for avr.
* gcc.dg/lto/20091013-1_2.c: xfail for avr.
>> If it is just because of the warnings, can't you
>> add
>> -Wno-int-to-point
This patch to the Go frontend avoids some compiler crashes on erroneous
inputs. It also makes some minor fixes to some comments. Bootstrapped
and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to
mainline.
Ian
diff -r 4f7eddf7a851 go/backend.h
--- a/go/backend.h Tue Feb 14 12:39:26 20
On 02/15/12 03:48, Richard Guenther wrote:
On Tue, Feb 14, 2012 at 6:47 PM, Richard Henderson wrote:
On 02/14/2012 08:46 AM, Aldy Hernandez wrote:
The call to ipa_tm_diagnose_tm_safe() does nothing because there are no longer
any calls in the function, since the function call has been inlined
This patch causes all tm tests to fail on arm-none-eabi target that doesn't
support -pthread command line option:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52259
> -Original Message-
> From: Eric Botcazou [mailto:ebotca...@adacore.com]
> Sent: 11 February 2012 19:24
> To: Jack Howarth
>
On 02/15/2012 01:14 PM, Richard Guenther wrote:
This patch by Tobias fixes PR50561.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Thanks Richard.
Tobi
This patch by Tobias fixes PR50561.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2012-02-15 Tobias Grosser
PR tree-optimization/50561
* graphite-flattening.c (lst_project_loop): Do not
remove old scattering dimensions after flattenin
On 14/02/12 18:30, Richard Earnshaw wrote:
s/obsolete peepholes/obsolete flag-clobbering peepholes/
Otherwise ok for stage1
Thanks.
The 'Similarly' that you delete suggests there might still be more of
them...
You'd think, but I can't figure out what that refers to.
There are some other p
I noticed when working on GAS for darwin that we fail to prepend the
user label prefix for extended identifiers.
fixed as below,
OK for trunk/when?
Iain
gcc:
* config/darwin.h (ASM_OUTPUT_LABELREF): Add user label prefix
for extended identifiers.
diff --git a/gcc/config/darwi
On Wed, Feb 15, 2012 at 8:23 AM, Ye Joey wrote:
> Ping^2
All hunks apart from the expr.c hunk from 182545 are ok (thus, all hunks
that are explicitly guarded with -fstrict-volatile-bitfields).
Please use -p for diffs, I couldn't spot the place in expr.c you patch,
the line-numbers are off and th
On Wed, Feb 15, 2012 at 10:04:15AM +, Paulo J. Matos wrote:
> It doesn't look like you can actually do that because, according to
> the internals manual:
> "For a named pattern, the condition (if present) may not depend on
> the data in the insn being matched, but only the target-machine-type
>
On Wed, Feb 15, 2012 at 4:47 AM, Kenny Simpson
wrote:
> Are any other recent trunk fixes to be backported - like the wrong-code bugs:
> PR tree-optimization/50444, PR tree-optimization/51528, or PR
> tree-optimization/46886 ?
I'll backport 46886 but like to defer 50444 and 51528 to 4.6.4 - they
On 14/02/12 14:10, Jakub Jelinek wrote:
On Tue, Feb 14, 2012 at 02:05:30PM +, Paulo J. Matos wrote:
I think the register allocator will generate good code using % if you
make your predicate nonimmediate_operand in operand 1:
(define_insn "iorqi3"
[(set (match_operand:QI 0 "register_operand"
On Tue, Feb 14, 2012 at 6:47 PM, Richard Henderson wrote:
> On 02/14/2012 08:46 AM, Aldy Hernandez wrote:
>> The call to ipa_tm_diagnose_tm_safe() does nothing because there are no
>> longer any calls in the function, since the function call has been inlined:
>>
>> f ()
>> {
>> :
>> __asm__ __v
Hello,
This patch fixes a regression when folding a cast cond expression to a
constant in case label.
The problem came from the removal of the lines in
c-common.c:check_case_value
/* ??? Can we ever get nops here for a valid case value? We
shouldn't for C. */
STRIP_TYPE_NOPS (value);
On Wed, 15 Feb 2012, Eric Botcazou wrote:
> > This fixes PR52178, the failure to bootstrap Ada with LTO (well,
> > until you hit the next problem). A self-referential DECL_QUALIFIER
> > makes us think that a QUAL_UNION_TYPE type is of variable-size which
> > makes us stream that type locally, wre
This is a regression present on the mainline: the compiler aborts on a deferred
constant initialized with an aggregate and whose type is a discriminated
record type with a variant part and which, ultimately, contains a component
whose type is controlled or tagged.
Tested on i586-suse-linux, app
This is a regression present on the mainline and which stems from enabling
-feliminate-unused-debug-types by default in Ada. We fail to emit the
debugging information for tagged types when only the associated class-wide
type is referenced in the source code.
Tested on i586-suse-linux, applied
On Tue, Feb 14, 2012 at 8:52 PM, Richard Henderson wrote:
> On 02/14/2012 10:26 AM, Uros Bizjak wrote:
>> #ifdef __x86_64__
>> + cfi_def_cfa(%rsi, 0)
>> movq (%rsi), %rcx
>> movq 8(%rsi), %rbx
>> movq 16(%rsi), %rbp
>> @@ -119,20 +120,21 @@
>> movq 32(%rsi)
This is a regression present on the mainline: the compiler aborts on the access
to a component of the value returned by a function call, if the return type is
a volatile record type that isn't controlled or tagged.
Tested on i586-suse-linux, applied on the mainline.
2012-02-15 Eric Botcazou
This fixes the regression in the frame unwinding support on SPARC/Solaris 11
present on the mainline and 4.6 branch. Tested (by me) on 6 differents OS
versions of Solaris 8, 9 and 10 and on Solaris 11 by Ian and Rainer.
Applied on the mainline and 4.6 branch.
2012-02-15 Eric Botcazou
74 matches
Mail list logo
