Re: [PATCH][ARM] Enable code hoisting with -Os (PR80155)

2019-10-10 Thread Joey Ye
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

[committed] Create ARM/Arm-9-branch

2019-10-03 Thread Joey Ye
Branch created for Arm GCC releases. - Joey

Re: [PATCH][OBVIOUS] Close file on return from verify-intermediate

2018-09-13 Thread Joey Ye
: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-interm

[PATCH][OBVIOUS] Close file on return from verify-intermediate

2018-09-05 Thread Joey Ye
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

Re: [PATCH AArch64]Fix test failure for pr84682-2.c

2018-08-30 Thread Joey Ye
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 version

Re: [PATCH AArch64]Fix test failure for pr84682-2.c

2018-08-30 Thread Joey Ye
6d67 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: > > > > Joey Ye

Re: [PATCH AArch64]Fix test failure for pr84682-2.c

2018-08-29 Thread Joey Ye
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, > > >> > >

RE: [ARM] mno-pic-data-is-text-relative & msingle-pic-base

2016-08-05 Thread Joey Ye
adhakrishnan; 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/2016-05/msg0

RE: [ARM] no-data-is-text-relative & msingle-pic-base

2016-07-12 Thread Joey Ye
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 someone us

RE: [ARM] mno-pic-data-is-text-relative & msingle-pic-base

2016-05-11 Thread Joey Ye
> -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,

RE: [ARM] mno-pic-data-is-text-relative & msingle-pic-base

2016-05-09 Thread Joey Ye
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-is-text-relati

[patch] [testsuite, arm] Missing test case for thumb2 pop single

2015-07-24 Thread Joey Ye
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 joey...@arm.com * gcc.target/arm/thumb2-pop-single.c: New

Re: [patch, arm] Minor optimization on thumb2 tail call

2015-01-12 Thread Joey Ye
Ping On Wed, Nov 19, 2014 at 10:43 AM, Joey Ye joey...@arm.com 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 -mcpu=cortex-m3

Re: [PATCH, ARM] Fix PR63718, Thumb1 bootstrap -- disable fuse-caller-save for Thumb1

2014-11-27 Thread Joey Ye
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 tom_devr...@mentor.com wrote: Richard, This patch fixes PR63718, which currently

Re: [PATCH, ARM, libgcc] New aeabi_idiv function for armv6-m

2014-11-26 Thread Joey Ye
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 hale.w...@arm.com wrote: Hi, This patch ports the aeabi_idiv routine from Linaro Cortex-Strings

[patch, arm] Minor optimization on thumb2 tail call

2014-11-18 Thread Joey Ye
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

Re: [PATCH, trivial][AArch64] Fix mode iterator for *aarch64_simd_ld1rmode pattern

2014-11-13 Thread Joey Ye
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) felix.y...@huawei.com wrote: Hi, We find that the VALLDI mode iterator used in *aarch64_simd_ld1rmode pattern is not appropriate.

Re: 答复: [PATCH, trivial][AArch64] Fix mode iterator for *aarch64_simd_ld1rmode pattern

2014-11-13 Thread Joey Ye
Can a new case be rewritten then? - Joey On Fri, Nov 14, 2014 at 9:32 AM, Yangfei (Felix) felix.y...@huawei.com 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

Re: [PATCH 3/5] IPA ICF pass

2014-11-05 Thread Joey Ye
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

Re: [PATCH, PR61605, 2/2] Use fuse-caller-save info in pass_cprop_hardreg

2014-11-03 Thread Joey Ye
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 l...@redhat.com wrote: On 10/16/14 03:14, Tom de Vries wrote: Eric, this patch is the second half of the fix

[patch][plugin] Fix PR59335 - missing plugin headers again

2014-09-04 Thread Joey Ye
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

Re: [PATCH] Don't set dir prefix twice (PR middle-end/60484)

2014-08-18 Thread Joey Ye
OK to 4.8 then? On Thu, Aug 14, 2014 at 6:36 PM, Richard Biener richard.guent...@gmail.com wrote: On Thu, Aug 14, 2014 at 7:34 AM, Joey Ye joey.ye...@gmail.com 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

Re: [PATCH] Don't set dir prefix twice (PR middle-end/60484)

2014-08-13 Thread Joey Ye
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 jos...@codesourcery.com wrote: On Fri, 14 Mar 2014, Marek Polacek wrote: This patch makes sure that we set the

Re: [PATCH, PR61219]: Fix sNaN handling in ARM float to double conversion

2014-05-19 Thread Joey Ye
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 aurel...@aurel32.net wrote: On ARM soft-float, the float to double conversion doesn't convert a sNaN to qNaN as the IEEE Std 754 standard

