On Mar 6, 2012, at 6:34 PM, Joseph S. Myers wrote:
On Tue, 6 Mar 2012, Tristan Gingold wrote:
The patch is simple: the C front-end will now calls c_build_pointer_type
(instead of build_pointer_type), which in turn calls
build_pointer_type_for_mode using the right mode.
There seem to
On 7 March 2012 05:22, Terry Guo wrote:
Hello,
Can anybody please review and approve the following simple patch? Thanks
very much.
http://gcc.gnu.org/ml/libstdc++/2011-08/msg00063.html
I'll test it on x86_64-linux as soon as trunk is able to build again.
On Wed, 7 Mar 2012, Ralf Corsepius wrote:
On 03/06/2012 10:43 AM, Richard Guenther wrote:
On Mon, 5 Mar 2012, Ralf Corsépius wrote:
Hi,
The patch below addresses an issue with gcc-4.7.0 the issue I had reported
in
http://gcc.gnu.org/ml/gcc/2012-03/msg00035.html
and
On Tue, Mar 6, 2012 at 9:56 PM, Torvald Riegel trie...@redhat.com wrote:
On Tue, 2012-03-06 at 21:18 +0100, Richard Guenther wrote:
On Tue, Mar 6, 2012 at 6:55 PM, Aldy Hernandez al...@redhat.com wrote:
On 02/29/12 03:22, Richard Guenther wrote:
So fixing up individual passes is easier - I
On Tue, 6 Mar 2012, Eric Botcazou wrote:
Well. I suppose fixing that negative DECL_FIELD_OFFSET thing should
be #1 priority.
OK, let me try over the next few days.
Thanks.
Btw, making [s]bitsizetype have TYPE_PRECISION of [s]sizetype plus
log2 (BITS_PER_UNIT) solves quite some issues
When the optab causes the actual mode to be wider we have to re-check
whether we still fulfill the gimple constraints we verify.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2012-03-07 Richard Guenther rguent...@suse.de
* tree-ssa-math-opts.c
On Tue, Mar 6, 2012 at 11:08 PM, Eric Botcazou ebotca...@adacore.com wrote:
If you try to bootstrap the GNAT 4.7.0 compiler on IA-64/Linux with
non-default
options (-gnatpg replaced with -gnatpgn), you get another comparison failure
caused by debug insns, stemming from the machine-specific
On Wed, Mar 07, 2012 at 10:27:22AM +0100, Richard Guenther wrote:
On Tue, Mar 6, 2012 at 11:08 PM, Eric Botcazou ebotca...@adacore.com wrote:
If you try to bootstrap the GNAT 4.7.0 compiler on IA-64/Linux with
non-default
options (-gnatpg replaced with -gnatpgn), you get another comparison
Hi,
as from Dave's side there is no objection, I committed this patch to
4.8 at revision 185027.
Regards,
Kai
2012/3/3 Kai Tietz ktiet...@googlemail.com:
Hi,
this patch allows that relocated const data is placed into .rdata. To provide
old behavior for older runtimes not supporting
This patch restores x86_64-linux-gnu bootstrap after my patch for 52372.
Applied as obvious.
Sorry for the breakage.
Richard
gcc/
PR middle-end/52515
* rtl.h (pc_rtx, cc0_rtx, ret_rtx, simple_return_rtx): Add GTY markers.
Index: gcc/rtl.h
On Wed, Mar 07, 2012 at 09:52:51AM +, Richard Sandiford wrote:
This patch restores x86_64-linux-gnu bootstrap after my patch for 52372.
Applied as obvious.
Sorry for the breakage.
Was it really necessary to move these out of the global rtx array?
Now you completely unnecessarily have 4
On Wed, Mar 7, 2012 at 10:35 AM, Jakub Jelinek ja...@redhat.com wrote:
On Wed, Mar 07, 2012 at 10:27:22AM +0100, Richard Guenther wrote:
On Tue, Mar 6, 2012 at 11:08 PM, Eric Botcazou ebotca...@adacore.com wrote:
If you try to bootstrap the GNAT 4.7.0 compiler on IA-64/Linux with
On Wed, Mar 7, 2012 at 10:28 AM, Uros Bizjak ubiz...@gmail.com wrote:
+ if (TARGET_X32)
+ operands[0] = convert_memory_address (word_mode, operands[0]);
This addition to indirect_jump and tablejump should be the only
change, needed in i386.md now. Please write the condition
if (Pmode
Jakub Jelinek ja...@redhat.com writes:
On Wed, Mar 07, 2012 at 09:52:51AM +, Richard Sandiford wrote:
This patch restores x86_64-linux-gnu bootstrap after my patch for 52372.
Applied as obvious.
Sorry for the breakage.
Was it really necessary to move these out of the global rtx array?
On Wed, Mar 7, 2012 at 11:07 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Wed, Mar 7, 2012 at 10:28 AM, Uros Bizjak ubiz...@gmail.com wrote:
+ if (TARGET_X32)
+ operands[0] = convert_memory_address (word_mode, operands[0]);
This addition to indirect_jump and tablejump should be the only
On Wed, Mar 7, 2012 at 11:11 AM, Richard Sandiford
rdsandif...@googlemail.com wrote:
Jakub Jelinek ja...@redhat.com writes:
On Wed, Mar 07, 2012 at 09:52:51AM +, Richard Sandiford wrote:
This patch restores x86_64-linux-gnu bootstrap after my patch for 52372.
Applied as obvious.
Sorry
On Wed, Mar 7, 2012 at 11:07 AM, Uros Bizjak ubiz...@gmail.com wrote:
Ah, I vaguely remember that indirect call/jmp is invalid on X32 for
This should read ... indirect call/jmp FROM MEMORY is invalid on X32
It looks I've had too much morning coffee already ;)
Uros.
Richard Guenther richard.guent...@gmail.com writes:
On Wed, Mar 7, 2012 at 11:11 AM, Richard Sandiford
rdsandif...@googlemail.com wrote:
Jakub Jelinek ja...@redhat.com writes:
On Wed, Mar 07, 2012 at 09:52:51AM +, Richard Sandiford wrote:
This patch restores x86_64-linux-gnu bootstrap
On Wed, Mar 07, 2012 at 11:57:18AM +0100, Richard Guenther wrote:
Logically, the target structure should be for target-specific rtl.
Target-independent rtl like constants, (cc0), (pc), etc., should be
shared between targets. It seems a shame if GC inefficiencies
force us to use a
Hello guys,
I am attaching initial patch which enables TSX's HLE [1] prefixes in
GCC. Since we have no official intrinsics declarations, I want to hear
your comments about the patch
Note, there is no option '-mhle' and no tests (I'll do that after)
[1] -
On Wed, Mar 07, 2012 at 03:05:58PM +0400, Kirill Yukhin wrote:
Hello guys,
I am attaching initial patch which enables TSX's HLE [1] prefixes in
GCC. Since we have no official intrinsics declarations, I want to hear
your comments about the patch
I think this is a wrong approach. Instead we
Hello,
This PR is again about insns that are recog'ed as =0 but do not change the
processor state. As explained in
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52203#c8, I've tried
experimenting with an attribute marking those empty insns in MD files and
asserting that all other insns do
On Wed, 7 Mar 2012, Andrey Belevantsev wrote:
Hello,
This PR is again about insns that are recog'ed as =0 but do not change the
processor state. As explained in
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52203#c8, I've tried experimenting
with an attribute marking those empty insns in
On Wed, Mar 07, 2012 at 11:01:36AM +0100, Richard Guenther wrote:
Hmm, but then this complicates and slows down the generic alias machinery.
Of course IMHO the RTL alias machinery should be conservative with respect
to what the RTL IL allows - so the question is are non-legitimate addresses
This makes us use build_nonstandard_integer_type in
signed_or_unsigned_type_for and adjusts the function to return
NULL_TREE for non-sensical inputs (only allowing pointer and
integeral types). This way we make sure that the precision of
the result type matches that of the input - something
Hmm, but isn't the bug that we feed debug-insn mems to memrefs_conflict_p?
We don't. The addresses come from regular insns, but cselib is able to
equivalence one of them with an address that is already in its hashtable
because of a debug insn (see cselib.c:promote_debug_loc).
Or that we
This patch adds memory barriers to
__builtin_avr_nop
__builtin_avr_sei
__builtin_avr_cli
__builtin_avr_wdr
__builtin_avr_sleep
__builtin_avr_delay_cycles
so that their code cannot be dragged over memory accesses.
Ok for trunk?
PR target/52496
* config/avr/avr.c
CCing Alex. I think we feed debug insn mems in the scheduler to be able to
find out what debug insns need to be invalidated and what can be kept.
And any address expressions are legitimate for debug insns, why should we
be limited by what the ISA allows? All we are limited is if we can
Prototyped in tree.h, called from the middle-end but implemented
in convert.c. That looks wrong.
Now, convert.c is used from all frontends to implement convert ()
(that looks backwards - the language convert should be a langhook,
called from convert implemented in convert.c). But well, I aint
The type_for_size langhook should go (it does not handle non-mode
precision well, at least it handles it unexpectedly to most callers).
Instead callers that want to call a langhook should use type_for_mode.
The following patch makes some direct uses of the langhook from the
middle-end use
On Wed, Mar 07, 2012 at 01:53:12PM +0100, Richard Guenther wrote:
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Jakub, are the OMP bits ok?
Yeah.
Jakub
I'll apply this as obvious after including it in a bootstrap / regtest.
Richard.
2012-03-07 Richard Guenther rguent...@suse.de
* c-typeck.c (pointer_diff): Use c_common_type_for_size.
Index: gcc/c-typeck.c
===
---
Nearly obvious. One question would be whether IVOPTs wants to
use SImode and word_mode instead of int/long. And the dojump.c
code is dead - TYPE_PRECISION (type) TYPE_PRECISION (TREE_TYPE (exp))
cannot ever hold here, but I'm double-checkign with an assert
(consider the patch adjusted before
On Wed, Mar 7, 2012 at 1:49 AM, Sriraman Tallam tmsri...@google.com wrote:
Patch for CPU detection at run-time.
===
Patch for CPU detection at run-time, to be used in dispatching of
multi-versioned functions. Please see this discussion:
http://gcc.gnu.org/viewcvs?view=revisionrevision=185043
This wrong code bug for __memx space was caused by a missing R22 in the
register footprint of xloadmode_A split.
Johann
PR target/52484
* config/avr/avr.md (xloadmode_A): Add R22... to register footprint.
Index:
This corrects the order of Input and Output operands in the documentation on
machine language insertions.
Tested on x86_64-pc-linux-gnu, committed on trunk
2012-03-07 Eric Botcazou ebotca...@adacore.com
* gnat_ugn.texi (Inline Assembler): Fix swapping of Input and
Output
This patch corrects the machinery which detects whether one node appears
earlier in the tree with respect to another node.
Tested on x86_64-pc-linux-gnu, committed on trunk
2012-03-07 Hristian Kirtchev kirtc...@adacore.com
* sem_ch12.adb (Earlier): When two nodes come from the same
Hi,
On Wed, 7 Mar 2012, Richard Guenther wrote:
FAIL: gcc.dg/tree-ssa/pr31261.c scan-tree-dump-times original return
\\(char\\)
-\\(unsigned char\\) c 31; 1
FAIL: gcc.dg/tree-ssa/pr31261.c scan-tree-dump-times original return
\\(int\\)
\\(12 - \\(unsigned int\\) d\\) 7; 1
because
On Wed, 7 Mar 2012, Michael Matz wrote:
Hi,
On Wed, 7 Mar 2012, Richard Guenther wrote:
FAIL: gcc.dg/tree-ssa/pr31261.c scan-tree-dump-times original return
\\(char\\)
-\\(unsigned char\\) c 31; 1
FAIL: gcc.dg/tree-ssa/pr31261.c scan-tree-dump-times original return
\\(int\\)
David Daney david.da...@cavium.com writes:
I'd have expected regeneration to use GCJ built to use ECJ, though I don't
know.
I've never tried this. Given that the .class file lives below
libjava/classpath and has to be synced with upstream Classpath anyway, I
hope the Java maintainers will
This patch increases the maximum number of instantiations allowed
per compilation.
Tested on x86_64-pc-linux-gnu, committed on trunk
2012-03-07 Bob Duff d...@adacore.com
* hostparm.ads (Max_Instantiations): Increase parameter.
Index: hostparm.ads
This patch increases the efficiency of bounded strings by removing an
unnecessary default.
No change in functional behavior.
Tested on x86_64-pc-linux-gnu, committed on trunk
2012-03-07 Bob Duff d...@adacore.com
* a-strsup.ads, a-stwisu.ads, a-stzsup.ads (Super_String):
Remove
The differences are not significant enough to require a separate version.
Pthread_Self is now an imported subprogram on Alpha too. Could be replaced
by an inline assembler on both platforms in case of performace issue.
Tested on x86_64-pc-linux-gnu, committed on trunk
2012-03-07 Tristan
Just in order to ease debugging (at pthread level).
No functional change.
Committed on trunk
2012-03-07 Tristan Gingold ging...@adacore.com
* s-taprop-vms.adb (Create_Task): set thread name.
* s-osinte-vms.ads (pthread_attr_setname_np): Declare.
Index: s-taprop-vms.adb
On 03/07/12 03:18, Richard Guenther wrote:
On Tue, Mar 6, 2012 at 9:56 PM, Torvald Riegeltrie...@redhat.com wrote:
On Tue, 2012-03-06 at 21:18 +0100, Richard Guenther wrote:
On Tue, Mar 6, 2012 at 6:55 PM, Aldy Hernandezal...@redhat.com wrote:
On 02/29/12 03:22, Richard Guenther wrote:
So
On Tue, Mar 6, 2012 at 10:27 PM, H.J. Lu hjl.to...@gmail.com wrote:
+ case '^':
+ if (TARGET_64BIT Pmode == SImode)
+ {
+ fputs (addr32, file);
+#ifndef HAVE_AS_IX86_REP_LOCK_PREFIX
+ if (ASSEMBLER_DIALECT == ASM_ATT)
+ fputs (addr32; ,
On Wed, Mar 7, 2012 at 1:28 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Tue, Mar 6, 2012 at 9:57 PM, H.J. Lu hjl.to...@gmail.com wrote:
(define_insn *call
- [(call (mem:QI (match_operand:P 0 call_insn_operand czw))
+ [(call (mem:QI (match_operand:C 0 call_insn_operand czw))
This patch to the Go compiler fixes a bug when a struct inherits from an
interface with a varargs method. I added a test for this to the master
Go testsuite, which will be copied into gcc in due course. Bootstrapped
and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to
mainline.
Ian
It's useful to set task/thread's name for debugging.
Under Linux, there is a prctl() system call to do that. Note that we don't
want to use pthread_setname_np since only very recent glibc (= 2.12) provide
this function.
Tested on x86_64-pc-linux-gnu, committed on trunk
2012-03-07 Arnaud Charlet
Richard Kenner wrote:
Given the current set of results, since I do not have any way to verify
whether my simplify_set changes would actually trigger correctly, I'd
rather propose to just remove the SUBREG case in apply_distributive_law
(i.e. only apply the first patch below).
Richard Sandiford rdsandif...@googlemail.com writes:
Definitely. Thanks for saving me the legwork :-)
It was both easy to do and a prerequisite for the MIPS_DEBUGGING_INFO
removal :-)
There are also some SDB_OUTPUT_SOURCE_LINE references in mips.c and
mips.h that could go. Removing those
This patch fixes an unwinding bug for functions with dynamic stack
frames. We stop generating REG_CFA_* notes for stack pointer, and at
the end of unwinding we restore the stack pointer by adjusting it by
EH_RETURN_STACKADJ_RTX.
Walter
diff --git a/gcc/config/tilegx/tilegx.c
On Wed, 7 Mar 2012, Tristan Gingold wrote:
On Mar 6, 2012, at 6:34 PM, Joseph S. Myers wrote:
On Tue, 6 Mar 2012, Tristan Gingold wrote:
The patch is simple: the C front-end will now calls c_build_pointer_type
(instead of build_pointer_type), which in turn calls
On Wed, 7 Mar 2012, Richard Guenther wrote:
Now, convert.c is used from all frontends to implement convert ()
(that looks backwards - the language convert should be a langhook,
called from convert implemented in convert.c). But well, I aint
not touching this beast ;)
I don't think convert
There is a single call to unshare_expr in gigi and it is actually superfluous
if you do things properly.
Tested on i586-suse-linux, applied on the mainline.
2012-03-07 Eric Botcazou ebotca...@adacore.com
* gcc-interface/trans.c (Identifier_to_gnu): Don't unshare initializer.
On Tue, Mar 6, 2012 at 8:44 PM, H.J. Lu hjl.to...@gmail.com wrote:
This is the last patch for Pmode == SImod in x32. In x32, the return value
of the symbol address must be zero-extended to DImode, This patch adds
*zero_extendsidi2_x32 to load the address of a symbol in SImode and
zero-extend
It was needed for STABS, but is totally obsolete for DWARF.
Tested on i586-suse-linux, applied on the mainline.
2012-03-07 Eric Botcazou ebotca...@adacore.com
* gcc-interface/gigi.h (rest_of_type_decl_compilation): Delete.
* gcc-interface/decl.c (defer_finalize_level):
This patch renames some internal atomic macros to have a less generic
prefix.
Walter
* config/tilepro/atomic.c: Rename atomic_ prefix to
arch_atomic_.
(atomic_xor): Rename and move definition to
config/tilepro/atomic.h.
(atomic_nand): Ditto.
*
On 3/7/2012 1:01 PM, Walter Lee wrote:
This patch fixes an unwinding bug for functions with dynamic stack
frames. We stop generating REG_CFA_* notes for stack pointer, and at
the end of unwinding we restore the stack pointer by adjusting it by
EH_RETURN_STACKADJ_RTX.
I forgot to attach the
The previous patch has exposed a latent issue: we sometimes set some flags on a
DECL node using information that come from another entity.
Tested on i586-suse-linux, applied on the mainline.
2012-03-07 Eric Botcazou ebotca...@adacore.com
* gcc-interface/decl.c (gnat_to_gnu_entity):
I've merged trunk revision 185072 to the gccgo branch.
Ian
In c++/52521 it was pointed out that our mangling of user-defined
literals was wrong.
Tested x86_64-pc-linux-gnu, applying to 4.7 and trunk.
commit 287cd9ecf4877db64774f5a29828081a62a53f5b
Author: Jason Merrill ja...@redhat.com
Date: Wed Mar 7 14:01:56 2012 -0500
PR c++/52521
*
This patch to the Go frontend avoids a crash if an error causes us to
write out type functions too late. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r 90c26bab1684 go/gogo.h
--- a/go/gogo.h Wed Mar 07 08:04:32 2012 -0800
+++ b/go/gogo.h Wed
On Wed, Mar 7, 2012 at 5:03 PM, H.J. Lu hjl.to...@gmail.com wrote:
(define_insn *call
- [(call (mem:QI (match_operand:P 0 call_insn_operand czw))
+ [(call (mem:QI (match_operand:C 0 call_insn_operand czw))
(match_operand 1 ))]
- !SIBLING_CALL_P (insn)
+ !SIBLING_CALL_P (insn)
Hi,
The attached patch is to fix PR target/52503 which is a build failure
for sh-wrs-vxworks. We've defined too many target option masks.
Tested with usual tests on sh4-unknown-linux-gnu and with a cc1-only
build on sh-wrs-vxworks. Applied on trunk.
Regards,
kaz
--
2012-03-07 Oleg
I committed this patch to google/main.
I have created a new patch for review for trunk here :
http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00458.html
Thanks,
-Sri.
On Thu, Mar 1, 2012 at 2:08 PM, Sriraman Tallam tmsri...@google.com wrote:
Removing [google] prefix from the subject line.
On
-Original Message-
From: Georg-Johann Lay [mailto:a...@gjlay.de]
Sent: Wednesday, March 07, 2012 5:40 AM
To: gcc-patches@gcc.gnu.org
Cc: Denis Chertykov; Weddington, Eric
Subject: [Patch,AVR]: Fix PR52496: Add memory barriers to built-ins
This patch adds memory barriers to
The discussion of 52521 also noted that the demangler doesn't support
literal operators yet.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit ee622f8167f66b6ae9fff1526688a2dfdd84a8be
Author: Jason Merrill ja...@redhat.com
Date: Wed Mar 7 17:50:19 2012 -0500
* cp-demangle.c
IainS,
Your patch completely broke libffi on AIX and your changes were not
Darwin-specific, despite Mike Stump's comment with his approval.
AIX shares ffi_darwin.c, as one should be able to see from all of the
AIX comments in the file.
aix_closure.S expects ffi_closure_helper_DARWIN to return
On 4 March 2012 12:56, Jonathan Wakely wrote:
PR libstdc++/52433
* include/debug/safe_iterator.h (_Safe_iterator): Add move
constructor and move assignment operator.
* testsuite/23_containers/vector/debug/52433.cc: New.
Tested 'make check check-debug' on x86_64
The Go frontend crashed doing an array assignment of two values of type
[2][]int. This patch fixes it. Normal array assignments worked because
gcc uses the same structure for all arrays with the same length and
element type. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu.
This patch applies the same logic to aix_closure.S that was modified
in darwin_closure.S, returning from 100% failure to:
=== libffi Summary ===
# of expected passes1638
# of unexpected failures13
# of unsupported tests 55
*
David discovered that there was a thinko in the logic of the powerpc setting
for the default tuning option. In theory, the compiler was supposed to use
PROCESSOR_DEFAULT for 32-bit targets and PROCESSOR_DEFAULT64 for 64-bit targets
if no cpu or tuning option was used in configuring the compiler.
-Original Message-
From: Richard Guenther [mailto:richard.guent...@gmail.com]
Sent: Tuesday, March 06, 2012 9:12 PM
To: Jiangning Liu
Cc: gcc-patches@gcc.gnu.org
Subject: Re: [PATCH] Improve SCEV for array element
On Fri, Jan 20, 2012 at 10:06 AM, Jiangning Liu
74 matches
Mail list logo