Fix e500 vector ICE with string constants

2012-06-19 Thread Joseph S. Myers
mr %0,%1 {l%U1%X1|lwz%U1%X1} %0,%1") +(define_insn "*mov_si_e500_subreg4_elf_low" + [(set (subreg:SI (match_operand:SPE64TF 0 "register_operand" "+r") 4) + (lo_sum:SI (match_operand:SI 1 "gpc_reg_operand" "r") + (match_operand 2 "" "")))] + "((TARGET_E500_DOUBLE && (mode == DFmode || mode == TFmode)) +|| (TARGET_SPE && mode != DFmode && mode != TFmode)) + && TARGET_ELF && !TARGET_64BIT" + "{ai|addic} %0,%1,%K2") + (define_insn "*mov_si_e500_subreg4_2" [(set (match_operand:SI 0 "rs6000_nonimmediate_operand" "+r,m") (subreg:SI (match_operand:SPE64TF 1 "register_operand" "r,r") 4))] -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH, gdc] - Merging gdc (GNU D Compiler) into gcc

2012-06-19 Thread Joseph S. Myers
On Mon, 18 Jun 2012, Iain Buclaw wrote: > http://www.gdcproject.org/files/gdc_testsuite.patch.gz I have no comments on this patch for now. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH, gdc] - Merging gdc (GNU D Compiler) into gcc

2012-06-19 Thread Joseph S. Myers
27;s not. No runtime libraries should go under gcc/ any more at all. > +dnl Copied from libstdc++-v3/acinclude.m4. Indeed, multilib will not work Refactor into the config/ directory, don't copy. > \ No newline at end of file Add any missing newlines to text files in all patches. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH, gdc] - Merging gdc (GNU D Compiler) into gcc

2012-06-19 Thread Joseph S. Myers
ver for D. > +$(D_DRIVER_NAME)$(exeext): $(D_GCC_OBJS) $(D_DRIVER_OBJS) $(EXTRA_GCC_OBJS) > $(D_EXTRA_SPEC_LIBS) $(LIBDEPS) > + $(CC) $(ALL_CFLAGS) $(LDFLAGS) -o $@ \ I think you mean +$(LINKER) not $(CC). The '+' is I think related to LTO, the use of $(LINKER) is needed to link appropriately with host libstdc++ in the presence of various options for building as C++. I don't see d.pdf or d.html targets in your Make-lang.in - make sure you're up to date with the full current set of hooks front ends should define. > diff -Naur gcc-4.8-20120617/gcc/d/rdmd.1 gcc-4.8/gcc/d/rdmd.1 Same comments as above. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH 4/4] - Merging gdc (GNU D Compiler) into gcc

2012-06-19 Thread Joseph S. Myers
trib.texi in this patch. I'd also expect contrib/gcc_update to be updated to handle timestamp ordering for generated files in libphobos. Are you volunteering to be appointed maintainer for this front end by the SC? -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH, gdc] - Merging gdc (GNU D Compiler) into gcc

2012-06-19 Thread Joseph S. Myers
testsuite.patch.gz > http://www.gdcproject.org/files/gdc_gcc.patch.gz Please provide GNU ChangeLog entries for each patch, for each relevant ChangeLog file. It would be best to post those in plain text to the list, even if the patches themselves are too big. -- Joseph S. Myers jos...@codesourcery.com

Re: [patch] Remove VMS_TARGET

2012-06-18 Thread Joseph S. Myers
me to convert return values from main into whatever form the underlying platform expects, rather than for the compiler to generate something different from what "return 0;" would do. -- Joseph S. Myers jos...@codesourcery.com

Re: [patch] Remove NO_IMPLICIT_EXTERN_C target macro

2012-06-18 Thread Joseph S. Myers
ible. Marc, could you have a > look at this, please? What targets (such as AIX?) did not explicitly undefine it, but did not get a definition from any header they used either? It's quite likely some such targets *should* have had NO_IMPLICIT_EXTERN_C defined, but not necessarily all of them. -- Joseph S. Myers jos...@codesourcery.com