RE: [patch] Shorten Windows path

2014-04-25 Thread Joey Ye
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 find answers and new version below: -Original

RE: [patch, testsuite] Fix fragile case nsdmi-union5

2014-04-24 Thread Joey Ye
-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 joey...@arm.com wrote: Resulting

[patch, testsuite] Fix fragile case nsdmi-union5

2014-04-17 Thread Joey Ye
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 === ---

RE: [patch] Disable if_conversion2 for Og

2014-04-16 Thread Joey Ye
-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. Why does if-conversion not have the same problem

RE: [patch] Disable if_conversion2 for Og

2014-04-16 Thread Joey Ye
-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 breakpointed conditional

RE: [patch] Disable if_conversion2 for Og

2014-04-16 Thread Joey Ye
-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

RE: [patch] Disable if_conversion2 for Og

2014-04-16 Thread Joey Ye
-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- From: Richard Earnshaw Sent: Wednesday

RE: [patch] Disable if_conversion2 for Og

2014-04-15 Thread Joey Ye
-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 joey...@arm.com wrote: If-converstion

[patch] Disable if_conversion2 for Og

2014-04-14 Thread Joey Ye
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

RE: [patch] Shorten Windows path

2014-04-01 Thread Joey Ye
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, Mar 25, 2014 at 1:58 AM, Joey Ye

RE: [patch] Shorten Windows path

2014-03-25 Thread Joey Ye
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 exceed in a complicated project

Re: [PATCH] Fix incorrect byte swap detection (PR tree-optimization/60454)

2014-03-11 Thread Joey Ye
4.8 also has this bug. OK to backport? On Tue, Mar 11, 2014 at 6:59 PM, Jakub Jelinek ja...@redhat.com 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

RE: [patch] [arm] Fix PR60169 - thumb1 far jump

2014-02-27 Thread Joey Ye
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/msg01229.html introduced this ICE

[PATCH] [libgcc,arm] Fix PR 60166 - NAN fraction bits

2014-02-27 Thread Joey Ye
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

RE: [PATCH][4.8] Backport strict-volatile-bitfields fixes to 4.8

2014-02-26 Thread Joey Ye
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 -Original Message- From: Joey Ye

RE: [PATCH][4.8] Backport strict-volatile-bitfields fixes to 4.8

2014-02-26 Thread Joey Ye
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 strict-volatile-bitfields fixes to 4.8

[patch] [arm] Fix PR60169 - thumb1 far jump

2014-02-21 Thread Joey Ye
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

RE: [patch] [arm] Fix PR60169 - thumb1 far jump

2014-02-21 Thread Joey Ye
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

RE: [PATCH][4.8] Backport strict-volatile-bitfields fixes to 4.8

2014-02-19 Thread Joey Ye
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 important to 4.8 ARM embedded users

[patch] Shorten Windows path

2014-02-18 Thread Joey Ye
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

RE: [PATCH][4.8] Backport strict-volatile-bitfields fixes to 4.8

2014-02-13 Thread Joey Ye
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 Subject: RE

RE: [PATCH][4.8] Backport strict-volatile-bitfields fixes to 4.8

2014-02-07 Thread Joey Ye
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 Message- From: Joey Ye

RE: [PATCH][4.8] Backport strict-volatile-bitfields fixes to 4.8

2014-01-19 Thread Joey Ye
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 that can be fixed

[PATCH][4.8] Backport strict-volatile-bitfields fixes to 4.8

2014-01-16 Thread Joey Ye
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 san...@codesourcery.com gcc/ * expr.h

Re: [GCC, ARM] Backport trunk patch to 4.8 to reclassify ARM preload insn

2014-01-16 Thread Joey Ye
ChangeLog is messed up with other one. On Thu, Jan 16, 2014 at 3:33 PM, Terry Guo terry@arm.com 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

[PATCH] [doc] Update plugin doc

2014-01-08 Thread Joey Ye
Update plugin document after switching to C++, also make it more friendly to cross-build. ChangeLog: 2014-01-08 Joey Ye joey...@arm.com 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

[patch] [plugin] Fix PR 59335 plugin build

2014-01-08 Thread Joey Ye
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 joey...@arm.com PR plugin/59335

Re: [arm-embedded] Backport trunk thumb1 pic fix to embedded-4_8-branch

2013-11-27 Thread Joey Ye
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 terry@arm.com 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

RE: [PATCH, libgcc] Disable JCR section when java is not enabled

2013-11-18 Thread Joey Ye
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; H.J. Lu; gcc-patches; 'Ian

RE: [patch] [arm] ARM Cortex-M3/M4 tuning

