This patch replaces (const_int 0) in vector patterns with its
const_vector representation. Backported to 4.7 and 4.8.
* config/tilegx/tilegx.md (insn_mnz_mode): Replaced by ...
(insn_mnz_v8qi): ... this ...
(insn_mnz_v4hi): ... and this. Replace (const_int 0) with the
This patch fixes the type attribute for jr. Backported to 4.7 and
4.8.
* config/tilegx/tilegx.md (*sibcall_insn): Fix type atribute
for jr.
(*sibcall_value): Ditto.
--- gcc/config/tilegx/tilegx.md (revision 197135)
+++ gcc/config/tilegx/tilegx.md (working copy)
@@
This patch deletes an extra tab in the asm addr vectors. Backported
to 4.7 and 4.8.
* config/tilegx/tilegx.h (ASM_OUTPUT_ADDR_VEC_ELT): Delete
extra tab.
(ASM_OUTPUT_ADDR_DIFF_ELT): Ditto.
--- gcc/config/tilegx/tilegx.h (revision 197134)
+++ gcc/config/tilegx/tilegx.h
I've committed the attached patch.
2013-03-27 Andreas Krebbel andreas.kreb...@de.ibm.com
* config/s390/s390.h (TARGET_FLT_EVAL_METHOD): Define.
---
gcc/config/s390/s390.h |5 +
1 file changed, 5 insertions(+)
Index: gcc/config/s390/s390.h
-Original Message-
From: Richard Earnshaw
Sent: Tuesday, March 26, 2013 6:57 PM
To: Bin Cheng
Cc: gcc-patches@gcc.gnu.org
Subject: Re: [PATCH GCC/ARM]Fix rtx cost for Thumb1
On 26/03/13 08:34, Bin Cheng wrote:
Hi,
As reported in PR56102, arm back end returns wrong rtx cost
Hi,
as I looked to compare_string I discovered that it could be
optimized. This speeds up case when strings are equal but we must check
padding where checking it byte by byte is suboptimal.
Ondra
2013-03-27 Ondřej Bílka nel...@seznam.cz
*
On Tue, Mar 26, 2013 at 11:53:27PM +0200, Janne Blomqvist wrote:
On Mon, Mar 25, 2013 at 7:00 PM, Thomas Koenig tkoe...@netcologne.de wrote:
Hello world,
this patch uses memcpy() directly when comparing two kind=1 strings of
equal and constant lengths. The test case modification depends
Il 26/03/2013 21:48, Gabriel Dos Reis ha scritto:
Hi Paolo,
The patchlet below allows uses of source file with .cc extension.
This comes out of work being done on the C++ front-end and has merit of
its own. OK to apply?
Thanks,
-- Gaby
2013-03-26 Gabriel Dos Reis
Having half-word accesses into the minipool is generally a bad idea.
The limited offset range that's supported by these instructions means
it's much more likely that we'll end up with a pool after a conditional
branch or, worse, in the middle of a linear code sequence. That means
we have to
(The email below was only sent to gcc-patches@; I now also CC fortran@ -
sorry for the full quote)
Regarding the below patch: I think it does not work as-is for Unicode
strings (UCS4, character(kind=4)), where each character is 4 bytes wide
and a space does not consist of sequences of four '
On Wed, Mar 27, 2013 at 12:50 AM, Lawrence Crowl cr...@googlers.com wrote:
On 3/26/13, Lawrence Crowl cr...@googlers.com wrote:
On 3/26/13, Richard Biener richard.guent...@gmail.com wrote:
On Mar 25, 2013 Lawrence Crowl cr...@googlers.com wrote:
On 3/25/13, Richard Biener
On Sat, Mar 23, 2013 at 11:36 PM, Lawrence Crowl cr...@googlers.com wrote:
This patch is a consolodation of the hash_table patches to the
cxx-conversion branch.
Update various hash tables from htab_t to hash_table.
Modify types and calls to match.
Ugh. Can you split it up somewhat ... like
Hi Tobias,
I think you need to deallocate array_size in gfc_simplify_sizeof
unconditionally.
+ if (x-rank x-expr_type != EXPR_ARRAY
+ gfc_array_size (x, array_size) == FAILURE)
+return NULL;
Here, it is allocated on success of gfc_array_size.
[...]
And here, it is freed only if
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 instead of INTVAL with it. And:
+
** PING **
And an updated patch. Changes:
- Updated isym handling due to the ISO_C_BINDING patch
- Fixed some bugs in the generated code for finalizing arrays (mainly
missing gfc_copy_expr)
Build and tested on x86-64-gnu-linux.
OK for the trunk?
Tobias
PS: Regarding true FINAL support: The
Otherwise OK.
Thanks,
Hi, chacked into trunk: http://gcc.gnu.org/ml/gcc-cvs/2013-03/msg00785.html
Thanks, K
This patch passes bootstrap+testsuite on x86_64-linux-gnu. Using the
opposite arbitrary order in compare_commutative_operands_precedence
(exchange x and y in the line with GET_CODE) passes as well. The
simplify-rtx bit is because I get an infinite recursion otherwise.
Surprisingly, that
Ok with that change.
Checked into trunk: http://gcc.gnu.org/ml/gcc-cvs/2013-03/msg00786.html
Thanks, K
Hello, gentle maintainer.
This is a message from the Translation Project robot. (If you have
any questions, send them to coordina...@translationproject.org.)
A new POT file for textual domain 'gcc' has been made available
to the language teams for translation. It is archived as:
Hi,
Thank you for your review. Since the patch was aimed to fix tests
and it is pretty small, is it posible to backport it to 4.8 branch? So we
would not have those testfails on Android with new release.
Alexander
2013/3/27 Kirill Yukhin kirill.yuk...@gmail.com:
Ok with that change.
Checked
Hello,
According to recent Spec, we have intrinsic
_mm256_broadcastsi128_si256, not _mm_broadcastsi128_si256.
Attached patch makes avx2intrin.h consistent with Spec in that way.
ChangeLog:
2013-03-27 Kirill Yukhin kirill.yuk...@intel.com
* gcc/config/i386/avx2intrin.h
Am 27.03.2013 10:47, schrieb Thomas Koenig:
I think you need to deallocate array_size in gfc_simplify_sizeof
unconditionally.
Actually, when I applied the more than two years old draft patch, I
thought likewise - but that leads to segfaults. If one reads the code
more carefully, one sees
On Wed, Mar 27, 2013 at 12:22 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote:
According to recent Spec, we have intrinsic
_mm256_broadcastsi128_si256, not _mm_broadcastsi128_si256.
Attached patch makes avx2intrin.h consistent with Spec in that way.
ChangeLog:
2013-03-27 Kirill Yukhin
This makes the vectorizer handle loads of SLP groups with
trailing gaps. That is, a (for now know) gap between
the groups between loop iterations as in the testcase
+ double self[1024];
+ double a[1024][1024];
+ double b[1024];
+
+ void __attribute__((noinline,noclone))
+ foo (void)
+ {
+ int
Hi,
On Sat, Mar 23, 2013 at 03:36:35PM -0700, Lawrence Crowl wrote:
This patch is a consolodation of the hash_table patches to the
cxx-conversion branch.
I can't approve anything but I am fine with
* tree-sra.c (candidates):
Change type to hash_table. Update dependent calls
This fixes ICEs when dumping PTA pass details (I'm pretty sure
it will not fix PR56716, but this issue is reported there, too).
Bootstrap and regtest running on x86_64-unknown-linux-gnu.
Richard.
2013-03-27 Richard Biener rguent...@suse.de
PR tree-optimization/56716
*
On Wed, Feb 27, 2013 at 1:22 AM, Kenneth Zadeck
zad...@naturalbridge.com wrote:
Here is the first of my wide int patches with joseph's comments and the
patch rot removed.
I would like to get these pre approved for the next stage 1.
+ int shift = HOST_BITS_PER_WIDE_INT - (prec
On Wed, Feb 27, 2013 at 1:27 AM, Kenneth Zadeck
zad...@naturalbridge.com wrote:
Here is the second of my wide int patches with the patch rot removed.
I would like to get these pre approved for the next stage 1.
On 10/05/2012 06:48 PM, Kenneth Zadeck wrote:
This patch adds machinery to
On Wed, Feb 27, 2013 at 1:37 AM, Kenneth Zadeck
zad...@naturalbridge.com wrote:
the original patch 3 was accepted and committed. These were subsequent
comments. This could likely be just checked in the next stage 1 without
approval.
Ok.
Thanks,
Richard.
On 03/27/2013 10:18 AM, Richard Biener wrote:
On Wed, Feb 27, 2013 at 1:27 AM, Kenneth Zadeck
zad...@naturalbridge.com wrote:
Here is the second of my wide int patches with the patch rot removed.
I would like to get these pre approved for the next stage 1.
On 10/05/2012 06:48 PM, Kenneth
My change to treat SCOPE_REFs in a template as instantiation-dependent
due to access issues incorrectly affected scoped enums; we don't need to
worry about access for them, since there are no access-specifiers in an
enum.
Tested x86_64-pc-linux-gnu, applying to trunk and 4.8.
commit
On Wed, Feb 27, 2013 at 2:59 AM, Kenneth Zadeck
zad...@naturalbridge.com wrote:
This patch contains a large number of the changes requested by Richi. It
does not contain any of the changes that he requested to abstract the
storage layer. That suggestion appears to be quite unworkable.
I
On Wed, Mar 27, 2013 at 3:23 PM, Kenneth Zadeck
zad...@naturalbridge.com wrote:
On 03/27/2013 10:18 AM, Richard Biener wrote:
On Wed, Feb 27, 2013 at 1:27 AM, Kenneth Zadeck
zad...@naturalbridge.com wrote:
Here is the second of my wide int patches with the patch rot removed.
I would like
On 03/26/2013 03:29 PM, Eric Botcazou wrote:
Heh. We've actually fixed this now -- unwind info generation aware of
the cfg is exactly what pass_dwarf2_frame does. So I guess this comment
has been out of date since gcc 4.7.
I see, thanks. So what do you suggest doing at this point?
Implements reduction of requirements into the constraints language:
logical formulas comprised of atomic propositions. Calls to constraint
predicates are recursively inlined in the resulting expression. All
other calls are treated as atoms.
2013-03-01 Andrew Sutton andrew.n.sut...@gmail.com
Not sure if this went out or not...
-Original Message-
From: Iyer, Balaji V
Sent: Wednesday, March 27, 2013 11:25 AM
To: 'GCC Patches (gcc-patches@gcc.gnu.org)'
Subject: [patch][cilkplus-merge] Fix a bug in one of the test cases
Hello Everyone,
This patch will fix a bug in
This fixes PR56695 where we ICEd because expand_vec_cond_expr
doesn't expect that the result of a vector comparison is of
TYPE_UNSIGNED type. So we unconditionally build a signed type now.
We know that the result of vec comparison is e.g. { -1, 0, -1}
and that all elements have to be SI types.
I'd suggest rewriting this expression in some easier way:
p += (*p == '%' *(p + 1)) ? 2 : 1;
I'd prefer
if (*p == '%')
p++;
p++;
However, that could be only my taste:)
On 26 March 2013 15:10, Maksim Kuznetsov maks.kuznet...@gmail.com
Iyer, Balaji V balaji.v.i...@intel.com writes:
Not sure if this went out or not...
-Original Message-
From: Iyer, Balaji V
Sent: Wednesday, March 27, 2013 11:25 AM
To: 'GCC Patches (gcc-patches@gcc.gnu.org)'
Subject: [patch][cilkplus-merge] Fix a bug in one of the test cases
On Tue, Mar 26, 2013 at 4:10 AM, Maksim Kuznetsov
maks.kuznet...@gmail.com wrote:
Thanks for the explanation, now I understand it. I fixed the patch
according to your remarks. I removed %| support since we don't
actually need it in i386 right now, it was added for the purpose of
possible
On 3/27/13, Richard Biener richard.guent...@gmail.com wrote:
On Mar 27, 2013, Lawrence Crowl cr...@googlers.com wrote:
Patch with rename to debug attached.
Tested on x86_64.
Add uniform debug dump function names.
Add some overloaded functions that provide uniform debug dump
function
While debugging some code I noticed that dump_generic_stmt() does not
work on ARRAY_NOTATION_REFs. Attached is a patch adding the smarts to
tree-pretty-print.
There is no testcase because ARRAY_NOTATION_REFs are expanded by the
parser, so by the time the tree dumps happen, there are no more
On 3/27/13, Richard Biener richard.guent...@gmail.com wrote:
On Mar 23, 2013 Lawrence Crowl cr...@googlers.com wrote:
This patch is a consolodation of the hash_table patches to the
cxx-conversion branch.
Update various hash tables from htab_t to hash_table.
Modify types and calls to
Yes, it looks OK.
Thanks,
Balaji V. Iyer.
-Original Message-
From: Aldy Hernandez [mailto:al...@redhat.com]
Sent: Wednesday, March 27, 2013 12:32 PM
To: gcc-patches; Iyer, Balaji V
Subject: [cilkplus-merge] Handle generic pretty printing of
ARRAY_NOTATION_REF
While debugging
Isn't pass_dwarf2_frame enabled whenever we're generating any unwind info?
Essentially, yes:
static bool
gate_dwarf2_frame (void)
{
#ifndef HAVE_prologue
/* Targets which still implement the prologue in assembler text
cannot use the generic dwarf2 unwinding. */
return false;
#endif
cpplib-4.8.0.zh_TW.po.gz
Description: Binary data
The Translation Project robot, in the
name of your translation coordinator.
coordina...@translationproject.org
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'cpplib' has been submitted
by the Chinese (traditional) team of translators. The file is available at:
http://translationproject.org/latest/cpplib/zh_TW.po
(This file,
Ping. Thanks, Teresa
On Fri, Dec 14, 2012 at 6:11 PM, Teresa Johnson tejohn...@google.com wrote:
This patch enables the gcov-dump tool to optionally compute and dump
the working set information from the counter histogram, via a new -w option.
This is useful to help understand and tune how the
On Wed, Mar 27, 2013 at 10:20:59AM +0100, Tobias Burnus wrote:
(The email below was only sent to gcc-patches@; I now also CC
fortran@ - sorry for the full quote)
Regarding the below patch: I think it does not work as-is for
Unicode strings (UCS4, character(kind=4)), where each character is 4
So I committed the patch with two modifications at revision 197168 as
* config/i386/cygwin-stdint.h: Add support for cygwin x64 target.
* config/i386/t-cygwin-w64: New file.
* config/i386/cygwin-w64.h: New file.
* config/i386/cygwin.h (EXTRA_OS_CPP_BUILTINS):
On 03/27/13 12:01, Iyer, Balaji V wrote:
Yes, it looks OK.
Thanks,
Balaji V. Iyer.
Ok, thanks. Committed to branch.
Adjust kernel.org link to use https.
Applied.
Gerald
Index: criteria.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-3.1/criteria.html,v
retrieving revision 1.35
diff -u -3 -p -r1.35 criteria.html
--- criteria.html 29 Mar 2009
Applied.
Gerald
Index: index.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/projects/cxx-reflection/index.html,v
retrieving revision 1.5
diff -u -3 -p -r1.5 index.html
--- index.html 29 Dec 2012 00:24:56 - 1.5
+++ index.html
Both of these are currently getting a permanent redirect.
Applied.
Index: frontends.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/frontends.html,v
retrieving revision 1.37
diff -u -3 -p -r1.37 frontends.html
--- frontends.html 28
Hi Joseph.
As you have requested, I have rewritten the __sec_reduce* builtins to
use the built-in .def infrastructure. You mentioned that the
__sec_implicit_index built-in may have to be rewritten as a keyword,
provided that relevant changes are made to the specification. For now,
I have
Hi,
This small patch enables libsanitizer on ARM.
It has been tested successfully on cortex-a9 hardware (via the GCC testsuite).
I have chosen to bundle -funwind-table with -fsanitize=* so that a
useful backtrace can be printed to the user in case of error,
otherwise the reporting is limited to
Applied
Index: readings.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/readings.html,v
retrieving revision 1.221
diff -u -3 -p -r1.221 readings.html
--- readings.html 1 Nov 2012 23:41:38 - 1.221
+++ readings.html 27
On Wed, 27 Mar 2013, Aldy Hernandez wrote:
Is this OK for the branch? I'd like to get a nod of approval before
committing to the branch, as to aid in review, but if you prefer that I just
commit at will, and then post a big cilk plus array notation patch to the
list, I can do this instead.
2013-03-27 Gerald Pfeifer ger...@pfeifer.com
* doc/invoke.texi (AVR Options): Tweak link for AVR-LibC user manual.
* doc/extend.texi (Named Address Spaces): Ditto.
(Variable Attributes): Ditto.
Installed on HEAD; will also apply to GCC 4.8 in a day or two.
Gerald
On 03/27/13 17:10, Joseph S. Myers wrote:
On Wed, 27 Mar 2013, Aldy Hernandez wrote:
Is this OK for the branch? I'd like to get a nod of approval before
committing to the branch, as to aid in review, but if you prefer that I just
commit at will, and then post a big cilk plus array notation
*Ping*
On Sat, Mar 23, 2013 at 5:15 PM, Steven Bosscher wrote:
Hello,
This patch replaces all tests I could find, where the code is looking
for jump table data with GET_CODE(..)=ADDR_VEC and similar with
JUMP_TABLE_DATA_P tests.
Most replacements are mechanical, but a few are not:
* The
Hi,
this is almost obvious, I guess: Manuel kindly explained in the audit
trail of c++/56725 that if a permerror returns false no diagnostic was
actually emitted thus any accompanying inform should not be called,
otherwise with -fpermissive -w only the latter are wrongly emitted.
Tested
OK.
Jason
I am not familiar how to use define_subst, so I write a patch that
changes define_insn_and_split to define_insn. bootstrapped and
regression tested on x86_64-unknown-linux-gnu.
A question is: after that change, Is there anyway I can make
targetm.rtx_costs() aware about the truncation, .i.e the
64 matches
Mail list logo