Re: long long availability in host compiler (Re: constant that doesn't fit in 32bits in alpha.c)

2012-06-15 Thread Joseph S. Myers
On Fri, 15 Jun 2012, Mike Stump wrote: > On Jun 15, 2012, at 2:46 PM, Joseph S. Myers wrote: > > HOST_WIDE_INT is an abstraction about the *target*; the target determines > > the required properties. The salient properties include: > > > > * At least as wide as tar

Re: long long availability in host compiler (Re: constant that doesn't fit in 32bits in alpha.c)

2012-06-15 Thread Joseph S. Myers
GCC - that it is an integer or size existing in some way on the target - and code needs to address the human reader, not just the compiler compiling it. -- Joseph S. Myers jos...@codesourcery.com

Re: long long availability in host compiler (Re: constant that doesn't fit in 32bits in alpha.c)

2012-06-15 Thread Joseph S. Myers
t; then HOST_WIDE_INT is what we have at present. If it's "fast on the host, but size doesn't matter", then HOST_WIDEST_FAST_INT. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] Add a new option "-fstack-protector-strong" (issue 6303078)

2012-06-14 Thread Joseph S. Myers
Please resubmit a self-contained patch submission to gcc-patches. -- Joseph S. Myers jos...@codesourcery.com

Re: [Ada] Add support for --program-prefix

2012-06-14 Thread Joseph S. Myers
same processing. Does this fix bug 864? If so, you should mention PR ada/864 in the ChangeLog entry, and resolve that bug as fixed. -- Joseph S. Myers jos...@codesourcery.com

Re: [testsuite] lib/dg-pch.exp: distinguish between two compilations in summary

2012-06-13 Thread Joseph S. Myers
rent, or use "-Dwith_PCH" and "-Dwithout_PCH" in the flags > that will appear in the test summary. Do you have a preference? I have no preference here. -- Joseph S. Myers jos...@codesourcery.com

Re: [testsuite] lib/dg-pch.exp: distinguish between two compilations in summary

2012-06-13 Thread Joseph S. Myers
as known incomplete. Anyway, check for open "testsuite" bugs for the issues you are fixing. -- Joseph S. Myers jos...@codesourcery.com

Re: constant that doesn't fit in 32bits in alpha.c

2012-06-13 Thread Joseph S. Myers
On Wed, 13 Jun 2012, Richard Henderson wrote: > On 2012-06-12 12:44, Joseph S. Myers wrote: > > I'd rather have a macro HOST_WIDE_INT_C in hwint.h (like INTMAX_C etc. in > > stdint.h). HOST_WIDE_INT_1 is already defined in hwint.h to either 1L or > > 1LL; I'd s

Re: [PATCH 3/7] Add stdint.h wrapper for VxWorks.

2012-06-13 Thread Joseph S. Myers
You have to use the non-typedef versions, with the keywords in the correct order - see the documentation of SIZE_TYPE in tm.texi for details. -- Joseph S. Myers jos...@codesourcery.com

Re: Make timevar phases mutually exclusive. (issue6302064)

2012-06-13 Thread Joseph S. Myers
On Wed, 13 Jun 2012, Diego Novillo wrote: > The LTO bits are fine. I would prefer if an FE maintainer takes a second look > over the other bits. Jason, Joseph? The c-decl.c changes are fine with me. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] Option to build bare-metal ARM cross-compiler for arm-none-eabi target without libunwind

2012-06-13 Thread Joseph S. Myers
own versions of __aeabi_div0 / __aeabi_ldiv0 doesn't suffice, then as long as your code doesn't raise exceptions it should be safe for you to stub out the __aeabi_unwind_cpp_pr* functions. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH 3/7] Add stdint.h wrapper for VxWorks.

2012-06-12 Thread Joseph S. Myers
On Tue, 12 Jun 2012, rbmj wrote: > On 06/12/2012 11:47 AM, Joseph S. Myers wrote: > > On Wed, 6 Jun 2012, rbmj wrote: > > > > > The stdint.h doesn't have all the typedefs needed for standards > > > compliance, so add a hack that adds all of the needed typed

Re: constant that doesn't fit in 32bits in alpha.c

2012-06-12 Thread Joseph S. Myers
ditionally). -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH 3/7] Add stdint.h wrapper for VxWorks.

2012-06-12 Thread Joseph S. Myers
config.gcc if you want to use GCC's stdint.h for freestanding compilations.) -- Joseph S. Myers jos...@codesourcery.com

Re: [patch][gcc47] Deprecate -fconserve-space

2012-06-12 Thread Joseph S. Myers
> to be a.out-specific. :) I commented in <http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01218.html> on how arm/aout.h could safely and usefully be merged into arm/arm.h (and on a few other oddities in the configuration of various ARM targets). -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] Option to build bare-metal ARM cross-compiler for arm-none-eabi target without libunwind

