[PATCH] Don't issue array bound warnings on zero-length arrays

2013-08-30 Thread Meador Inge
by bailing out when a zero-length array is encountered. OK for trunk? gcc/ 2013-08-30 Meador Inge * tree-vrp.c (check_array_ref): Bail out no emtpy arrays. gcc/testsuite/ 2013-08-30 Meador Inge * gcc.dg/Warray-bounds-11.c: New testcase. Index: gcc/testsuite/gcc.dg

Re: [PATCH] Don't issue array bound warnings on zero-length arrays

2013-09-03 Thread Meador Inge
On 09/02/2013 04:27 AM, Richard Biener wrote: > On Fri, Aug 30, 2013 at 5:13 PM, Meador Inge wrote: >> Hi All, >> >> This patch fixes a minor issue that can occur when issuing array bounds >> warnings. In GNU C mode we allow empty lists and their upper bound is

Re: [PATCH] Don't issue array bound warnings on zero-length arrays

2013-09-03 Thread Meador Inge
On 09/03/2013 10:45 AM, Jakub Jelinek wrote: > On Tue, Sep 03, 2013 at 10:40:16AM -0500, Meador Inge wrote: >>> And I fail to see why the testcase should >>> not warn. Clearly you have a definition of a here and it doesn't have >>> an element >>> so t

[PATCH] gcc-{ar,nm,ranlib}: Find binutils binaries relative to self

