Hi,
This patch includes the main logic to expand ccmp instructions.
In the patch,
* ccmp_candidate_p is used to identify the CCMP candidate
* expand_ccmp_expr is the main entry, which calls expand_ccmp_expr_1
to expand CCMP.
* expand_ccmp_expr_1 uses a recursive algorithm to expand CCMP
Hi,
The patches defines ccmp operand predicate for AARCH64.
OK for trunk?
Thanks!
-Zhenqiang
ChangeLog:
2014-06-23 Zhenqiang Chen
* config/aarch64/aarch64-protos.h (aarch64_uimm5): New prototype.
* config/aarch64/constraints.md (Usn): Immediate for ccmn.
* config/aar
Hi,
The patches add a set of CC mode for AARCH64, which is similar as them for ARM.
OK for trunk?
Thanks!
-Zhenqiang
ChangeLog:
2014-06-23 Zhenqiang Chen
* config/aarch64/aarch64-modes.def: Define new CC modes for ccmp.
* config/aarch64/aarch64.c (aarch64_get_condition_code_
Hi,
The patch adds three help functions to output ccmp instructions.
OK for trunk?
Thanks!
-Zhenqiang
ChangeLog:
2014-06-23 Zhenqiang Chen
* config/aarch64/aarch64-protos.h (aarch64_output_ccmp): New prototype.
* config/aarch64/aarch64.c (aarch64_code_to_nzcv): New function.
Hi,
The patches implements the two hooks for AARCH64 to generate ccmp instructions.
Bootstrap and no make check regression on qemu.
OK for trunk?
Thanks!
-Zhenqiang
ChangeLog:
2014-06-23 Zhenqiang Chen
* config/aarch64/aarch64.c (aarch64_code_to_ccmode): New function.
(aarc
Hi,
The patch enhances ifcvt to handle conditional compare instruction
(ccmp) to make it work with cmov. For ccmp, ALLOW_CC_MODE is set to
TRUE when calling canonicalize_condition. And the backend does not
need to generate additional "compare (CC, 0)" for it.
Bootstrap and no check regression on
Hi,
The patch adds two insn patterns for ccmp instructions.
cbranchcc4 is introduced to generate optimized conditional branch
without an additional compare against the result of ccmp.
OK for trunk?
Thanks!
-Zhenqiang
ChangeLog:
2014-06-23 Zhenqiang Chen
* config/aarch64/aarch64.md
On Mon, Jun 23, 2014 at 12:01 AM, Zhenqiang Chen
wrote:
> Hi,
>
> The patch enhances ifcvt to handle conditional compare instruction
> (ccmp) to make it work with cmov. For ccmp, ALLOW_CC_MODE is set to
> TRUE when calling canonicalize_condition. And the backend does not
> need to generate additio
On Mon, Jun 23, 2014 at 12:09 AM, Andrew Pinski wrote:
> On Mon, Jun 23, 2014 at 12:01 AM, Zhenqiang Chen
> wrote:
>> Hi,
>>
>> The patch enhances ifcvt to handle conditional compare instruction
>> (ccmp) to make it work with cmov. For ccmp, ALLOW_CC_MODE is set to
>> TRUE when calling canonicali
On Tue, Jun 03, 2014 at 04:00:17PM +0200, Marc Glisse wrote:
> Ping?
Ok for trunk, sorry for the delay.
Jakub
On 23 June 2014 15:09, Andrew Pinski wrote:
> On Mon, Jun 23, 2014 at 12:01 AM, Zhenqiang Chen
> wrote:
>> Hi,
>>
>> The patch enhances ifcvt to handle conditional compare instruction
>> (ccmp) to make it work with cmov. For ccmp, ALLOW_CC_MODE is set to
>> TRUE when calling canonicalize_conditio
> On Jun 23, 2014, at 12:37 AM, Zhenqiang Chen
> wrote:
>
>> On 23 June 2014 15:09, Andrew Pinski wrote:
>> On Mon, Jun 23, 2014 at 12:01 AM, Zhenqiang Chen
>> wrote:
>>> Hi,
>>>
>>> The patch enhances ifcvt to handle conditional compare instruction
>>> (ccmp) to make it work with cmov. For
On Mon, 23 Jun 2014, Jan Hubicka wrote:
> > > On Fri, 13 Jun 2014, Jan Hubicka wrote:
> > >
> > > > >
> > > > > When you extract the address and use it. For example when you
> > > > > do auto-parallelization and outline a part of your function it
> > > > > passes arrays as addresses.
> > > > >
Hi,
On Fri, 20 Jun 2014 13:44:18, Martin Jambor wrote:
>
> Hi,
>
> On Thu, Jun 19, 2014 at 06:18:47PM +0200, Bernd Edlinger wrote:
>> Hi,
>>
>> from a recent discussion on g...@gcc.gnu.org I have learned that the default
>> of
>> --param allow-store-data-races is still 1, and it is causing probl
On 19 June 2014 14:12, James Greenhalgh wrote:
> This has been sitting waiting for comment for a while now. If we do need a
> mechanism to describe individual costs for alternatives, it will need
> applied to all the existing uses in aarch64.md/aarch64-simd.md. I think
> solving that problem (if
On Fri, Jun 20, 2014 at 12:41 PM, Thomas Preud'homme
wrote:
>> From: Richard Biener [mailto:richard.guent...@gmail.com]
>> Sent: Tuesday, June 10, 2014 5:05 PM
>>
>> Backports are welcome - please post a patch.
>>
>
> Sorry for the delay. Here you are:
>
> diff --git a/gcc/testsuite/gcc.c-torture/
Hi,
On Mon, Jun 23, 2014 at 04:55:36AM +0200, Jan Hubicka wrote:
> > > On Fri, 13 Jun 2014, Jan Hubicka wrote:
> > >
> > > > >
> > > > > When you extract the address and use it. For example when you
> > > > > do auto-parallelization and outline a part of your function it
> > > > > passes arrays
On Fri, Jun 20, 2014 at 6:52 PM, Bernd Edlinger
wrote:
> Hi,
>
> I have noticed there is a minor flaw with the include path when cloog is
> installed in-tree.
>
> That is, the cloog-include directory is added twice, first with absolute
> path, and then
> again with relative path, but with one ".
On 20 June 2014 15:14, Kyrill Tkachov wrote:
> Sure, but it depends on
> https://gcc.gnu.org/ml/gcc-patches/2014-06/msg00779.html.
> Is it ok to backport that one as well?
This can be backported as well.
/Marcus
On Mon, Jun 23, 2014 at 10:18:16AM +0200, Richard Biener wrote:
> > --- a/gcc/tree-ssa-math-opts.c
> > +++ b/gcc/tree-ssa-math-opts.c
> > @@ -1741,6 +1741,8 @@ find_bswap_1 (gimple stmt, struct symbolic_number *n,
> > int limit)
> > if (n->size % BITS_PER_UNIT != 0)
> > retur
On Sun, Jun 22, 2014 at 9:14 AM, Bernd Edlinger
wrote:
> Hi,
>
> I noticed that several testcases in the GMP-4.3.2 test suite are failing now
> which
> did not happen with GCC 4.9.0. I debugged the first one, mpz/convert, and
> found
> the file mpn/generic/get_str.c was miscompiled.
>
> mpn/get
On Mon, Jun 23, 2014 at 7:32 AM, Chung-Lin Tang wrote:
> Hi Richard,
>
> In this change:
> https://gcc.gnu.org/ml/gcc-patches/2014-06/msg01278.html
>
> where substitute_and_fold() was changed to use a dom walker, the calls
> to purge dead EH edges during the walk can alter the dom-tree, and have
>
On Mon, Jun 23, 2014 at 10:02 AM, Bernd Edlinger
wrote:
> Hi,
>
>
> On Fri, 20 Jun 2014 13:44:18, Martin Jambor wrote:
>>
>> Hi,
>>
>> On Thu, Jun 19, 2014 at 06:18:47PM +0200, Bernd Edlinger wrote:
>>> Hi,
>>>
>>> from a recent discussion on g...@gcc.gnu.org I have learned that the
>>> default o
> From: Jakub Jelinek [mailto:ja...@redhat.com]
> Sent: Monday, June 23, 2014 4:37 PM
>
> On Mon, Jun 23, 2014 at 10:18:16AM +0200, Richard Biener wrote:
> > > --- a/gcc/tree-ssa-math-opts.c
> > > +++ b/gcc/tree-ssa-math-opts.c
> > > @@ -1741,6 +1741,8 @@ find_bswap_1 (gimple stmt, struct
> symbol
On Wed, Jun 18, 2014 at 3:07 PM, Prathamesh Kulkarni
wrote:
> Put a check for capture index.
>
> * genmatch.c (parse_capture): Add condition to check capture index.
> (capture_max): New constant.
> (stdlib.h): Include.
I'd rather record the maximum seen c
On Mon, Jun 23, 2014 at 04:50:49PM +0800, Thomas Preud'homme wrote:
> > Sent: Monday, June 23, 2014 4:37 PM
> >
> > On Mon, Jun 23, 2014 at 10:18:16AM +0200, Richard Biener wrote:
> > > > --- a/gcc/tree-ssa-math-opts.c
> > > > +++ b/gcc/tree-ssa-math-opts.c
> > > > @@ -1741,6 +1741,8 @@ find_bswap
Hi Richard,
Thanks for your comments. I am working on the review comments, and will share
the reworked patch soon.
However, here is clarification on some of the issues raised.
> > + if (TARGET_FIX_24K && TUNE_P5600)
> > +error ("unsupported combination: %s", "-mtune=p5600 -mfix-24k");
> > +
> From: Jakub Jelinek [mailto:ja...@redhat.com]
> Sent: Monday, June 23, 2014 4:59 PM
>
> Host could e.g. in theory have CHAR_BIT 32, while target BITS_PER_UNIT 8
> (otherwise bswap pass would give up). sizeof (unsigned HOST_WIDE_INT)
> could
> very well be 2 in that case.
In this case the pass
On 20/06/14 21:28, Richard Henderson wrote:
There aren't too many users of the cmpelim pass, and previously they were all
small embedded targets without an FPU.
I'm a bit surprised that Ramana decided to enable this pass for aarch64, as
that target is not so limited as the block comment for th
Please add a FIXME note in graphite_regenerate_ast_isl saying that
this is not yet a full implementation of the code generator with ISL
ASTs.
It would be useful to make the current graphite_regenerate_ast_isl
working by calling graphite_regenerate_ast_cloog and adding the fixme
note above saying th
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> ow...@gcc.gnu.org] On Behalf Of Thomas Preud'homme
>
> However
> although the original comments on struct symbolic_number implies that
> there is a mapping between host bytes (the bytes of the symbolic number)
> and target bytes, it isn'
Thanks Sebastian for the review! It is good to see you again on the
mailing list!
On 23/06/2014 11:29, Sebastian Pop wrote:
Please add a FIXME note in graphite_regenerate_ast_isl saying that
this is not yet a full implementation of the code generator with ISL
ASTs.
It would be useful to make th
Sameera Deshpande writes:
>> > + if (TARGET_FIX_24K && TUNE_P5600)
>> > +error ("unsupported combination: %s", "-mtune=p5600 -mfix-24k");
>> > +
>> >/* Save the base compression state and process flags as though we
>> > were generating uncompressed code. */
>> >mips_base_compre
Hi,
For below simplified case:
#define LEN (32000)
__attribute__((aligned(16))) float a[LEN],b[LEN];
int foo (int M)
{
for (int i = 0; i < M; i++)
a[i+M] = a[i] + b[i];
}
Compiling it with command like:
$ aarch64-elf-gcc -O3 -S foo.c -o foo.S -std=c99
The assembly code of vectorized loop
As for 4.8, I'm intending to backport the ZIP/UZP/TRN fix for ARM big-endian in
r211369 of mainline. That patches arm_neon.h, so again we need to remove the
OCAML code by which that file is autogenerated...ok?
--Alancommit e83cb5fff3687316ff391e9e7a8c65df2d35c880
Author: Alan Lawrence
Date:
Richard Sandiford writes:
> Sorry for the slow review.
And my slow response :-)
> Matthew Fortune writes:
> > The initial support for MIP64r6 is intentionally minimal to make
> review
> > easier. Performance enhancements and use of new MIPS64r6 features will
> > be introduced separately. The cu
Hi,
this fixes a potential out-of-bound access in unwind-seh's GetGr/SetGr function.
ChangeLog
2014-06-23 Kai Tietz
PR libgcc/61585
* unwind-seh.c (_Unwind_GetGR): Check for proper
index range.
(_Unwind_SetGR): Likewise.
I will apply this patch after successful build and te
On Sat, Jun 21, 2014 at 10:28:41AM +0200, Tobias Burnus wrote:
> Jakub Jelinek wrote:
> >Bootstrap/regtest pending, does this look ok?
>
> Except for the module/resolved issues discussed elsewhere, it look good to
> me.
So, either we need something like the following patch (incremental), or
anoth
On 23/06/14 09:26, Marcus Shawcroft wrote:
On 20 June 2014 15:14, Kyrill Tkachov wrote:
Sure, but it depends on
https://gcc.gnu.org/ml/gcc-patches/2014-06/msg00779.html.
Is it ok to backport that one as well?
This can be backported as well.
/Marcus
Thanks, I've backported to 4.9 the above
On 23/06/14 11:52, Kyrill Tkachov wrote:
On 23/06/14 09:26, Marcus Shawcroft wrote:
On 20 June 2014 15:14, Kyrill Tkachov wrote:
Sure, but it depends on
https://gcc.gnu.org/ml/gcc-patches/2014-06/msg00779.html.
Is it ok to backport that one as well?
This can be backported as well.
/Marcus
This fixes errors when using 0s or 1'000s to create durations.
Tested x86_64-linux, committed to trunk.
commit c57e645222ac9bf476ef5b6414773b5f4e2b86fc
Author: Jonathan Wakely
Date: Sun Jun 22 18:06:42 2014 +0100
* include/bits/parse_numbers.h (_Number_help): Fix divide-by-zero.
*
Hi all,
Here's my new patch for PR 61561
(https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61561).
Which fixes ICE appeared due to QI/HI pattern lack in arm.md for stack
pointer register.
Reg. tested on arm-v7.
--Marat
arm.diff
Description: Binary data
On Sun, Jun 22, 2014 at 10:33:57PM +0200, Gerald Pfeifer wrote:
> On Mon, 2 Jun 2014, Marek Polacek wrote:
> > * c-typeck.c (parser_build_binary_op): Warn when logical not is used
> > on the left hand side operand of a comparison.
>
> This...
>
> > +/* Warn about logical not used on the
On 22/06/14 22:40 +0200, Gerald Pfeifer wrote:
On Mon, 9 Jun 2014, Jonathan Wakely wrote:
This fairly tedious patch refreshes the FAQ, including adding some
notes saying "This answer is old and probably no longer relevant" to
several answers referring to problems in the GCC 3.x era.
I'm wonder
On 2 May 2014 10:04, Kugan wrote:
> Thanks for spotting it. Here is the updated patch that changes it to
> ARM_FE_*.
> +2014-05-02 Kugan Vivekanandarajah
> +
> + * config/arm/arm.c (TARGET_ATOMIC_ASSIGN_EXPAND_FENV): New define.
> + (arm_builtins) : Add ARM_BUILTIN_GET_FPSCR and AR
On Thu, Jun 19, 2014 at 12:49:55PM +0100, Martin Jambor wrote:
> Hi,
>
> On Wed, Jun 18, 2014 at 06:12:34PM +0200, Bernhard Reutner-Fischer wrote:
> > On 18 June 2014 10:24:16 Martin Jambor wrote:
> >
> > >@@ -3002,10 +3014,8 @@ try_make_edge_direct_virtual_call (struct
> > >cgraph_edge *ie,
> >
Thank you for the review!
--
Cheers, Roman Gareev
ChangeLog_entry1
Description: Binary data
ChangeLog_entry2
Description: Binary data
patch1
Description: Binary data
patch2
Description: Binary data
Hi James,
On 19/06/14 14:12, James Greenhalgh wrote:
diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md
>index
266d7873a5a1b8dbb7f955c3f13cf370920a9c4a..7c5b5a566ebfd907b83b38eed2e214738e7e9bd4
100644
>--- a/gcc/config/aarch64/aarch64.md
>+++ b/gcc/config/aarch64/aarch
This removes the non-DT matching code.
Committed.
Richard.
2014-06-23 Richard Biener
* genmatch.c (operand::gen_gimple_match): Remove.
(predicate::gen_gimple_match): Likewise.
(expr::gen_gimple_match): Likewise.
(c_expr::gen_gimple_match): Likewise.
(
* match.pd: Mark operators in some bitwise and plus-minus
patterns to be commutative.
Thanks and Regards,
Prathamesh
Index: gcc/match.pd
===
--- gcc/match.pd (revision 211893)
+++ gcc/match.pd (working copy)
@@ -138,7 +138,7 @@ along
Hi Martin,
>>
>> Well actually, I am not sure if we ever wanted to have a race condition here.
>> Have you seen any impact of --param allow-store-data-races on any benchmark?
>
> It's trivially to write one. The only pass that checks the param is
> tree loop invariant motion and it does that when
> It seems the patch1/patch2 files you attach have the Content-Type:
> application/octet-stream. This makes it impossible to view them inline.
> Could you send them as text files? Just calling them patch1.patch or
> patch1.txt should make this work.
Yes, sure.
diff --git a/gcc/graphite-clast-to-gi
* genmatch.c (gen_gimple_match_fail): Remove.
(expr::gen_gimple_transform): Remove call to gen_gimple_match_fail.
Change fprintf (f, "if (!res)") to fprintf (f, "if (!res) return false;\n")
Thanks and Regards,
Prathamesh
Index: gcc/genmatch.c
=
On 06/23/2014 02:29 AM, Ramana Radhakrishnan wrote:
>
>
> On 20/06/14 21:28, Richard Henderson wrote:
>> There aren't too many users of the cmpelim pass, and previously they were all
>> small embedded targets without an FPU.
>>
>> I'm a bit surprised that Ramana decided to enable this pass for aa
On Mon, Jun 23, 2014 at 3:32 PM, Prathamesh Kulkarni
wrote:
> * match.pd: Mark operators in some bitwise and plus-minus
> patterns to be commutative.
/* A - (A +- B) -> -+ B */
(match_and_simplify
- (minus @0 (plus @0 @1))
+ (minus @0 (plus:c @0 @1))
(negate @0))
seems pointless
/* ~x &
On 06/20/2014 02:59 PM, Kai Tietz wrote:
> So I suggest following change of passes.def:
>
> Index: passes.def
> ===
> --- passes.def (Revision 211850)
> +++ passes.def (Arbeitskopie)
> @@ -384,7 +384,6 @@ along with GCC; see the fil
On Mon, 23 Jun 2014, Richard Biener wrote:
On Mon, Jun 23, 2014 at 3:32 PM, Prathamesh Kulkarni
wrote:
* match.pd: Mark operators in some bitwise and plus-minus
patterns to be commutative.
/* A - (A +- B) -> -+ B */
(match_and_simplify
- (minus @0 (plus @0 @1))
+ (minus @0 (plus:c @0 @1))
On 06/20/2014 01:42 PM, Marek Polacek wrote:
> 2014-06-20 Marek Polacek
>
> * genpreds.c (verify_rtx_codes): New function.
> (main): Call it.
> * rtl.h (RTX_FLD_WIDTH, RTX_HWINT_WIDTH): Define.
> (struct rtx_def): Use them.
Looks pretty good. Just a few nits.
> +stati
> I noticed that several testcases in the GMP-4.3.2 test suite are failing now
> which did not happen with GCC 4.9.0. I debugged the first one,
> mpz/convert, and found the file mpn/generic/get_str.c was miscompiled.
>
> mpn/get_str.c.132t.dse2:
> pretmp_183 = (sizetype) chars_per_limb_80;
>
Hi,
On Mon, 23 Jun 2014 10:40:53, Richard Biener wrote:
>
> On Sun, Jun 22, 2014 at 9:14 AM, Bernd Edlinger
> wrote:
>> Hi,
>>
>> I noticed that several testcases in the GMP-4.3.2 test suite are failing now
>> which
>> did not happen with GCC 4.9.0. I debugged the first one, mpz/convert, and
>>
Hi all,
This patch implements some absolute compare intrinsics in arm_neon.h.
Execution tests are added.
Tested aarch64-none-elf, aarch64_be-none-elf, bootstrapped on aarch64 linux
Ok for trunk?
2014-06-23 Kyrylo Tkachov
* config/aarch64/arm_neon.h (vcage_f64): New intrinsic.
(vcag
On Mon, Jun 23, 2014 at 4:13 PM, Richard Henderson wrote:
> On 06/20/2014 02:59 PM, Kai Tietz wrote:
>> So I suggest following change of passes.def:
>>
>> Index: passes.def
>> ===
>> --- passes.def (Revision 211850)
>> +++ passes.def
On Mon, Jun 23, 2014 at 3:43 PM, Prathamesh Kulkarni
wrote:
> * genmatch.c (gen_gimple_match_fail): Remove.
> (expr::gen_gimple_transform): Remove call to gen_gimple_match_fail.
> Change fprintf (f, "if (!res)") to fprintf (f, "if (!res) return
> false;\n")
Thanks, committed.
Richard.
>
On Mon, Jun 23, 2014 at 4:23 PM, Marc Glisse wrote:
> On Mon, 23 Jun 2014, Richard Biener wrote:
>
>> On Mon, Jun 23, 2014 at 3:32 PM, Prathamesh Kulkarni
>> wrote:
>>>
>>> * match.pd: Mark operators in some bitwise and plus-minus
>>> patterns to be commutative.
>>
>>
>> /* A - (A +- B) -> -+ B *
We ICEd on the following testcase since the void type has a NULL
TYPE_SIZE_UNIT. I took Andrew's patch from gcc@ ML and added
a testcase.
Regtested/bootstrapped on x86_64-linux, ok for trunk?
2014-06-23 Marek Polacek
Andrew MacLeod
PR c/61553
* c-common.c (get_a
On Mon, Jun 23, 2014 at 4:25 PM, Richard Henderson wrote:
> On 06/20/2014 01:42 PM, Marek Polacek wrote:
>> 2014-06-20 Marek Polacek
>>
>> * genpreds.c (verify_rtx_codes): New function.
>> (main): Call it.
>> * rtl.h (RTX_FLD_WIDTH, RTX_HWINT_WIDTH): Define.
>> (struct r
On Mon, 23 Jun 2014, Marek Polacek wrote:
> I think the latter is better, incidentally, g++ doesn't warn either.
> The following one liner makes cc1 behave as cc1plus. Thanks for the
> report.
>
> Regtested/bootstrapped on x86_64. Joseph, is this ok?
>
> 2014-06-23 Marek Polacek
>
>
On 11 June 2014 00:03, Ramana Radhakrishnan wrote:
> On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon
> wrote:
>> This is patch series is a more complete version of the patch I sent
>> some time ago:
>> https://gcc.gnu.org/ml/gcc-patches/2013-10/msg00624.html
>>
>> I have created a series of patc
We see the following on a 32bit gcc installed on 64 bit host:
Reading symbols from ./i586-pokymllib32-linux-gcc...done.
(gdb) run
Starting program:
x86-pokymllib32-linux/lib32-gcc/4.9.0-r0/image/usr/bin/i586-pokymllib32-linux-gcc
Program received signal SIGSEGV, Segmentation fault.
0xf
Looks like there is a broken link on
http://www.mail-archive.com/ftikumt2001@yahoogroups.com/msg00234.html to
http://www.cs.cmu.edu/books.html. You might also want to add some of these
resources I found useful...
http://online.sju.edu/resource/engineering-technology/trends-in-instructional-desi
On 06/22/2014 10:42 AM, Paolo Carlini wrote:
I think the below would be most of it. Today, however, I did some
archeology, noticed that we would essentially revert to pre-PR9278
behavior (thus, per its audit trail, make again unhappy some people in
the template metaprogramming world?!? Was that k
This fixes a bootstrap compare failure on current mainline and 4.9
branch configured with --disable-checking, caused by losing value
range info when outputting debug info. Lack of value range info leads
to loop bounds not being calculated, which in turn means a "j < n"
test is not converted to "j
Hi,
Here is a patch that restructures neon builtins to use vector types based on
standard base types. We previously defined arm_neon.h's neon vector
types(int8x8_t) using gcc's front-end vector extensions. We now move away from
that and use types built internally(e.g. __Int8x8_t). These inter
On Mon, 23 Jun 2014, Jakub Jelinek wrote:
Ok for trunk, sorry for the delay.
Thanks. Richard has moved the passes a bit since then, but I still have
exactly one spot where the testsuite is ok :-) I need strlen to be after
dom (for calloc.C) and before vrp (for several strlenopt-*.c). I'll co
On 23/06/14 15:01, Richard Henderson wrote:
On 06/23/2014 02:29 AM, Ramana Radhakrishnan wrote:
On 20/06/14 21:28, Richard Henderson wrote:
There aren't too many users of the cmpelim pass, and previously they were all
small embedded targets without an FPU.
I'm a bit surprised that Ramana d
On Mon, Jun 23, 2014 at 01:53:28PM +0100, Kyrill Tkachov wrote:
> On 19/06/14 14:12, James Greenhalgh wrote:
> > diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md
> > >index
> > >266d7873a5a1b8dbb7f955c3f13cf370920a9c4a..7c5b5a566ebfd907b83b38eed2e214738e7e9bd4
> > > 10064
On June 23, 2014 5:51:30 PM CEST, Marc Glisse wrote:
>On Mon, 23 Jun 2014, Jakub Jelinek wrote:
>
>> Ok for trunk, sorry for the delay.
>
>Thanks. Richard has moved the passes a bit since then, but I still have
>
>exactly one spot where the testsuite is ok :-) I need strlen to be
>after
>dom (for
> I don't like this very much. It's fragile and it will be very hard to
> detect bugs caused by it.
>
> Please don't spread uses of the DECL_NONALIASED "hack".
>
> If we are only concerned about LTO I'd rather have a in_lto_p check
> in may_be_aliased and trust TREE_ADDRESSABLE there.
I do not
On June 23, 2014 6:15:10 PM CEST, Jan Hubicka wrote:
>> I don't like this very much. It's fragile and it will be very hard
>to
>> detect bugs caused by it.
>>
>> Please don't spread uses of the DECL_NONALIASED "hack".
>>
>> If we are only concerned about LTO I'd rather have a in_lto_p check
>>
On Mon, 23 Jun 2014, Richard Biener wrote:
On June 23, 2014 5:51:30 PM CEST, Marc Glisse wrote:
On Mon, 23 Jun 2014, Jakub Jelinek wrote:
Ok for trunk, sorry for the delay.
Thanks. Richard has moved the passes a bit since then, but I still have
exactly one spot where the testsuite is ok :
Hi,
pr61510 is a case where cgraphunit.c::analyze_functions can end up
dereferencing a NULL pointer. This is, to me, the obvious way to avoid
dereferencing NULL.
However, I'm not very confident that this isn't just masking some
horrible and subtle bug, so I'd like some review feedback on the pat
> On June 23, 2014 6:15:10 PM CEST, Jan Hubicka wrote:
> >> I don't like this very much. It's fragile and it will be very hard
> >to
> >> detect bugs caused by it.
> >>
> >> Please don't spread uses of the DECL_NONALIASED "hack".
> >>
> >> If we are only concerned about LTO I'd rather have a in
On 06/23/14 08:32, Richard Biener wrote:
On Mon, Jun 23, 2014 at 4:13 PM, Richard Henderson wrote:
On 06/20/2014 02:59 PM, Kai Tietz wrote:
So I suggest following change of passes.def:
Index: passes.def
===
--- passes.def (Revis
On 06/22/14 01:14, Bernd Edlinger wrote:
Hi,
I noticed that several testcases in the GMP-4.3.2 test suite are failing now
which
did not happen with GCC 4.9.0. I debugged the first one, mpz/convert, and found
the file mpn/generic/get_str.c was miscompiled.
It's interesting you stumbled on this.
On Sun, Jun 22, 2014 at 11:23 AM, Uros Bizjak wrote:
> On Sat, Jun 21, 2014 at 8:07 PM, Jakub Jelinek wrote:
>
>>> > --- gcc/config/i386/driver-i386.c.jj2014-05-14 14:45:54.0
>>> > +0200
>>> > +++ gcc/config/i386/driver-i386.c 2014-06-20 18:59:57.805006358
>>> > +0200
>>> > @@
> "Mark" == Mark Wielaard writes:
Mark> The following is just a prototype to try out a new qualifier type tag
Mark> proposed for DWARFv5. There is not even a draft yet of DWARFv5, so this
Mark> is just based on a proposal that might or might not be adopted and/or
Mark> changed http://dwarfstd
On Mon, 23 Jun 2014, James Greenhalgh wrote:
>
> Hi,
>
> pr61510 is a case where cgraphunit.c::analyze_functions can end up
> dereferencing a NULL pointer. This is, to me, the obvious way to avoid
> dereferencing NULL.
>
> However, I'm not very confident that this isn't just masking some
> horr
On Mon, Jun 23, 2014 at 4:38 PM, Richard Biener
wrote:
> On Mon, Jun 23, 2014 at 4:23 PM, Marc Glisse wrote:
>> On Mon, 23 Jun 2014, Richard Biener wrote:
>>
>>> On Mon, Jun 23, 2014 at 3:32 PM, Prathamesh Kulkarni
>>> wrote:
* match.pd: Mark operators in some bitwise and plus-minus
>>
On 06/23/2014 09:22 AM, Jeff Law wrote:
> On 06/23/14 08:32, Richard Biener wrote:
>> Btw, there is now no DCE after peephole2? Is peephole2 expected to
>> cleanup after itself?
> There were cases where we wanted to change the insns we would output to fit
> into the 4:1:1 issue model of the PPro,
On Mon, Jun 23, 2014 at 6:19 PM, Marc Glisse wrote:
> On Mon, 23 Jun 2014, Richard Biener wrote:
>
>> On June 23, 2014 5:51:30 PM CEST, Marc Glisse
>> wrote:
>>>
>>> On Mon, 23 Jun 2014, Jakub Jelinek wrote:
>>>
Ok for trunk, sorry for the delay.
>>>
>>>
>>> Thanks. Richard has moved the pas
The tests gcc.dg/globalalias-2.c and gcc.dg/localalias-2.c fail on darwin with
/opt/gcc/work/gcc/testsuite/gcc.dg/globalalias-2.c:20:2: warning: alias
definitions not supported in Mach-O; ignored
I think they should be protected by
/* { dg-require-alias "" } */
Dominique
On Wed, 18 Jun 2014, Jeff Law wrote:
> On 06/18/14 04:42, Richard Biener wrote:
> >
> > The following aims at reducing the number of pointless passes we run
> > on functions containing no loops. Those are at least two copyprop
> > and one dce pass (two dce passes when vectorization is enabled,
>
On Mon, 23 Jun 2014, Marek Polacek wrote:
> We ICEd on the following testcase since the void type has a NULL
> TYPE_SIZE_UNIT. I took Andrew's patch from gcc@ ML and added
> a testcase.
>
> Regtested/bootstrapped on x86_64-linux, ok for trunk?
>
> 2014-06-23 Marek Polacek
> Andrew
On Fri, Jun 20, 2014 at 12:52 PM, wrote:
> From: Trevor Saunders
>
> Hi,
>
> This patch adds a hash_map class so we can consolidate the boiler plate around
> using hash_table as a map, it also allows us to get rid of pointer_map which I
> do in this patch by converting its users to hash_map.
>
>
Hi,
when I applied this patch (r211846), I made a little mistake in output
test patterns. This patch fixes this.
Tested on x86_64-unknown-linux-gnu.
Ok to commit?
-Maxim
gcc/testsuite/ChangeLog:
2014-06-23 Max Ostapenko
* c-c++-common/asan/strlen-overflow-1.c: Change match patterns.
d
On Fri, Jun 20, 2014 at 12:52 PM, wrote:
> From: Trevor Saunders
>
> Hi,
>
> this patch allows you to define the type the hash table stores as elements
> instead of the type elements point at by having your hash descriptor define
> the
> type store_values_directly. It turns out trying to imple
The issue fixed by the attached test case was that an automatic array
wasn't properly initialized with -fcoarray=lib. As the code is for
explicit-size arrays, it isn't reachable for allocatable coarrays. On
the other hand, for coarrays with SAVE attribute, already the condition
before should be
On Mon, 2014-06-23 at 10:31 -0600, Tom Tromey wrote:
> > "Mark" == Mark Wielaard writes:
>
> Mark> The following is just a prototype to try out a new qualifier type tag
> Mark> proposed for DWARFv5. There is not even a draft yet of DWARFv5, so this
> Mark> is just based on a proposal that mig
On Mon, Jun 23, 2014 at 4:28 PM, Bernd Edlinger
wrote:
> Hi,
>
> On Mon, 23 Jun 2014 10:40:53, Richard Biener wrote:
>>
>> On Sun, Jun 22, 2014 at 9:14 AM, Bernd Edlinger
>> wrote:
>>> Hi,
>>>
>>> I noticed that several testcases in the GMP-4.3.2 test suite are failing
>>> now which
>>> did not
Hi,
On Mon, 23 Jun 2014 10:27:59, Jeff Law wrote:
>
> On 06/22/14 01:14, Bernd Edlinger wrote:
>> Hi,
>>
>> I noticed that several testcases in the GMP-4.3.2 test suite are failing now
>> which
>> did not happen with GCC 4.9.0. I debugged the first one, mpz/convert, and
>> found
>> the file mpn/
1 - 100 of 144 matches
Mail list logo
