Mikael Morin wrote:
there was no specific handling for absent arrays passed as argument
to elemental procedures. So, because of scalarisation, we were passing
an array element reference of a NULL pointer which was failing.
These patches add a conditional to pass NULL when the data pointer
is
On Fri, 10 Feb 2012, Richard Henderson wrote:
On 02/10/2012 01:44 AM, Richard Guenther wrote:
What is the reason to keep a GIMPLE_TRANSACTION stmt after
TM lowering and not lower it to a builtin function call?
Because real optimization hasn't happened yet, and we hold
out hope that we'll
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
ow...@gcc.gnu.org] On Behalf Of Jiangning Liu
Sent: Friday, January 20, 2012 5:07 PM
To: 'Richard Guenther'
Cc: gcc-patches@gcc.gnu.org
Subject: RE: [PATCH] Improve SCEV for array element
It's
On Fri, Feb 10, 2012 at 10:02 PM, Andrew Pinski pins...@gmail.com wrote:
On Fri, Feb 10, 2012 at 12:46 PM, Michael Meissner
meiss...@linux.vnet.ibm.com wrote:
I was looking at the routelookup EEMBC benchmark and it has code of the form:
while ( this_node-cmpbit next_node-cmpbit )
{
On Sat, Feb 11, 2012 at 12:49 PM, Jakub Jelinek ja...@redhat.com wrote:
Hi!
In July Richard changed reduce_bit_field BIT_NOT_EXPR expansion from
NOT unop to XOR with all the bits in the bitfield's precision set.
Unfortunately that is correct for unsigned bitfields only, for signed
bitfields,
On Mon, Feb 13, 2012 at 10:54 AM, Jiangning Liu jiangning@arm.com wrote:
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
ow...@gcc.gnu.org] On Behalf Of Jiangning Liu
Sent: Friday, January 20, 2012 5:07 PM
To: 'Richard Guenther'
Cc:
Committed as obvious.
Richard.
2012-02-13 Richard Guenther rguent...@suse.de
PR translation/52211
* passes.c (enable_disable_pass): Fix typo.
Index: gcc/passes.c
===
--- gcc/passes.c(revision 184151)
+++
On 08/02/12 08:29, Bin Cheng wrote:
Hi,
Julian Brown once posted a patch fixing ARM EABI violation, which I think
also essential to 4.6 branch.
I created a patch against 4.6 branch as attached. Is it ok to back port?
You can refer following link for original patch.
On Thu, Feb 2, 2012 at 11:44 AM, Tom de Vries tom_devr...@mentor.com wrote:
Richard,
this patch fixes PR52801.
Consider test-case pr51879-12.c:
...
__attribute__((pure)) int bar (int);
__attribute__((pure)) int bar2 (int);
void baz (int);
int x, z;
void
foo (int y)
{
int a = 0;
On Tue, 7 Feb 2012 15:44:04 +0200
Ira Rosen i...@il.ibm.com wrote:
Hi,
I am starting to work on a new project and won't be able to continue
with vectorizer maintenance.
I'd like to thank all the people I had a chance to work with for
making my GCC experience so enjoyable.
Thanks for
On 13/02/12 12:54, Richard Guenther wrote:
On Thu, Feb 2, 2012 at 11:44 AM, Tom de Vries tom_devr...@mentor.com wrote:
Richard,
this patch fixes PR52801.
Consider test-case pr51879-12.c:
...
__attribute__((pure)) int bar (int);
__attribute__((pure)) int bar2 (int);
void baz (int);
int
-Original Message-
From: Gerald Pfeifer
Sent: Sunday, February 12, 2012 3:17 PM
To: Georg-Johann Lay
Cc: gcc-patches@gcc.gnu.org; Denis Chertykov; Weddington, Eric
Subject: Re: [Patch,wwwdocs,AVR]: AVR release notes
This looks like an impressive release for AVR!
Gerald
Jakub asked to have a closer look at the problem, and I found we could
do somewhat better. The first thing I noticed was that the problem was
that, in each block that computed a (base+const), we created a new VALUE
for the expression (with the same const and global base), and a new
reverse
On Mon, Feb 13, 2012 at 12:27:35PM -0200, Alexandre Oliva wrote:
Jakub asked to have a closer look at the problem, and I found we could
do somewhat better. The first thing I noticed was that the problem was
that, in each block that computed a (base+const), we created a new VALUE
for the
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, wrecking type merging and later
ICEing in the type verifier.
On 02/12/2012 04:48 PM, Gerald Pfeifer wrote:
On Wed, 8 Feb 2012, Andrew MacLeod wrote:
Checked in the shortened version andcode changes. How thats?
seems better :-)
Yep, thanks! There is just a minor grammor I went ahead fixing.
On the title page, I was thinking to refer to the release
Ping. Can someone please review these ports? Here is a summary of
the submission.
Summary of changes in latest submit:
http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01854.html
Latest submit:
1/6 toplevel: http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01860.html
2/6 contrib:
Committed to mainline.
2012-02-13 Andreas Krebbel andreas.kreb...@de.ibm.com
* gcc.c-torture/execute/pr51933.c: Modify for s390 31 bit.
---
gcc/testsuite/gcc.c-torture/execute/pr51933.c |8
1 file changed, 8 insertions(+)
Index:
This patch set removes __builtin_avr_map8 __builtin_avr_map16 built-ins and
implements a built-in __builtin_avr_insert_bits instead.
This has several reasons:
* From user feedback I learned that speed matters more than size here
* I found that the new built-in has better usability and fits
On 02/13/2012 01:35 AM, Richard Guenther wrote:
On Fri, 10 Feb 2012, Richard Henderson wrote:
On 02/10/2012 01:44 AM, Richard Guenther wrote:
What is the reason to keep a GIMPLE_TRANSACTION stmt after
TM lowering and not lower it to a builtin function call?
Because real optimization hasn't
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'gcc' has been submitted
by the German team of translators. The file is available at:
http://translationproject.org/latest/gcc/de.po
(This file, 'gcc-4.7-b20120128.de.po',
On Mon, 13 Feb 2012, Richard Guenther wrote:
Indeed. But note that the transform is not valid as *this_node may cross
a page boundary and thus either pointer load may trap if the other does not
(well, unless the C standard (and thus our middle-end) would require that
iff ptr-component does
Gerald Pfeifer ger...@pfeifer.com writes:
On Sun, 5 Feb 2012, Richard Sandiford wrote:
I've committed this patch to describe the MIPS changes in GCC 4.7.
Corrections, comments, and help with wordsmithing are all welcome.
Nice! How about the small follow-up below?
The first definitely looks
Hi,
the attached patch changes the low-level libgfortran IO dispatching
mechanism to use shared vtables for each stream type, instead of all
the function pointers being replicated for each unit. This is similar
to e.g. how the C++ frontend implements vtables. The benefits are:
- Slightly smaller
Hi!
The PR50912 changed vect_get_and_check_slp_defs dt from
array into scalar, which fails when calling vect_model_simple_cost
which looks at two array members. I believe even 4.6 checked just
the first operand, as it called it when processing the first operand,
so IMHO this patch doesn't
Hi!
As the testcase shows, deciding on whether to convert an argument or not
based on TYPE_SIZE is wrong. While the old __sync_* builtins in the
_[1248]/_16 variants only had a VPTR as first argument and optionally
I[1248]/I16 argument or arguments that should be converted, the new
__atomic_*
Ping 5...
Ping 4...
Ping 3? It's been months with no feedback...
Ping 2 ?
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01889.html
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg02555.html
http://gcc.gnu.org/ml/gcc-patches/2012-01/msg00529.html
Hi!
While debugging PR52172, I've noticed ICE when dumping RTL, all of
cselib seems to test setting_insn for NULL, but this spot doesn't.
Ok for trunk?
2012-02-13 Jakub Jelinek ja...@redhat.com
* cselib.c (dump_cselib_val): Don't assume l-setting_insn is
non-NULL.
---
On Wed, Jan 04, 2012 at 05:21:38PM +, Marcus Shawcroft wrote:
Alias analysis by DSE based on CSELIB expansion assumes that
references to the stack frame from different base registers (ie FP, SP)
never alias.
The comment block in cselib explains that cselib does not allow
substitution of
Hello!
alpha linux does not have expected /proc/net/igmp and
/proc/net/igmp6 files, so func interfaceMulticastAddrTable(ifindex
int) from interface_linux.go always returns (nil, nil), failing
net/test with:
--- FAIL: net.TestListenMulticastUDP (4.71 seconds)
???:1: IPv4 multicast
Hi!
If omp for loop body doesn't fallthru (which doesn't make much sense),
then we would call GOMP_loop_static_start with wrong number of arguments
if collapse is 1, static scheduling without chunk size and no ordered
clause.
Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux,
On 02/13/2012 07:42 AM, Walter Lee wrote:
1/6 toplevel: http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01860.html
2/6 contrib: http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01855.html
3/6 gcc: http://gcc.gnu.org/ml/gcc-patches/2012-01/msg01494.html
4/6 libcpp:
On 02/13/2012 11:43 AM, Jakub Jelinek wrote:
* cselib.c (dump_cselib_val): Don't assume l-setting_insn is
non-NULL.
Ok.
r~
On 02/13/2012 11:54 AM, Jakub Jelinek wrote:
* cselib.c (expand_loc): Return sp, fp, hfp or cfa base reg right
away if seen.
Looks good.
r~
On 02/13/2012 11:42 AM, Jakub Jelinek wrote:
2012-02-13 Jakub Jelinek ja...@redhat.com
PR c++/52215
* c-common.c (sync_resolve_params): Don't decide whether to convert
or not based on TYPE_SIZE comparison, convert whenever arg_type
is unsigned INTEGER_TYPE.
On 02/12/2012 12:15 PM, Eric Botcazou wrote:
2012-02-12 Eric Botcazou ebotca...@adacore.com
* configure.tgt (target_cpu): Handle sparc and sparc64 sparcv9.
* config/sparc/cacheline.h: New file.
* config/sparc/target.h: Likewise.
* config/sparc/sjlj.S: Likewise.
Hello world,
the attached patch xfails the offencing test case on architectures
which do not allow unaligned access for vecorization. OK for trunk?
Any other architectures which should be XFAILed?
Regression-tested on powerpc64-unknown-linux-gnu. OK for trunk?
Thomas
2012-02-13
Alexandre Oliva aol...@redhat.com writes:
Jakub asked to have a closer look at the problem, and I found we could
do somewhat better. The first thing I noticed was that the problem was
that, in each block that computed a (base+const), we created a new VALUE
for the expression (with the same
On Mon, Feb 13, 2012 at 7:20 PM, Janne Blomqvist
blomqvist.ja...@gmail.com wrote:
Hi,
the attached patch changes the low-level libgfortran IO dispatching
mechanism to use shared vtables for each stream type, instead of all
the function pointers being replicated for each unit. This is similar
This patch re-writes the streaming of binding levels to guarantee that
the whole tree of binding levels in each file is written and merged-in
before anything else.
With this re-factoring, we now write all the binding levels, the merge
keys for symbols/types and their other contents at the start
On 02/11/2012 06:14 AM, Eric Botcazou wrote:
2012-02-11 Eric Botcazou ebotca...@adacore.com
* gcc.c (LINK_COMMAND_SPEC): Deal with -fgnu-tm.
(GTM_SELF_SPECS): Define if not already defined.
(driver_self_specs): Add GTM_SELF_SPECS.
* config/darwin.h
On 2/13/2012 3:02 PM, Richard Henderson wrote:
On 02/13/2012 07:42 AM, Walter Lee wrote:
1/6 toplevel: http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01860.html
2/6 contrib: http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01855.html
3/6 gcc: http://gcc.gnu.org/ml/gcc-patches/2012-01/msg01494.html
2012/2/12 Gerald Pfeifer ger...@pfeifer.com:
On Fri, 27 Jan 2012, Fabien Chêne wrote:
I get back to you for the snippet about deprecated access
declarations. I would also find it sensible to advertise about the fix
of c++/14258, a popular bug I have hit myself many times. OK to commit
the
This patch adds a new TM method, ml_wt, which uses an array of locks
with version numbers and runs a write-through algorithm with time-based
validations and snapshot time extensions.
patch1 adds xcalloc as a helper function for allocations (used in the
new TM method).
patch2 improves TM method
The following tiny patch fixes testsuite regressions on x86-64 occurred
after latest merge (this weekend).
The patch was successfully bootstrapped on x86/x86-64.
Committed as rev. 184173.
2012-02-13 Vladimir Makarov vmaka...@redhat.com
* lra.c (check_rtl): Ignore addr with UNSPEC.
On Mon, Feb 13, 2012 at 10:50 PM, Vladimir Makarov vmaka...@redhat.com wrote:
The following tiny patch fixes testsuite regressions on x86-64 occurred
after latest merge (this weekend).
Hello Vladimir,
Could you please also update http://gcc.gnu.org/svn.html#devbranches ?
It still mentions ira
Per http://gcc.gnu.org/codingconventions.html we should use
dependent, not dependant. This fixes this for the new GCC 4.7
porting notes as well as one old news entry.
Committed.
Gerald
Index: gcc-4.7/porting_to.html
===
RCS file:
On 02/13/2012 05:07 PM, Steven Bosscher wrote:
On Mon, Feb 13, 2012 at 10:50 PM, Vladimir Makarovvmaka...@redhat.com wrote:
The following tiny patch fixes testsuite regressions on x86-64 occurred
after latest merge (this weekend).
Hello Vladimir,
Could you please also update
Mikael,
This is OK for trunk with one proviso; could you move
is_class_container_ref to gfc_is_class_container_ref in class.c?
Thanks for the patch
Paul
On Sun, Feb 12, 2012 at 10:11 PM, Mikael Morin mikael.mo...@sfr.fr wrote:
Hello,
this is the next PR50981 fix:
when passing polymorphic
On Mon, 13 Feb 2012, Fabien Chêne wrote:
Do we need an update for http://gcc.gnu.org/gcc-4.7/porting_to.html
as well?
I don't know. The deprecation of access declarations only raises a
warning -- unless -Werror is used.
Is porting_to.html appropriate to describe the way to fix the warning ?
On 02/13/2012 01:47 PM, Torvald Riegel wrote:
+ else {
Watch the formatting.
+ // Location-to-orec mapping. Stripes of 16B mapped to 2^19 orecs.
+ static const gtm_word L2O_ORECS = 1 19;
+ static const gtm_word L2O_SHIFT = 4;
Is it just easier to say 16B or did we really want
Hello!
We can jump indirect from memory address, sparing a couple of cycles.
2012-02-14 Uros Bizjak ubiz...@gmail.com
* config/x86/target.h (GTM_longjmp): Jump indirect from memory address.
Tested on x86_64-pc-linux-gnu {,-m32}.
OK for mainline?
Uros.
Index: config/x86/sjlj.S
On 02/13/2012 02:54 PM, Uros Bizjak wrote:
- movq56(%rsi), %rdx
movl%edi, %eax
cfi_def_cfa(%rcx, 0)
- cfi_register(%rip, %rdx)
movq%rcx, %rsp
- jmp *%rdx
+ jmp *56(%rsi)
If you're going to do that, the correct fix for the unwind info is
On 02/11/2012 12:56 AM, Uros Bizjak wrote:
FWIW, the mode of flags in users doesn't matter at all on x86, but
which way is correct?
As far as I know, it doesn't matter anywhere. We don't even bother to have
perfect harmony between integer modes in hard registers -- think about what
happens
This patch fixes a race condition in how transactions previously chose
the dispatch at transaction begin: default_dispatch in retry.cc was
read by transaction before they became either serial or nonserial
transactions (with the serial_lock). A concurrent change of
default_dispatch was possible
On 02/02/2012 11:01 AM, Richard Sandiford wrote:
Ping for:
http://gcc.gnu.org/ml/gcc-patches/2012-01/msg01564.html
which fixes a MIPS va_arg regression (admittedly a long-standing one)
on zero-sized types. There are no functional changes to other targets
and I'm as confident as I can
On Tue, Feb 14, 2012 at 12:00 AM, Richard Henderson r...@redhat.com wrote:
On 02/11/2012 12:56 AM, Uros Bizjak wrote:
FWIW, the mode of flags in users doesn't matter at all on x86, but
which way is correct?
As far as I know, it doesn't matter anywhere. We don't even bother to have
perfect
On 02/13/2012 03:03 PM, Torvald Riegel wrote:
-// The default TM method used when starting a new transaction.
-static GTM::abi_dispatch* default_dispatch = 0;
+// The default TM method used when starting a new transaction. Initialized
+// in number_of_threads_changed() below.
+static
Hi!
To avoid -fcompare-debug failures, we promote_debug_loc VALUEs looked
up from DEBUG_INSNs when they are looked from some other insns.
Unfortunately, the scheduler after cselib_lookup_from_insn from
DEBUG_INSN calls cselib_subst_to_values, which may e.g. cselib_lookup_mem
(with create=0). As
On 02/13/2012 03:17 PM, Jakub Jelinek wrote:
2012-02-13 Jakub Jelinek ja...@redhat.com
PR bootstrap/52172
* cselib.h (cselib_subst_to_values_from_insn): New prototype.
* cselib.c (cselib_subst_to_values_from_insn): New function.
* sched-deps.c
On Feb 13, 2012, at 1:43 PM, Walter Lee wrote:
Thanks for the review. Do I have permission to commit,
Yes, you do. Richard can approve this, and when he says, Ok., you're good to
go.
or is there anything else I need to do?
Nope. (Assuming you have write after approval to the tree.)
Hi,
this patch fixes typo I introduced in my patch fixing infinte recursion of
predict_paths_for_bb.
While converting the check from aux pointers to bitmaps, I got bitmap_set_bit
wrong.
Bootstrapped/regtested x86_64-linux, comitted.
PR middle-end/52214
* predict.c
On Sat, Feb 11, 2012 at 1:12 AM, Richard Henderson r...@redhat.com wrote:
Seeing as how Uros is starting to go down the path of cleaning up the
flags handling for x86, I thought I'd go ahead and knock up the idea
that I've been tossing around to help automate the process of building
patterns
On Mon, Feb 13, 2012 at 11:57 PM, Richard Henderson r...@redhat.com wrote:
On 02/13/2012 02:54 PM, Uros Bizjak wrote:
- movq 56(%rsi), %rdx
movl %edi, %eax
cfi_def_cfa(%rcx, 0)
- cfi_register(%rip, %rdx)
movq %rcx, %rsp
- jmp *%rdx
+ jmp
On 02/13/2012 04:09 PM, Uros Bizjak wrote:
On Mon, Feb 13, 2012 at 11:57 PM, Richard Henderson r...@redhat.com wrote:
On 02/13/2012 02:54 PM, Uros Bizjak wrote:
- movq56(%rsi), %rdx
movl%edi, %eax
cfi_def_cfa(%rcx, 0)
- cfi_register(%rip, %rdx)
movq
Uros Bizjak ubiz...@gmail.com writes:
alpha linux does not have expected /proc/net/igmp and
/proc/net/igmp6 files, so func interfaceMulticastAddrTable(ifindex
int) from interface_linux.go always returns (nil, nil), failing
net/test with:
--- FAIL: net.TestListenMulticastUDP (4.71 seconds)
PR 50654 points out that many Go tests fail on systems that use emutls.
This turns out to be a subtle issue involving the use of setcontext and
getcontext. When a particular invocation is moved to run on a different
thread via getcontext and setcontext, it must reload the thread-local
variables m
OK.
Jason
This patch fixes this particular bug, but there are some issues.
First, non_static_member_function_p only checks the first function in
the overload set, which may not be representative of all of them. It
really shouldn't look through OVERLOADs, we need to defer this decision
until
From: Eric Botcazou ebotca...@adacore.com
Date: Sun, 12 Feb 2012 21:15:26 +0100
+ load[%o1 + OFFSET (JB_CFA)], %fp
+ cfi_def_cfa(%fp, 0)
+#if STACK_BIAS
+ sub %fp, STACK_BIAS, %fp
+ cfi_def_cfa_offset(STACK_BIAS)
+#endif
I think you really need to put the proper
From: Eric Botcazou ebotca...@adacore.com
Date: Sun, 12 Feb 2012 21:15:26 +0100
+static inline void
+cpu_relax (void)
+{
+ __asm volatile ( : : : memory);
+}
We probably want to do some nop'ish thing here which will yield the
cpu thread on Niagara cpus, I'd recommend something along the
On Sat, 11 Feb 2012, Eric Botcazou wrote:
Hi,
this completes the half-implemented linking scheme of libitm and makes it
mimic
that of libgomp entirely. We need the -pthread thing on Solaris 8.
It broke all targets that don't implement threads and as such
don't support -pthread. And you
Hi,
This patch adds support for powerpc*-grtev2-linux-gnu. The changes
include:
1. Relocating the dynamic linker using a run-time root prefix.
2. Using different library setting in static linking.
This is tested by building PowerPC64 and PowerPC toolchains and ran
some tests with the resulting
On Mon, Feb 13, 2012 at 6:41 PM, Doug Kwan dougk...@google.com wrote:
Hi,
This patch adds support for powerpc*-grtev2-linux-gnu. The changes
include:
1. Relocating the dynamic linker using a run-time root prefix.
2. Using different library setting in static linking.
This is tested by
Thanks Andrew. I will take a look at that.
-Doug
On Mon, Feb 13, 2012 at 6:45 PM, Andrew Pinski pins...@gmail.com wrote:
On Mon, Feb 13, 2012 at 6:41 PM, Doug Kwan dougk...@google.com wrote:
Hi,
This patch adds support for powerpc*-grtev2-linux-gnu. The changes
include:
1. Relocating
The patch uses the weak version of compare_exchange universally, which
is incorrect in a number of cases. You wouldn't see this on x86_64;
you'd have to use a ll/sc target such as powerpc.
In addition to changing several uses to strong compare_exchange, I
also optimize the idiom
-Original Message-
From: Richard Earnshaw
Sent: Monday, February 13, 2012 7:37 PM
To: Bin Cheng
Cc: gcc-patches@gcc.gnu.org
Subject: Re: [PATCH ARM] backport r174803 from trunk to 4.6 branch
On 08/02/12 08:29, Bin Cheng wrote:
Hi,
Julian Brown once posted a patch fixing ARM
Hello Gerald,
Thank you for this suggestion.
I have not worked on these changes before. Therefore, can you please review
the attached patch and let me know if any changes are required to be done in
it.
Thanks and Regards,
Jayant Sonar
[KPIT Cummins, Pune]
cr16-htdocs2.diff
Description:
Committed.
2012-02-14 Walter Lee w...@tilera.com
* MAINTAINERS (Write After Approval): Add myself.
Index: MAINTAINERS
===
--- MAINTAINERS (revision 184193)
+++ MAINTAINERS (working copy)
@@ -428,6 +428,7 @@ Asher Langton
Hi,
This patch was submitted as part of PR 52235.
It increases the preference of a register for first operand of a
commutative operator.
2012-02-13 Paulo Matos paulo.ma...@csr.com
* gcc/rtlanal.c: Increase preference of a register for the
first operand in a commutative
On Tue, Feb 14, 2012 at 1:15 AM, Richard Henderson r...@redhat.com wrote:
- movq 56(%rsi), %rdx
movl %edi, %eax
cfi_def_cfa(%rcx, 0)
- cfi_register(%rip, %rdx)
movq %rcx, %rsp
- jmp *%rdx
+ jmp *56(%rsi)
If you're going to do that, the
On Tue, Feb 14, 2012 at 8:39 AM, Uros Bizjak ubiz...@gmail.com wrote:
- cfi_register(%rip, %rdx)
+ cfi_offset(%rip, 56)
Hm, we just defined new CFA as rcx+0, so we should define location of
rip relative to new CFA. Since CFA points to stack slot just before
return address was pushed, new
82 matches
Mail list logo