PING.
On 24 September 2012 11:34, Matthew Gretton-Dann
wrote:
> On Wednesday 05 September 2012 13:47:19 Steven Bosscher wrote:
>> On Wed, Sep 5, 2012 at 1:25 PM, Matthew Gretton-Dann wrote:
>> > + /* If the two blocks are in different partitions we do not want to mark
&g
.
Is this OK to be backported to 4.7? Cross tested arm-none-linux-gnueabi.
Thanks,
Matt
2012-10-15 Matthew Gretton-Dann
Backported from mainline
2012-07-23 Ulrich Weigand
* config/arm/arm.c (arm_reorg): Ensure all insns are split.
--
Matthew Gretton-Dann
Linaro
the canonical way to detect being
in a different partition should it be moved out into a query function,
and all of cfgrtl.c updated to use it?
[Note I am not a maintainer and so can't approve/reject your patch].
Thanks,
Matt
--
Matthew Gretton-Dann
Linaro Toolchain Working Group
matthew.gretton-d...@linaro.org
ml/gcc-patches/2012-09/msg00274.html
> or are there others?
That is one of two. The other one is:
http://gcc.gnu.org/ml/gcc-patches/2012-09/msg00275.html
I'd be careful with the second one - the original patch posted (in the
link) fixes the issue I was seeing, but Stephen Bosscher suggested I
made some changes, and I reposted a patch with some incorrect logic
(which unfortunately also fixes the issue but more by luck than
judgement). I haven't had the time to fully work through updating and
testing a reworked patch yet.
Thanks,
Matt
--
Matthew Gretton-Dann
Linaro Toolchain Working Group
matthew.gretton-d...@linaro.org
changes are probably on all
cores without further testing.
Thanks,
Matt
--
Matthew Gretton-Dann
Linaro Toolchain Working Group
matthew.gretton-d...@linaro.org
On 16 November 2012 12:22, Bin Cheng wrote:
>
>
>> -Original Message-----
>> From: Matthew Gretton-Dann [mailto:matthew.gretton-d...@linaro.org]
>> Sent: Friday, November 16, 2012 6:30 PM
>> To: Bin Cheng
>> Cc: gcc-patches@gcc.gnu.o
One question I do have having done this work - is there a canonical way to
test for the arm*-*-linux-gnueabi triplet (or variants)? Various configure
and testsuite files test for this, but there doesn't seem to be a consistent
method.
OK for 4.7?
Thanks,
Matt
2012-11-08 Matthew Gr
On 21 November 2012 00:05, Matthias Klose wrote:
> Am 20.11.2012 21:34, schrieb Matthew Gretton-Dann:
>> All,
>>
>> This patch backports Matthais Klose's arm*-*-linux-gnueabihf triplet
>> support patch of 2012-10-15 to 4.7.
>>
>> The backport wa
testcase which will reliably
test this condition.
OK for trunk, 4.7, and 4.6?
Thanks,
Matt
gcc/ChangeLog:
2012-11-21 Matthew Gretton-Dann
PR target/54974
* config/arm/arm.md (thumb1_extendhisi2): Reduce Thumb
pool_range.
(arm_movdi): Likewise.
(th
On 24 November 2012 00:27, Ramana Radhakrishnan
wrote:
> On Wed, Nov 21, 2012 at 7:59 PM, Matthew Gretton-Dann
> wrote:
[snip]
>> The fix is to decrease the pool_range of all insns by 2 when generating
>> Thumb code. There is no need to change neg_pool_range values as rounding
PING
On 20 November 2012 20:34, Matthew Gretton-Dann
wrote:
> All,
>
> This patch backports Matthais Klose's arm*-*-linux-gnueabihf triplet
> support patch of 2012-10-15 to 4.7.
>
> The backport was not clean as 4.8 has obsoleted various arm*-*-*
> triplets which ar
PING^2
On 29 November 2012 14:45, Matthew Gretton-Dann
wrote:
> PING
>
> On 20 November 2012 20:34, Matthew Gretton-Dann
> wrote:
>> All,
>>
>> This patch backports Matthais Klose's arm*-*-linux-gnueabihf triplet
>> support patch of 2012-10-15 to 4.7.
&g
GS.
It looks to me as if this should also be applied to the 4.8 branch - Xinyu
do you agree?
If so is the backport OK for 4.8?
Thanks,
Matt
--
Matthew Gretton-Dann
Toolchain Working Group, Linaro
, 198142, 198176, 198298,
198302-198306, 198316, 198394, 198396-198400, 198402-198404, 198406, 198412,
198424-198425, 198443
Thanks,
Matt
--
Matthew Gretton-Dann
Toolchain Working Group, Linaro
All,
I have just backported the following revisions from trunk to
linaro/gcc-4_8-branch: 197838, 198191, 198490-198496, 198575-198575, and 198677.
I have also merged the gcc-4_8-branch into linaro/gcc-4_8-branch up to
revision 198615.
Thanks,
Matt
--
Matthew Gretton-Dann
Toolchain
eferring one form over the
other.
In conclusion - I prefer the PROFILE_HOOK method because it makes the
compiler do all the work, and results in less impact on stack usage
and performance. FUNCTION_PROFILER may impact the code generated by
the compiler less and produce a smaller overall image - but I'm not
sure that's more beneficial.
Thanks,
Matt
--
Matthew Gretton-Dann
Linaro Toolchain Working Group
matthew.gretton-d...@linaro.org
options are related.
Tested cross arm-none-linux-gnueabi.
OK for trunk?
Thanks,
Matt
gcc/ChangeLog:
2013-08-19 Matthew Gretton-Dann
* config/arm/linux-elf.h (MULTILIB_DEFAULTS): Remove definition.
* config/arm/t-linux-eabi: (MULTILIB_OPTIONS): Document association
- fprintf (stream, wc_reg_names [INTVAL (x)]);
+ fputs (wc_reg_names [INTVAL (x)], stream);
}
return;
--
Matthew Gretton-Dann
Toolchain Working Group, Linaro
: Likewise.
* gcc.target/arm/vselvsdf.c: Likewise.
* gcc.target/arm/vselvssf.c: Likewise.
--
Matthew Gretton-Dann
Toolchain Working Group, Linaro
. (Backports are revisions
197599:197609).
Thanks,
Matt
--
Matthew Gretton-Dann
Toolchain Working Group, Linaro
On 17 December 2012 14:28, Richard Earnshaw wrote:
> On 21/11/12 11:48, Matthew Gretton-Dann wrote:
>>
>> On 21 November 2012 00:05, Matthias Klose wrote:
>>>
>>>
>>> looks fine, except one missing chunk from my original patch. maybe left
>
On 29/11/12 14:42, Matthew Gretton-Dann wrote:
On 24 November 2012 00:27, Ramana Radhakrishnan
wrote:
On Wed, Nov 21, 2012 at 7:59 PM, Matthew Gretton-Dann
wrote:
[snip]
The fix is to decrease the pool_range of all insns by 2 when generating
Thumb code. There is no need to change
building a cross
arm-none-linux-gnueabi compiler with DEBUG_RELOAD forced on.
OK for trunk?
gcc/ChangeLog:
2013-02-02 Matthew Gretton-Dann
* gcc/reload.c (subst_reloads): Fix DEBUG_RELOAD build issue.
--
Matthew Gretton-Dann
Linaro Toolchain Working Group
matthew.gretton-d...@linaro.org
I want to backport early to this branch.
Thanks,
Matt
--
Matthew Gretton-Dann
Toolchain Working Group, Linaro
? htdocs/.svn.html.swp
? htdocs/svn.html~
Index: htdocs/svn.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/svn.html,v
64-unknown-linux-gnu, i386-apple-darwin9.8.0,
and powerpc-apple-darwin9.8.0.
Rainer
This also breaks arm-none-eabi builds (fails to find unwind-arm-common.h
from gcc/ginclude). I have raised PR50978 for this
(http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50978).
Thanks,
Matt
--
Matth
someone please review and comment.
Thanks,
Matt
gcc/ChangeLog:
2011-11-07 Matthew Gretton-Dann
* config/arm/arm-cores.def: Add -mcpu=cortex-a7.
* config/arm/arm-tables.opt: Regenerate.
* config/arm/arm-tune.md: Likewise.
* config/arm/bpabi.h (BE8_LINK_SPEC
TL pieces are least ready for this.
I would change this to say something like (using C language terms):
The compiler should provide enough information to allow breakpoints to
be set at each sequence point, and that the state of the machine is
such that everything before that sequence point will ha
have seen too many
cases where people debug at -O0/-O1 and then build a release with a
-O2/-O3 build and get bitten by undefined behaviour issues. The more
optimised -Og code is the less the reason to release a build at a
higher optimisation level.
Thanks,
Matt
--
Matthew Gretton-Dann
Linaro Toolchain Working Group
matthew.gretton-d...@linaro.org
On 5 September 2012 09:55, Steven Bosscher wrote:
> On Wed, Sep 5, 2012 at 10:46 AM, Matthew Gretton-Dann
> wrote:
>>> Please, no inlining. Think of stack back-traces and their use
>>> when debugging.
>>
>> I would argue [without sufficient knowledge of how
nted is to have rtl_tidy_fallthru not do anything
if the fallthru edge crosses a partition boundary.
OK?
Thanks,
Matt
gcc/ChangeLog:
2012-09-05 Matthew Gretton-Dann
* cfgrtl.c (rtl_tidy_fallthru_edge): Don't tidy edges which
cross partitions.
diff --git a/gcc/cfgrtl.c b/gcc/cf
Not that this would have helped
in this particular instance.
Tested cross arm-linux-none-gnueabi with in progress ARM -freorder-blocks-and-
partition enabling patch.
OK?
Thanks,
Matt
gcc/ChangeLog:
2012-09-05 Matthew Gretton-Dann
* postreload-gcse.c (eliminate_partially_red
On 5 September 2012 13:45, Richard Earnshaw wrote:
> On 05/09/12 13:02, Steven Bosscher wrote:
>> On Wed, Sep 5, 2012 at 1:42 PM, Matthew Gretton-Dann wrote:
>>> Whilst this fix works for this particular case I am not sure it is the
>>> best fix for the general iss
o other approaches could be to see what adding
/* { dg-require-effective-target arm_vfp_ok } */
after dg-add-options achieves.
If that doesn't work I would suggest you add an arm_arch_v6_vfp_v2_ok
set of effective-targets instead of restricting the current
arm_arch_v6 effective target.
Thanks,
Matt
--
Matthew Gretton-Dann
Linaro Toolchain Working Group
matthew.gretton-d...@linaro.org
On Wednesday 05 September 2012 13:47:19 Steven Bosscher wrote:
> On Wed, Sep 5, 2012 at 1:25 PM, Matthew Gretton-Dann wrote:
> > + /* If the two blocks are in different partitions we do not want to mark
> > + this as a fallthru edge. */
> > + if (BB_PARTITION (
On Wednesday 05 September 2012 17:40:23 Steven Bosscher wrote:
> On Wed, Sep 5, 2012 at 3:18 PM, Matthew Gretton-Dann
>
> wrote:
> > On 5 September 2012 13:45, Richard Earnshaw wrote:
> >> On 05/09/12 13:02, Steven Bosscher wrote:
> >>> On Wed, Sep 5, 2012 at
PING.
On Mon, May 28, 2012 at 10:51:27AM +0100, Matthew Gretton-Dann wrote:
> All,
>
> This patch adds a variety of predefined macros to reveal the presence of
> various features of the ARM architecture. These are detailed in the ARM
> C Language Extensions specification, availab
Further testing has found a couple of failures to build with a C++ compiler,
and trunk has moved on a bit so the patch doesn't apply cleanly.
An updated patch is attached.
OK for trunk?
Same ChangeLog as before.
Thanks,
Matt
On 20/06/12 11:18, Matthew Gretton-Dann wrote:
PING.
O
have been tested as a whole using QEmu targetting
arm-none-eabi, and with an arm-linux-gnueabi bootstrap.
They require the ACLE feature test macros patch currently under review here:
http://gcc.gnu.org/ml/gcc-patches/2012-06/msg01592.html
Thanks,
Matt
--
Matthew Gretton-Dann
Principal Engineer
All,
This patch adds support to the ARM backend for generating floating-point
fused multiply-accumulate.
OK?
gcc/ChangeLog:
2012-06-25 Matthew Gretton-Dann
* config/arm/iterators.md (SDF): New mode iterator.
(V_if_elem): Add support for SF and DF modes.
(V_reg
All,
This patch adds vectoriser support for VFMA to the ARM Neon backend.
Note that the VFP VFNMA and VFNMS instructions do not have Neon
equivalents.
OK?
gcc/ChangeLog:
2012-06-25 Matthew Gretton-Dann
* config/arm/neon.md (fma4): New pattern.
(*fmsub4): Likewise.
2012
/ChangeLog:
2012-06-25 Matthew Gretton-Dann
* config/arm/arm.c (neon_builtin_data): Add vfma and vfms
builtins.
* config/arm/neon-docgen.ml (intrinsic_groups): Add
fused-multiply-* groups.
* config/neon-gen.ml (print_feature_test_start): New function
All,
This patch enables the dump-noaddr test to work in out-of-build-tree testing.
It does this by making sure that the dump files generated during the
test are created under $tmpdir.
gcc/testsuite/ChangeLog:
2012-06-27 Matthew Gretton-Dann
* gcc.c-torture/unsorted/dump-noaddr.x
On 27/06/12 21:35, Andrew Pinski wrote:
On Wed, Jun 27, 2012 at 3:33 AM, Matthew Gretton-Dann
wrote:
All,
This patch enables the dump-noaddr test to work in out-of-build-tree
testing.
[snip]
I created a much simpler patch which I have been meaning to submit.
I attached it for reference
On 28/06/12 14:38, Mike Stump wrote:
On Jun 28, 2012, at 1:28 AM, Matthew Gretton-Dann
> wrote:
On 27/06/12 21:35, Andrew Pinski wrote:
On Wed, Jun 27, 2012 at 3:33 AM, Matthew Gretton-Dann
wrote:
All,
This patch enables the dump-noaddr test to work in out-of-build-tree
testing.
[s
On 26/06/12 14:44, Richard Earnshaw wrote:
On 25/06/12 15:59, Matthew Gretton-Dann wrote:
All,
This patch adds support to the ARM backend for generating floating-point
fused multiply-accumulate.
OK?
gcc/ChangeLog:
2012-06-25 Matthew Gretton-Dann
* config/arm/iterators.md (SDF
and to something other than `1', we also update
cpp.texi to reflect this fact.
OK?
Thanks,
Matt
gcc/ChangeLog:
2012-05-28 Matthew Gretton-Dann
James Greenhalgh
* config/arm/arm.h (TARGET_CPU_CPP_BUILTINS): Add new built-ins.
(TARGET_FMA): New macro.
On 28/05/12 12:27, Joseph S. Myers wrote:
On Mon, 28 May 2012, Matthew Gretton-Dann wrote:
This patch adds a variety of predefined macros to reveal the presence of
various features of the ARM architecture. These are detailed in the ARM
C Language Extensions specification, available here:
http
-fno-tree-tail-merge to the test options.
Tested arm-none-eabi.
Can someone review and approve please?
Thanks,
Matt
gcc/testsuite/ChangeLog:
2011-09-28 Matthew Gretton-Dann
* gcc.target/arm/pr42835.c: Add -fno-tree-tail-merge.
--
Matthew Gretton-Dann
Principal Engineer, PD Soft
t trunk (r179378) for arm-none-eabi.
Please see http://gcc.gnu.org/PR50576.
Thanks,
Matt
--
Matthew Gretton-Dann
Principal Engineer, PD Software - Tools, ARM Ltd
if (!is_widening_mult_p (rhs2_stmt,&type1,&mult_rhs1,
+ if (!is_widening_mult_p (type, rhs2_stmt,&type1,&mult_rhs1,
&type2,&mult_rhs2))
return false;
add_rhs = rhs1;
--
Matthew Gretton-Dann
Principal Engineer, PD Software - Tools, ARM Ltd
ite/ChangeLog:
2011-10-13 Matthew Gretton-Dann
* gcc.dg/vect/pr30858.c: Update expected output for
architectures with multiple vector sizes.
--
Matthew Gretton-Dann
Principal Engineer, PD Software - Tools, ARM Ltddiff --git a/gcc/testsuite/gcc.dg/vect/pr30858.c
b/gcc/t
eon.h generators and ensuring
no
changes in the generated testsuite/header.
OK for trunk?
OK for backporting to GCC 4.7?
Thanks,
Matt
gcc/ChangeLog:
2012-03-12 Matthew Gretton-Dann
* config/arm/neon.ml (ops): Fixup expected instructions for
unsigned vector compares.
--
Matthew Gr
27;ve also manually
inspected libgcc to make sure the functions are being built as expected.
Please can someone review?
Thanks,
Matt
libgcc/ChangeLog:
2011-11-15 Matthew Gretton-Dann
* config/arm/lib1funcs.asm (udivsi3): Add support for divide
functions.
(aeabi_uid
apply the patch?
Thanks,
Matt
ChangeLog:
2011-11-21 Matthew Gretton-Dann
* htdocs/gcc-4.7/changes.html: Document -mcpu=cortex-a7.
--
Matthew Gretton-Dann
Principal Engineer, PD Software - Tools, ARM LtdIndex: htdocs/gcc-4.7/changes.html
someone please review the patch?
Thanks,
Matt
gcc/testsuite/ChangeLog:
2011-11-21 Matthew Gretton-Dann
* gcc.dg/vshift-1.c (main): Call rand instead of random.
* gcc.dg/vshift-3.c (main): Likewise.
--
Matthew Gretton-Dann
Principal Engineer, PD Software - Tools, ARM Ltddiff
me popular embedded benchmarks shows the pipeline
description gives a 1.8% performance improvement on (geomean) average.
The testsuite shows no regressions targetting arm-none-eabi and using QEmu.
Can someone please review.
Thanks,
Matt
gcc/ChangeLog:
2011-11-28 Matthew Gretton-Dann
| grep -v arm\\.md`.
Tested by doing a arm-none-eabi build.
Can someone please review, and if appropriate apply?
Thanks,
Matt
gcc/ChangeLog:
2011-11-29 Matthew Gretton-Dann
* config/arm/t-arm (MD_INCLUDES): Ensure all md files are
listed.
--
Matthew Gretton-Dann
Principal
All,
I have just committed the attached patch to add myself to the
write-after-approval section of the MAINTAINERS file.
Thanks,
Matt
ChangeLog:
2011-11-30 Matthew Gretton-Dann
* MAINTAINERS (write-after-approval): Add self.
--
Matthew Gretton-Dann
Principal Engineer, PD
On 29/11/11 11:04, Ramana Radhakrishnan wrote:
gcc/ChangeLog:
2011-11-28 Matthew Gretton-Dann
* config/arm/arm.c (arm_issue_rate): Cortex-A15 can triple
issue.
* config/arm/arm.md (mul64): Add new attribute.
(generic_sched): Cortex-A15 is not scheduled
All,
The attached patch changes another use of random in the testsuite to rand.
Tested on an arm-none-eabi target using QEmu.
Please can someone review.
Thanks,
Matt
gcc/testsuite/ChangeLog:
2011-11-30 Matthew Gretton-Dann
* gcc.dg/torture/vec-cvt-1.c (FLTTEST): Call rand
, and otherwise takes the generic values. Tuning for other
optimisations (notably preload insertion) is still to be done.
Thanks,
Matt
gcc/ChangeLog:
2012-01-06 Matthew Gretton-Dann
* config/arm/arm-cores.def (cortex-a15): Use cortex_a15_tune for
tuning parameters
All,
The attached patch corrects the mov_notscc pattern in arm.md.
This issue also exists in 4.5 and 4.6, is it okay for me to backport the
fix to those branches, as well as trunk?
OK?
Thanks,
Matt
gcc/ChangeLog:
2012-01-10 Matthew Gretton-Dann
* config/arm/arm.md (mov_notscc
Hi,
This patch fixes the thumb2_mov_notscc pattern in the same way as the
ARM state mov_notscc pattern was fixed earlier in January.
This was highlighted by the gcc.target/arm/20120111-1.c testcase.
OK?
Thanks,
Matt
gcc/ChangeLog:
2012-01-30 Matthew Gretton-Dann
config/arm
rther changes that could be
made which would improve the code generated which will come in a
separate patch.
This issue does not affect the auto-vectorizer.
OK?
Matt
gcc/ChangeLog:
2012-02-21 Matthew Gretton-Dann
PR target/51534
* config/arm/arm.c (neon_builtin_data): Add entrie
Matthew Gretton-Dann
Revert r183011
* config/arm/arm-cores.def (cortex-a15): Use generic Cortex tuning
parameters.
* config/arm/arm.c (arm_cortex_a15_tune): Remove.
--
Matthew Gretton-Dann
Principal Engineer, PD Software, ARM Ltd.diff --git a/gcc/config/arm/arm
fpv3-d16" } */
> +
> +__extension__ typedef unsigned long long int uint64_t;
> +
> +static int
> +func2 (int a, int b)
> +{
> + return a == 0 ? a : a / b;
> +}
> +
> +int array1[1];
> +const uint64_t array2[1] = { 1 };
> +
> +void
> +foo (void)
&g
66 matches
Mail list logo
