RFA: Fix mips-sde-elf fallout from 4.8 extv/extzv changes

2013-02-02 Thread Richard Sandiford
This patch fixes a regression caused by my ext and extzv changes. The original idea was that the extract side could pass the number of significant bits in the structure to get_best_reg_extraction_insn, ignoring any bits higher than the field we're extracting: /* Return true if an instruction exist

Re: [PATCH, RFC] PR 55403 + 55391

2013-02-02 Thread Richard Sandiford
Sorry, I realised when posting the expmed patch earlier today that I hadn't finished this off. Richard Sandiford writes: > Richard Henderson writes: >> On 11/20/2012 11:24 AM, Richard Henderson wrote: >>> On 11/20/2012 09:29 AM, Richard Sandiford wrote: Gah. How about this patch, currently

Re: [RFC patch] testsuite: Workaround issues with GCC 4.8.0pre + gdb.trace new KFAIL

2013-02-02 Thread Jan Kratochvil
Hi, just forgot also about gdb.trace/unavailable.exp, equivalent to previous gdb.trace/collection.exp workarounds. Jan gdb/testsuite/ 2013-02-02 Jan Kratochvil Workaround GCC PR debug/55056 and GDB PR server/15081. * gdb.base/restore.c (caller3): Protect l1 by GCC_PR_55056

Re: [patch, fortran] Fix PR 50627

2013-02-02 Thread Paul Richard Thomas
Dear Thomas, I wrote this almost immediately after you posted the patch, got disturbed and forgot about it - my apologies. Note Dominique's reply after you corrected the PR number. OK for trunk. The part in decl.c is exactly what I was thinking to do. Thanks for the patch Paul

[MIPS, committed] Include mips-sde-elf in vect_float

2013-02-02 Thread Richard Sandiford
I noticed while looking at some -mgp32 -mfp64 vect testsuite failures on mips-sde-elf that we weren't running the more interesting tests. Like mipsisa64-elf, hard-float mips-sde-elf ought to have full support for paired single, and mips-sde-elf is the easiest way of testing mismatched register size

Re: [patch, fortran] Fix PR 50627

2013-02-02 Thread Thomas Koenig
Hi Paul, OK for trunk. The part in decl.c is exactly what I was thinking to do. Thanks for the patch Thanks a lot for your review! Committed to trunk as rev. 195684, 4.7 and 4.6 to follow. Thomas

Re: [PATCH 1/6] [AArch64-4.7] Fix warning - Initialise generic_tunings.

2013-02-02 Thread Richard Earnshaw
On 01/02/13 17:42, James Greenhalgh wrote: Hi, This patch moves the various tuning parameter data structures further up config/aarch64/aarch64.c and then uses them to initialise the generic_tunings variable. This mirrors their position on trunk. This fixes the warning: config/aarch64/aarch64.

Re: [PATCH 2/6] [AArch64-4.7] Fix warning - aarch64_add_constant mixed code and declarations.

2013-02-02 Thread Richard Earnshaw
On 01/02/13 17:42, James Greenhalgh wrote: Hi, In config/aarch64/aarch64.c::aarch64_add_constant `shift' was declared after we started writing code. C90 doesn't like this, so split the declaration and the assignment. This fixes the warning: config/aarch64/aarch64.c: In function ‘aarch64_add_c

Re: [PATCH 3/6] [AArch64-4.7] Fix warning - aarch64_legitimize_reload_address passes the wrong type to push_reload.

2013-02-02 Thread Richard Earnshaw
On 01/02/13 17:42, James Greenhalgh wrote: Hi, push_reload takes an `enum reload_type' as its final argument. On trunk we just cast the int we have to the correct type, so we do that here to mirror trunk and correct the warning. We can't fix this by changing the type of the argument we take as

Re: [PATCH 4/6] [AArch64-4.7] Fix warning - aarch64_trampoline_init passes the wrong type to emit_library_call.

2013-02-02 Thread Richard Earnshaw
On 01/02/13 17:42, James Greenhalgh wrote: Hi, emit_library_call takes an `enum library_type` as its second argument. Currently aarch64-4.7-branch passes it an int 0. This patch fixes this, mirroring trunk, by passing LCT_NORMAL instead. This patch fixes the warning: config/aarch64/aarch64.c

Re: [PATCH 5/6] [AArch64-4.7] Fix warning - Mixed code and declarations in aarch64_simd_const_bounds.

2013-02-02 Thread Richard Earnshaw
On 01/02/13 17:42, James Greenhalgh wrote: Hi, aarch64_simd_const_bounds declares `lane' after an assert. This patch moves the declaration above the assert. This patch fixes the warning: config/aarch64/aarch64.c: In function ‘aarch64_simd_const_bounds’: config/aarch64/aarch64.c:6412:3: warnin

Re: [PATCH 6/6] [AArch64-4.7] Backport: Fix warning in aarch64.md

2013-02-02 Thread Richard Earnshaw
On 01/02/13 17:42, James Greenhalgh wrote: Hi, This patch is a backport of one approved here: http://gcc.gnu.org/ml/gcc-patches/2012-12/msg01135.html The patch fixes the warning: config/aarch64/aarch64.md:840: warning: source missing a mode? Regression tested with no regressions on aarch64-n

libgo patch committed: Correct runtime.Callers skip argument

2013-02-02 Thread Ian Lance Taylor
With the change to using the libbacktrace code, inlined functions now picked up. Previously the skip parameter to runtime.Callers was adjusted based on inlining that was ignored. This patch corrects it so that the right thing happens. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu

libgo patch committed: Only run syscall creds_test on GNU/Linux

2013-02-02 Thread Ian Lance Taylor
PR 56171 points out that the syscall/creds_test.go test fails on Solaris. In fact that test is only intended to be run on GNU/Linux anyhow. Solaris supports a similar mechanism, but it works differently. This patch changes the libgo Makefile to let packages specify the list of tests to run, and c

[patch] libffii, fix build for powerpc where DOUBLE equal LONGDOUBLE

2013-02-02 Thread Andreas Tobler
Hi, I'm going to apply the patch below if there are no objections. On PowerPC FreeBSD we have DOUBLE = LONGDOUBLE, so w/o the below the compilation fails with a double 'case' entry. Something went wrong with the latest upstream patch. And I didn't have a chance to test earlier. Ok? TIA, Andreas

[RFA] Fix DEBUG_RELOAD support

2013-02-02 Thread Matthew Gretton-Dann
Hi, Whilst debugging a reload issue I tried enabling DEBUG_RELOAD, only to find that this caused GCC to fail to build. I think this failure was introduced during the change to vec being a C++ type, as DEBUG_RELOAD is normally forced off. The attached patch fixes the build issue. Tested by build

hasher speed traits

2013-02-02 Thread François Dumont
Hi Here is the last patch I can think of for 4.8. Thanks to it default performance reported in performance/23_containers/insert/54075.cc and performance/23_containers/insert_erase/41975.cc are always the best: 54075.cc std::unordered_set without hash code cached 3

[PATCH] fix PR sanitizer/55617

2013-02-02 Thread Jack Howarth
Currently darwin is unable to utilize libasan with constructors due to the lack of constructor priority support on that target. The asan_finish_file routine inserts an essential __asan_init into the array of constructors (via the __mod_init_func section). However the insertion occurs at the e