2013-11-17 Thread Joey Ye
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: Re: [patch] [arm] ARM

RE: [patch] [arm] New option for PIC offset unfixed

2013-11-14 Thread Joey Ye
-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 this: + Target Report Var(arm_pic_data_is_text_relative) Init

RE: [patch] [arm] New option for PIC offset unfixed

2013-11-14 Thread Joey Ye
-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: -Original Message- From: Richard Earnshaw Sent

RE: [patch] [arm] ARM Cortex-M3/M4 tuning

2013-11-14 Thread Joey Ye
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: [patch] [arm] ARM Cortex

RE: [patch] [arm] New option for PIC offset unfixed

2013-11-13 Thread Joey Ye
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/13 10:20, Joey Ye wrote: +@item -mpic

RE: [patch] [arm] New option for PIC offset unfixed

2013-11-12 Thread Joey Ye
-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 the option's concept is confusing

RE: [patch] [arm] ARM Cortex-M3/M4 tuning

2013-11-12 Thread Joey Ye
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 Cortex-M3/M4 tuning

[patch] [arm] New option for PIC offset unfixed

2013-11-11 Thread Joey Ye
its behavior unchanged. Tested with arm-none-eabi make and VxWorks RTP small case OK to trunk? ChangeLog: 2013-11-12 Joey Ye joey...@arm.com * config/arm/arm.c (arm_option_override): Error if -mpic-offset-unfixed without -fpic, and set for VxWorks RTP

Re: [C] Fix PR57258: unused variable warning is emitted for volatile variables

2013-11-07 Thread Joey Ye
- 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 prefer

RE: [patch] [arm] ARM Cortex-M3/M4 tuning

2013-11-07 Thread Joey Ye
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.html and * Merged with latest

[PATCH] [libiberty] MAX_PATH problems with mingw gcc

2013-11-05 Thread Joey Ye
Ping ChangeLog 2013-10-27 Vladimir Simonov vladimir.simo...@acronis.com (include) filename.h (FILENAME_NORMALIZE): New macro. (filename_normalize): New declare. (libiberty) filename_cmp.c (memmove_left): New function. (filename_normalize): Likewise.

Re: [PATCH 1/n] Add conditional compare support

2013-11-04 Thread Joey Ye
+ 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 zhenqiang.c...@arm.com wrote: -Original Message- From: Richard Henderson

[patch] [arm] ARM Cortex-M3/M4 tuning

2013-10-31 Thread Joey Ye
: Likewise. ChangeLog: 2013-11-01 Julian Brown jul...@codesourcery.com Joey Ye joey...@arm.com * 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

Re: FW: MAX_PATH problems with mingw gcc

2013-10-20 Thread Joey Ye
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

Re: FW: MAX_PATH problems with mingw gcc

2013-10-17 Thread Joey Ye
On Thu, Oct 17, 2013 at 4:18 PM, Vladimir Simonov vladimir.simo...@acronis.com 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

Re: FW: MAX_PATH problems with mingw gcc

2013-10-16 Thread Joey Ye
Thanks for contribution. See review comments at following. On Wed, Oct 16, 2013 at 6:45 PM, Vladimir Simonov vladimir.simo...@acronis.com wrote: Hi, Resending filename-normalize patch to correct list gcc-patches@gcc.gnu.org. All context, please, see below. +extern void filename_normalize

Re: FW: MAX_PATH problems with mingw gcc

2013-10-16 Thread Joey Ye
On Wed, Oct 16, 2013 at 6:45 PM, Vladimir Simonov vladimir.simo...@acronis.com 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

RE: [PATCH, libgcc] Disable JCR section when java is not enabled

2013-10-12 Thread Joey Ye
-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] Disable JCR section when java is not enabled

RE: [PATCH, libgcc] Disable JCR section when java is not enabled

