RE: [PATCH][ARM] AArch32 vmaxnm, vminnm support

2012-12-04 Thread Kyrylo Tkachov
Ping. Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 26 November 2012 14:40 To: gcc-patches@gcc.gnu.org Cc: Ramana Radhakrishnan; Richard Earnshaw Subject: [PATCH][ARM] AArch32 vmaxnm

[PATCH][ARM][testsuite] Add testsuite options for AArch32 NEON

2012-12-11 Thread Kyrylo Tkachov
-a is not the default target. No regressions otherwise on arm-none-eabi. Ok for trunk? Thanks, Kyrill gcc/testsuite/ChangeLog 2012-12-11 Kyrylo Tkachov kyrylo.tkac...@arm.com * lib/target-supports.exp (add_options_for_arm_v8_neon): Add -march=armv8-a.diff --git a/gcc/testsuite/lib

[PATCH][ARM] ce_count attribute for thumb2 abssi2 patterns

2012-12-11 Thread Kyrylo Tkachov
affected patterns each produce two conditional instructions. A new testcase is added. No regressions on arm-none-eabi with qemu. Ok for trunk? Thanks, Kyrill gcc/ChangeLog 2012-12-11 Kyrylo Tkachov kyrylo.tkac...@arm.com PR target/55642 * config/arm/thumb2.md (*thumb2_abssi2

[PATCH][ARM] Add copyright notice to unspecs.md

2012-12-13 Thread Kyrylo Tkachov
Hi all, This patch adds a copyright notice to the recently added unspecs.md file in the arm backend (r193204). The file just gathered some unspecs definitions into one place. Ok for trunk? Thanks, Kyrill gcc/ChangeLog 2012-12-13 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm

RE: [PATCH][ARM][testsuite] Add testsuite options for AArch32 NEON

2012-12-17 Thread Kyrylo Tkachov
-Original Message- From: Richard Earnshaw Sent: 17 December 2012 14:13 To: Kyrylo Tkachov Cc: gcc-patches@gcc.gnu.org; Ramana Radhakrishnan Subject: Re: [PATCH][ARM][testsuite] Add testsuite options for AArch32 NEON On 11/12/12 12:30, Kyrylo Tkachov wrote: Hi all, Since

[PATCH][ARM][0/3] Add vectorization support for rounding functions

2012-12-18 Thread Kyrylo Tkachov
Hi all, The three following patches add support for the vectorisation of the rounding functions: floorf, ceilf, truncf, roundf. The first patch does some refactoring of the NEON builtins, allowing us to give them symbolic names and thus paving the way for the definition of

[PATCH][ARM][1/3] Add vectorization support for rounding functions

2012-12-18 Thread Kyrylo Tkachov
, since various functions that deal with the initialisation and expansion of builtins use it. No regressions on arm-none-eabi with model. Ok for trunk? Thanks, Kyrill gcc/ChangeLog 2012-12-18 Kyrylo Tkachov kyrylo.tkachov at arm.com * config/arm/arm_neon_builtins.def: New file

[PATCH][ARM][2/3] Add vectorization support for rounding functions

2012-12-18 Thread Kyrylo Tkachov
and TARGET_VECTORIZE_BUILTIN_VECTORIZED_FUNCTION macros and the function arm_builtin_vectorized_function that returns the decl of the vector form of a builtin function, or NULL_TREE if no vector variant exists. No regressions on arm-none-eabi with AEM. Ok for trunk? gcc/ChangeLog 2012-12-18 Kyrylo Tkachov kyrylo.tkachov at arm.com

[PATCH][ARM][3/3] Add vectorization support for rounding functions

2012-12-18 Thread Kyrylo Tkachov
/testuite/ChangeLog 2012-12-18 Kyrylo Tkachov kyrylo.tkachov at arm.com * lib/target-supports.exp (check_effective_target_arm_v8_neon_hw): New procedure. (check_effective_target_vect_call_btruncf): Add check for arm and ARMv8 NEON

[PATCH][ARM] Add support for Cortex-A7 to GCC -mcpu=native option.

2012-12-18 Thread Kyrylo Tkachov
Hi all, This one-line patch adds support for Cortex A-7 to the -mcpu=native option. Ok for trunk? No regressions on arm-none-eabi. Thanks, Kyrill gcc/ChangeLog 2012-12-18 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/driver-arm.c (arm_cpu_table): Add Cortex-A7.diff

RE: [PATCH][ARM] AArch32 vmaxnm, vminnm support

2012-12-19 Thread Kyrylo Tkachov
Ping. http://gcc.gnu.org/ml/gcc-patches/2012-11/msg02097.html Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 26 November 2012 14:40 To: gcc-patches@gcc.gnu.org Cc: Ramana Radhakrishnan

RE: [PATCH][ARM][1/3] Add vectorization support for rounding functions

2013-01-07 Thread Kyrylo Tkachov
Ping. http://gcc.gnu.org/ml/gcc-patches/2012-12/msg0.html Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 18 December 2012 13:33 To: gcc-patches@gcc.gnu.org Cc: Ramana Radhakrishnan

RE: [PATCH][ARM][2/3] Add vectorization support for rounding functions

2013-01-07 Thread Kyrylo Tkachov
Ping. http://gcc.gnu.org/ml/gcc-patches/2012-12/msg01113.html Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 18 December 2012 13:34 To: gcc-patches@gcc.gnu.org Cc: Ramana Radhakrishnan

RE: [PATCH][ARM][1/3] Add vectorization support for rounding functions

2013-01-14 Thread Kyrylo Tkachov
Ping. Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 07 January 2013 10:35 To: gcc-patches@gcc.gnu.org Cc: Ramana Radhakrishnan; Richard Earnshaw Subject: RE: [PATCH][ARM][1/3] Add

RE: [PATCH][ARM][2/3] Add vectorization support for rounding functions

2013-01-14 Thread Kyrylo Tkachov
Ping. Thanks, Kyrill -Original Message- From: Kyrylo Tkachov Sent: 07 January 2013 10:35 To: Kyrylo Tkachov; gcc-patches@gcc.gnu.org Cc: Ramana Radhakrishnan; Richard Earnshaw Subject: RE: [PATCH][ARM][2/3] Add vectorization support for rounding functions Ping. http

RE: [PATCH][ARM][1/3] Add vectorization support for rounding functions

2013-01-22 Thread Kyrylo Tkachov
Ping^3? Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 14 January 2013 17:32 To: gcc-patches@gcc.gnu.org Cc: Ramana Radhakrishnan; Richard Earnshaw Subject: RE: [PATCH][ARM][1/3] Add

RE: [PATCH][ARM][2/3] Add vectorization support for rounding functions

2013-01-22 Thread Kyrylo Tkachov
Ping^3? Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 14 January 2013 17:32 To: gcc-patches@gcc.gnu.org Cc: Ramana Radhakrishnan; Richard Earnshaw Subject: RE: [PATCH][ARM][2/3] Add

[PATCH][ARM] use vsel instruction for floating point conditional moves in ARMv8

2013-01-30 Thread Kyrylo Tkachov
in the set {GE, GT, EQ, VS, LT, LE, NE, VC}. New compilation tests are added. They pass on a model and no new regressions on arm-none-eabi with qemu. Ok for trunk? Thanks, Kyrill gcc/ChangeLog 2013-01-30 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm.md (f_sels, f_seld): New types

[PATCH][ARM][1/2] Load-acquire, store-release atomics in AArch32 ARMv8

2013-01-30 Thread Kyrylo Tkachov
will come in a separate patch soon. No regressions on arm-none-eabi. Bootstrap on armv7l-unknown-linux-gnueabihf successful. Ok for trunk (now or when stage 1 reopens)? Thanks, Kyrill gcc/ChangeLog 2013-01-23 Kyrylo Tkachov kyrylo.tkachov at arm.com * config/arm/arm.c

[PATCH][ARM][2/2] Load-acquire, store-release atomics in AArch32 ARMv8

2013-02-01 Thread Kyrylo Tkachov
2013-01-25 Kyrylo Tkachov kyrylo.tkachov at arm.com * gcc.target/aarch64/atomic-comp-swap-release-acquire.c: Move test body from here... * gcc.target/aarch64/atomic-comp-swap-release-acquire.x: ... to here. * gcc.target/aarch64/atomic-op-acq_rel.c: Move test body

RE: [PATCH][ARM][2/2] Load-acquire, store-release atomics in AArch32 ARMv8

2013-02-01 Thread Kyrylo Tkachov
Ummm... forgot the patch, sorry! -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 01 February 2013 17:37 To: gcc-patches@gcc.gnu.org Cc: Ramana Radhakrishnan; Richard Earnshaw; Marcus Shawcroft Subject

RE: [PATCH][ARM][2/3] Add vectorization support for rounding functions

2013-02-05 Thread Kyrylo Tkachov
Ping? Thanks, Kyrill -Original Message- From: Kyrylo Tkachov [mailto:kyrylo.tkac...@arm.com] Sent: 07 January 2013 10:35 To: Kyrylo Tkachov; gcc-patches@gcc.gnu.org Cc: Ramana Radhakrishnan; Richard Earnshaw Subject: RE: [PATCH][ARM][2/3] Add vectorization support for rounding

RE: [PATCH][ARM][1/3] Add vectorization support for rounding functions

2013-02-05 Thread Kyrylo Tkachov
Ping? Thanks, Kyrill -Original Message- From: Kyrylo Tkachov [mailto:kyrylo.tkac...@arm.com] Sent: 07 January 2013 10:35 To: gcc-patches@gcc.gnu.org Cc: Ramana Radhakrishnan; Richard Earnshaw Subject: RE: [PATCH][ARM][1/3] Add vectorization support for rounding functions Ping

RE: [PATCH][ARM][3/3] Add vectorization support for rounding functions

2013-02-07 Thread Kyrylo Tkachov
/ChangeLog 2013-02-07 Kyrylo Tkachov kyrylo.tkac...@arm.com * lib/target-supports.exp (check_effective_target_arm_v8_neon_hw): New procedure. (check_effective_target_arm_v8_neon_ok_nocache): Likewise. (check_effective_target_arm_v8_neon_ok): Change to use

RE: [PATCH][ARM] use vsel instruction for floating point conditional moves in ARMv8

2013-02-15 Thread Kyrylo Tkachov
Ping http://gcc.gnu.org/ml/gcc-patches/2013-01/msg01418.html Probably for 4.9 now... Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 30 January 2013 09:25 To: gcc-patches@gcc.gnu.org Cc

[PATCH, TESTSUITE] Add -fno-short-enums to pr51712

2012-09-11 Thread Kyrylo Tkachov
/testsuite 2012-09-11 Kyrylo Tkachov kyrylo.tkac...@arm.com * c-c++-common/pr51712.c: Add -fno-short-enums flag to test.--- a/gcc/testsuite/c-c++-common/pr51712.c +++ b/gcc/testsuite/c-c++-common/pr51712.c @@ -1,6 +1,6 @@ /* PR c/51712 */ /* { dg-do compile } */ -/* { dg-options -Wtype-limits

RE: [PATCH, TESTSUITE] Add -fno-short-enums to pr51712

2012-09-11 Thread Kyrylo Tkachov
now passes. Comment? Ok for trunk? Thanks, Kyrill gcc/testsuite 2012-09-11 Kyrylo Tkachov kyrylo.tkac...@arm.com * c-c++-common/pr51712.c: Add -fno-short-enums flag to test. -Original Message- From: Jakub Jelinek [mailto:ja...@redhat.com] Sent: 11 September 2012 13:50

RE: [PATCH, TESTSUITE] Add -fno-short-enums to pr51712

2012-09-13 Thread Kyrylo Tkachov
Can I get this commited please? Thanks, K. -Original Message- From: Mike Stump [mailto:mikest...@comcast.net] Sent: 11 September 2012 18:41 To: Kyrylo Tkachov Cc: 'Jakub Jelinek'; gcc-patches@gcc.gnu.org Subject: Re: [PATCH, TESTSUITE] Add -fno-short-enums to pr51712 On Sep 11, 2012

[PATCH][Resend][tree-optimization] Fix PR58088

2013-09-09 Thread Kyrylo Tkachov
and bootstraps cleanly on both. Bootstrapped on x86_64-linux-gnu and tested arm-none-eabi on qemu. Ok for trunk and 4.8? Thanks, Kyrill 2013-09-09 Kyrylo Tkachov kyrylo.tkac...@arm.com PR tree-optimization/58088 * fold-const.c (mask_with_trailing_zeros): New function

RE: [PATCH][AArch64] Fix FAIL: gcc.target/aarch64/cmn.c scan-assembler cmn\tw[0-9]

2013-09-09 Thread Kyrylo Tkachov
Hi Richard, On 29/07/13 14:58, Kyrylo Tkachov wrote: Hi all, Now that combine emits the canonical form for (eq (neg x) (y)) instead of (eq (x) (neg y)), this patch fixes up the corresponding pattern in aarch64 to match that. This enables combine to properly generate the cmn

[PATCH][ARM] Improve cond_exec opportunities for immediate shifts for -mrestrict-it

2013-09-09 Thread Kyrylo Tkachov
Tkachov kyrylo.tkac...@arm.com * config/arm/arm.md (arm_shiftsi3): New alternative l/l/N. * config/arm/constraints.md (N): Remove !TARGET_32BIT condition. 2013-09-09 Kyrylo Tkachov kyrylo.tkac...@arm.com * gcc.target/arm/thumb-ifcvt-2.c: New test. cond

[PATCH][ARM][1/2] ARMv8 aarch32 round to integral instructions

2012-11-14 Thread Kyrylo Tkachov
tested using AEM for the new functionality. Also, no regressions when testing for ARMv7-a (for which these changes should not have any effect) on qemu. Ok for trunk? Thanks, Kyrill gcc/ChangeLog 2012-11-14 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm.h (TARGET_FPU_ARMV8): New

[PATCH][ARM][2/2] ARMv8 aarch32 round to integral instructions

2012-11-14 Thread Kyrylo Tkachov
-11-14 Kyrylo Tkachov kyrylo.tkac...@arm.com * lib/target-supports.exp (check_effective_target_arm_v8_vfp_ok): New procedure. (add_options_for_arm_v8_vfp): New procedure. * gcc.target/arm/vrintaf32.c: New test. * gcc.target/arm/vrintaf64.c: Likewise

RE: [PATCH][ARM][1/2] ARMv8 aarch32 round to integral instructions

2012-11-14 Thread Kyrylo Tkachov
Sorry, the ChangeLog entry for vfp.md in the end should say: (vrint_patternSDF:mode2): New pattern. Instead of (VRINT:patternSDF:mode2): New pattern. -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 14

[PATCH][ARM] Define predicable attribute for arm_abssi2 and arm_neg_abssi2

2012-11-19 Thread Kyrylo Tkachov
predication is defined as predicable. No regressions on arm-none-eabi with qemu. Ok for trunk? Thanks, Kyrill gcc/ChangeLog 2012-11-19 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm.md (*arm_abssi2): Define predicable attribute. (*arm_neg_abssi2): Define predicable

RE: [PATCH][ARM] Define predicable attribute for arm_abssi2 and arm_neg_abssi2

2012-11-20 Thread Kyrylo Tkachov
://gcc.gnu.org/ml/gcc-patches/2012-11/msg01578.html No regressions for arm-none-eabi for qemu. Ok for trunk? Thanks, Kyrill gcc/ChangeLog 2012-11-20 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm.md (*arm_abssi2): Define predicable attribute. (*arm_neg_abssi2): Likewise

RE: [PATCH][ARM][1/2] ARMv8 aarch32 round to integral instructions

2012-11-21 Thread Kyrylo Tkachov
Ping? Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 14 November 2012 13:52 To: gcc-patches@gcc.gnu.org Cc: Richard Earnshaw; Ramana Radhakrishnan Subject: [PATCH][ARM][1/2] ARMv8 aarch32

RE: [PATCH][ARM][2/2] ARMv8 aarch32 round to integral instructions

2012-11-21 Thread Kyrylo Tkachov
Ping? Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 14 November 2012 13:52 To: gcc-patches@gcc.gnu.org Cc: mikest...@comcast.net; Ramana Radhakrishnan; Richard Earnshaw Subject: [PATCH][ARM

RE: [PATCH][ARM] Define predicable attribute for arm_abssi2 and arm_neg_abssi2

2012-11-22 Thread Kyrylo Tkachov
Thanks for the review. As I do not have write access, can someone please apply? http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01655.html Thanks, Kyrill -Original Message- From: Richard Earnshaw Sent: 20 November 2012 10:32 To: Kyrylo Tkachov Cc: gcc-patches@gcc.gnu.org; Ramana

[PATCH][ARM] AArch32 vmaxnm, vminnm support

2012-11-26 Thread Kyrylo Tkachov
: f_minmaxs and f_minmaxd. New compilation tests are added. They pass and no regressions on arm-none-eabi. Ok for trunk? Thanks, Kyrill gcc/ChangeLog 2012-11-26 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm.md (f_minmaxs, f_minmaxd): New types. * config/arm/vfp.md

[0/3][ARM] AArch32 NEON vrint builtins and intrinsics

2012-11-29 Thread Kyrylo Tkachov
Hi all, This set of patches adds support for the AArch32 vrint NEON builtins. It also adds related NEON intrinsics. It is organised in 3 patches: * The first patch adds support for the builtin versions of the instructions. * The second is a small testsuite patch that adds the effective target

[PATCH][ARM][2/3] AArch32 NEON vrint builtins and intrinsics

2012-11-29 Thread Kyrylo Tkachov
Kyrylo Tkachov kyrylo.tkac...@arm.com * doc/sourcebuild.texi: Document arm_v8_neon_ok. gcc/testsuite/ChangeLog 2012-11-29 Kyrylo Tkachov kyrylo.tkac...@arm.com * lib/target-supports.exp (check_effective_target_arm_v8_neon_ok): New procedure

[PATCH][ARM][3/3] AArch32 NEON vrint builtins and intrinsics

2012-11-29 Thread Kyrylo Tkachov
tests pass, no regressions (once the effective target checks patch is added). Ok for trunk? Thanks, Kyrill 2012-11-29 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/neon.ml (opcode): Add Vrintn, Vrinta, Vrintp, Vrintm, Vrintz to type. (type features): Add Requires_arch

[PATCH][Committed] Add self to write-after-approval section of MAINTAINERS file

2012-11-30 Thread Kyrylo Tkachov
Hi all, I have committed the attached patch, adding myself to the Write After Approval section of the MAINTAINERS file. Thanks, Kyrill ChangeLog: 2012-11-30 Kyrylo Tkachov kyrylo.tkac...@arm.com * MAINTAINERS (Write After Approval): Add myself.Index: MAINTAINERS

[PATCH][committed]: Fix typo in predict.c

2013-04-29 Thread Kyrylo Tkachov
Hi all, I've committed this typo fix in predict.c as r198408. Thanks, Kyrill 2013-04-29 Kyrylo Tkachov kyrylo.tkac...@arm.com * predict.c: Fix typo in comment above #define PROB_VERY_UNLIKELY. predict-spelling.patch Description: Binary data

[PATCH][ARM] Restrict store_minmaxsi

2013-04-29 Thread Kyrylo Tkachov
, Kyrill 2013-04-29 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm.md (store_minmaxsi): Use only when optimize_insn_for_size_p. disable_store_minmaxsi.patch Description: Binary data

RE: [PATCH][ARM] Fix PR 56809

2013-04-30 Thread Kyrylo Tkachov
-Original Message- From: Ramana Radhakrishnan Sent: 03 April 2013 16:16 To: Kyrylo Tkachov Cc: GCC Patches Subject: Re: [PATCH][ARM] Fix PR 56809 On 04/03/13 16:07, Kyrylo Tkachov wrote: Hi all, This patch fixes an ICE that we encounter when building gcc on arm targets

[PATCH][gensupport] Add optional attributes field to define_cond_exec

2013-05-20 Thread Kyrylo Tkachov
with and without the patch and didn't see anything bad (the way define_cond_exec is parsed changes due to the format change) Any other feedback on the patch would, of course, be very appreciated. Ok for trunk if no objections? Thanks, Kyrill 2013-05-20 Kyrylo Tkachov kyrylo.tkac...@arm.com

RE: [PATCH][gensupport] Add optional attributes field to define_cond_exec

2013-05-21 Thread Kyrylo Tkachov
Hi Steven, This would allow us to, for example, disable the predicable variant of an insn based on a non-constant variable. Is there a reason why you can't use attribute enabled for this? The problem stems from the fact that the predicable attribute cannot have a dynamic value.

RE: [PATCH][gensupport] Add optional attributes field to define_cond_exec

2013-05-21 Thread Kyrylo Tkachov
Hi Richard, On 05/20/2013 09:55 AM, Kyrylo Tkachov wrote: For example, a setup like: (define_attr predicated yes,no (const_string no)) (define_attr control_attr yes,no (const_string yes)) (define_attr enabled no,yes (cond [(and (eq_attr control_attr

RE: [PATCH][gensupport] Add optional attributes field to define_cond_exec

2013-05-22 Thread Kyrylo Tkachov
We have a new define_subst which may help here. I *think* that define_cond_exec is (or is nearly) a subset of define_subst. On my medium term to-do list is to support define_cond_exec within gensupport via the define_subst infrastructure, removing everything except the actual parsing

RE: [PATCH][gensupport] Add optional attributes field to define_cond_exec

2013-05-23 Thread Kyrylo Tkachov
Hi Richard, No, define_subst works across patterns, keyed by attributes. Exactly like cond_exec, really. But what you ought to be able to do right now is (define_subst ds_predicable [(match_operand 0)] [(cond_exec (blah) (match_dup 0))]) (define_subst_attr

RE: [PATCH][gensupport] Add optional attributes field to define_cond_exec

2013-05-23 Thread Kyrylo Tkachov
with cond_exec's. Thanks, Kyrill Thanks, Michael On 23 May 2013 12:56, Kyrylo Tkachov kyrylo.tkac...@arm.com wrote: Hi Richard, No, define_subst works across patterns, keyed by attributes. Exactly like cond_exec, really. But what you ought to be able to do right now is (define_subst

RE: [PATCH][gensupport] Add optional attributes field to define_cond_exec

2013-05-24 Thread Kyrylo Tkachov
On 23 May 2013 20:53, Kyrylo Tkachov kyrylo.tkac...@arm.com wrote: Hi Michael, Hi Kyrylo, Richard, What would be the function of (set_attr ds_predicable yes) ? Doesn't the use of ds_predicable_enabled already trigger the substitution? To use define subst one doesn't need to write

RE: [PATCH][gensupport] Add optional attributes field to define_cond_exec

2013-05-24 Thread Kyrylo Tkachov
suggested (I mean proper use of 'control_attr' in 'enabled' attribute). Thanks, Kyrill Thanks, Michael On 24 May 2013 14:11, Kyrylo Tkachov kyrylo.tkac...@arm.com wrote: Hi Michael, - What about define_insn_and_split? Currently, we can define predicable

RE: [PATCH][gensupport] Add optional attributes field to define_cond_exec

2013-05-24 Thread Kyrylo Tkachov
these examples could help. Thanks, Kyrill Thanks, Michael On 24 May 2013 15:39, Kyrylo Tkachov kyrylo.tkac...@arm.com wrote: Unfortunately, that is a strong point against define_subst in my case, since on arm we have more than 400 predicable patterns, of we which we might want

RE: [PATCH][gensupport] Add optional attributes field to define_cond_exec

2013-05-28 Thread Kyrylo Tkachov
I agree that it is possible to use define_subst (apart from the fact that it doesn't support define_insn_and_split yet). It's just that I think it looks hacky and makes the patterns look less readable if the reader has to remember that predication is implicit when a subst_attr is

[PATCH][ARM] PR56315 Improve code generation for DImode IOR

2013-05-31 Thread Kyrylo Tkachov
prefer to trust the optimisers to make the correct decision, therefore I didn't copy over the '?'. Ok for trunk? Thanks, Kyrill gcc/ 2013-05-31 Kyrylo Tkachov kyrylo.tkac...@arm.com PR target/56315 * config/arm/arm.c (const_ok_for_dimode_op): Handle IOR. * config/arm/arm.md

RE: [PATCH][gensupport] Add optional attributes field to define_cond_exec

2013-06-03 Thread Kyrylo Tkachov
Any comments/suggestions on my implementation of the idea are very welcome. http://gcc.gnu.org/ml/gcc-patches/2013-05/msg01093.html Otherwise, OK for trunk? Ping? Thanks, Kyrill

[PATCH][ARM][1/n] Partial IT block deprecation in ARMv8 AArch32

2013-06-04 Thread Kyrylo Tkachov
behaviour is controlled by a new variable, arm_restrict_it. We stop the merging of cond_execs into multi-instruction IT blocks when arm_restrict_it is enabled. Bootstrapped on a Cortex-A15, regtested arm-none-eabi on qemu and a model. Ok for trunk? Thanks, Kyrill 2013-06-04 Kyrylo Tkachov

RE: [PATCH][ARM][1/n] Partial IT block deprecation in ARMv8 AArch32

2013-06-04 Thread Kyrylo Tkachov
Hi Joseph, It creates a command line option, -mrestrict-it, that can be used to Command-line options need documenting in invoke.texi. I've added some documentation in invoke.texi Confirmed that documentation builds ok. Thanks, Kyrill 2013-06-04 Kyrylo Tkachov kyrylo.tkac...@arm.com

[2/n] Partial IT block deprecation in ARMv8 AArch32 - define_cond_exec machinery

2013-06-05 Thread Kyrylo Tkachov
the predicable_short_it attribute to no. The enabled_for_depr_it attribute is used to disable alternatives in some cases where explicit IT blocks are used in the output template. Ok for trunk? 2013-06-05 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm.md (enabled_for_depr_it): New

[PATCH][ARM][3/n] Partial IT block deprecation in ARMv8 AArch32 - atomics patterns

2013-06-05 Thread Kyrylo Tkachov
for ARMv8 Thumb mode is pretty straightforward. Tested together with the other patches in the series by bootstrap on Cortex-A15 and regtest arm-none-eabi on qemu and model. Ok for trunk? Thanks, Kyrill 2013-06-05 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/sync.md

[PATCH][ARM][4/n] Partial IT block deprecation in ARMv8 AArch32 - load/store multiple patterns

2013-06-05 Thread Kyrylo Tkachov
is regenerated. The changes are quite straightforward. Tested together with the other patches in the series by bootstrap on a Cortex-A15 and regtest arm-none-eabi on qemu and model. Ok for trunk? Thanks, Kyrill 2013-06-05 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm-ldmstm.ml: Set

[PATCH][ARM][5/n] Partial IT block deprecation in ARMv8 AArch32 - load/store multiple

2013-06-06 Thread Kyrylo Tkachov
not be conditionalised by the new rules. Tested together with the other patches in the series by bootstrap on a Cortex-A15 and regtest arm-none-eabi on qemu and model. Ok for trunk? Thanks, Kyrill 2013-06-06 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm-fixed.md (addmode3

[PATCH][ARM] Fix iordi3_insn constraints

2013-06-06 Thread Kyrylo Tkachov
in some cases. This patch adds a new constraint for use by iordi3_insn to ensure that the appropriate immediates are accepted. Tested arm-none-eabi with and without -mthumb on qemu. Ok for trunk? Thanks, Kyrill 2013-06-06 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm

[PATCH][ARM] Cleanup anddi3 constraints

2013-06-06 Thread Kyrylo Tkachov
-06 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm.md (anddi3_insn): Remove duplicate alternatives. Clean up. cleanup_anddi3.patch Description: Binary data

RE: [PATCH][ARM][5/n] Partial IT block deprecation in ARMv8 AArch32 - load/store multiple

2013-06-07 Thread Kyrylo Tkachov
On 06/06/13 17:26, Richard Henderson wrote: On 06/06/2013 08:02 AM, Richard Earnshaw wrote: (define_insn addmode3 - [(set (match_operand:FIXED 0 s_register_operand =r) -(plus:FIXED (match_operand:FIXED 1 s_register_operand r) -(match_operand:FIXED 2

[PATCH][ARM] PR56315 Improve codegen for DImode immediates - XOR

2013-06-07 Thread Kyrylo Tkachov
on qemu with and without -mthumb. Ok for trunk? Thanks, Kyrill gcc/ 2013-06-07 Kyrylo Tkachov kyrylo.tkac...@arm.com PR target/56315 * config/arm/arm.md (*xordi3_insn): Change to insn_and_split. (xordi3): Change operand 2 constraint to arm_xordi_operand

RE: [PATCH][ARM] PR56315 Improve codegen for DImode immediates - XOR

2013-06-07 Thread Kyrylo Tkachov
Oops, once again with patch attached... gcc/ 2013-06-07 Kyrylo Tkachov kyrylo.tkac...@arm.com PR target/56315 * config/arm/arm.md (*xordi3_insn): Change to insn_and_split. (xordi3): Change operand 2 constraint to arm_xordi_operand. * config/arm/constraints.md

[PATCH][ARM][6/n] Partial IT block deprecation in ARMv8 AArch32 - VFP patterns

2013-06-10 Thread Kyrylo Tkachov
that is available in ARMv8 when appropriate. Tested arm-none-eabi on qemu and model as part of the whole series and also independently bootstrapped with -mrestrict-it enabled on a Cortex-A15. Ok for trunk? Thanks, Kyrill 2013-06-10 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/predicates.md

RE: [PATCH][ARM][6/n] Partial IT block deprecation in ARMv8 AArch32 - VFP patterns

2013-06-17 Thread Kyrylo Tkachov
Ping? http://gcc.gnu.org/ml/gcc-patches/2013-06/msg00493.html Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 10 June 2013 11:52 To: gcc-patches@gcc.gnu.org Cc: Richard Earnshaw; Ramana

[PATCH][ARM] Fix FAIL: gcc.target/arm/unaligned-memcpy-2.c scan-assembler-times stmia 1

2013-06-17 Thread Kyrylo Tkachov
Hi all, This arm testsuite patch initialises an array in the unaligned-memcpy-2.c test to ensure alignment, making the test pass. This is similar to the patch http://gcc.gnu.org/ml/gcc-patches/2013-06/msg00683.html. Ok for trunk? Thanks, Kyrill gcc/testsuite/ 2013-06-17 Kyrylo Tkachov

RE: [PATCH, ARM][2 of 2] Enable shrink-wrap for ARM

2013-06-17 Thread Kyrylo Tkachov
Hi Zhenqiang, This patch causes PR57637 (miscompare in SPEC2000). I'll try to reduce a testcase. Thanks, Kyrill The *arm_simple_return is the same as simple_return used by shrink-wrap. *arm_return and *arm_simple_return are not merged since *arm_return is for Often the return insn will be

[PATCH][ARM][7/n] Partial IT block deprecation in ARMv8 AArch32 - straightforward arm.md changes

2013-06-18 Thread Kyrylo Tkachov
-A15 and tested arm-none-eabi on a model on armv7-a and armv8-a architecture levels. Ok for trunk? Thanks, Kyrill 2013-06-18 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm.md (arm_mulsi3_v6): Add alternative for 16-bit encoding. (mulsi3addsi_v6): Disable predicable

[PATCH][ARM][8/n] Partial IT block deprecation in ARMv8 AArch32 - mov* patterns and splitters

2013-06-20 Thread Kyrylo Tkachov
and qemu on arm-none-eabi for ARMv7 and ARMv8. Ok for trunk? Thanks, Kyrill 2013-06-20 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/constraints.md (Ts): New constraint. * config/arm/arm.md (arm_movqi_insn): Add alternatives for 16-bit encodings. (compare_scc

[PATCH][ARM][9/n] Partial IT block deprecation in ARMv8 AArch32 - bitwise ops

2013-06-20 Thread Kyrylo Tkachov
on a Cortex-A15. Ok for trunk? Thanks, Kyrill 2013-06-20 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm.md (arm_andsi3_insn): Add alternatives for 16-bit encoding. (iorsi3_insn): Likewise. (arm_xorsi3): Likewise. 10-bitwise-ops.patch Description: Binary data

[PATCH][ARM] Fix FAIL pr46975

2013-06-21 Thread Kyrylo Tkachov
-code setting. Ok for trunk? Thanks, Kyrill gcc/ 2013-06-21 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm.md (peepholes for eq (reg1) (reg2/imm)): Use the flag setting variants when optimising for size. gcc/testsuite/ 2013-06-21 Kyrylo Tkachov kyrylo.tkac

[PATCH][ARM][10/n] Partial IT block deprecation in ARMv8 AArch32 - thumb2.md changes

2013-06-21 Thread Kyrylo Tkachov
is added that is a splitter version of thumb2_cond_arith but produces patterns appropriate for arm_restrict_it. Bootstrapped on Cortex-A15, tested on model and qemu with ARMv7 and ARMv8 architecture levels. Ok for trunk? Thanks, Kyrill 2013-06-21 Kyrylo Tkachov kyrylo.tkac...@arm.com

[PATCH][ARM][11/n] Partial IT block deprecation in ARMv8 AArch32 - final arm.md changes

2013-06-21 Thread Kyrylo Tkachov
. Bootstrapped on a Cortex-A15, tested on model and qemu with ARMv7 and ARMv8 architecture levels. Ok for trunk? Thanks, Kyrill 2013-06-21 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/constraints.md (Pd): Allow TARGET_THUMB instead of TARGET_THUMB1. (Pz): New constraint

RE: [PATCH][ARM] Fix FAIL pr46975

2013-07-01 Thread Kyrylo Tkachov
used when optimising for speed, since the large code should be more efficient because the scheduler won't have to deal with condition-code setting. Ok for trunk? Thanks, Kyrill gcc/ 2013-06-21 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm.md (peepholes for eq

RE: [PATCH][ARM][11/n] Partial IT block deprecation in ARMv8 AArch32 - final arm.md changes

2013-07-02 Thread Kyrylo Tkachov
Hi Richard, Thanks for the review. (define_insn_and_split *arm_cmpdi_unsigned [(set (reg:CC_CZ CC_REGNUM) - (compare:CC_CZ (match_operand:DI 0 s_register_operand r) -(match_operand:DI 1 arm_di_operand rDi)))] + (compare:CC_CZ (match_operand:DI 0

RE: [PATCH][ARM][10/n] Partial IT block deprecation in ARMv8 AArch32 - thumb2.md changes

2013-07-11 Thread Kyrylo Tkachov
-a with and without -mthumb. Bootstrapped on a Cortex-A15. Thanks, Kyrill 2013-07-11 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/predicates.md (shiftable_operator_strict_it): New predicate. * config/arm/thumb2.md (thumb_andsi_not_shiftsi_si): Disable cond_exec version

RE: PATCH (c,c++) for c++/57793

2013-07-17 Thread Kyrylo Tkachov
/pr42611.c, because the error message now appears on the line of the definition of the struct. This patch fixes the testcase. Ok to apply to trunk? Thanks, Kyrill 2013-07-17 Kyrylo Tkachov kyrylo.tkac...@arm.com * gcc.dg/pr42611.c: Move dg-error to correct line. pr42611.patch

[PATCH][4.8 backport] Fix PR57735

2013-07-22 Thread Kyrylo Tkachov
. Use gen_int_mode rather than GEN_INT. Ok to backport to the 4.8 branch? I've attached the testcase that exposed the ICE. I think the ChangeLog would look like this: 2013-07-22 Kyrylo Tkachov kyrylo.tkac...@arm.com PR target/57735 Backport from mainline 2013-04-30

RE: [PATCH][4.8 backport] Fix PR57735

2013-07-23 Thread Kyrylo Tkachov
Hi Richard, Richard Sandiford rdsandif...@googlemail.com writes: Kyrylo Tkachov kyrylo.tkac...@arm.com writes: Hi all, The fix for PR57735 is in current trunk (for a different issue I think), just needs a backport to 4.8. It is r198462 by Richard Sandiford: 2013-04-30 Richard

[PATCH][ARM] Fix bug in add patterns due to commutativity modifier

2013-07-25 Thread Kyrylo Tkachov
of the sources being the same as the destination. Bootstrapped on arm-linux-gnueabihf and regtested on arm-none-eabi. Ok for trunk? Thanks, Kyrill 2013-07-25 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm.md (arm_addsi3, addsi3_carryin_optab and addsi3_carryin_alt2_optab

[PATCH][ARM] Update two fixed point arithmetic patterns for -mrestrict-it

2013-07-25 Thread Kyrylo Tkachov
armv7-a and armv8-a. Ok for trunk? Thanks, Kyrill 2013-07-25 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm-fixed.md (ssmulsa3, usmulusa3): Adjust for arm_restrict_it. Remove trailing whitespace. 13-arm-fixed.patch Description: Binary data

[PATCH][ARM][combine] Improve opportunities to generate cmn instructions

2013-07-25 Thread Kyrylo Tkachov
-linux-gnueabihf and x86_64-linux-gnu. Regtested on arm-none-eabi on qemu. Ok for trunk? 2013-07-25 Kyrylo Tkachov kyrylo.tkac...@arm.com Richard Earnshaw richard.earns...@arm.com * combine.c (simplify_comparison): Re-canonicalize operands where appropriate

[PATCH][ARM] Fix gcc.target/arm/minmax_minus.c

2013-07-26 Thread Kyrylo Tkachov
instructions instead of rsb when compiling for Thumb2. They perform the same functionality, but makes the testcase fail. This patch fixes the testacase by scanning for the absence of moves instead. Ok for trunk? Thanks, Kyrill 2013-07-26 Kyrylo Tkachov kyrylo.tkac...@arm.com * gcc.target/arm

RE: [PATCH][4.8 backport] Fix PR57735

2013-07-29 Thread Kyrylo Tkachov
Ping? Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 23 July 2013 10:09 To: 'Richard Sandiford' Cc: gcc-patches; mi...@it.uu.se; 'Richard Biener' Subject: RE: [PATCH][4.8 backport] Fix

[PATCH][AArch64] Fix FAIL: gcc.target/aarch64/cmn.c scan-assembler cmn\tw[0-9]

2013-07-29 Thread Kyrylo Tkachov
for trunk? Thanks, Kyrill 2013-07-29 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/aarch64/aarch64.md (compare_negmode): Match canonical RTL form. aarch64-cmn.patch Description: Binary data

RE: [PATCH][ARM] Fix FAIL pr46975

2013-07-31 Thread Kyrylo Tkachov
Hi all, Resurrecting this thread... On 01/07/13 12:05, Kyrylo Tkachov wrote: Hi Richard, This hurts code size. Therefore I've disabled the new peephole2 for optimize_insn_for_size_p so that the original peephole before r200197 is used when optimising for size. I've also added

RE: [PATCH][ARM] Fix FAIL pr46975

2013-08-01 Thread Kyrylo Tkachov
for the add-with-carry case in thumb2_reorg so that the flag-setting variant of ADC is used. Now for the testcase, when optimising for size we get only 3 16-bit instructions. Tested arm-none-eabi on qemu with and without -mthumb. Ok? Thanks, Kyrill 2013-08-01 Kyrylo Tkachov kyrylo.tkac

[PATCH][ARM] Emit canonical form for constant subtraction in minmax_arithsi_non_canon

2013-08-01 Thread Kyrylo Tkachov
? Thanks, Kyrill 2013-08-01 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm.md (minmax_arithsi_non_canon): Emit canonical RTL form when subtracting a constant. minmax_canon_minus.patch Description: Binary data

[PATCH][ARM] Delete gcc.target/arm/neon-for-64bits-2.c test

2013-08-02 Thread Kyrylo Tkachov
generation of 64 bit. The -mneon-for-64bits option is not supposed to force generation of NEON instructions and therefore the compiler might choose to not use them, making this testcase invalid. This patch removes it. Ok for trunk? Thanks, Kyrill 2013-08-02 Kyrylo Tkachov kyrylo.tkac...@arm.com

RE: [PATCH][4.8 backport] Fix PR57735

2013-08-05 Thread Kyrylo Tkachov
Ping? Ping^2 Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 23 July 2013 10:09 To: 'Richard Sandiford' Cc: gcc-patches; mi...@it.uu.se; 'Richard Biener' Subject: RE

[PATCH][tree-optimization] Fix PR58088

2013-08-07 Thread Kyrylo Tkachov
for double_int, so this patch will need to be reworked for 4.8. In the meantime, Ok for trunk? Bootstrapped on x86_64-linux-gnu and tested arm-none-eabi on qemu. Thanks, Kyrill 2013-08-07 Kyrylo Tkachov kyrylo.tkac...@arm.com * gcc/fold-const.c (mask_with_trailing_zeros): New function

RE: [PATCH][tree-optimization] Fix PR58088

2013-08-07 Thread Kyrylo Tkachov
This issue is present in 4.8.1 as well as trunk. However, I think 4.8 uses a different API for double_int, so this patch will need to be reworked for 4.8. Actually, I was confused. This patch applies to 4.8.1 as well and fixes the issue. Passes bootstrap on x86_64-linux-gnu and testing

RE: [PATCH][tree-optimization] Fix PR58088

2013-08-08 Thread Kyrylo Tkachov
Ok for trunk? Bootstrapped on x86_64-linux-gnu and tested arm-none-eabi on qemu. Thanks, Kyrill * gcc.c-torture/compile/pr58088.c: New test. Also, the ChangeLog entries should be: 2013-08-08 Kyrylo Tkachov kyrylo.tkac...@arm.com PR tree-optimization/58088

  1   2   3   4   5   6   7   8   9   10   >