[PATCH] RDSEED-builtin Description Fix

Hi, Here is an obvious fix for a mistake in description of __builtin_ia32_rdseed_di_step. Bootstrap and rdseed-* tests are ok. Ok for commit to trunk? Changelog entry: 2012-08-10 Michael Zolotukhin * config/i386/i386.c (ix86_init_mmx_sse_builtins): Fix description of __builtin

Re: [PATCH] Intrinsics for ADCX

Thanks! On 9 August 2012 18:36, Kirill Yukhin wrote: >> >> Ok. > > Checked in: > http://gcc.gnu.org/ml/gcc-cvs/2012-08/msg00231.html > > Thanks, K -- --- Best regards, Michael V. Zolotukhin, Software Engineer Intel Corporation.

Re: libgo patch committed: Support NumCPU on more platforms

On Tue, Aug 7, 2012 at 5:21 AM, Richard Earnshaw wrote: > > Wouldn't it be more useful on Linux to check the task's affinity > settings? Then when a task is locked to a limited set of cores it won't > overload those cores with threads. This suggestion was implemented by Shenghou Ma, as attached.

RE: [PATCH,i386] fma,fma4 and xop flags

> -mxop implies -mfma4, but reverse is not true. I think this handling went in for bdver1. But, with bdver2, we have both fma and fma4. So for bdver2, -mxop should not be enabling one of them. > if someone set -mfma4 together > with -mfma on the command line, we should NOT disable selected ISA >

Re: [PATCH, testsuite] New effective target long_neq_int

On 08/09/2012 06:46 PM, William J. Schmidt wrote: > As suggested by Janis regarding testsuite/gcc.dg/tree-ssa/slsr-30.c, > this patch adds a new effective target for machines having long and int > of differing sizes. > > Tested on powerpc64-unknown-linux-gnu, where the test passes for -m64 > and i

Re: [PATCH][7/6] Allow anonymous SSA names

