I've committed the attached patch.
2013-03-27 Andreas Krebbel
* 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
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
* libgfortran/intrinsics/string_intrinsics_inc.c (compare_strin
On Tue, Mar 26, 2013 at 11:53:27PM +0200, Janne Blomqvist wrote:
> On Mon, Mar 25, 2013 at 7:00 PM, Thomas Koenig wrote:
> > Hello world,
> >
> > this patch uses memcpy() directly when comparing two kind=1 strings of
> > equal and constant lengths. The test case modification depends
> > on the pr
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
(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 wrote:
> On 3/26/13, Lawrence Crowl wrote:
>> On 3/26/13, Richard Biener wrote:
>>> On Mar 25, 2013 Lawrence Crowl wrote:
>>> > On 3/25/13, Richard Biener wrote:
>>> > > You add a not used new interface. What for?
>>> >
>>> > So that people can
On Sat, Mar 23, 2013 at 11:36 PM, Lawrence Crowl 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 split target b
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 o
> 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:
+ u
** 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 c
>
> 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 i
> 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 .)
A new POT file for textual domain 'cpplib' has been made available
to the language teams for translation. It is archived as:
http://translationproject.org/POT-files/c
Hello, gentle maintainer.
This is a message from the Translation Project robot. (If you have
any questions, send them to .)
A new POT file for textual domain 'gcc' has been made available
to the language teams for translation. It is archived as:
http://translationproject.org/POT-files/gcc-
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 :
>> Ok with that change.
>
> Checked into trunk: http://
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
* gcc/config/i386/avx2intrin.h (_mm256_broadcastsi128_si256):
F
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 that
On Wed, Mar 27, 2013 at 12:22 PM, Kirill Yukhin 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
>
> * gcc/
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)
+ {
+ in
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 cal
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
PR tree-optimization/56716
* tree-ssa-structalias.c (
On Wed, Feb 27, 2013 at 1:22 AM, Kenneth Zadeck
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 &
(HOST_BITS_PER_WIDE_INT - 1));
On Wed, Feb 27, 2013 at 1:27 AM, Kenneth Zadeck
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 genmodes.c so that larg
On Wed, Feb 27, 2013 at 1:37 AM, Kenneth Zadeck
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
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 add
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 279da5952b
On Wed, Feb 27, 2013 at 2:59 AM, Kenneth Zadeck
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 believe that the wide
On Wed, Mar 27, 2013 at 3:23 PM, Kenneth Zadeck
wrote:
>
> On 03/27/2013 10:18 AM, Richard Biener wrote:
>>
>> On Wed, Feb 27, 2013 at 1:27 AM, Kenneth Zadeck
>> wrote:
>>>
>>> Here is the second of my wide int patches with the patch rot removed.
>>>
>>> I would like to get these pre approved for
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
* gcc/cp/Makefile.la
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
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.
Th
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 wrote:
>> Thanks for the
"Iyer, Balaji V" 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
>>
>> Hello Eve
On Tue, Mar 26, 2013 at 4:10 AM, Maksim Kuznetsov
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 generalization.
>>
>>
On 3/27/13, Richard Biener wrote:
> On Mar 27, 2013, Lawrence Crowl 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 names. These names are:
>>
>>
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 wrote:
> On Mar 23, 2013 Lawrence Crowl 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
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 debu
I found that the node weight updates on cloned nodes during ipa-cp were
leading to incorrect/insane weights. Both the original and new node weight
computations used truncating divides, leading to a loss of total node weight.
I have fixed this by making both rounding integer divides.
Bootstrapped a
> 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.
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, 'cpp
Ping. Thanks, Teresa
On Fri, Dec 14, 2012 at 6:11 PM, Teresa Johnson 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 compiler will use
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
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): Extend
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 2
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 J
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 on
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 inste
2013-03-27 Gerald Pfeifer
* 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
Index: doc/extend.texi
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" p
*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:
>
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 x86_
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 cos
63 matches
Mail list logo
