On Wed, 27 Mar 2013, Eric Botcazou wrote:
int is getting small to store one bit per vector element (V32QI...) so I
switched to hwint after checking that Zadeck's patches don't touch this.
unsigned HOST_WIDE_INT is indeed the correct type to use for mask manipulation
but please use UINTVAL
On 27.02.2013 13:03, Andrey Belevantsev wrote:
Hello,
For this volatile-related issue (no volatile bits on volatile fields of a
non-volatile struct) AFAIU there is an agreement of fixing the front-ends
if needed, but anyways the patch to the selective scheduler is required
that properly merges
On 22.02.2013 17:30, Andrey Belevantsev wrote:
Hello,
As found by Jakub and explained in the PR audit trail by Alexander, this
patch fixes the selective scheduler merge glitch of 2008 that added the
unnecessary JUMP_P check to the flush_pending_lists call. I have removed
the check and expanded
On 19.02.2013 17:13, Andrey Belevantsev wrote:
Hello,
As we discussed in the PR, the problem here is that the selective scheduler
does not expect that renaming a hard register to a pseudo would result in
extra dependencies. The dependencies come from implicit clobbers code of
sched-deps.c, so
Hi,
On 04/01/2013 02:00 AM, Gerald Pfeifer wrote:
Andi's patch broke bootstrap on all FreeBSD platforms, which took me
a bit to realize since he did not update the ChangeLog:
2013-03-23 Andi Kleen a...@my.domain.org
* local_atomic (__always_inline): Add.
This page now goes to the general AMD press release page, which we
don't want to link to (and which does not have the original information
anymore). So, gone it is.
Gerald
Index: readings.html
===
RCS file:
Fix test case sra-13.c that assumed int is always 4 bytes.
Regards,
Pitchumani
2013-04-01 Pitchumani Sivanupandi pitchuman...@atmel.com
testsuite
* gcc.dg/tree-ssa/sra-13.c: Fix for 16 bit int
--- gcc/testsuite/gcc.dg/tree-ssa/sra-13.c (revision 197081)
+++
Ping!
(I hope I didn't anti-advertise this patch too much ;)
2013/3/20 Janus Weil ja...@gcc.gnu.org:
Hi all,
here is a simple patch which fixes some problems with IMPLICT
CLASS(...) statements. Actually that's not a feature I would seriously
recommend anyone to use, but the Fortran
On 04/01/2013 07:03 AM, Janus Weil wrote:
Ping!
(I hope I didn't anti-advertise this patch too much ;)
OK for trunk.
Jerry
Hi Janus,
Ping!
(I hope I didn't anti-advertise this patch too much ;)
2013/3/20 Janus Weil ja...@gcc.gnu.org:
Hi all,
here is a simple patch which fixes some problems with IMPLICT
CLASS(...) statements. Actually that's not a feature I would seriously
recommend anyone to use, but the
2013-04-01 Catherine Moore c...@codesourcery.com
* config/mips/mips.md (*movhi_internal, *movqi_internal): New
operands. Record compression.
Index: mips.md
===
--- mips.md (revision 197114)
+++ mips.md
On Android pthread is integrated into libc.
Attached patch fixes configures for this case by trying to build test
without -pthread -lpthread.
2013-04-01 Pavel Chupin pavel.v.chu...@intel.com
Fix libatomic and libgomp configure for systems without libpthread
*
Hello!
A part of an error has been demoted to a note recently. Attached
patch fixes a testcase in the obj-c++ testsuite.
2013-04-01 Uros Bizjak ubiz...@gmail.com
* obj-c++.dg/try-catch-13.mm: Use dg-message for
initializing argument note.
Tested on x86_64-pc-linux-gnu.
OK for
Ping!
(I hope I didn't anti-advertise this patch too much ;)
2013/3/20 Janus Weil ja...@gcc.gnu.org:
Hi all,
here is a simple patch which fixes some problems with IMPLICT
CLASS(...) statements. Actually that's not a feature I would seriously
recommend anyone to use, but the Fortran
Fix an error I made in one of the non-mechanical changes of r197234.
PR middle-end/56798
* cfgbuild.c (inside_basic_block_p): Restore check broken at r197234.
Index: cfgbuild.c
===
--- cfgbuild.c (revision 197267)
On Sat, 30 Mar 2013, Marc Glisse wrote:
* tree-flow-inline.h (get_addr_base_and_unit_offset_1): Handle
BIT_FIELD_REF.
I wrote a safer version of this for PR52436:
case BIT_FIELD_REF:
- return NULL_TREE;
+ {
+ HOST_WIDE_INT this_off =
Hi,
three items:
- Remove DECL_UNBOUND_CLASS_TEMPLATE_P, unused outside cp-tree.h.
- Use get_containing_scope in 4 places (obvious I guess)
- Use existing predicates in 2 places (likewise)
Tested x86_64-linux. Ok?
Thanks,
Paolo.
PS: In my opinion we should also rename DECL_FUNCTION_MEMBER_P
On 04/01/2013 12:41 PM, Paolo Carlini wrote:
+#define DECL_FUNCTION_TEMPLATE_P(NODE) \
+ (TREE_CODE (NODE) == TEMPLATE_DECL\
+DECL_TEMPLATE_RESULT (NODE) != NULL_TREE \
Do we need the NULL_TREE check? That is, do we
Hi,
On 04/01/2013 06:50 PM, Jason Merrill wrote:
On 04/01/2013 12:41 PM, Paolo Carlini wrote:
+#define DECL_FUNCTION_TEMPLATE_P(NODE) \
+ (TREE_CODE (NODE) == TEMPLATE_DECL \
+DECL_TEMPLATE_RESULT (NODE) != NULL_TREE\
Do we need the NULL_TREE check?
Hi again
On 04/01/2013 06:55 PM, Paolo Carlini wrote:
Hi,
On 04/01/2013 06:50 PM, Jason Merrill wrote:
On 04/01/2013 12:41 PM, Paolo Carlini wrote:
+#define DECL_FUNCTION_TEMPLATE_P(NODE) \
+ (TREE_CODE (NODE) == TEMPLATE_DECL \
+DECL_TEMPLATE_RESULT (NODE) !=
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.8.0.de.po', has just
On 04/01/2013 01:11 PM, Paolo Carlini wrote:
We have an ICE for g++.dg/template/qualttp17.C. Is this something we
want to further investigate now or shall I just leave the check in?
Leave the check. The patch is OK.
Jason
Hi all,
When doing namelist reads, nml_read_obj calls itself recursively to read through
arrays. Short lists are allowed so we have to have a way to detect if we have a
short read or a real error.
We do this by flagging errors and then backing out of the read and checking to
see if what we
Am 01.04.2013 20:33, schrieb Jerry DeLisle:
With this particular bug, nml_read_obj was clearing the error flag itself with
the read so that rather then bailing out, it tried to continue reading data
until it was done, then the subsequent read failed looking for a valid name,
which had been
I'm not sure what this is supposed to do. How is pat == target ever
going to be true when ANY_RETURN_P (pat) is true? Isn't target supposed
to be a CODE_LABEL or NULL? What am I missing?
The simple_return change from Bernd. The JUMP_LABEL of a JUMP_INSN is now
either a CODE_LABEL or a
On 03/29/13 16:57, Iyer, Balaji V wrote:
Hello Joseph, Aldy et al.,
I reworded couple comments (e.g changed builtin with built-in, etc) and
added a header comment to the c-array-notation.c that explains the overall
process. I am attaching a fixed patch.
Ok, this latest patch that
On Sun, Mar 31, 2013 at 12:13 AM, Jason Merrill ja...@redhat.com wrote:
On 03/29/2013 10:56 PM, Gabriel Dos Reis wrote:
int wanted = num_template_headers_for_class (ctx);
I think you want to add one to wanted if decl is a primary template.
OK but I am a little bit dense here: why is
On 04/01/2013 04:53 PM, Gabriel Dos Reis wrote:
On Sun, Mar 31, 2013 at 12:13 AM, Jason Merrill ja...@redhat.com wrote:
On 03/29/2013 10:56 PM, Gabriel Dos Reis wrote:
int wanted = num_template_headers_for_class (ctx);
I think you want to add one to wanted if decl is a primary
On 03/29/13 16:57, Iyer, Balaji V wrote:
+bool
+find_rank (location_t loc, tree orig_expr, tree array, bool ignore_builtin_fn,
+ size_t *rank)
+{
+ tree ii_tree;
Balaji, I believe what Joseph meant with saving the location_t is so we
can give meaningful error messages when
Hi all,
here is a small patch which does two things:
1) It fixes the ICE in the subject line (in a rather obvious way).
2) It warns about alternate-return arguments (which is an obsolescent
feature), and adds -std=legacy to some test cases to suppress the
warning.
Regarding the second point, one
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
ow...@gcc.gnu.org] On Behalf Of Aldy Hernandez
Sent: Monday, April 01, 2013 5:02 PM
To: Iyer, Balaji V
Cc: 'Joseph Myers'; 'gcc-patches'
Subject: Re: [patch] cilkplus: Array notation for C patch
On
On Mon, Apr 1, 2013 at 3:58 PM, Jason Merrill ja...@redhat.com wrote:
On 04/01/2013 04:53 PM, Gabriel Dos Reis wrote:
On Sun, Mar 31, 2013 at 12:13 AM, Jason Merrill ja...@redhat.com wrote:
On 03/29/2013 10:56 PM, Gabriel Dos Reis wrote:
int wanted = num_template_headers_for_class
We try to resolve types at template definition time if expressions don't
depend on template parameters, but this testcase shows a case of
dependency we weren't considering: the bounds of an array can depend on
the length of the initializer.
Tested x86_64-pc-linux-gnu, applying to 4.7, 4.8,
Here, when we saw a.B::Y we got confused because B is not a class. But
this seems to me like a reasonable thing to say.
Tested x86_64-pc-linux-gnu, applying to trunk and 4.8.
commit 2ea3a64073bdfe37cfd812e5b297fa9831d10e71
Author: Jason Merrill ja...@redhat.com
Date: Mon Apr 1 16:16:49 2013
Here trying to process the initializer at template time didn't work;
let's not bother in this case.
Tested x86_64-pc-linux-gnu, applying to trunk and 4.8.
commit 48ec5c5cc4c8f5640dd25b3b455141cde868361c
Author: Jason Merrill ja...@redhat.com
Date: Mon Apr 1 16:30:48 2013 -0400
PR
Janus Weil wrote:
here is a small patch which does two things:
1) It fixes the ICE in the subject line (in a rather obvious way).
2) It warns about alternate-return arguments (which is an obsolescent
feature), and adds -std=legacy to some test cases to suppress the
warning.
I would prefer if
Tobias Burnus wrote:
This patch adds a rank field to the descriptor and uses it. Before
the information was in the dtype. At the same time, it bumps the
maximal rank from 7 to 15 as required by Fortran 2008.
I have now committed the patch as Rev. 197323 to Fortran-dev - and
updated it to the
here is a small patch which does two things:
1) It fixes the ICE in the subject line (in a rather obvious way).
2) It warns about alternate-return arguments (which is an obsolescent
feature), and adds -std=legacy to some test cases to suppress the
warning.
I would prefer if you could use
Hello,
The SET_INSN_DELETED vs. set_insn_deleted thing is just an incomplete cleanup.
Using set_block_for_insn instead of using BLOCK_FOR_INSN is the proper way.
So, mechanical replacements. If no-one objects and the usual
bootstrap+testing passes, I'll commit this later this week as obvious.
We need to do EXPR_LOCATION (array2) and then issue the error
note: original mismatching rank at:
array[:][:] = array2[:];
^
Yes.
Or something similar... the note can be emitted with inform(). So you need to
save the original rank location to emit the note.
At Gerald's suggestion this reworks the libstdc++ contributing docs to
refer to the general GCC contributing docs instead of repeating the
same info.
* doc/xml/manual/appendix_contributing.xml: Remove broken link and
defer to general documentation on contributing to GCC.
*
On 04/01/2013 01:47 PM, Eric Botcazou wrote:
I'm not sure what this is supposed to do. How is pat == target ever
going to be true when ANY_RETURN_P (pat) is true? Isn't target supposed
to be a CODE_LABEL or NULL? What am I missing?
The simple_return change from Bernd. The JUMP_LABEL of a
On Sat, Mar 30, 2013 at 06:29:36PM -0400, David Edelsohn wrote:
How can we make progress to get this patch committed on trunk, 4.8 and 4.7?
I have OKs for the config/i386/winnt.c and config/rs6000/rs6000.c
parts. I just need someone who is authorized to review patches to
varasm.c, and is
1.c attached.
On Mon, Apr 1, 2013 at 10:43 PM, Wei Mi w...@google.com wrote:
I attached the patch.4 based on r197308. r197308 changes shift-and
type truncation from define_insn_and_split to define_insn. patch.4
changes ix86_rtx_costs for shift-and type rtx to get the correct cost
for the
44 matches
Mail list logo