> > This converts most users of create_tmp_{var,reg} to use anonymous > SSA names. To give you one more reason to look at 6/6 ;) > > Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Very cool. Thanks for the hard work. Did you have time to test the memory use effets? (I have to

Re: [PATCH 3/3] Compute predicates for phi node results in ipa-inline-analysis.c

> Hi, > > this third patch is basically a proof-of-concept aiming at alleviating > the following code found in Fortran functions when they look at the > contents of array descriptors: > > : > stride.156_7 = strain_tensor_6(D)->dim[0].stride; > if (stride.156_7 != 0) > goto ; >

Re: [PATCH 2/3] Incorporate aggregate jump functions into inlining analysis

> *** inline_merge_summary (struct cgraph_edge > *** 2639,2655 > int count = ipa_get_cs_argument_count (args); > int i; > > ! evaluate_properties_for_edge (edge, true, &clause, NULL, NULL); > if (count) > ! VEC_safe_grow_cleared (int, heap, oper

Re: [PATCH 2/3] Incorporate aggregate jump functions into inlining analysis

> Hi, > > this patch uses the aggregate jump functions created by the previous > patch in the series to determine benefits of inlining a particular > call graph edge. It has not changed much since the last time I posted > it, except for the presence of by_ref flags and removal of checks > require

Re: [cxx-conversion] Make double_int a class with methods and operators. (issue6443093)

On Aug 9, 2012, at 5:00 PM, Michael Matz wrote: > On Thu, 9 Aug 2012, Mike Stump wrote: > >> On Aug 9, 2012, at 8:19 AM, Michael Matz wrote: >>> Hmm. And maintaining a cache is faster than >>> passing/returning/manipulating two registers? >> >> For the most part, we merely mirror existing code,

[PATCH 3/7] s390: Use risbgz for AND.

--- gcc/config/s390/s390.md | 107 +++ 1 files changed, 62 insertions(+), 45 deletions(-) diff --git a/gcc/config/s390/s390.md b/gcc/config/s390/s390.md index 2677fb2..6474023 100644 --- a/gcc/config/s390/s390.md +++ b/gcc/config/s390/s390.md @@ -5946,

[PATCH 6/7] s390: Generate rxsbg, and shifted forms of rosbg

--- gcc/config/s390/s390.md | 63 +- 1 files changed, 56 insertions(+), 7 deletions(-) diff --git a/gcc/config/s390/s390.md b/gcc/config/s390/s390.md index ae004ac..d733062 100644 --- a/gcc/config/s390/s390.md +++ b/gcc/config/s390/s390.md @@ -384,6 +

[PATCH 1/7] s390: Constraints, predicates, and op letters for contiguous bitmasks

--- gcc/config/s390/constraints.md | 11 - gcc/config/s390/predicates.md |6 +++ gcc/config/s390/s390.c | 92 +++- gcc/config/s390/s390.md| 48 + 4 files changed, 90 insertions(+), 67 deletions(-) diff --git a/

[PATCH 5/7] s390: Implement extzv for z10

--- gcc/config/s390/predicates.md |4 +++ gcc/config/s390/s390-protos.h |1 + gcc/config/s390/s390.c| 16 gcc/config/s390/s390.md | 55 +++-- 4 files changed, 68 insertions(+), 8 deletions(-) diff --git a/gcc/config/s390/p

[PATCH 4/7] s390: Add mode attribute for mode bitsize

Constant fold, and less typing than, GET_MODE_BITSIZE with another mode substitution. --- gcc/config/s390/s390.md | 24 1 files changed, 12 insertions(+), 12 deletions(-) diff --git a/gcc/config/s390/s390.md b/gcc/config/s390/s390.md index 6474023..b6e1535 100644 --- a/

[PATCH 7/7] s390: Generate rnsbg

--- gcc/config/s390/s390.md | 55 +++ 1 files changed, 55 insertions(+), 0 deletions(-) diff --git a/gcc/config/s390/s390.md b/gcc/config/s390/s390.md index d733062..182e7b1 100644 --- a/gcc/config/s390/s390.md +++ b/gcc/config/s390/s390.md @@ -3462,6

[PATCH 2/7] s390: Only use lhs zero_extract in word_mode

This means that anything targeting extimm or z10 must therefore imply zarch, which implies word_mode == DImode. Then, now that *insv_z10 is no longer dependent on mode, let gas do some arithmetic, rather than doing it in C and generating new rtl. --- gcc/config/s390/s390.md | 45 +++

[PATCH 0/7] s390 improvements with r[ioxn]sbg

Only "tested" visually, by examining assembly diffs of the runtime libraries between successive patches. All told it would appear to be some remarkable code size improvements. Please test. r~ Richard Henderson (7): s390: Constraints, predicates, and op letters for contiguous bitmasks s390

s390: Use VOIDmode with gen_rtx_SET

Committed as obvious. r~ * config/s390/s390.c (s390_expand_insv): Use VOIDmode in gen_rtx_SET. diff --git a/gcc/config/s390/s390.c b/gcc/config/s390/s390.c index 0ae77a2..d67c0eb 100644 --- a/gcc/config/s390/s390.c +++ b/gcc/config/s390/s390.c @@ -4684,9 +4684,8 @@ s390_expand_insv (rtx

[PATCH, testsuite] New effective target long_neq_int

As suggested by Janis regarding testsuite/gcc.dg/tree-ssa/slsr-30.c, this patch adds a new effective target for machines having long and int of differing sizes. Tested on powerpc64-unknown-linux-gnu, where the test passes for -m64 and is skipped for -m32. Ok for trunk? Thanks, Bill doc: 2012-

[PATCH] Fix PR54211

Fix a thinko in strength reduction. I was checking the type of the wrong operand to determine whether address arithmetic should be used in replacing expressions. This produced a spurious POINTER_PLUS_EXPR when an address was converted to an unsigned long and back again. Bootstrapped and tested o

[rl78] add some checks

RTL checking pointed out a couple of cases where rl78.c was extracting info from rtx without checking the rtx type first. Applied. 2012-08-09 DJ Delorie * config/rl78/rl78.c (rl78_alloc_physical_registers): Check for SET before extracting SET_SRC. (rl78_remove_unused_

Re: [cxx-conversion] Make double_int a class with methods and operators. (issue6443093)

Hi, On Thu, 9 Aug 2012, Mike Stump wrote: > On Aug 9, 2012, at 8:19 AM, Michael Matz wrote: > > Hmm. And maintaining a cache is faster than > > passing/returning/manipulating two registers? > > For the most part, we merely mirror existing code, check out > lookup_const_double and immed_double

Re: Value type of map need not be default copyable

On 08/09/2012 11:22 PM, Marc Glisse wrote: I don't know if std:: is needed, but it looks strange to have it only on some functions: std::forward_as_tuple(forward(__k)), Looking at this line again, you seem to be using std::forward on something that is not a deduced parameter type. I guess it i

Re: PATCH: PR bootstrap/54209: [4.8 Regression] Failed to build gcc for Android/x86

On Thu, Aug 9, 2012 at 4:01 PM, H.J. Lu wrote: > On Thu, Aug 9, 2012 at 3:17 PM, Ian Lance Taylor wrote: >> On Thu, Aug 9, 2012 at 9:39 AM, H.J. Lu wrote: >>> >>> Bionic C library doesn't provide link.h. >> >> Does Bionic provide dl_iterate_phdr? If it does, I'll just note in >> passing that it

Re: PATCH: PR bootstrap/54209: [4.8 Regression] Failed to build gcc for Android/x86

On Thu, Aug 9, 2012 at 3:17 PM, Ian Lance Taylor wrote: > On Thu, Aug 9, 2012 at 9:39 AM, H.J. Lu wrote: >> >> Bionic C library doesn't provide link.h. > > Does Bionic provide dl_iterate_phdr? If it does, I'll just note in > passing that it would be straightforward to simply incorporate the > re

[v3] fix references to C++11 standard

This just updates some comments to refer to the section numbers in the final C++11 standard. * acinclude.m4: Update references to final C++11 standard. * include/bits/shared_ptr.h: Likewise. * include/bits/shared_ptr_base.h: Likewise. * include/bits/unique_ptr.h: Li

Re: [SH] PR 54089 - Reinstate T_REG clobber for left shifts

Oleg Endo wrote: > Removing the T_REG clobber from the left shift patterns entirely wasn't > such a good idea. Especially if dynamic shifts are not available > (anything < SH3) incorrect code may be generated. > The attached patch adds a T_REG clobbering version of the left shift > insn "ashlsi3_

Re: [patch] Fix a couple of VEC_reserve uses, speed up update_ssa a bit

On Fri, Aug 10, 2012 at 12:15 AM, Richard Henderson wrote: > On 08/09/2012 03:06 PM, Steven Bosscher wrote: >> + unsigned old_len = name_to_id ? VEC_length (unsigned, name_to_id) : 0; >> + VEC_reserve (unsigned, heap, name_to_id, num_ssa_names - old_len); > > VEC_length already handles N

Re: PATCH: PR bootstrap/54209: [4.8 Regression] Failed to build gcc for Android/x86

On Thu, Aug 9, 2012 at 9:39 AM, H.J. Lu wrote: > > Bionic C library doesn't provide link.h. Does Bionic provide dl_iterate_phdr? If it does, I'll just note in passing that it would be straightforward to simply incorporate the required types and constants in unwind-dw2-fde-dip.c directly, and avo

Re: [patch] Fix a couple of VEC_reserve uses, speed up update_ssa a bit

On 08/09/2012 03:06 PM, Steven Bosscher wrote: > + unsigned old_len = name_to_id ? VEC_length (unsigned, name_to_id) : 0; > + VEC_reserve (unsigned, heap, name_to_id, num_ssa_names - old_len); VEC_length already handles NULL input. r~

Re: [PATCH] Set correct source location for deallocator calls

On 08/08/2012 12:32 PM, Richard Henderson wrote: On 08/08/2012 09:27 AM, Dehao Chen wrote: Then we should probably assign UNKNOWN_LOCATION for these destructor calls, what do you guys think? I think it's certainly plausible. I can't think what other problems such a change would cause. Jason?

[patch] Fix a couple of VEC_reserve uses, speed up update_ssa a bit

Hello, VEC_reserve allocates an *extra* number of slots. There is unfortunately no VEC_resize op (one of the first things to add after the merge of the cxx branch, I suppose...), so to "grow" a VEC without increasing the used slots count (the VEC_length) it's necessary to compute the number of ext

[google/main, google/gcc-4_7] Fix segfault in linemap lookup

This patch is for the google/main and google/gcc-4_7 branches. New code in GCC 4.7 is calling linemap_lookup with a location_t that may still represent a location-with-discriminator. Before using a location_t value to lookup the line number, it needs to be mapped to a real location_t value. Test

[patch] Use SBITMAP_SIZE in a few places

Hello, SBITMAP_SIZE should be used to get the current size of an sbitmap. Bootstrapped&tested on powerpc64-unknown-linux-gnu. Will commit as obvious. Ciao! Steven sbitmap_size.diff Description: Binary data

Re: Value type of map need not be default copyable

On Thu, 9 Aug 2012, François Dumont wrote: Here is an updated version considering the good catch from Marc. However I prefer to use an explicit instantiation of tuple rather than using cref that would have imply inclusion of in addition to . I wouldn't have used make_tuple at all (tuple<>

[cxx-conversion] Avoid overloaded double_int 'constructor'. (issue6441127)

Convert overloaded double_int::make to non-overloaded from_signed and from_unsigned. This change is intended to preserve the exact semantics of the existing expressions using shwi_to_double_int and uhwi_to_double_int. Tested on x86_64. Index: gcc/ChangeLog 2012-08-09 Lawrence Crowl

Re: Value type of map need not be default copyable

On 08/09/2012 10:35 AM, Paolo Carlini wrote: Hi, On 08/09/2012 09:14 AM, Marc Glisse wrote: On Wed, 8 Aug 2012, François Dumont wrote: On 08/08/2012 03:39 PM, Paolo Carlini wrote: On 08/08/2012 03:15 PM, François Dumont wrote: I have also introduce a special std::pair constructor for contai

Re: [PATCH][7/6] Allow anonymous SSA names

On 12-08-09 09:20 , Richard Guenther wrote: if (interm_type) { /* Create a type conversion HALF_TYPE->INTERM_TYPE. */ ! tmp = create_tmp_reg (interm_type, NULL); ! new_oprnd = make_ssa_name (tmp, NULL); new_stmt = gimple_build_as

Re: [PATCH, libjava] Use accessor functions to manipulate xmlOutputBuffer

- Original Message - > Andrew Hughes writes: > > > Don't worry about reverting it. I'll add it to Classpath now, then > > they'll be in sync when we do the next merge. > > Thank you. > Done: http://git.savannah.gnu.org/cgit/classpath.git/commit/?id=4d4db712cf4df4feb4d7b98bb1b5b4482

Re: [PATCH,i386] fma,fma4 and xop flags

On Wed, Aug 8, 2012 at 1:31 PM, wrote: > Bdver2 cpu supports both fma and fma4 instructions. > Previous to patch, option "-mno-xop" removes "-mfma4". > Similarly, option "-mno-fma4" removes "-mxop". It looks to me that there is some misunderstanding. AFAICS: -mxop implies -mfma4, but reverse i

Re: [google/gcc-4_7] XFAIL libitm failures

On 12-08-09 15:42 , Ollie Wild wrote: * testsuite-management/x86_64-grtev3-linux-gnu.xfail: XFAIL libitm failuires. OK. Diego.

[google/gcc-4_7] XFAIL libitm failures

As discussed, this patch XFAILs the libitm failures uncovered by http://gcc.gnu.org/viewcvs?view=revision&revision=190233. OK for google/gcc-4_7? Ollie 2012-08-09 Ollie Wild * testsuite-management/x86_64-grtev3-linux-gnu.xfail: XFAIL libitm failuires. commit 8d78568138de78f11

RE: PATCH: PR bootstrap/54209: [4.8 Regression] Failed to build gcc for Android/x86

On Thu, 9 Aug 2012, Fu, Chao-Ying wrote: > How about this patch? Just enable it for MIPS that provides link.h in > Android NDK. > Thanks a lot! Please don't put this sort of architecture conditional in an architecture-independent source file. In this case it should be fine for libgcc's con

Re: [google/gcc-4_7] Fix problems with -fdebug-types-section and local types

On 12-08-08 19:17 , Cary Coutant wrote: 2012-08-07 Cary Coutant gcc/ * dwarf2out.c (clone_as_declaration): Copy DW_AT_abstract_origin attribute. (generate_skeleton_bottom_up): Remove DW_AT_object_pointer attribute from original DIE. (clone_tree_hash):

RE: PATCH: PR bootstrap/54209: [4.8 Regression] Failed to build gcc for Android/x86

> > Where does mips link.h come from? I didn't see it in AOSP > Bionic C library. > > -- > H.J. > It's from development/ndk/platforms/android-9/arch-mips/include/link.h from AOSP checkout. Regards, Chao-ying

Re: PATCH: PR bootstrap/54209: [4.8 Regression] Failed to build gcc for Android/x86

On Thu, Aug 9, 2012 at 11:11 AM, Fu, Chao-Ying wrote: >> > Hi, >> > >> > Bionic C library doesn't provide link.h. This patch >> reverts revision >> > 186788: >> > >> > http://gcc.gnu.org/ml/gcc-cvs/2012-04/msg00740.html >> > >> > OK to install? >> > >> > Thanks. >> > >> > H.J. >> > --- >> > 2012-

[PATCH, i386]: Improve LIMIT_RELOAD_CLASSES

On Sat, Aug 4, 2012 at 2:26 PM, Uros Bizjak wrote: >>> Without this, on the new testcase we hit the assert in >>> inline_secondary_memory_needed. The comment before the function states: >>> >>> The macro can't work reliably when one of the CLASSES is class >>> containing registers from multip

RE: PATCH: PR bootstrap/54209: [4.8 Regression] Failed to build gcc for Android/x86

> Hi, > > Bionic C library doesn't provide link.h. This patch reverts revision > 186788: > > http://gcc.gnu.org/ml/gcc-cvs/2012-04/msg00740.html > > OK to install? > > Thanks. > > H.J. > --- > 2012-08-09 H.J. Lu > > PR bootstrap/54209 > * unwind-dw2-fde-dip.c (USE_PT_GNU_EH_FR

RE: PATCH: PR bootstrap/54209: [4.8 Regression] Failed to build gcc for Android/x86

> > Hi, > > > > Bionic C library doesn't provide link.h. This patch > reverts revision > > 186788: > > > > http://gcc.gnu.org/ml/gcc-cvs/2012-04/msg00740.html > > > > OK to install? > > > > Thanks. > > > > H.J. > > --- > > 2012-08-09 H.J. Lu > > > > PR bootstrap/54209 > > * unwin

Re: [PATCH][7/6] Allow anonymous SSA names

On 08/09/2012 06:20 AM, Richard Guenther wrote: > This converts most users of create_tmp_{var,reg} to use anonymous > SSA names. To give you one more reason to look at 6/6 ;) Wow, there's some really nice cleanups in there. r~

Re: [cxx-conversion] Make double_int a class with methods and operators. (issue6443093)

On Aug 9, 2012, at 8:19 AM, Michael Matz wrote: > Hmm. And maintaining a cache is faster than > passing/returning/manipulating two registers? For the most part, we merely mirror existing code, check out lookup_const_double and immed_double_const. If the existing code is wrong, love to have so

Re: s390: Avoid CAS boolean output inefficiency

> This was caused (or perhaps abetted by) the representation of EQ > as NE ^ 1. With the subsequent truncation and zero-extend, I > think combine reached its insn limit of 3 before seeing everything > it needed to see. This can be 4 now, if you tweak the initial heuristic. -- Eric Botcazou

Re: Commit: RL78: Include tree-pass.h

> The issue is that using the plugin interface makes breakage only > detectable when you are able to test a target, not by merely > building it. You just described *most* of the bugs I have to deal with.

[SH] PR 54089 - Reinstate T_REG clobber for left shifts

Hello, Removing the T_REG clobber from the left shift patterns entirely wasn't such a good idea. Especially if dynamic shifts are not available (anything < SH3) incorrect code may be generated. The attached patch adds a T_REG clobbering version of the left shift insn "ashlsi3_n". While at it, I

PATCH: PR bootstrap/54209: [4.8 Regression] Failed to build gcc for Android/x86

Hi, Bionic C library doesn't provide link.h. This patch reverts revision 186788: http://gcc.gnu.org/ml/gcc-cvs/2012-04/msg00740.html OK to install? Thanks. H.J. --- 2012-08-09 H.J. Lu PR bootstrap/54209 * unwind-dw2-fde-dip.c (USE_PT_GNU_EH_FRAME): Don't define for

Re: [PATCH, MIPS] fix MIPS16 hard-float function stub bugs

On 08/08/2012 03:07 AM, Richard Sandiford wrote: It looks like this patch might have been written before: http://gcc.gnu.org/ml/gcc-patches/2012-01/msg00756.html which added: /* If we're calling a locally-defined MIPS16 function, we know that it will return values in both the "sof

Re: MIPS Android patch

On Thu, Aug 9, 2012 at 8:45 AM, H.J. Lu wrote: > On Fri, Apr 20, 2012 at 6:15 PM, Maxim Kuvyrkov > wrote: >> On 20/04/2012, at 1:34 PM, Fu, Chao-Ying wrote: >> >>> Hi Maxim, Richard, >>> >>> I built cross-toolchains for 3 different targets as follows. >>> 1. mips-linux-gnu >>> 2. mips-linux-gnu

Re: MIPS Android patch

On Fri, Apr 20, 2012 at 6:15 PM, Maxim Kuvyrkov wrote: > On 20/04/2012, at 1:34 PM, Fu, Chao-Ying wrote: > >> Hi Maxim, Richard, >> >> I built cross-toolchains for 3 different targets as follows. >> 1. mips-linux-gnu >> 2. mips-linux-gnu --enable-targets=all >> 3. mips64-linux-gnu >> >> These ta

Re: [cxx-conversion] Make double_int a class with methods and operators. (issue6443093)

Hi, On Thu, 9 Aug 2012, Mike Stump wrote: > > Ah. For simple objects like double_int I prefer to have either all > > ops mutating or all ops non-mutating. > > wide_int, which replaces double_int for int types, is always > non-mutating, by value interface. In C++, it will be const & input >

Re: [cxx-conversion] Make double_int a class with methods and operators. (issue6443093)

On Aug 9, 2012, at 1:22 AM, Richard Guenther wrote: > Ah. For simple objects like double_int I prefer to have either all ops > mutating > or all ops non-mutating. wide_int, which replaces double_int for int types, is always non-mutating, by value interface. In C++, it will be const & input par

Re: [PATCH, libjava] Use accessor functions to manipulate xmlOutputBuffer

Andrew Hughes writes: > Don't worry about reverting it. I'll add it to Classpath now, then > they'll be in sync when we do the next merge. Thank you. > In future, please post changes to files under the libjava/classpath directory > to > classp...@gnu.org and feel free to ping me directly if y

Ping Re: Add --no-sysroot-suffix driver option

Ping. This patch is pending review. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] Set current_function_decl in {push,pop}_cfun and push_struct_function

On Thu, Aug 9, 2012 at 4:26 PM, Martin Jambor wrote: > Hi, > > I've always found it silly that in order to change the current > function one has to call push_cfun and pop_cfun which conveniently set > and restore the value of cfun and in addition to that also set > current_function_decl and usuall

Re: PATCH: PR rtl-optimization/54157: [x32] -maddress-mode=long failures

On Wed, Aug 8, 2012 at 8:11 AM, Richard Sandiford wrote: > "H.J. Lu" writes: >> On Wed, Aug 8, 2012 at 6:43 AM, Uros Bizjak wrote: >>> Probably we need to backport this patch to 4.7, where x32 is >>> -maddress-mode=long by default. >>> >> >> It doesn't fail on 4.7 branch since checking mode on P

Re: [PATCH, alpha]: Prevent another case of wrong relaxation with exception handler

On 08/08/2012 11:23 PM, Uros Bizjak wrote: > 2012-08-09 Uros Bizjak > > * config/alpha/alpha.c (alpha_pad_noreturn): Rename to ... > (alpha_pad_function_end): ... this. Also insert NOP between > sibling call and GP load. > (alpha_reorg): Update call to alpha_pad

Re: [PATCH, libjava] Use accessor functions to manipulate xmlOutputBuffer

- Original Message - > Andrew Hughes writes: > > >> OK. > >> > > > > As this is a GNU Classpath change, it should go in there first to > > avoid creating > > a divergence which will cause later problems in merging. Classpath > > is regularly > > merged into gcj as a whole. > > > > I fou

Re: [PATCH] Intrinsics for ADCX

> > Ok. Checked in: http://gcc.gnu.org/ml/gcc-cvs/2012-08/msg00231.html Thanks, K

[PATCH] Set current_function_decl in {push,pop}_cfun and push_struct_function

Hi, I've always found it silly that in order to change the current function one has to call push_cfun and pop_cfun which conveniently set and restore the value of cfun and in addition to that also set current_function_decl and usually also cache its old value to restore it back afterwards. I also

Re: [PATCH] Intrinsics for ADCX

On 08/09/2012 05:21 AM, Michael Zolotukhin wrote: > Changelog entry: > 2012-08-09 Michael Zolotukhin > > * config/i386/adxintrin.h: Remove guarding __ADX__ check. > * config/i386/x86intrin.h: Likewise. > * config/i386/i386.c (ix86_init_mmx_sse_builtins): Remove >

[PATCH] Fix PR54027

This fixes PR54027, VRP treating overflow in signed left-shifts undefined. Bootstrap and regtest pending on x86_64-unknown-linux-gnu. Richard. 2012-08-09 Richard Guenther PR tree-optimization/54027 * tree-vrp.c (extract_range_from_binary_expr_1): Merge RSHIFT_EXPR an

Re: [PATCH, libjava] Use accessor functions to manipulate xmlOutputBuffer

Andrew Hughes writes: >> OK. >> > > As this is a GNU Classpath change, it should go in there first to avoid > creating > a divergence which will cause later problems in merging. Classpath is > regularly > merged into gcj as a whole. > > I found several patches during the last merge which had

Re: [SH] PR 51244 - Improve store of floating-point comparison

Oleg Endo wrote: > This patch mainly improves stores of negated/inverted floating point > comparison results in regs and removes a useless zero-extension after > storing the negated T bit in a reg. [snip] > Tested on rev 190151 with > make -k check RUNTESTFLAGS="--target_board=sh-sim > \{-m2/-ml,

Re: [SH] PR 39423

Oleg Endo wrote: > How about the attached patch? > Is that way of dealing with the mems OK? > What could be a possible test case for the alias info issue? > > Tested on rev 190151 with > make -k check RUNTESTFLAGS="--target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-

Re: [SH] PR 50751

Oleg Endo wrote: > This patch fixes a minor issue related to the displacement addressing > patterns, which leads to useless movt exts.* sequences and one of the > predicates wrongly accepting non-mem ops. > > Tested on rev 190151 with > make -k check RUNTESTFLAGS="--target_board=sh-sim > \{-m2/-

Re: Fix PR 53701

On Thu, 9 Aug 2012, Andrey Belevantsev wrote: > Hello, > > The problem in question is uncovered by the recent speculation patch, it is in > the handling of expressions blocked by bookkeeping. Those are expressions > that become unavailable due to the newly created bookkeeping copies. In the >

Re: [PATCH, alpha]: Prevent another case of linker issues with exception handler

On Thu, Aug 9, 2012 at 3:04 PM, Uros Bizjak wrote: > 2012-08-09 Uros Bizjak > > * config/alpha/alpha.c (alpha_pad_noreturn): Rename to ... > (alpha_pad_function_end): ... this. Also insert NOP between > sibling call and GP load. > (alpha_reorg): Update call to

Re: [cxx-conversion] Support garbage-collected C++ templates

On Thu, Aug 9, 2012 at 2:44 PM, Diego Novillo wrote: > On Thu, Aug 9, 2012 at 5:03 AM, Richard Guenther > wrote: > >> But now with doing it in stages you end up with (this) first stage >> that complicates >> gengtype to support a very small subset of C++ types (namely the one special >> case you

[PATCH, alpha]: Prevent another case of linker issues with exception handler

Hello! This problem is similar to [1], but in this case issue occurs when exception handler immediately follows sibcall function. This happens in libstdc++, ./include/ext/pb_ds/detail/binary_heap_/split_join_fn_imps.hpp: 130 This is the reason for testcase failure in [2]: Running target unix FA

Test...

.

Re: [cxx-conversion] Support garbage-collected C++ templates

On Thu, Aug 9, 2012 at 5:03 AM, Richard Guenther wrote: > But now with doing it in stages you end up with (this) first stage > that complicates > gengtype to support a very small subset of C++ types (namely the one special > case you need for vec.h). Exactly what I did _not_ want! No. It suppor

Re: [PATCH] Strength reduction part 3 of 4: candidates with unknown strides

On Wed, 2012-08-08 at 19:22 -0700, Janis Johnson wrote: > On 08/08/2012 06:41 PM, William J. Schmidt wrote: > > On Wed, 2012-08-08 at 15:35 -0700, Janis Johnson wrote: > >> On 08/08/2012 03:27 PM, Andrew Pinski wrote: > >>> On Wed, Aug 8, 2012 at 3:25 PM, H.J. Lu wrote: > On Wed, Aug 1, 2012

Re: [PATCH] Intrinsics for ADCX

Hi guys, This patch generalizes recently commited addcarryx-intrinsic so that it could be generated either via ADCX or common ADC instruction. ADX-* tests are ok, bootstrap is passed. Is it ok for trunk? Changelog entry: 2012-08-09 Michael Zolotukhin * config/i386/adxintrin.h: Remove g

Re: [Patch, Fortran] PR40881 - Add two F95 obsolescence warnings

On 08/08/2012 19:12, Tobias Burnus wrote: > With this patch, I think the only unimplemented obsolescence warning is for > "(8) Fixed form source -- see B.2.7." > > For the latter, I would like to see a possibility to silence that > warning, given that there is substantial code around, which is in

Re: Value type of map need not be default copyable

On 9 August 2012 09:35, Paolo Carlini wrote: > > When it does, and the corresponding PR will be *ready* we'll reconsider the > issue. After all the *months and months and months* spent by the LWG adding > and removing members from pair and tweaking everything wrt the containers > and issues *still*

Fix PR 53701

Hello, The problem in question is uncovered by the recent speculation patch, it is in the handling of expressions blocked by bookkeeping. Those are expressions that become unavailable due to the newly created bookkeeping copies. In the original algorithm the supported insns and transformatio

Re: [Patch, Fortran] PR54199 improve warning "is also the name of an intrinsic" for internal procedures

On 09/08/2012 11:12, Tobias Burnus wrote: > This patch makes the warning for internal procedures whose name is the > same as the one of an intrinsic clearer. Initially, I though that one > shouldn't warn for internal procedures, but others disagree. In any > case, the warning text is better than or

[PATCH][5/n] Allow anonymous SSA names

Another set of small changes. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2012-08-09 Richard Guenther * tree.h (SSA_VAR_P): Simplify. * tree-ssanames.c (make_ssa_name_fn): Strengthen assert. * fold-const.c (fold_comparison): Check for defau

[AArch64] Merge from upstream trunk r189905

Hi, I've just merged upstream trunk on the aarch64-branch up to r189905. Thanks Sofiane

[Patch, Fortran] PR54199 improve warning "is also the name of an intrinsic" for internal procedures

This patch makes the warning for internal procedures whose name is the same as the one of an intrinsic clearer. Initially, I though that one shouldn't warn for internal procedures, but others disagree. In any case, the warning text is better than original one. Build and regstested on x86-64-li

Re: [cxx-conversion] Support garbage-collected C++ templates

On Wed, Aug 8, 2012 at 11:27 PM, Diego Novillo wrote: > On 12-08-08 17:25 , Gabriel Dos Reis wrote: > >> Aha, so it is an ordering issue, e.g. declarations being generated >> after they have been seen used in an instantiation. >> >> We might want to consider including the header file (that contai

Re: Value type of map need not be default copyable

Hi, On 08/09/2012 09:14 AM, Marc Glisse wrote: On Wed, 8 Aug 2012, François Dumont wrote: On 08/08/2012 03:39 PM, Paolo Carlini wrote: On 08/08/2012 03:15 PM, François Dumont wrote: I have also introduce a special std::pair constructor for container usage so that we do not have to include th

Re: [cxx-conversion] Make double_int a class with methods and operators. (issue6443093)

On Thu, Aug 9, 2012 at 3:22 AM, Richard Guenther wrote: > On Thu, Aug 9, 2012 at 12:25 AM, Lawrence Crowl wrote: >> On 8/8/12, Richard Guenther wrote: >>> On Aug 7, 2012 Lawrence Crowl wrote: >>> > We should probably think about naming conventions for mutating >>> > operations, as I expect we w

Re: [PATCH,i386] fma,fma4 and xop flags

On Thu, Aug 9, 2012 at 7:55 AM, Gopalasubramanian, Ganesh wrote: >> Otherwise, what does -mno-fma4 -mxop do? >> (it should enable both xop and fma4!) what should -mfma4 -mno-xop do >> (it should disable both xop and fma4!). > > Yes! that's what GCC does now. > Some flags are coupled (atleast for

Re: [cxx-conversion] Make double_int a class with methods and operators. (issue6443093)

On Thu, Aug 9, 2012 at 12:25 AM, Lawrence Crowl wrote: > On 8/8/12, Richard Guenther wrote: >> On Aug 7, 2012 Lawrence Crowl wrote: >> > We should probably think about naming conventions for mutating >> > operations, as I expect we will want them eventually. >> >> Right. In the end I would pref

Re: [cxx-conversion] Make double_int a class with methods and operators. (issue6443093)

On Thu, Aug 9, 2012 at 12:17 AM, Lawrence Crowl wrote: > On 8/7/12, Mike Stump wrote: >> On Aug 7, 2012, at 11:38 AM, Lawrence Crowl wrote: >> > Hm. There seems to be significant opinion that there should not be any >> > implicit conversions. I am okay with operations as above, but would like >

Re: Commit: RL78: Include tree-pass.h

On Wed, Aug 8, 2012 at 5:29 PM, Richard Henderson wrote: > On 08/08/2012 07:19 AM, Ian Lance Taylor wrote: >>> > I was suggesting to for example register a 2nd mdreorg-like pass and >>> > add a 2nd target hook. regstack should get the same treatment. >> If the mechanism is a proliferation of mdre

Re: [PATCH] Strength reduction part 3 of 4: candidates with unknown strides

On Wed, 8 Aug 2012, H.J. Lu wrote: > On Wed, Aug 1, 2012 at 10:36 AM, William J. Schmidt > wrote: > > Greetings, > > > > Thanks for the review of part 2! Here's another chunk of the SLSR code > > (I feel I owe you a few beers at this point). This performs analysis > > and replacement on groups

Re: Value type of map need not be default copyable

On Wed, 8 Aug 2012, François Dumont wrote: On 08/08/2012 03:39 PM, Paolo Carlini wrote: On 08/08/2012 03:15 PM, François Dumont wrote: I have also introduce a special std::pair constructor for container usage so that we do not have to include the whole tuple stuff just for associative contain

Re: [cxx-conversion] Support garbage-collected C++ templates

Diego - It's all good changes and your plan for future improvements sounds good, including the part where gengtype is killed with fire. > - Functions should be emitted in files that have access to the > structure where they were defined. I'm not convinced that the > current multiplicity of g

  1   2   >