Committed to ARM/arm-9-branch
On Wed, Sep 11, 2019 at 4:50 PM Wilco Dijkstra wrote:
>
> While code hoisting generally improves codesize, it can affect performance
> negatively. Benchmarking shows it doesn't help SPEC and negatively affects
> embedded benchmarks, so only enable code hoisting with
Branch created for Arm GCC releases.
- Joey
18 03:29 PM, Joey Ye wrote:
> > This is a fix to an obvious issue in gcov.exp, where proc
> > verify-intermediate returns without closing the open file.
> >
> > This can be a possible fix to PR85871. gcov-8.C diffs to other gcov
> > testcases that it invokes verify-i
regression will disappear completely one month after this patch
committed to trunk.
Tested with make check with no new regressions.
OK to trunk?
testsuite/ChangeLog:
2018-09-05 Joey Ye
* lib/gcov.exp (verify-intermediate): Add missing close.
gcov-20180905.patch
Description: gcov
typo: s/reorg.c/recog.c/g
On Thu, Aug 30, 2018 at 11:20 AM Joey Ye wrote:
>
> Hi Bin & Richard,
>
> It is not as simple as keeping the assertion, which still fails even
> with the change in reorg.c. The testing result is as following:
>
> I. With Bin's patch versio
/gcc/gcc/cfgexpand.c:3572
0x766d67 expand_gimple_stmt
/build/trunk/src/gcc/gcc/cfgexpand.c:3734
0x768ce7 expand_gimple_basic_block
More places need to be patched.
Thanks,
Joey
On Thu, Aug 30, 2018 at 2:02 AM Bin.Cheng wrote:
>
> On Thu, Aug 30, 2018 at 2:47 AM Richard Sandiford
> wrote:
Ping^2 for Bin
The ICE is still there annoyingly.
Thanks,
Joey
On Wed, May 16, 2018 at 9:21 AM Kyrill Tkachov
wrote:
>
> Hi Bin,
>
>
> On 22/03/18 11:07, Bin.Cheng wrote:
> > On Sat, Mar 17, 2018 at 8:54 AM, Richard Sandiford
> > wrote:
> > > Kyrill Tkachov writes:
> > >> Hi Bin,
> > >>
> >
: Ramana Radhakrishnan; Nathan Sidwell; Joey Ye; GCC Patches
> Cc: Richard Earnshaw
> Subject: Re: [ARM] mno-pic-data-is-text-relative & msingle-pic-base
>
> Ramana,
>
> I saw some correspondence between you and Nathan on his patch
> [https://gcc.gnu.org/ml/gcc-patches/20
nt: 12 July 2016 17:07
> To: Ramana Radhakrishnan
> Cc: GCC Patches; Joey Ye; nd
> Subject: Re: [ARM] no-data-is-text-relative & msingle-pic-base
>
> On 07/12/16 12:01, Ramana Radhakrishnan wrote:
>
> > I am also slightly inclined to go further and error out if someo
> -Original Message-
> From: Nathan Sidwell [mailto:nathanmsidw...@gmail.com] On Behalf Of
> Nathan Sidwell
> Sent: 09 May 2016 18:22
> To: Joey Ye; Richard Earnshaw; GCC Patches
> Subject: Re: [ARM] mno-pic-data-is-text-relative & msingle-pic-base
>
> Joey,
gic that this whole patch stands
on.
Thanks,
Joey
> -Original Message-
> From: Nathan Sidwell [mailto:nathanmsidw...@gmail.com] On Behalf Of
> Nathan Sidwell
> Sent: 09 May 2016 15:07
> To: Richard Earnshaw; GCC Patches
> Cc: Joey Ye
> Subject: [ARM] mno-pic-data
Find a missing test case for
https://gcc.gnu.org/ml/gcc-patches/2015-01/msg00789.html left at the corner.
Merged with the latest trunk. New test case does not fail on
thumb1/thumb2/arm targets.
ChangeLog:
2015-07-24: Joey Ye
* gcc.target/arm/thumb2-pop-single.c: New test.
diff --git
Ping
On Wed, Nov 19, 2014 at 10:43 AM, Joey Ye wrote:
> Current thumb2 -Os generates suboptimal code for following tail call case:
>
> int f4(int b, int a, int c, int d);
> int g(int a, int b, int c, int d)
> { return f4(b, a, c, d); }
>
> arm-none-eabi-gcc -Os -mthumb -
This work around brings thumb1 bootstrap back. Though I cannot
approve, I would like it in stage 3. A complete fix in thumb1 pattern
will be welcomed.
Thanks,
Joey
On Thu, Nov 20, 2014 at 7:54 PM, Tom de Vries wrote:
> Richard,
>
> This patch fixes PR63718, which currently breaks Thumb1 bootstra
OK applying to arm/embedded-4_9-branch, though you still need
maintainer approval into trunk.
- Joey
On Wed, Nov 26, 2014 at 11:43 AM, Hale Wang wrote:
> Hi,
>
> This patch ports the aeabi_idiv routine from Linaro Cortex-Strings
> (https://git.linaro.org/toolchain/cortex-strings.git), which was
Current thumb2 -Os generates suboptimal code for following tail call case:
int f4(int b, int a, int c, int d);
int g(int a, int b, int c, int d)
{ return f4(b, a, c, d); }
arm-none-eabi-gcc -Os -mthumb -mcpu=cortex-m3 test.c
push
{r4, lr}
mov r4, r1
mov r1, r0
mov r0, r4
pop {r4, lr}
b f4
Ther
Can a new case be rewritten then?
- Joey
On Fri, Nov 14, 2014 at 9:32 AM, Yangfei (Felix) wrote:
> No, we noticed this issue when improving the vld1(q?)_dup intrinsics. Thanks.
>
>
>> Is there a case or PR to demonstrate the issue? If yes, better to include it
>> as a test
>> case.
>>
>> Thank
Is there a case or PR to demonstrate the issue? If yes, better to
include it as a test case.
Thanks,
Joey
On Thu, Nov 13, 2014 at 2:14 PM, Yangfei (Felix) wrote:
> Hi,
>
> We find that the VALLDI mode iterator used in *aarch64_simd_ld1r
> pattern is not appropriate.
> The reason is that it'
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63747 is likely caused by
this patch. compare_gimple_switch does not check CASE_LOW and
CASE_HIGH, resulting merging functions not identical.
Interestingly in the first a few versions of this patch CASE_LOW/HIGH
were checked. But last versions only chec
Tom,
This patch broke arm thumb1 bootstrap. Please check details at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63718
Best Regards
Joey
On Fri, Oct 17, 2014 at 5:57 AM, Jeff Law wrote:
> On 10/16/14 03:14, Tom de Vries wrote:
>>
>> Eric,
>>
>> this patch is the second half of the fix for PR61
Trunk fails to build plugin again due to missing plugin header files. This
patch fixes it.
OK to trunk?
ChangeLog:
PR plugin/59335
* Makefile.in (PLUGIN_HEADERS): Add wide-int.h, signop.h, hash-map.h,
hash-set.h.
diff --git a/gcc/Makefile.in b/gcc/Makefile.in
index 63124f8..8e7aada 10064
OK to 4.8 then?
On Thu, Aug 14, 2014 at 6:36 PM, Richard Biener
wrote:
> On Thu, Aug 14, 2014 at 7:34 AM, Joey Ye wrote:
>> PR60484 is marked as 4.7/4.8 regression and it is reported against 4.8
>> recently by an user.
>>
>> OK backporting to 4.7/4.8?
>
> The
PR60484 is marked as 4.7/4.8 regression and it is reported against 4.8
recently by an user.
OK backporting to 4.7/4.8?
- Joey
On Sat, Mar 15, 2014 at 1:43 AM, Joseph S. Myers
wrote:
> On Fri, 14 Mar 2014, Marek Polacek wrote:
>
>> This patch makes sure that we set the directory prefix of
>> dum
If f2d need fix, then please fix d2f too as current implementation for
both behave similarly.
- Joey
On Mon, May 19, 2014 at 5:23 AM, Aurelien Jarno wrote:
> On ARM soft-float, the float to double conversion doesn't convert a sNaN
> to qNaN as the IEEE Std 754 standard mandates:
>
> "Under defau
Ping
> -Original Message-
> From: Joey Ye [mailto:joey...@arm.com]
> Sent: Tuesday, April 01, 2014 6:18 PM
> To: 'Ian Lance Taylor'
> Cc: gcc-patches
> Subject: RE: [patch] Shorten Windows path
>
> Ian, thanks for your comments. Please
> -Original Message-
> From: Mike Stump [mailto:mikest...@comcast.net]
> Sent: Monday, April 21, 2014 11:39 PM
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [patch, testsuite] Fix fragile case nsdmi-union5
>
> On Apr 17, 2014, at 10:28 PM, Joey Ye w
Resulting from discussion here:
http://gcc.gnu.org/ml/gcc/2014-04/msg00125.html
ChangeLog:
* g++.dg/cpp0x/nsdmi-union5.C: Change to runtime test.
Index: gcc/testsuite/g++.dg/cpp0x/nsdmi-union5.C
===
--- gcc/testsuite/g++.dg/c
> -Original Message-
> From: Richard Earnshaw
> Sent: Wednesday, April 16, 2014 6:21 PM
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [patch] Disable if_conversion2 for Og
>
> On 16/04/14 11:17, Joey Ye wrote:
> >> -Original Message
> -Original Message-
> From: Richard Earnshaw
> Sent: Wednesday, April 16, 2014 6:04 PM
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [patch] Disable if_conversion2 for Og
>
> On 16/04/14 11:02, Joey Ye wrote:
> >
> >
> >>
> -Original Message-
> From: Richard Earnshaw
> Sent: Wednesday, April 16, 2014 5:44 PM
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [patch] Disable if_conversion2 for Og
>
> Arguably, this is a bug in gdb. The debugger should understand when a
&g
> -Original Message-
> From: Joey Ye [mailto:joey...@arm.com]
> Sent: Tuesday, April 15, 2014 6:37 PM
> To: 'Richard Biener'
> Cc: GCC Patches
> Subject: RE: [patch] Disable if_conversion2 for Og
>
> > Ok for trunk and branches after a while. Wh
> -Original Message-
> From: Richard Biener [mailto:richard.guent...@gmail.com]
> Sent: Tuesday, April 15, 2014 4:05 PM
> To: Joey Ye
> Cc: GCC Patches
> Subject: Re: [patch] Disable if_conversion2 for Og
>
> On Tue, Apr 15, 2014 at 3:59 AM, Joey Ye wrote:
>
If-converstion is harmful to optimized debugging as it generates conditional
execution instructions with line number information, which resulted in a
dillusion to developers that both then-else branches are executed.
For example:
test.c:
1: unsigned oldest_sequence;
2:
3: unsigned foo(unsigned seq
Ian, thanks for your comments. Please find answers and new version below:
> -Original Message-
> From: Ian Lance Taylor [mailto:i...@google.com]
> Sent: 25 March 2014 21:09
> To: Joey Ye
> Cc: gcc-patches
> Subject: Re: [patch] Shorten Windows path
>
> On Tue,
Ping
> -Original Message-
> From: Joey Ye [mailto:joey...@arm.com]
> Sent: 19 February 2014 15:45
> To: gcc-patches@gcc.gnu.org; Ian Lance Taylor (i...@google.com)
> Subject: [patch] Shorten Windows path
>
> Max length of path on Windows is 255, which is easy to exce
4.8 also has this bug. OK to backport?
On Tue, Mar 11, 2014 at 6:59 PM, Jakub Jelinek wrote:
> On Tue, Mar 11, 2014 at 06:48:37PM +0800, Thomas Preud'homme wrote:
>> I also added a typedef unsigned uint32_t for when sizeof(unsigned) == 4. I
>> hope it's right.
>
> In theory you could have __CHAR
This patch is a mirror copy from approved patch in glibc:
http://sourceware.org/ml/libc-alpha/2014-02/msg00741.html
OK to trunk, 4.8 and 4.7?
ChangeLog.libgcc:
* config/arm/sfp-machine.h (_FP_NANFRAC_H,
_FP_NANFRAC_S, _FP_NANFRAC_D, _FP_NANFRAC_Q):
Set to zero.
diff --git a/libgcc/config/ar
Ping. OK for trunk and 4.8?
> -Original Message-
> From: Joey Ye [mailto:joey...@arm.com]
> Sent: 21 February 2014 19:32
> To: gcc-patches@gcc.gnu.org
> Subject: [patch] [arm] Fix PR60169 - thumb1 far jump
>
> Patch http://gcc.gnu.org/ml/gcc-patches/2012-12/msg
Committed to ARM/embedded-4_8-branch
Still pending to gcc-4_8-branch.
> -Original Message-
> From: Joey Ye [mailto:joey...@arm.com]
> Sent: 27 February 2014 13:53
> To: 'ja...@redhat.com'; 'gcc-patches@gcc.gnu.org'
> Subject: RE: [PATCH][4.8] Backport
Ping ^ 5
> -Original Message-
> From: Joey Ye [mailto:joey...@arm.com]
> Sent: 19 February 2014 17:22
> To: 'ja...@redhat.com'; gcc-patches@gcc.gnu.org
> Subject: RE: [PATCH][4.8] Backport strict-volatile-bitfields fixes to 4.8
>
> Ping ^ 4
>
>
OK to trunk and 4.8?
-Original Message-
From: Joey Ye [mailto:joey...@arm.com]
Sent: 2014年2月21日 19:32
To: gcc-patches@gcc.gnu.org
Subject: [patch] [arm] Fix PR60169 - thumb1 far jump
Patch http://gcc.gnu.org/ml/gcc-patches/2012-12/msg01229.html introduced
this ICE:
1. thumb1 estimate
Patch http://gcc.gnu.org/ml/gcc-patches/2012-12/msg01229.html introduced
this ICE:
1. thumb1 estimate if far_jump is used based on function insn size
2. During reload, after stack layout finalized, it does reload_as_needed. It
however increases insn size that changes estimation result of far_jump,
Ping ^ 4
> -Original Message-
> From: Joey Ye [mailto:joey...@arm.com]
> Sent: Friday, February 14, 2014 9:58
> To: gcc-patches@gcc.gnu.org
> Subject: RE: [PATCH][4.8] Backport strict-volatile-bitfields fixes to 4.8
>
> Ping ^3
>
> These fixes are very im
Max length of path on Windows is 255, which is easy to exceed in a
complicated project. Ultimate solution may be complex but canonizing the
path and skipping the ".."s in path is helpful.
Relative discussion in gcc-patches:
http://gcc.gnu.org/ml/gcc-patches/2013-11/msg00582.html
OK to trunk stage
Ping ^3
These fixes are very important to 4.8 ARM embedded users, as they rely on
strict volatile bitfields a lot. Please let them in 4.8.
> -Original Message-
> From: Joey Ye [mailto:joey...@arm.com]
> Sent: Saturday, February 08, 2014 10:42
> To: gcc-patches@gcc.gnu.org
&
Ping ^ 2
OK to 4.8?
> -Original Message-
> From: Joey Ye [mailto:joey...@arm.com]
> Sent: Monday, January 20, 2014 10:47
> To: gcc-patches@gcc.gnu.org
> Subject: RE: [PATCH][4.8] Backport strict-volatile-bitfields fixes to 4.8
>
> Ping
>
> > -Original
Ping
> -Original Message-
> From: Joey Ye [mailto:joey...@arm.com]
> Sent: Thursday, January 16, 2014 16:28
> To: gcc-patches@gcc.gnu.org
> Subject: [PATCH][4.8] Backport strict-volatile-bitfields fixes to 4.8
>
> 4.8 has a number of strict-volatile-bitfields issues
ChangeLog is messed up with other one.
On Thu, Jan 16, 2014 at 3:33 PM, Terry Guo wrote:
> Hi,
>
> Current 4.8 branch will assign alu_reg attribute to the type of arm preload
> insn, which is clearly wrong. The attached patch intends to back port trunk
> patch to reclassify the type attribute as
4.8 has a number of strict-volatile-bitfields issues that can be fixed by
following patches.
trunk@205899, 205898, 205897, 205896, 203003
Tested on x86_64 and arm without regression.
OK to 4.8?
2013-09-28 Sandra Loosemore
gcc/
* expr.h (extract_bit_field): Remove packedp par
Fix trunk plugin build by adding missing headers and remove headers no
longer exist.
Test passed:
- arm-none-eabi build --enable-plugins
- build test plugin
- x86_64 bootstrap --enable-plugins
OK to trunk?
ChangeLog.gcc
2013-11-19 Joey Ye
PR plugin/59335
Update plugin document after switching to C++, also make it more friendly to
cross-build.
ChangeLog:
2014-01-08 Joey Ye
doc/plugin.texi (Building GCC plugins): Update to C++.
OK to trunk?
diff --git a/gcc/doc/plugins.texi b/gcc/doc/plugins.texi
index fc2d754..e668de6 100644
--- a/gcc
Terry, this is a bug fix to pic register. I feel it should also be in
gcc-4_8-branch.
- Joey
On Wed, Nov 27, 2013 at 11:45 AM, Terry Guo wrote:
> Hi,
>
> This patch back ported trunk fix at r205391 to arm/embedded-4_8-branch.
>
> BR,
> Terry
>
> gcc/ChangeLog.arm
> 2013-11-27 Terry Guo
>
>
Ping, as wasting 8 bytes of RAM isn't ignorable on embedded system.
OK to trunk stage 1?
> -Original Message-
> From: Tom Tromey [mailto:tro...@redhat.com]
> Sent: Thursday, October 10, 2013 21:32
> To: Jakub Jelinek
> Cc: Joey Ye; p...@bothner.com; a...@redhat.com;
Sorry about this. I should have run x86 make check.
- Joey
> -Original Message-
> From: Richard Biener [mailto:richard.guent...@gmail.com]
> Sent: Thursday, November 14, 2013 22:16
> To: H.J. Lu
> Cc: Joey Ye; Janis Johnson; GCC Patches; Ramana Radhakrishnan
> Subject
In mainline and arm/embedded-4_8-branch now.
> -Original Message-
> From: Janis Johnson [mailto:janis_john...@mentor.com]
> Sent: Thursday, November 14, 2013 1:45
> To: Joey Ye; jani...@codesourcery.com
> Cc: gcc-patches@gcc.gnu.org; Ramana Radhakrishnan
> Subject: Re:
> -Original Message-
> From: Richard Earnshaw
> Sent: Thursday, November 14, 2013 18:00
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [patch] [arm] New option for PIC offset unfixed
>
> On 14/11/13 08:23, Joey Ye wrote:
> >> -Origina
> -Original Message-
> From: Richard Earnshaw
> Sent: Thursday, November 14, 2013 0:57
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [patch] [arm] New option for PIC offset unfixed
>
> > So you are suggesting change like th
This patch address all comments.
Thanks,
Joey
> -Original Message-
> From: Richard Earnshaw
> Sent: Wednesday, November 13, 2013 19:07
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [patch] [arm] New option for PIC offset unfixed
>
> On 13/11
> -Original Message-
> From: Richard Earnshaw
> Sent: Wednesday, November 13, 2013 17:49
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [patch] [arm] New option for PIC offset unfixed
>
> On 13/11/13 06:18, Joey Ye wrote:
> >> -Origina
Janis, can you please take a look at test case changes.
Thanks,
Joey
> -Original Message-
> From: Ramana Radhakrishnan
> Sent: Friday, November 08, 2013 17:11
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org; jani...@codesourcery.com
> Subject: Re: [patch] [arm] ARM C
> -Original Message-
> From: Richard Earnshaw
> Sent: Tuesday, November 12, 2013 18:49
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [patch] [arm] New option for PIC offset unfixed
>
> The name of the option and the documentation highlights that t
keep its behavior unchanged.
Tested with arm-none-eabi make and VxWorks RTP small case
OK to trunk?
ChangeLog:
2013-11-12 Joey Ye
* config/arm/arm.c (arm_option_override): Error if
-mpic-offset-unfixed without -fpic, and set for
VxWorks RTP.
(legitimize_pic_ad
Ping
> -Original Message-
> From: Joey Ye [mailto:joey...@arm.com]
> Sent: Friday, November 01, 2013 1:00
> To: gcc-patches@gcc.gnu.org
> Subject: [patch] [arm] ARM Cortex-M3/M4 tuning
>
> Based on Julian's http://gcc.gnu.org/ml/gcc-patches/2012-07/msg01006.htm
- warning_at (DECL_SOURCE_LOCATION (p),
- OPT_Wunused_but_set_variable,
- "variable %qD set but not used", p);
+ {
+if (!TREE_THIS_VOLATILE (p))
+ warning_at (DECL_SOURCE_LOCATION (p),
+OPT_Wunused_but_set_variable,
+"variable %qD set but not used", p);
+ }
I'd prefe
Ping
ChangeLog
2013-10-27 Vladimir Simonov
(include)
filename.h (FILENAME_NORMALIZE): New macro.
(filename_normalize): New declare.
(libiberty)
filename_cmp.c (memmove_left): New function.
(filename_normalize): Likewise.
getpwd.c (getpwd): Use FILENAME_NOR
+ "TARGET_ARM || TARGET_THUMB2"
TARGET_32BIT
+static const char *const ite = "it\t%d4";
+static const int cmp_idx[9] = {0, 0, 1, 0, 1};
s/9/5/
On Wed, Oct 30, 2013 at 5:32 PM, Zhenqiang Chen wrote:
>
>> -Original Message-
>> From: Richard Henderson [mailto:r...@redhat.com]
>> Se
p-4.c: Likewise.
ChangeLog:
2013-11-01 Julian Brown
Joey Ye
* config/arm/arm.c (arm_cortex_m_branch_cost): New.
(arm_v7m_tune): New.
(arm_*_tune): Add comments for Sched adj cost.
* config/arm/arm-cores.def (cortex-m4, cortex-m3):
Use arm_v7m_tune tuning.
test
Vladimir,
I found no more issue on patch itself. But ChangeLogs are missing.
Please refer to format in /include/ChangeLog, and generate yours in
your next email for
/include
/libcpp
/libiberty
Maintainers of libiberty are in TO list.
- Joey
On Thu, Oct 17, 2013 at 4:18 PM, Vladimir Simonov
wrote:
>
> Thank you for pointing this problem.
>
> So, on file systems with symlinks support "playing" with filenames as strings
> is impossible.
> This means that filename_normalize name is too pretentious - it will do
> nothing for most of gcc
On Wed, Oct 16, 2013 at 6:45 PM, Vladimir Simonov
wrote:
> There are many pro and contras, i.e. it adds additional, probably unnecessary
> work on Linux
> time but makes filenames shorter, it affects libiberty which is shared
> between gcc/binutils/gdb,
> but it may be useful in other packages,
Thanks for contribution. See review comments at following.
On Wed, Oct 16, 2013 at 6:45 PM, Vladimir Simonov
wrote:
> Hi,
>
> Resending filename-normalize patch to correct list gcc-patches@gcc.gnu.org.
> All context, please, see below.
>
> +extern void filename_normalize (char *f);
> +#define FI
> -Original Message-
> From: Jakub Jelinek [mailto:ja...@redhat.com]
> Sent: Thursday, October 10, 2013 16:48
> To: Joey Ye
> Cc: p...@bothner.com; a...@redhat.com; Tom Tromey; H.J. Lu; gcc-patches;
> 'Ian Lance Taylor'
> Subject: Re: [PATCH, libgcc] Disab
Dear Java maintainers, are you OK with this patch?
- Joey
> -Original Message-
> From: Ian Lance Taylor [mailto:i...@google.com]
> Sent: Thursday, September 12, 2013 3:28
> To: Joey Ye
> Cc: gcc-patches; H.J. Lu; p...@bothner.com; a...@redhat.com; Tom Tromey
> Subject:
Updated to http://gcc.gnu.org/ml/gcc-patches/2012-12/msg01097.html
Build passes on arm-none-eabi and bootstrap passes on x86.
OK to trunk?
ChangeLog
* libgcc/Makefile.in: Include JAVA_IS_ENABLED in CFLAGS.
* libgcc/configure.ac (java_is_enabled): New variable.
* libgcc/configur
OK for embedded 4.8 branch
- Joey
> -Original Message-
> From: Terry Guo
> Sent: Tuesday, August 06, 2013 14:17
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: [arm-embedded] Request to backport thumb1 far jump patch to
> embedded 4.8 branch
>
> Hell
OK to embedded 4.8 branch.
> -Original Message-
> From: Terry Guo
> Sent: Tuesday, August 06, 2013 11:59
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: [arm-embedded] Request to back port Cortex-R7 option support
> patch
>
> Hi Joey,
>
> Attach
OK
- Joey
> -Original Message-
> From: Terry Guo
> Sent: Wednesday, July 24, 2013 16:15
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: [arm-embedded] Patch to define multilibs for arm embedded-4_8-
> branch
>
> Hi Joey,
>
> This patch is to defi
Committed as 199680
> -Original Message-
> From: Ramana Radhakrishnan
> Sent: Friday, May 10, 2013 18:13
> To: Matthew Gretton-Dann
> Cc: gcc-patches@gcc.gnu.org; Richard Earnshaw; d...@canonical.com; Patch
> Tracking; Richard Biener; Joey Ye
> Subject: Re: [RFA/ARM/4.7] Fix PR54974: Thum
> -Original Message-
> From: Ramana Radhakrishnan
> Sent: Thursday, April 11, 2013 4:40 PM
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH][ARM][thumb1] Reduce lr save for leaf function with
> non-far jump
>
> On 12/20/12 09:53, Joey Ye wr
Ping ^ 2
> -Original Message-
> From: Joey Ye
> Sent: Saturday, January 05, 2013 3:41 PM
> To: Ramana Radhakrishnan
> Cc: Joey Ye; gcc-patches@gcc.gnu.org
> Subject: RE: [PATCH][ARM][thumb1] Reduce lr save for leaf function with
> non-far jump
>
> Ping
&
Committed as r196535.
> -Original Message-
> From: Georg-Johann Lay [mailto:g...@gcc.gnu.org]
> Sent: Monday, March 04, 2013 02:42
> To: Joey Ye
> Cc: 'Joseph Myers'; gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH] Fix PR50293 - LTO plugin with space in path
>
> Joey Ye schr
> -Original Message-
> From: Joseph Myers [mailto:jos...@codesourcery.com]
> Sent: Monday, March 04, 2013 00:49
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: RE: [PATCH] Fix PR50293 - LTO plugin with space in path
>
> On Mon, 18 Feb 2013, Joey Ye wr
Ping
> -Original Message-
> From: Joey Ye [mailto:joey...@arm.com]
> Sent: Monday, February 18, 2013 11:32
> To: 'Joseph Myers'
> Cc: gcc-patches@gcc.gnu.org
> Subject: RE: [PATCH] Fix PR50293 - LTO plugin with space in path
>
> Joseph, Thanks for your v
Committed as r196116.
Joseph, Thanks for your valuable comments. See my reply and new patch below.
> -Original Message-
> From: Joseph Myers [mailto:jos...@codesourcery.com]
> Sent: Monday, February 18, 2013 06:16
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH] Fix PR50293
Mainline and 4.7 failed to use LTO when toolchain is installed to a path
with space in it. Resulting in error message like:
/ld.exe: c:/program: error loading plugin
collect2.exe: error: ld returned 1 exit status
Root cause is when GCC driver process specs, it doesn't handle plugin file
name
> -Original Message-
> From: Janis Johnson [mailto:janis_john...@mentor.com]
> Sent: Thursday, January 17, 2013 10:41
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH] [testsuite] [arm] Test thumb1 far jump
>
> On 01/16/2013 06:05 PM, Joey Ye w
Test cases for previous patch "no lr save for non-far branches in leaf
function".
* gcc.target/arm/thumb1-far-jump-1.c: New.
* gcc.target/arm/thumb1-far-jump-2.c: New.
Index: gcc/testsuite/gcc.target/arm/thumb1-far-jump-2.c
=
Ping^2
> -Original Message-
> From: Joey Ye
> Sent: Saturday, January 05, 2013 15:41
> To: Ramana Radhakrishnan
> Cc: Joey Ye; gcc-patches@gcc.gnu.org
> Subject: RE: [PATCH][ARM][thumb1] Reduce lr save for leaf function with
> non-far jump
>
> Ping
&
Ping
> -Original Message-
> From: Joey Ye
> Sent: Thursday, December 20, 2012 17:53
> To: gcc-patches@gcc.gnu.org; Ramana Radhakrishnan
> Cc: Joey Ye
> Subject: [PATCH][ARM][thumb1] Reduce lr save for leaf function with non-
> far jump
>
> Current GCC thumb1 h
e of them
persuaded myself.
Tests passed:
* build libgcc, libstdc++, newlib, libm
* make check-gcc with cpu=cortex-m0
* Small and extreme test cases
ChangeLog:
2012-12-20 Joey Ye
* config/arm/arm.c(thumb1_final_prescan_insn):
Assert lr save for real far jump.
(thumb
Current GCC thumb1 has an annoying problem that always assuming far branch.
So it forces to save lr, even when unnecessarily. The most extreme case
complained by partner is:
// compiled with "-mthumb -mcpu=cortex-m0 -Os".
void foo() { for (;;); }
=>
foo:
push{lr} // Crazy!!!
.L2:
> -Original Message-
> From: H.J. Lu [mailto:hjl.to...@gmail.com]
> Sent: Tuesday, December 18, 2012 12:10
> To: Joseph Prostko
> Cc: Joey Ye; gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH, libgcc] Make possible to disable JCR in crtstuff.c
>
> On Mon, Dec 17,
> -Original Message-
> From: H.J. Lu [mailto:hjl.to...@gmail.com]
> Sent: Saturday, December 15, 2012 01:20
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org; Joseph Prostko
> Subject: Re: [PATCH, libgcc] Make possible to disable JCR in crtstuff.c
> Can't you do
>
> -Original Message-
> From: H.J. Lu [mailto:hjl.to...@gmail.com]
> Sent: Friday, December 14, 2012 11:55
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org; Joseph Prostko
> Subject: Re: [PATCH, libgcc] Make possible to disable JCR in crtstuff.c
>
&g
> -Original Message-
> From: H.J. Lu [mailto:hjl.to...@gmail.com]
> Sent: Tuesday, November 27, 2012 12:56
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH, libgcc] Make possible to disable JCR in crtstuff.c
> >> >
> >> > OK t
Ping^2
> -Original Message-
> From: Joey Ye
> Sent: Tuesday, November 20, 2012 10:09
> To: gcc-patches@gcc.gnu.org
> Cc: Joey Ye
> Subject: RE: [PATCH, libgcc] Make possible to disable JCR in crtstuff.c
>
> Ping, as Joseph Prostko is saying that this patch shall
Ping, as Joseph Prostko is saying that this patch shall solve the same
problem he's facing.
> -Original Message-
> From: Joey Ye [mailto:joey...@arm.com]
> Sent: Friday, September 21, 2012 15:42
> To: gcc-patches@gcc.gnu.org
> Subject: [PATCH, libgcc] Make possib
1 - 100 of 140 matches
Mail list logo