Re: Fix std::pair std::is_copy_assignable behavior

2013-04-13 Thread Gabriel Dos Reis
> Does DR 1402 resolution generalization need a Standard committee validation > first ? I cannot see why we would want otherwise :-) -- Gaby

[PATCH, tree-ssa] Avoid -Wuninitialized warning in try_unroll_loop_completely()

2013-04-13 Thread Chung-Ju Wu
Hi, I noticed there is an uninitialized variable warning when compiling tree-ssa-loop-ivcanon.c file. Attached patch is a slight modification to avoid the warning and a plaintext ChangeLog is as below. Is it OK for trunk? 2013-04-14 Chung-Ju Wu * tree-ssa-loop-ivcanon.c (try_unroll

Re: Fwd: Fix std::pair std::is_copy_assignable behavior

2013-04-13 Thread Paolo Carlini
On 04/13/2013 09:21 PM, François Dumont wrote: Does DR 1402 resolution generalization need a Standard committee validation first ? In my opinion, it's much more clear to send the C++ front-end patch *separately* together with a simple C++-only (no library) testcase. I would also CC Jason. Pao

[PATCH] V2DI zero constant in GPR (PR target/56948)

2013-04-13 Thread David Edelsohn
V2DI mode is allowed in GPRs and the pattern predicate allows easy vector constants but the pattern in vsx.md does not provide an alternative for that case, which can lead to an ICE where the insn does not satisfy its constraints. The following patch adds an alternative for this case. I also noti

Fwd: Fix std::pair std::is_copy_assignable behavior

2013-04-13 Thread François Dumont
Hider Here is a patch already posted to libstdc++ mailing but I am resending following libstdc++ maintainers advises to add gcc-patches mailing list. This patch proposal is to fix the behavior of std::pair regarding the std::is_*_assignable meta programming functions. As annou

[patch] fix PR52139 correctly

2013-04-13 Thread Steven Bosscher
Hello, The fix for PR52139 only papered over another problem: That things from a basic block header were emitted into the insns stream. When going out of cfglayout mode, these header-insn will be lost. It's probably possible to construct a test case where e.g. a NOTE_INSN_DELETED_DEBUG_LABEL is lo

[PATCH] Enable java for aarch64

2013-04-13 Thread Andreas Schwab
This enables building java for aarch64. Most of the configuration bits were copied from arm. === libjava Summary === # of expected passes2533 # of unexpected failures29 # of untested testcases 25 Andreas. * configure.ac (aarch64-*-*): Don't d

[patch] committed: minor sched-int.h and sched-deps.c fixes

2013-04-13 Thread Steven Bosscher
Hello, In sched-deps.c:deps_analyze_insn there's no need to look for EH_REGION notes, they don't exist until just before final. This assert, and some code in alpha.c (PR56858), are the only remaining meaningful references outside final.c In sched-int.h, the header is only non-empty if INSN_SCHEDU

Re: [Patch, Fortran, OOP] PR 56266: ICE on invalid in gfc_match_varspec

2013-04-13 Thread Mikael Morin
Le 13/04/2013 16:02, Janus Weil a écrit : > Hi Mikael, > >> So, it seems that EXPR_FUNCTION is acceptable in gfc_match_varspec. >> And then, there is nothing preventing 'c(i)' in 'c(i)%encM()' from being >> parsed as a function. Is this supported? > > I think this is forbidden by the Fortran st

Re: [Patch, fortran] PR 56919 SYSTEM_CLOCK on Windows

2013-04-13 Thread Janne Blomqvist
On Sat, Apr 13, 2013 at 1:02 AM, Tobias Burnus wrote: > Janne Blomqvist wrote: >> >> the attached patch implements the SYSTEM_CLOCK intrinsics on the MinGW >> and Cygwin targets using the GetTickCount/GetTickCount64 functions. >> These should be quite robust monotonic clocks and AFAICS are the bes

Re: [Patch, fortran] PR 56919 SYSTEM_CLOCK on Windows

2013-04-13 Thread Janne Blomqvist
On Fri, Apr 12, 2013 at 11:49 PM, Dave Korn wrote: > On 12/04/2013 19:47, Janne Blomqvist wrote: > >> As I don't have a Windows system to test on, I would appreciate if somebody >> more familiar with that platform could take a quick look. In particular, I >> *think* it should be Ok to use win32 AP

Re: [PATCH][ARM][1/2] Add support for vcvt_f16_f32 and vcvt_f32_f16 NEON intrinsics

2013-04-13 Thread Julian Brown
On Fri, 12 Apr 2013 20:09:39 +0100 Julian Brown wrote: > On Fri, 12 Apr 2013 15:19:18 +0100 > Kyrylo Tkachov wrote: > > > Hi all, > > > > This patch adds the vcvt_f16_f32 and vcvt_f32_f16 NEON intrinsic > > to arm_neon.h through the generator ML scripts and also adds the > > built-ins to which

Re: [Patch, Fortran, OOP] PR 56266: ICE on invalid in gfc_match_varspec

2013-04-13 Thread Janus Weil
Hi Mikael, >> Unless someone has a better idea how to treat this, I will commit the >> attached patch as obvious. >> > Not really a better idea, but it seems to me that function calls can > have trailing sub-references, so that gfc_match_varspec could be called > on them. > > gfc_match_rvalue has:

Re: [PATCH] Reset `used' flags after verify RTL sharing

2013-04-13 Thread Eric Botcazou
> 2013-04-11 Andreas Krebbel > > * emit-rtl.c (reset_all_used_flags): New function. > (verify_rtl_sharing): Call reset_all_used_flags before and after > performing the checks. OK for mainline and 4.8 branch (verify_rtl_sharing isn't invoked in release mode) modulo: > ! /* G

Re: [Patch, Fortran, OOP] PR 56266: ICE on invalid in gfc_match_varspec

2013-04-13 Thread Mikael Morin
Hello, Le 12/04/2013 20:38, Janus Weil a écrit : > Unless someone has a better idea how to treat this, I will commit the > attached patch as obvious. > Not really a better idea, but it seems to me that function calls can have trailing sub-references, so that gfc_match_varspec could be called on t

Re: [patch, fortran, backport, 4.8] PR51825 - Fortran runtime error: Cannot match namelist object name

2013-04-13 Thread Janus Weil
Hi Tilo, > I would like to backport the fix for PR51825 I posted here > > http://gcc.gnu.org/ml/gcc-patches/2013-03/msg00316.html > > to the 4.8 branch. well, the usual gfortran policy is to only do backports of regression fixes. In exceptional cases, also non-regression fixes can be backported (

[Patch, Fortran, OOP] PR 55959: ICE in in gfc_simplify_expr, at fortran/expr.c:1920

2013-04-13 Thread Janus Weil
Hi all, here is another trivial ICE-on-invalid fix that I will commit later today (as obvious & regtested on x86_64-unknown-linux-gnu). Cheers, Janus 2013-04-13 Janus Weil PR fortran/55959 * expr.c (gfc_simplify_expr): Branch is not unreachable. 2013-04-13 Janus Weil PR fo

Re: [patch][DF] do not call df_insn_delete in remove_insn, only unlink the insn

2013-04-13 Thread Paolo Bonzini
Il 13/04/2013 02:02, Steven Bosscher ha scritto: > > * emit-rtl.c (remove_insn): Do not call df_insn_delete here. > * cfgrtl.c (delete_insn): Call it here instead. > * lra-spills.c (lra_final_code_change): Use delete_insn. > * haifa-sched.c (sched_remove_insn): Likewise. >