2012-10-04 Thread Meador Inge
copyright dates from collect2.c. 2012-10-04 Meador Inge * collect2.c (main): Call find_file_set_debug. (find_a_find, add_prefix, prefix_from_env, prefix_from_string): Factor out into ... * file-find.c (New file): ... here and ... * file-find.h (New file

Re: [PATCH] gcc-{ar,nm,ranlib}: Find binutils binaries relative to self

2012-10-09 Thread Meador Inge
Ping. On 10/04/2012 03:45 PM, Meador Inge wrote: > Hi All, > > Currently the gcc-{ar,nm,ranlib} utilities assume that binutils is in > path when invoking the wrapped binutils program. This goes against the > accepted practice in GCC to find sub-programs relative to where the >

Re: [PATCH] gcc-{ar,nm,ranlib}: Find binutils binaries relative to self

2012-10-18 Thread Meador Inge
Ping ^ 2. On 10/09/2012 09:44 PM, Meador Inge wrote: > Ping. > > On 10/04/2012 03:45 PM, Meador Inge wrote: >> Hi All, >> >> Currently the gcc-{ar,nm,ranlib} utilities assume that binutils is in >> path when invoking the wrapped binutils program. This goes ag

Re: [PATCH] gcc-{ar,nm,ranlib}: Find binutils binaries relative to self

2012-10-19 Thread Meador Inge
On 10/18/2012 01:33 PM, Bernhard Reutner-Fischer wrote: > On 18 October 2012 17:30:20 Meador Inge wrote: >> Ping ^ 2 > > Been a while but wasn't --with-build-sysroot for exactly this? AFAICT, no. --with-build-sysroot seems to be used for setting a different sysroot to use

Re: [PATCH] gcc-{ar,nm,ranlib}: Find binutils binaries relative to self

2012-10-19 Thread Meador Inge
CC'ing the LTO maintainers. On 10/18/2012 10:30 AM, Meador Inge wrote: > Ping ^ 2. > > On 10/09/2012 09:44 PM, Meador Inge wrote: >> Ping. >> >> On 10/04/2012 03:45 PM, Meador Inge wrote: >>> Hi All, >>> >>> Currently the gcc-{ar,nm

Re: [PATCH] gcc-{ar,nm,ranlib}: Find binutils binaries relative to self

2012-10-29 Thread Meador Inge
Ping ^ 3. On 10/18/2012 10:30 AM, Meador Inge wrote: > Ping ^ 2. > > On 10/09/2012 09:44 PM, Meador Inge wrote: >> Ping. >> >> On 10/04/2012 03:45 PM, Meador Inge wrote: >>> Hi All, >>> >>> Currently the gcc-{ar,nm,ranlib} utilities assume

Re: [patch] Shorten Windows path

2014-06-04 Thread Meador Inge
ptions to produce a canonical path name discussed? Nothing obvious jumped out at me. Despite the limits, using 'GetFullPathName' seems like the natural way to handle it because it knows about all the various Windows file system quirks. -- Meador Inge CodeSourcery / Mentor Embedded

Re: [PATCH] gcc-{ar,nm,ranlib}: Find binutils binaries relative to self

2012-11-07 Thread Meador Inge
Ping ^ 4. On 10/29/2012 10:46 AM, Meador Inge wrote: > Ping ^ 3. > > On 10/18/2012 10:30 AM, Meador Inge wrote: >> Ping ^ 2. >> >> On 10/09/2012 09:44 PM, Meador Inge wrote: >>> Ping. >>> >>> On 10/04/2012 03:45 PM, Meador Inge wrote:

Re: [PATCH] gcc-{ar,nm,ranlib}: Find binutils binaries relative to self

2012-11-27 Thread Meador Inge
On 11/26/2012 09:05 AM, Richard Biener wrote: > On Wed, Nov 7, 2012 at 10:51 PM, Meador Inge wrote: >> Ping ^ 4. > > Ok. Thanks for the review. Committed to trunk. -- Meador Inge CodeSourcery / Mentor Embedded http://www.mentor.com/embedded-software

[PATCH] ARM: Don't clobber CC reg when it is live after the peephole window

2013-05-29 Thread Meador Inge
suite. OK? gcc/ 2013-05-29 Meador Inge * config/arm/arm.md (conditional move peephole2): Only clobber CC register when it is dead after the peephole window. [1] http://gcc.gnu.org/ml/gcc-patches/2010-12/msg01336.html Index: gcc/config/arm/a

Re: [PATCH] ARM: Don't clobber CC reg when it is live after the peephole window

2013-06-05 Thread Meador Inge
Ping. On 05/29/2013 12:15 PM, Meador Inge wrote: > Hi All, > > This patch fixes a bug in one of the ARM peephole2 optimizations. The > peephole2 optimization in question was changed to use the CC-updating > form for all of the instructions produced by the peephole so that the >

Re: [PATCH] ARM: Don't clobber CC reg when it is live after the peephole window

2013-06-06 Thread Meador Inge
gt; > ... but what's this bit about? The original intent was to revert back to the original peephole pattern (pre-PR 46975) when the CC reg is still live, but that doesn't properly maintain the CC state either (it just happened to pass in the test case I was looking at because I only cared about the Z flag, which is maintained the same). OK with the above bit left out? -- Meador Inge CodeSourcery / Mentor Embedded

Re: [PATCH] ARM: Don't clobber CC reg when it is live after the peephole window

2013-06-10 Thread Meador Inge
On 06/06/2013 01:23 PM, Meador Inge wrote: > On 06/06/2013 08:11 AM, Richard Earnshaw wrote: > >> I understand (and agree with) this bit... >> >>> +(define_peephole2 >>> + [(set (reg:CC CC_REGNUM) >>> +(compare:CC (match_operand:SI 0 "regist

[PATCH][ARM][testsuite] Add 'dg-require-effective-target sync_*' to some atomic tests

2013-06-12 Thread Meador Inge
Hi All, This patch adds the needed 'dg-require-effective-target sync_*' lines to some of the atomic tests so that they will not be run if the appropriate atomic support is not available. OK for trunk? 2013-06-12 Meador Inge * gcc.dg/atomic-flag.c: Add dg-require-effect

Re: [PATCH] ARM: Don't clobber CC reg when it is live after the peephole window

2013-06-18 Thread Meador Inge
Ping. On 06/06/2013 01:23 PM, Meador Inge wrote: > On 06/06/2013 08:11 AM, Richard Earnshaw wrote: > >> I understand (and agree with) this bit... >> >>> +(define_peephole2 >>> + [(set (reg:CC CC_REGNUM) >>> +(com

Re: [PATCH][ARM][testsuite] Add 'dg-require-effective-target sync_*' to some atomic tests

2013-06-26 Thread Meador Inge
Ping. On 06/12/2013 11:46 AM, Meador Inge wrote: > Hi All, > > This patch adds the needed 'dg-require-effective-target sync_*' lines to some > of the atomic tests so that they will not be run if the appropriate atomic > support is not available. > > OK for trun

[PATCH] libgcc: Add DWARF info to aeabi_ldivmod and aeabi_uldivmod

2013-03-05 Thread Meador Inge
Hi All, This patch fixes a minor annoyance that causes backtraces to disappear inside of aeabi_ldivmod and aeabi_uldivmod due to the lack of appropriate DWARF information. I fixed the problem by adding the necessary cfi_* macros in these functions. OK? 2013-03-05 Meador Inge

Re: [PATCH] libgcc: Add DWARF info to aeabi_ldivmod and aeabi_uldivmod

2013-03-18 Thread Meador Inge
Ping. On 03/05/2013 12:15 PM, Meador Inge wrote: > Hi All, > > This patch fixes a minor annoyance that causes backtraces to disappear > inside of aeabi_ldivmod and aeabi_uldivmod due to the lack of appropriate > DWARF information. I fixed the problem by adding the necessary cf

[PATCH] Correct handling of gcc-[ar|nm|ranlib] exit codes

2012-09-27 Thread Meador Inge
and 'status' is only set for a successful return. This patch fixes the bug by appropriately checking the returned status and extracting the exit code when needed. Tested on GNU/Linux and Windows. OK? 2012-09-27 Meador Inge * gcc-ar.c (main): Handle the returning of the sub-p

Re: PATCH: Always create a new language function for nested functions

2012-06-17 Thread Meador Inge
On 06/17/2012 09:28 AM, Markus Trippelsdorf wrote: > On 2012.05.29 at 19:07 +, Joseph S. Myers wrote: >> On Tue, 29 May 2012, Meador Inge wrote: >> >>> 2012-05-29 Meador Inge >>> >>> * c-decl.c (c_push_function_context): Alw

[PATCH] PR c/53702: Fix -Wunused-local-typedefs with nested functions

2012-06-20 Thread Meador Inge
tml gcc/ 2012-06-20 Meador Inge PR c/53702 * c-decl.c (c_push_function_context): Restore the behavior to reuse the language function allocated for -Wunused-local-typedefs. (c_pop_function_context): If necessary, clear the language function created in c_p

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

2012-06-21 Thread Meador Inge
Subject says it all ... 2012-06-21 Meador Inge * MAINTAINERS (Write After Approval): Add myself. Index: MAINTAINERS === --- MAINTAINERS (revision 188858) +++ MAINTAINERS (working copy) @@ -392,6 +392,7 @@ Andrew John

[PATCH] PR rtl-optimization/53352

2012-05-15 Thread Meador Inge
essions). OK? (If this is OK, then can someone commit for me. I don't have write access). gcc/ 2012-05-15 Meador Inge PR rtl-optimization/53352 * cse.c (equiv_constant): Ignore paradoxical subregs. gcc/testsuite/ 2012-05-15 Meador Inge PR rtl-optimizatio

Re: [PATCH] PR rtl-optimization/53352

2012-05-16 Thread Meador Inge
I meant to CC the RTL optimization maintainers before ... On 05/15/2012 02:39 PM, Meador Inge wrote: > Hi All, > > As reported in PR rtl-optimization/53352 CSE currently trips up on a > paradoxical subreg case. When compiling for ARM GNU/Linux with -O3 > the expanded RTL of inte

Re: [PATCH] PR rtl-optimization/53352

2012-05-17 Thread Meador Inge
are undefined, so they cannot be said to equal anything. */ Sounds good to me. v2 OK? If so, would you mind committing for me? I don't have write access. -- Meador Inge CodeSourcery / Mentor Embedded http://www.mentor.com/embedded-software Index: testsuite/gcc.dg/pr53352.c ===

Re: [PATCH] PR rtl-optimization/53352

2012-05-18 Thread Meador Inge
On 05/18/2012 09:16 AM, H.J. Lu wrote: > On Thu, May 17, 2012 at 1:46 PM, Meador Inge wrote: >> On 05/17/2012 03:02 PM, Richard Sandiford wrote: >> >>> After agonising over this for a couple of days, I think it's probably >>> the correct fix. What we'

Re: [PATCH] PR rtl-optimization/53352

2012-05-18 Thread Meador Inge
On 05/18/2012 09:16 AM, H.J. Lu wrote: > On Thu, May 17, 2012 at 1:46 PM, Meador Inge wrote: >> On 05/17/2012 03:02 PM, Richard Sandiford wrote: >> >>> After agonising over this for a couple of days, I think it's probably >>> the correct fix. What we'

PATCH: Always create a new language function for nested functions

2012-05-29 Thread Meador Inge
and bootstrapping and running the full test suite for i686-pc-linux-gnu. OK? P.S. If it is OK, then can someone commit for me (I don't have write access)? 2012-05-29 Meador Inge * c-decl.c (c_push_function_context): Always create a new language function. (c_pop_

[PATCH] PR c++/52672

2012-03-26 Thread Meador Inge
tested this and saw no regressions. P.S. If it is OK can some please commit for me? I don't have write access. -- Meador Inge CodeSourcery / Mentor Embedded http://www.mentor.com/embedded-software cp/ 2012-03-26 Meador Inge PR c++/52672 * gcc/cp/semantics.c (cxx_fold_indirect_ref): Don

[PATCH] rs6000: Extend TARGET_NO_LWSYNC to cover more processors

2012-03-27 Thread Meador Inge
1238.html [2] https://www-01.ibm.com/chips/techlib/techlib.nsf/techdocs/852569B20050FF7785256996006F9795/$file/603e_um.pdf [3] http://cache.freescale.com/files/32bit/doc/ref_manual/MPC603EUM.pdf [4] http://gcc.gnu.org/ml/gcc/2008-06/msg00449.html -- Meador Inge CodeSourcery / Mentor Embedded http://www.

Re: [PATCH] rs6000: Extend TARGET_NO_LWSYNC to cover more processors

2012-03-28 Thread Meador Inge
On 03/28/2012 03:59 PM, David Edelsohn wrote: > On Tue, Mar 27, 2012 at 5:21 PM, Meador Inge wrote: >> Hi All, >> >> This patch fixes an issue reported by one of our customers where an >> instruction >> exception gets raised when using '__sync_fetch_and_ad

Re: [PATCH] PR c++/52672

2012-03-29 Thread Meador Inge
On 03/26/2012 09:14 AM, Meador Inge wrote: > Hi All, > > This patch fixes an ICE that occurs when attempting to fold nested > INDIRECT_REF > trees that have conversions in between the indirect references. For example: > > constexpr unsigned long b = *((ul_ptr)(*((ul_