2013-10-10 Thread Joey Ye
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: Re: [PATCH, libgcc

[PATCH, libgcc] Disable JCR section when java is not enabled

2013-09-10 Thread Joey Ye
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. *

RE: [arm-embedded] Request to backport thumb1 far jump patch to embedded 4.8 branch

2013-08-06 Thread Joey Ye
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 Hello Joey, The thumb1 far jump patch

RE: [arm-embedded] Request to back port Cortex-R7 option support patch

2013-08-05 Thread Joey Ye
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, Attached patch is a backport to support cortex-r7

RE: [arm-embedded] Patch to define multilibs for arm embedded-4_8-branch

2013-07-30 Thread Joey Ye
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 define multilibs for recently created embedded

[arm/embedded-4_7-branch] Merge with gcc-4_7-branch 199638

2013-06-04 Thread Joey Ye
Committed as 199680

RE: [RFA/ARM/4.7] Fix PR54974: Thumb literal pools don't handle PC rounding

2013-05-10 Thread Joey Ye
-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: Thumb literal pools don't handle

RE: [PATCH][ARM][thumb1] Reduce lr save for leaf function with non-far jump

2013-04-15 Thread Joey Ye
-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 wrote: Current GCC thumb1 has

RE: [PATCH][ARM][thumb1] Reduce lr save for leaf function with non-far jump

2013-04-11 Thread Joey Ye
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 -Original Message- From: Joey

[arm/embedded-4_7-branch] Merge with gcc-4_7-branch r196534

2013-03-08 Thread Joey Ye
Committed as r196535.

RE: [PATCH] Fix PR50293 - LTO plugin with space in path

2013-03-04 Thread Joey Ye
-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 wrote: +static char

RE: [PATCH] Fix PR50293 - LTO plugin with space in path

2013-03-04 Thread Joey Ye
-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 schrieb: Ping Subject: RE: [PATCH] Fix

RE: [PATCH] Fix PR50293 - LTO plugin with space in path

2013-02-24 Thread Joey Ye
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 valuable comments. See my reply and new

[arm/embedded-4_7-branch] Merge with gcc-4_7-branch r196107

2013-02-18 Thread Joey Ye
Committed as r196116.

RE: [PATCH] Fix PR50293 - LTO plugin with space in path

2013-02-17 Thread Joey Ye
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 - LTO plugin with space

[PATCH] Fix PR50293 - LTO plugin with space in path

2013-02-16 Thread Joey Ye
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

RE: [PATCH] [testsuite] [arm] Test thumb1 far jump

2013-01-16 Thread Joey Ye
-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 wrote: Test cases for previous

RE: [PATCH][ARM][thumb1] Reduce lr save for leaf function with non-far jump

2013-01-15 Thread Joey Ye
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 -Original Message- From: Joey Ye

RE: [PATCH][ARM][thumb1] Reduce lr save for leaf function with non-far jump

2013-01-04 Thread Joey Ye
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 has an annoying problem that always

[PATCH][ARM][thumb1] Reduce lr save for leaf function with non-far jump

2012-12-20 Thread Joey Ye
: * build libgcc, libstdc++, newlib, libm * make check-gcc with cpu=cortex-m0 * Small and extreme test cases ChangeLog: 2012-12-20 Joey Ye joey...@arm.com * config/arm/arm.c(thumb1_final_prescan_insn): Assert lr save for real far jump. (thumb_far_jump_used_p): Count

[PATCH][ARM][thumb1] Reduce lr save for leaf function with non-far jump

2012-12-19 Thread Joey Ye
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:

RE: [PATCH, libgcc] Make possible to disable JCR in crtstuff.c

2012-12-18 Thread Joey Ye
-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, 2012 at 5:23 PM, Joseph Prostko

RE: [PATCH, libgcc] Make possible to disable JCR in crtstuff.c

2012-12-17 Thread Joey Ye
-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 +# Disable jcr section if we're

RE: [PATCH, libgcc] Make possible to disable JCR in crtstuff.c

2012-12-13 Thread Joey Ye
-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 to trunk? 2012-09-21 Joey Ye joey...@arm.com

RE: [PATCH, libgcc] Make possible to disable JCR in crtstuff.c

2012-12-13 Thread Joey Ye
-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 2012-12-12 Joey Ye joey...@arm.com

RE: [PATCH, libgcc] Make possible to disable JCR in crtstuff.c

2012-11-26 Thread Joey Ye
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 solve the same problem he's

RE: [PATCH, libgcc] Make possible to disable JCR in crtstuff.c

2012-11-19 Thread Joey Ye
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 possible to disable JCR in crtstuff.c

[PATCH 4.7] Backport Don't pull in unwinder for 64-bit division routines

2012-09-27 Thread Joey Ye
OK backporting following patches to 4.7? http://gcc.gnu.org/ml/gcc-patches/2012-07/msg01193.html 2012-08-17 Julian Brown jul...@codesourcery.com * Makefile.in (LIB2_DIVMOD_EXCEPTION_FLAGS): Default to -fexceptions -fnon-call-exceptions if not defined. ($(lib2-divmod-o),

[PATCH, libgcc] Make possible to disable JCR in crtstuff.c

2012-09-21 Thread Joey Ye
TARGET_USE_JCR_SECTION to 0, but it has no crtbegin/end stuff. So also no impact. OK to trunk? 2012-09-21 Joey Ye joey...@arm.com * crtstuff.c: Check TARGET_USE_JCR_SECTION. Index: libgcc/crtstuff.c === --- libgcc/crtstuff.c (revision

  1   2   >