2012-06-12 Thread Joseph S. Myers
not avoiding it. -- Joseph S. Myers jos...@codesourcery.com

Re: [C PATCH] Diagnose compound literals with incomplete type (PR c/53532)

2012-06-12 Thread Joseph S. Myers
r duplicates of this bug (51034, 53196) also added to the testsuite. -- Joseph S. Myers jos...@codesourcery.com

Re: FYI: 1500+ typos, with suggested fixes

2012-05-29 Thread Joseph S. Myers
and disregard an overflow that occurred only because of the sign-extension change in the latter conversion. */ { tree t = build_binary_op (loc, -- Joseph S. Myers jos...@codesourcery.com

Re: FYI: 1500+ typos, with suggested fixes

2012-05-29 Thread Joseph S. Myers
> sed -i '2102s!hasnt!hasn't!' gcc/config/picochip/picochip.c Needs to allow for the ' in the replacement string. -- Joseph S. Myers jos...@codesourcery.com

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

2012-05-29 Thread Joseph S. Myers
Thanks, committed. -- Joseph S. Myers jos...@codesourcery.com

Re: [driver, LTO Patch]: Resurrect user specs support

2012-05-29 Thread Joseph S. Myers
On Tue, 29 May 2012, Christian Bruel wrote: > So I tested the following semantics, with the ones that you pointed. Thanks for the testing. The patch is OK with the ChangeLog conflict markers removed (and the dates on log entries updated to the date of commit). -- Joseph S. Myers

Re: FYI: 1500+ typos, with suggested fixes

2012-05-29 Thread Joseph S. Myers
ollowing variable: # -# MAN_FLAGS - The apropriate flags for installManPage +# MAN_FLAGS - The appropriate flags for installManPage # according to the user's selection. # # -- Joseph S. Myers jos...@codesourcery.com

Re: FYI: 1500+ typos, with suggested fixes

2012-05-29 Thread Joseph S. Myers
On Tue, 29 May 2012, Joseph S. Myers wrote: > For libjava and gcc/java changes, CC j...@gcc.gnu.org on the list (they > may well be able to deal with the classpath typos although properly it's a > separate project). For libstdc++-v3 changes (other than > libstdc++-v3/doc/ht

Re: FYI: 1500+ typos, with suggested fixes

2012-05-29 Thread Joseph S. Myers
== --- libcpp/ChangeLog(revision 187963) +++ libcpp/ChangeLog(working copy) @@ -1,3 +1,10 @@ +2012-05-29 Joseph Myers + + * directives.c: Fix typos. + * include/line-map.h: Fix typos. + * line-map.c: Fix typos. + * macro.c: Fix typos. + 2012-05-25 Dodji Seketeli PR bootstrap/53459 -- Joseph S. Myers jos...@codesourcery.com

Re: FYI: 1500+ typos, with suggested fixes

2012-05-29 Thread Joseph S. Myers
now updated texinfo.tex from upstream to fix these. 2012-05-29 Joseph Myers * doc/include/texinfo.tex: Update to version 2012-05-16.16. Tested with "make pdf". Diffs omitted. -- Joseph S. Myers jos...@codesourcery.com

Re: FYI: 1500+ typos, with suggested fixes

2012-05-29 Thread Joseph S. Myers
uot;,"sold!' libjava/classpath/NEWS classpath is an external project so changes should go upstream. > sed -i '5133s!Ammend!Amend!' libstdc++-v3/doc/html/ext/lwg-active.html These files are maintained by WG21 so changes should go there. > sed -i '34s!untill!until!' zlib/contrib/ada/zlib-streams.ads zlib is an external project so changes should go upstream. -- Joseph S. Myers jos...@codesourcery.com

Re: [driver, LTO Patch]: Resurrect user specs support

2012-05-29 Thread Joseph S. Myers
rer). Please do send such a patch (with an explanation in each case of how "validated" still gets set with that redundant setting removed). -- Joseph S. Myers jos...@codesourcery.com

Re: [driver, LTO Patch]: Resurrect user specs support

2012-05-29 Thread Joseph S. Myers
. If you don't think propagation is needed to do_spec functions, then it should be possible to remove the "validated" setting in there, with a proper explanation of the order in which the various relevant functions are called and where "validated" will previously or subsequently be set. -- Joseph S. Myers jos...@codesourcery.com

Re: [driver, LTO Patch]: Resurrect user specs support

2012-05-28 Thread Joseph S. Myers
On Mon, 28 May 2012, Christian Bruel wrote: > > > On 05/28/2012 01:11 PM, Joseph S. Myers wrote: > > On Mon, 28 May 2012, Christian Bruel wrote: > > > >> I shared the same concern, however, after playing bits with spec toys, I > >> couldn't a find

Re: [RFA/ARM] Add ACLE Predefined macro support

2012-05-28 Thread Joseph S. Myers
are known at compile time to be available? That would occasionally be useful (see my comments in <http://sourceware.org/ml/libc-ports/2012-04/msg00087.html>, for example). -- Joseph S. Myers jos...@codesourcery.com

Re: [driver, LTO Patch]: Resurrect user specs support

2012-05-28 Thread Joseph S. Myers
some setting of "validated" is redundant but the code still works correctly? -- Joseph S. Myers jos...@codesourcery.com

Re: RFC (c): PATCH for c++/53220 (array compound literals and C++)

2012-05-26 Thread Joseph S. Myers
The C front-end change is OK. -- Joseph S. Myers jos...@codesourcery.com

Re: [AARCH64] [PATCH 1/3] AArch64 Port

2012-05-25 Thread Joseph S. Myers
be directly represented in C inline functions. -- Joseph S. Myers jos...@codesourcery.com

Re: [cxx-conversion] New Hash Table (issue6244048)

2012-05-25 Thread Joseph S. Myers
longer recommand `' quoting in diagnostics.) -- Joseph S. Myers jos...@codesourcery.com

Re: [AARCH64] [PATCH 2/3] AArch64 Port

2012-05-25 Thread Joseph S. Myers
arts to fail. I advise splitting such tests up into lots of little tests, each testing just one intrinsic. -- Joseph S. Myers jos...@codesourcery.com

Re: [AARCH64] [PATCH 1/3] AArch64 Port

2012-05-25 Thread Joseph S. Myers
g00890.html>; don't add them back. > + %{!dynamic-linker:-dynamic-linker " GNU_USER_DYNAMIC_LINKER "} \ I removed %{!dynamic-linker:} uses in <http://gcc.gnu.org/ml/gcc-patches/2010-12/msg00194.html>; don't add them back. > @@ -278,6 +281,18 @@ > esac > > case ${host} in > +aarch64*-*-elf) > + extra_parts="$extra_parts crtbegin.o crtend.o crti.o crtn.o" > + tmake_file="${tmake_file} ${cpu_type}/t-aarch64" > + tmake_file="${tmake_file} t-softfp-sfdf t-softfp-excl" > + tmake_file="${tmake_file} ${cpu_type}/t-softfp t-softfp" > + ;; > +aarch64*-*-linux*) > + md_unwind_header=aarch64/linux-unwind.h > + tmake_file="${tmake_file} t-softfp-sfdf t-softfp-excl" > + tmake_file="${tmake_file} ${cpu_type}/t-softfp t-softfp" > + tmake_file="${tmake_file} ${cpu_type}/t-linux" > + ;; Wht are you using t-softfp-sfdf and t-softfp-excl when aarch64/t-softfp appears to override them completely? > +#ifdef __ELF__ > +#define TYPE(x) .type SYM(x),function > +#define SIZE(x) .size SYM(x), . - SYM(x) > +#define LSYM(x) .x Are you really planning to support non-ELF for AArch64? If no, leave out the conditionals. > +#endif > +#endif > \ No newline at end of file Add the missing newline. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] Add powerpc64-linux configuration options

2012-05-23 Thread Joseph S. Myers
for -mstrict-align, again if someone wants -mstrict-align libraries I think they should set up a multilib using that option (or a CPU option that implies it) rather than trying to make the libraries from a compiler configured for some CPU compatible with another CPU they wouldn't normally

Re: Request new specs string token for multilib_os_dir

2012-05-22 Thread Joseph S. Myers
On Fri, 27 Apr 2012, Steven Drake wrote: > 2012-04-27 Steven Drake > > * gcc.c (do_spec_1): Add %M spec token to output multilib_os_dir. Thanks, I've committed this patch with the addition of %M to the comment in gcc.c that documents specs. -- Joseph S. Myers jos...@codesourcery.com

Re: [driver, LTO Patch]: Resurrect user specs support

2012-05-22 Thread Joseph S. Myers
ion that sets "validated" based on a spec should be passed the information about whether it's a user spec or not. So validate_all_switches would need to pass that down to validate_switches_from_spec, for example - and do_spec_1 would also need to get that information. -- Joseph S. Myers jos...@codesourcery.com

Fix PR c/53148 (ICE with division by zero in conditional expression)

2012-05-21 Thread Joseph S. Myers
= remove_c_maybe_const_expr (op2); + } ret = build3 (COND_EXPR, result_type, ifexp, op1, op2); if (int_operands) ret = note_integer_operands (ret); -- Joseph S. Myers jos...@codesourcery.com

Re: [RFC] PR 53063 encode group options in .opt files

2012-05-17 Thread Joseph S. Myers
intainers for their views, rather than me. -- Joseph S. Myers jos...@codesourcery.com

Re: [RFC] PR 53063 encode group options in .opt files

2012-05-16 Thread Joseph S. Myers
On Thu, 17 May 2012, Chung-Lin Tang wrote: > 2012-05-17 Chung-Lin Tang > > * Makefile.in (options.c): Add options.h to included header > files, before tm.h. OK. -- Joseph S. Myers jos...@codesourcery.com

Re: EnabledBy cleanups

2012-05-16 Thread Joseph S. Myers
as_error): Do not special case Wuninitialized. > * optc-gen.awk: Add sanity checks. OK. -- Joseph S. Myers jos...@codesourcery.com

Re: PING [RFA] PowerPC e5500 and e6500 cores support

2012-05-16 Thread Joseph S. Myers
so CC relevant maintainers on pings. -- Joseph S. Myers jos...@codesourcery.com

Re: arm-rtems switch default target to EABI

2012-05-14 Thread Joseph S. Myers
-rtems). It would be a good idea to move them to using the arm_eabi effective-target. (Some of those tests also explicitly list arm*-*-symbianelf*, an existing EABI target not matching the arm*-*-*eabi* pattern; some do not.) -- Joseph S. Myers jos...@codesourcery.com

Re: [RFC] PR 51712 -Wtype-limits should not trigger for enum constants

2012-05-14 Thread Joseph S. Myers
redictability but false positives) or late (fewer false positives, less predictability). Or you could lower as needed but carry around both lowered and unlowered versions of an expression (so front-end IR would include a pointer to lowered IR if lowering has taken place on-demand). -- Joseph S. Myers jos...@codesourcery.com

Re: implement LangEnabledBy

2012-05-14 Thread Joseph S. Myers
abledby_lang[lang_index]] removed rather than commented out. +#print lang_name" " enabledby_name " " k " " Likewise here. -- Joseph S. Myers jos...@codesourcery.com

Re: more enabledby updates

2012-05-13 Thread Joseph S. Myers
ledBy. > * c-opts.c (c_common_post_options): Do not set here explicitly. > gcc/ > * common.opt (Wtype-limits): Use EnabledBy. OK. -- Joseph S. Myers jos...@codesourcery.com

Re: make conflicting help text an error

2012-05-13 Thread Joseph S. Myers
other flags). Not something I am > planning to work on in the future. > > OK? This is OK. I don't think we want to support different help strings for different languages; if an option is supported for multiple languages, we should have a generic description of that option that is

Re: [RFC] PR 53063 encode group options in .opt files

2012-05-12 Thread Joseph S. Myers
pens in optc-gen.awk), which seems ugly, but probably is the most > straightforward solution. Would that be ok? It seems reasonable to me. -- Joseph S. Myers jos...@codesourcery.com

Re: [RFC] PR 53063 encode group options in .opt files

2012-05-12 Thread Joseph S. Myers
ndle having multiple LangEnabledBy fields on the merged option produced by opt-gather.awk, but the principle is clear.) -- Joseph S. Myers jos...@codesourcery.com

Re: [RFC] PR 53063 encode group options in .opt files

2012-05-12 Thread Joseph S. Myers
On Sat, 12 May 2012, Manuel L?pez-Ib??ez wrote: > On 11 May 2012 21:23, Joseph S. Myers wrote: > > > > There's nothing wrong with having separate autogenerated functions for > > each language if you want to split things out that way, but it would seem > > simpler

Re: [RFC] PR 53063 encode group options in .opt files

2012-05-11 Thread Joseph S. Myers
central, whatever option it is (common or not) that is enabling other options. -- Joseph S. Myers jos...@codesourcery.com

Re: [RFC] PR 53063 encode group options in .opt files

2012-05-11 Thread Joseph S. Myers
only for some front ends? That's the only case that should need a language-dependent generated function here. -- Joseph S. Myers jos...@codesourcery.com

Re: [RFC] PR 53063 encode group options in .opt files

2012-05-10 Thread Joseph S. Myers
Include diagnostics.h instead of > diagnostics-core.h. > (set_Wall): Do not see warn_unused here. > (gfc_handle_option): Set it here using handle_generated_option. OK with the > \ No newline at end of file fixed (i.e. a newline added to the end of optc-gen.awk) and 2

Re: PR 40752: -Wconversion generates false warnings for operands not larger than target type

2012-05-09 Thread Joseph S. Myers
warning is not appropriate Yes, that's correct. -- Joseph S. Myers jos...@codesourcery.com

Re: [RFC] PR 53063 encode group options in .opt files

2012-05-08 Thread Joseph S. Myers
s everywhere. > > Any suggestions how to fix this? options.h already has some #if !defined(IN_LIBGCC2) && !defined(IN_TARGET_LIBS) && !defined(IN_RTS) conditionals, so you could arrange for some more such conditionals to be generated. -- Joseph S. Myers jos...@codesourcery.com

Re: [patch] support for multiarch systems

2012-05-08 Thread Joseph S. Myers
generic support has been reviewed by build system maintainers, and the x86 support by x86 maintainers and people familiar with all the applicable x86 ABIs, send patches for each other architecture (or architecture/OS combination), and the relevant architecture experts can review them to make su

Re: remove TARGET_E500 and factorize SPE defaults computation in powerpc ports

2012-05-07 Thread Joseph S. Myers
y tested > at this point, posted to get a first round of comments to validate that this > corresponds to what was suggested. I haven't checked the details of the patch, but it does look substantially along the lines of what I think the logic should be. -- Joseph S. Myers jos...@codesourcery.com

Re: [RFC] PR 53063 encode group options in .opt files

2012-05-06 Thread Joseph S. Myers
is a good idea? Where would be the best place to call this function? That certainly seems one reasonable way to handle implications. -- Joseph S. Myers jos...@codesourcery.com

Re: [RFC] PR 53063 encode group options in .opt files

2012-05-06 Thread Joseph S. Myers
(Wuninitialized): Use EnabledBy. Delete Init. > (Wunused-but-set-variable): Likewise. > (Wunused-function): Likewise. > (Wunused-label): Likewise. > (Wunused-value): Likewise. > (Wunused-variable): Likewise. > * opt-read.awk: Create opt_numbers array. OK. -- Joseph S. Myers jos...@codesourcery.com

Re: [RFC] PR 53063 encode group options in .opt files

2012-05-05 Thread Joseph S. Myers
g the linear search in find_flags_by_name, build up a mapping from option names to numbers in opt-read.awk. (All awk arrays are actually associative arrays, you just need to put opt_numbers[$1] = n_opts at an appropriate point when each option is read in.) -- Joseph S. Myers jos...@codesourcery.com

RE: Patch to enable --with-multilib-list for arm-none-eabi target

2012-05-04 Thread Joseph S. Myers
ho review this patch. -- Joseph S. Myers jos...@codesourcery.com

Re: fix spec function evaluations part of an outer string construction

2012-05-03 Thread Joseph S. Myers
On Thu, 3 May 2012, Olivier Hainque wrote: > 2012-05-03 Olivier Hainque > > * gcc.c (eval_spec_function): Finalize/restore the current string > obstack state as part of the context push/pop operations. OK. -- Joseph S. Myers jos...@codesourcery.com

RE: Propose to add a new argument MULTILIB_REQUIRED in multilib framework

2012-05-03 Thread Joseph S. Myers
yright dates on genmultilib and fragments.texi updated, in the absence of any build system maintainer objections within 72 hours. -- Joseph S. Myers jos...@codesourcery.com

RE: Propose to add a new argument MULTILIB_REQUIRED in multilib framework

2012-05-03 Thread Joseph S. Myers
ed from @code{MULTILIB_OPTIONS} will be > +filtered by @code{MULTILIB_EXCEPTIONS} and then by @code{MULTILIB_REQUIRED}. I think you need more details of the exact syntax of entries in MULTILIB_REQUIRED. Are they sets of options from MULTILIB_OPTIONS, separated by '/', with no wildcards or ot

Re: PR c/51712 -Wtype-limits should not trigger for types of implementation-defined signedness

2012-05-01 Thread Joseph S. Myers
nversion that generates no instruction but don't > + let the signedness change. */ "generates no instruction" is not appropriate semantics for something related to diagnostics. "any conversion that preserves the sign and precision"? OK with that version. -- Joseph S. Myers jos...@codesourcery.com

Re: PR c/51712 -Wtype-limits should not trigger for types of implementation-defined signedness

2012-05-01 Thread Joseph S. Myers
and it is to avoid warning for (expr_of_unsigned_enum_type) >= 0 because being unsigned there is an implementation-defined matter. -- Joseph S. Myers jos...@codesourcery.com

Re: PR c/51712 -Wtype-limits should not trigger for types of implementation-defined signedness

2012-05-01 Thread Joseph S. Myers
ther than STRIP_SIGN_NOPS here? If so, could you ensure there are comments explaining why removing sign changes is safe in this context? -- Joseph S. Myers jos...@codesourcery.com

Re: PR c/53066 Wshadow should not warn for shadowing an extern function

2012-04-25 Thread Joseph S. Myers
On Wed, 25 Apr 2012, Manuel López-Ibáñez wrote: > On 25 April 2012 00:01, Joseph S. Myers wrote: > > On Sun, 22 Apr 2012, Manuel López-Ibáñez wrote: > > > >> Wshadow warns whenever any declaration shadows a global function > >> declaration. This is almost alwa

Re: PR c/53066 Wshadow should not warn for shadowing an extern function

2012-04-24 Thread Joseph S. Myers
functions if the variable has any other type (because if the variable has some type that isn't a pointer-to-function, no confusion is likely without another error being given). -- Joseph S. Myers jos...@codesourcery.com

Re: [C] improve missing initializers diagnostics

2012-04-24 Thread Joseph S. Myers
on" formulation came in with a gcc2 merge in 1999 (r24879), replacing previous diagnostics that did identify the field directly in the message, and never really made sense to me; I think naming the field directly in the diagnostic is better for all such diagnostics. -- Joseph S. Myers jos...@codesourcery.com

Re: Go patch committed: Add explicit checks for division zero and overflow

2012-04-24 Thread Joseph S. Myers
th -fwrapv and the Go front end, to enable those checks in the language-independent compiler. See bug 30484. -- Joseph S. Myers jos...@codesourcery.com

Re: [C PATCH] Fix -Woverride-init (PR c/52880)

2012-04-24 Thread Joseph S. Myers
c.dg/pr52880.c: New test. OK. -- Joseph S. Myers jos...@codesourcery.com

Re: Propose to add a new argument MULTILIB_REQUIRED in multilib framework

2012-04-24 Thread Joseph S. Myers
On Tue, 17 Apr 2012, Terry Guo wrote: > 2012-04-17 Terry Guo > > * Makefile.in (s-mlib): Add new argument MULTILIB_REQUIRED. > * genmultilib (MULTILIB_REQUIRED): New. You need to add documentation in fragments.texi for MULTILIB_REQUIRED. -- Joseph

Re: [PATCH, RFC] Extend __attribute__((format)) with user-specified conversion sequences

2012-04-08 Thread Joseph S. Myers
nclination has been that we should make it possible for plugins to add new format checking types (but the details of the relevant interfaces would be unstable, so such plugins might need to change for each GCC version). That means a function for a plugin to register a new format type - and

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-04 Thread Joseph S. Myers
ink that's correct; the new path should be used independent of endian, just as the existing path is. But any multiarch support patch should presumably define separate multiarch paths for each endianness. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-04 Thread Joseph S. Myers
itectures did (I hope at least aarch64 will use it > by default). The AArch64 ABI (generic, not GNU/Linux, and draft, still subject to incompatible change) is public and used 128-bit long double the last time I checked. My presumption is that there has been no demand for long double wider tha

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-04 Thread Joseph S. Myers
o far). > > Agreed. Note that this loader path discussion is unrelated to > multiarch. It came from the same people so there's a family > resemblance. I think it's directly related, and that such a path is inappropriate by default; that ARM should be consistent wi

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-03 Thread Joseph S. Myers
one that properly accounts for my review comments on previous patch versions (failure to produce such a fixed patch being why Debian multiarch directory support has not got into GCC so far). -- Joseph S. Myers jos...@codesourcery.com

Re: PATCH: Add OPTION_MASK_ISA_X86_64 and support TARGET_BI_ARCH == 2

2012-03-28 Thread Joseph S. Myers
On Wed, 28 Mar 2012, H.J. Lu wrote: > Here is the updated patch. I will wait for OK from Joseph. I have no comments on this patch. -- Joseph S. Myers jos...@codesourcery.com

Re: PATCH: Add OPTION_MASK_ISA_X86_64 and support TARGET_BI_ARCH == 2

2012-03-28 Thread Joseph S. Myers
t; !TARGET_64BIT; \ > config/sol2-bi.h:#define WCHAR_TYPE (TARGET_64BIT ? "int" : "long int") > config/sol2-bi.h:#define WINT_TYPE (TARGET_64BIT ? "int" : "long int") I don't know exactly what TARGET_64BIT is intended to mean, architecture-independently, for Darwin and Solaris. -- Joseph S. Myers jos...@codesourcery.com

Re: PATCH: Remove MaskExists property from config/*/*.opt files

2012-03-27 Thread Joseph S. Myers
On Tue, 27 Mar 2012, H.J. Lu wrote: > Hi, > > This patch removes MaskExists property from config/*/*.opt files > since MaskExists handling has been removed. Tested on Linux/x86-64. > There is no difference between options.h before and after the patch. > OK for trunk? OK. -

Re: PATCH: Remove MaskExists property from option handling

2012-03-27 Thread Joseph S. Myers
On Mon, 26 Mar 2012, H.J. Lu wrote: > The same set of macros are provided. The only differences are the > order of bits used and macro definitions. I am doing a full bootstrap and > test on Linux/x86-64. OK for trunk if there are no regressions? OK. -- Joseph S.

Re: PATCH: Add MaskNeeded property to option handling

2012-03-26 Thread Joseph S. Myers
options use the same mask name), and MaskExists should be removed, rather than adding MaskNeeded - if I understood correctly the purpose for which you are adding MaskNeeded. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] PR bootstrap/52623 Disable libquadmath on AIX

2012-03-26 Thread Joseph S. Myers
. Please include this explanation in a comment on the powerpc-*-aix* | rs6000-*-aix* case. Existing directory-disabling cases may be uncommented, but new ones ought to have comments explaining the reason for disabling the library. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] Change VECTOR_CST representation from TREE_LIST to TREE_VEC-like

2012-03-15 Thread Joseph S. Myers
On Thu, 15 Mar 2012, Richard Guenther wrote: > c-family/ > * c-pretty-print.c (pp_c_initializer_list): Adjust. The c-family changes are OK. -- Joseph S. Myers jos...@codesourcery.com

Re: RFC: PATCH: Add -maddress-mode=short|long for x86

2012-03-14 Thread Joseph S. Myers
On Wed, 14 Mar 2012, Uros Bizjak wrote: > The patch is technically OK for mainline, Joseph - can you please > review option stuff? The option changes look fine to me. -- Joseph S. Myers jos...@codesourcery.com

Re: Remove DWARF_OFFSET_SIZE, DWARF_INITIAL_LENGTH_SIZE

2012-03-14 Thread Joseph S. Myers
On Wed, 14 Mar 2012, Rainer Orth wrote: > Jason Merrill writes: > > > On 03/12/2012 01:41 PM, Joseph S. Myers wrote: > >> As a *target macro* it makes sense to remove it - reomve it from > >> defaults.h, make it purely internal to dwarf2out.c. But I think it make

Re: [C] Handle #pragma before a parameter

2012-03-14 Thread Joseph S. Myers
gt; No regressions for c on x86_64-darwin. > > Ok for trunk ? OK. -- Joseph S. Myers jos...@codesourcery.com

Update contrib/gennews for 4.7

2012-03-13 Thread Joseph S. Myers
nd/or modify @@ -24,6 +24,7 @@ website=http://gcc.gnu.org/ files=" +gcc-4.7/index.html gcc-4.7/changes.html gcc-4.6/index.html gcc-4.6/changes.html gcc-4.5/index.html gcc-4.5/changes.html gcc-4.4/index.html gcc-4.4/changes.html -- Joseph S. Myers jos...@codesourcery.com

4.7 version of c99status.html

2012-03-13 Thread Joseph S. Myers
= --- ChangeLog (revision 185363) +++ ChangeLog (working copy) @@ -1,3 +1,8 @@ +2012-03-13 Joseph Myers + + * doc/invoke.texi (-std=c99), doc/standards.texi (C language): + Refer to GCC 4.7 version of c99status.html. + 2012-03-12 Michael Meissner Backport from mainline -- Joseph S. Myers jos...@codesourcery.com

<    5   6   7   8   9   10   11   12   13   14   >