[PATCH] Fix PR49603

This fixes PR49603. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2011-07-01 Richard Guenther PR tree-optimization/49603 * tree-vect-stmts.c (vectorizable_load): Remove unnecessary assert. * gcc.dg/torture/pr49603.c: New tes

Re: C++ PATCH for c++/49418 (lost cv-quals on template parameter type)

Hi Jason, I've had looked into the problem and I believe, that problem is somehow connected to FE. Moreover, it is really look like that it is connected to array-to-pointer conversion. I put lots of details in the bug: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49519 Here is very short descripti

Re: [PATCH] Handle vectorization of invariant loads (PR46787)

On Thu, Jun 30, 2011 at 8:24 AM, Richard Guenther wrote: > On Wed, 29 Jun 2011, Richard Guenther wrote: > >> >> The following patch makes us handle invariant loads during vectorization. >> Dependence analysis currently isn't clever enough to disambiguate them >> thus we insert versioning-for-alias

Re: [pph] Stream first/weak_object_global_name (issue4641086)

notice_global_symbol is actually called in the parser (before we stream out). I just confirmed this by setting a break point on the function in the pph generation of c1varoder.pph and hitting the function twice for the two variables defined in c1varorder.h It looks like rtl and assembler_name are

C++ PATCH for c++/49387 (strange error with typeid)

Here we were winding up in get_tinfo_decl with a template type that was still incomplete because nothing else had required it to be complete. So we initially decide to treat it as a class with no bases. But then the process of creating the initializer for the type_info node causes the type to

Re: [PATCH,c++] mark EXPR_PACK_EXPANSION as typed only, v2

On 05/09/2011 07:41 PM, Nathan Froyd wrote: I present the patch below. We have a new function, cp_tree_operand_length, and the code has been retrofitted to use it in various places. I've changed iterative_hash_template_arg to use it as well. Tested x86_64-pc-linux-gnu, applying to trunk. comm

C++ PATCH for c++/49355 (ICE with new T({""}))

The problem here was that stabilize_init called from build_new_1 wasn't doing anything to stabilize the elements of a CONSTRUCTOR, so the allocator argument wasn't preevaluated, and due to the cleanup for the data pointer its cleanup wasn't associated with the CLEANUP_POINT_EXPR, leading to a c

C++ PATCH for c++/49569 (ICE with implicitly-declared copy constructor)

If we aren't setting DECL_ARTIFICIAL on the rhs parm of an implicitly-declared copy constructor, we need to set DECL_PARM_INDEX for the sake of the constexpr expander. Tested x86_64-pc-linux-gnu, applying to trunk. commit eae21fea450d9fa1b9fce0544db5cf7fdce8f022 Author: Jason Merrill Date: T

Re: [pph] Stream first/weak_object_global_name (issue4641086)

On Thu, Jun 30, 2011 at 16:24, Gabriel Charette wrote: > first/weak_global_object_name are part of the global state. > > This seems to be used to produce the assembler names. They are set only once > as early as possible in the parsing; thus we should define it to be whatever > it was in the fir

PATCH: PR target/49600: Bad SSE2 int->float split in i386.md

In one SSE2 int->float split, when TARGET_USE_VECTOR_CONVERTS is true, TARGET_INTER_UNIT_MOVES is false and GENERAL_REG_P (op1) is true. we will get gcc_unreachable. This patch removes TARGET_INTER_UNIT_MOVES check. OK for trunk? Thanks. H.J. --- 2011-06-30 H.J. Lu PR target/49600

Remove config.gcc support for *local* configurations

config.gcc has special handling of triplets matching "*local*". I don't think this is particularly relevant to how GCC is used and built today; a good GCC port is expected to include many pieces in different places that are not covered by this generic code, and adding a config.gcc stanza is one of

Re: [PATCH, SRA] Total scalarization and padding

Hi, On Wed, Jun 29, 2011 at 02:52:36PM +0200, Richard Guenther wrote: > On Tue, 28 Jun 2011, Martin Jambor wrote: > > On Tue, Jun 28, 2011 at 03:01:17PM +0200, Richard Guenther wrote: > > > On Tue, Jun 28, 2011 at 2:50 PM, Martin Jambor wrote: > > > > Hi, > > > > > > > > at the moment SRA can get

Re: [testsuite, libmudflap] Only use --noinhibit-exec with GNU ld (PR libmudflap/49549)

On Jun 30, 2011, at 10:26 AM, Rainer Orth wrote: > Though I don't nominally need approval for this patch, I'll wait a day > if Mike or Frank have comments. Seems fine.

[PATCH] Use ira_reg_class_max_nregs array instead of CLASS_MAX_NREGS macro

Hello. This patch replaces the use of macro CLASS_MAX_NREGS to reading from ira_reg_class_max_nregs array. The ira_reg_class_max_nregs array is initialized from a macro CLASS_MAX_NREGS. This patch should speed up the compiler, a little. The patch has been bootstrapped on and regression tes

Re: [PATCH 0/4] Docs: extend.texi

On Thu, Apr 28, 2011 at 01:20, Michael Witten wrote: > See the following emails for a few inlined patches > to /trunk/gcc/doc/extend.texi (revision 172911): > > [1] Docs: extend.texi: Add missing semicolon for consistency > [2] Docs: extend.texi: Remove trailing blanks from lines > [3] Docs: e

C++ PATCH to avoid duplicate overload resolution (related to c++/48481)

While I was looking at 48481 a while back, I noticed that since Doug's change to implement DR 164, in a call subject to argument-dependent lookup we end up considering every function twice. This patch avoids this by using a pointer set to remember which functions we already have in the overloa

Re: [gcc patch] Re: C++ member function template id not matching linkage name (PR debug/49408)

On Thu, 30 Jun 2011 00:27:31 +0200, Jason Merrill wrote: > The earlier output was correct. We just don't want to print "g(int, > double)". OK, understood now. It got more clear with the new testcase: _Z1tIlEDTplcvT_Li5EclL_Z1qsELi6EEEv decltype (((long)(5))+((q(short))(6))) t() -> decltype (((

Re: PATCH [8/n]: Prepare x32: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

On Thu, Jun 30, 2011 at 12:02 PM, Richard Henderson wrote: > On 06/30/2011 11:23 AM, H.J. Lu wrote: >> +#ifdef REG_VALUE_IN_UNWIND_CONTEXT >> +typedef _Unwind_Word _Unwind_Context_Reg_Val; >> +/* Signal frame context.  */ >> +#define SIGNAL_FRAME_BIT ((_Unwind_Word) 1 >> 0) > > There's absolutely

Re: [testsuite, AVR]: Add some progmem test cases

Georg-Johann Lay schrieb: Some runtime and checks for error/warning for C/C++. ... testsuite/ * gcc.target/avr/torture/progmem.h: New file. * gcc.target/avr/torture/exit-abort.h: New file. maybe it's better to have the .h files one level up like so? * gcc.target/avr/

[pph] Stream first/weak_object_global_name (issue4641086)

first/weak_global_object_name are part of the global state. This seems to be used to produce the assembler names. They are set only once as early as possible in the parsing; thus we should define it to be whatever it was in the first pph (or even in the C file if it was set before any pph was e

Re: PATCH [8/n]: Prepare x32: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

On Thu, Jun 30, 2011 at 11:09 AM, Rainer Orth wrote: > "H.J. Lu" writes: > >>> Why all those contortions with i386/value-unwind.h?  It seems far >>> simpler to me to put this into libgcc/config/i386/linux-unwind.h and be >>> done with it. >> >> I did it this way so that other non-Linux x32 target

Re: [PATCH] [Annotalysis] Fixes virtual method calls when type is unknown

On Thu, Jun 30, 2011 at 12:09 PM, Diego Novillo wrote: > > On 11-06-30 13:01 , Delesley Hutchins wrote: >> >> Okay for branches/annotalysis and google/main? > > OK with some minor formatting nits below. Please also cherry-pick this into the google/gcc_4-6 branch. I've sent you instructions out-o

Re: [patch] Fix oversight in tuplification of DOM

> 2011-06-28 Eric Botcazou > > * tree-ssa-dom.c (initialize_hash_element): Fix oversight. As reported under PR tree-optimization/49572, using the type of the LHS like in the other cases leads to optimization regressions. The attached patch changes this to use the type of RHS instead in

Re: PATCH [8/n]: Prepare x32: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

On 06/30/2011 11:23 AM, H.J. Lu wrote: > +#ifdef REG_VALUE_IN_UNWIND_CONTEXT > +typedef _Unwind_Word _Unwind_Context_Reg_Val; > +/* Signal frame context. */ > +#define SIGNAL_FRAME_BIT ((_Unwind_Word) 1 >> 0) There's absolutely no reason to re-define this. So what if the value is most-significant

[PATCH] Improve CONSTRUCTOR printing in tree-pretty-print.c

Hi! As reporteed by Tobias, when printing array ctors the pretty printer would never print indexes or ranges, which means that e.g. {[2 ... 71]=7} ctor was printed as {7} and {[3]=1, [7]=2} ctor was printed as {1, 2} The following patch prints the index (if different from the last index + 1 resp.

Re: PATCH [8/n]: Prepare x32: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

On Thu, Jun 30, 2011 at 11:09 AM, Rainer Orth wrote: > "H.J. Lu" writes: > >>> Why all those contortions with i386/value-unwind.h?  It seems far >>> simpler to me to put this into libgcc/config/i386/linux-unwind.h and be >>> done with it. >> >> I did it this way so that other non-Linux x32 target

Re: [PATCH] Sign extend before converting constants to GMP values.

On 06/30/2011 11:01 AM, Sebastian Pop wrote: On Thu, Jun 30, 2011 at 10:03, Richard Guenther wrote: But what do you do for for (unsigned char i = 128; i< 255; ++i) ? You change 128 to -128 which is wrong. Yes, 128 gets translated to -128. And 255 gets translated to -1. And so the loop i

Re: [patch, fortran] Always return malloc(1) for empty arrays in the library

Hi Janne, I'd prefer to add the zero check to runtime/memory.c (internal_malloc_size), i.e. change if (size == 0) return NULL; to if (size == 0) size = 1; Good point. I have done so in the attached patch, plus removed all special cases for checking for zero size. Regression-tested.

Re: PATCH [8/n]: Prepare x32: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

"H.J. Lu" writes: >> Why all those contortions with i386/value-unwind.h?  It seems far >> simpler to me to put this into libgcc/config/i386/linux-unwind.h and be >> done with it. > > I did it this way so that other non-Linux x32 targets can use it. Such as? Currently, only Linux, Windows and So

Re: [Patch, AVR, longlong.h]: Implement __clrsbhi2.

2011/6/30 Georg-Johann Lay : > The new count leading signs is not implemented for HI: > > builtin-bitops-1.c:(.text+0xd1e): undefined reference to `__clrsbhi2' > FAIL: gcc.c-torture/execute/builtin-bitops-1.c compilation,  -O0 > > Extended clz/ctz support in longlong.h > > With the patch testcase p

Re: PATCH [8/n]: Prepare x32: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

On Thu, Jun 30, 2011 at 11:03 AM, Rainer Orth wrote: > "H.J. Lu" writes: > >>> gcc/ >>> >>> 2011-06-30  H.J. Lu   >>> >>>        * config.gcc (libgcc_tm_file): Add i386/value-unwind.h for >>>        Linux/x86. >>> >>>        * system.h (REG_VALUE_IN_UNWIND_CONTEXT): Poisoned. >>> >>>        * unw

Re: PATCH [5/n]: Prepare x32: PR middle-end/48016: Inconsistency in non-local goto save area

On Wed, Jun 29, 2011 at 9:16 AM, Michael Matz wrote: > Hi, > > On Wed, 29 Jun 2011, H.J. Lu wrote: > >> > diff --git a/gcc/function.c b/gcc/function.c >> > index 81c4d39..131bc09 100644 >> > --- a/gcc/function.c >> > +++ b/gcc/function.c >> > @@ -4780,7 +4780,7 @@ expand_function_start (tree subr)

Re: PATCH [8/n]: Prepare x32: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

"H.J. Lu" writes: >> gcc/ >> >> 2011-06-30  H.J. Lu   >> >>        * config.gcc (libgcc_tm_file): Add i386/value-unwind.h for >>        Linux/x86. >> >>        * system.h (REG_VALUE_IN_UNWIND_CONTEXT): Poisoned. >> >>        * unwind-dw2.c (_Unwind_Context_Reg_Val): New. >>        (_Unwind_Get_Un

[patch tree-optimization]: Fix typo in X bitwise-binary-op CST

Hello, this patch fixes a typo in type-sinking for bitwise-binary operation X op CST. ChangeLog gcc/ 2011-06-30 Kai Tietz * tree-ssa-forwprop.c (simplify_bitwise_binary): Fix typo. ChangeLog gcc/testsuite 2011-06-30 Kai Tietz * gcc.dg/tree-ssa/bitwise-sink.c: New test.

Use INCOMING_REGNO macro unconditionally in df-scan.c

Like in other files, since defaults.h ensures that it is always defined. Tested on i586-suse-linux, applied on the mainline. 2011-06-30 Eric Botcazou * df-scan.c (df_get_entry_block_def_set): Use INCOMING_REGNO macro unconditionally. -- Eric Botcazou Index: df-scan.c =

Re: [PATCH] Enable IPA-PTA for ltrans stage

On Thu, Jun 30, 2011 at 7:01 AM, Richard Guenther wrote: > > This patch makes IPA-PTA work during the ltrans stage of LTO. > It basically makes sure to honor the accessed-from-other-ltrans-unit > flags for variables and functions where appropriate. > > With this patch, IPA-PTA survives LTO bootstr

[testsuite, AVR]: Add some progmem test cases

Some runtime and checks for error/warning for C/C++. Is ./testsuite/gcc.target/avr/ realm of avr port maintainers? Johann testsuite/ * gcc.target/avr/avr.exp: Run over cpp files, too. * gcc.target/avr/torture/avr-torture.exp: Ditto. * gcc.target/avr/progmem-error-1.c: N

Re: PATCH [8/n]: Prepare x32: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

On Thu, Jun 30, 2011 at 10:37 AM, H.J. Lu wrote: > On Thu, Jun 30, 2011 at 9:08 AM, H.J. Lu wrote: >> On Thu, Jun 30, 2011 at 8:03 AM, Jason Merrill wrote: >>> On 06/30/2011 10:42 AM, H.J. Lu wrote: Register may be saved/restored either by address or value. My patch doesn't change

Re: PATCH [8/n]: Prepare x32: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

On Thu, Jun 30, 2011 at 9:08 AM, H.J. Lu wrote: > On Thu, Jun 30, 2011 at 8:03 AM, Jason Merrill wrote: >> On 06/30/2011 10:42 AM, H.J. Lu wrote: >>> >>> Register may be saved/restored either by address or value. My patch >>> doesn't change the reg field.  The other way will be >>> >>> #ifdef USE

Re: [PATCH 04/12] vax: Emit prologue as rtl.

On Jun 30, 2011, at 1:21 PM, Richard Henderson wrote: > On 06/29/2011 04:01 PM, Steven Bosscher wrote: >> How many TARGET_ASM_FUNCTION_{PRO,EPI}LOGUE targets are left anyway? > > Answering the question that it sounds like you actually asked: 2 or 3. > Not that it does us that much good because of

[testsuite, libmudflap] Only use --noinhibit-exec with GNU ld (PR libmudflap/49549)

As reported in the PR, libmudflap.cth/cthfrags.exp unconditionally uses -Wl,--noinhibit-exec, which is a GNU ld option unknown to Sun ld, even though the Solaris 11 ld has implemented many of the GNU ld options. This patch addresses this by providing an effective-target check for gld, similar to th

Re: [PATCH 04/12] vax: Emit prologue as rtl.

On 06/29/2011 04:01 PM, Steven Bosscher wrote: > How many TARGET_ASM_FUNCTION_{PRO,EPI}LOGUE targets are left anyway? Answering the question that it sounds like you actually asked: 2 or 3. Not that it does us that much good because of the other uses. FWIW, Alpha appears to be the only target that

Improve Solaris mudflap support (PR libmudflap/49550)

This is the first of two patches to get mudflap fully working on Solaris 11, both with Sun ld and GNU ld. It addresses a couple of testsuite failures: * Several tests fail with 3 unexpected register violations: *** mudflap violation 1 (register): time=1309356076.070433 ptr=21680 size=16 pc=7

Re: [PATCH] [Annotalysis] Fixes virtual method calls when type is unknown

On 11-06-30 13:01 , Delesley Hutchins wrote: This bug fixes a failure in annotalysis that is caused when gcc does not return the correct static type for the callee of a virtual method. Bootstrapped and passed GCC regression testsuite on x86_64-unknown-linux-gnu. Okay for branches/annotalysis an

Re: [PATCH] Add support on powerpc to change CASE_VALUES_THRESHOLD

On Thu, Jun 30, 2011 at 12:31:44PM +0200, Richard Guenther wrote: > On Thu, Jun 30, 2011 at 12:34 AM, Michael Meissner > wrote: > > On the powerpc, switch statements can be expensive, and we would like to be > > able to tune the threshold of when the compiler generates if statements > > vs. using

[PATCH] [Annotalysis] Fixes virtual method calls when type is unknown

This bug fixes a failure in annotalysis that is caused when gcc does not return the correct static type for the callee of a virtual method. Bootstrapped and passed GCC regression testsuite on x86_64-unknown-linux-gnu. Okay for branches/annotalysis and google/main? -DeLesley 2011-06-30 DeLe

Re: [Patch] Fix objc/48109

Hi Richard(s), thanks for the reviews. before re-jigging - a couple of questions / points below. On 30 Jun 2011, at 16:12, Richard Henderson wrote: On 06/30/2011 03:56 AM, Richard Guenther wrote: It is only used by darwin at present - so nothing is split out for any other target (the defau

Re: [PATCH, PR 49094] Refrain from creating misaligned accesses in SRA

Hi, On Thu, Jun 30, 2011 at 03:39:55PM +0200, Martin Jambor wrote: > Hi, > > I had to add a test that the analyzed expression is not an SSA name. > This has been approved by Rchi on IRC yesterday. Thus, I have > committed the following as revision 175703 after successful run of c > and c++ tests

Re: [PATCH][PING] Backport PR42333 fix to gcc 4.5

On Jun 19, 2011, at 7:07 PM, Jack Howarth wrote: > I requested a backport to gcc-4_5-branch of the fix for PR42333 awhile > back... > > http://gcc.gnu.org/ml/gcc-patches/2011-02/msg01098.html > > Can we get that committed? Yes. Applied for 4.5.4. Thanks for your dedication.

Re: [trans-mem] Beginning of refactoring

On Thu, 2011-06-30 at 07:59 -0700, Richard Henderson wrote: > I'm not sure what to make of your first two examples above, but > let's suppose that the third example is the real one. In which > case you don't actually need concatenation. Remove the ## from > the macro and then you can pass in > >

Re: [PATCH][Revised] Pass -no_pie on SYSTEMSPEC for darwin11

On Jun 19, 2011, at 4:55 PM, Jack Howarth wrote: >>> This is definitely a problem in current gcc-4_5-branch without the backport >>> of the SYSTEMSPEC fix... >> >> Excellent... and does the patch fix it? > > Yes... Ok, applied to the 4.5 branch. Scream loud and soon if for any reason you not

Re: PATCH [8/n]: Prepare x32: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

On Thu, Jun 30, 2011 at 8:03 AM, Jason Merrill wrote: > On 06/30/2011 10:42 AM, H.J. Lu wrote: >> >> Register may be saved/restored either by address or value. My patch >> doesn't change the reg field.  The other way will be >> >> #ifdef USE_UNWIND_WORD >>     _Unwind_Word reg[DWARF_FRAME_REGISTER

Re: [PATCH] Sign extend before converting constants to GMP values.

On Thu, Jun 30, 2011 at 10:03, Richard Guenther wrote: > But what do you do for > >  for (unsigned char i = 128; i < 255; ++i) > > ?  You change 128 to -128 which is wrong. Yes, 128 gets translated to -128. And 255 gets translated to -1. And so the loop iteration domain gets translated in the pol

Re: [PATCH] Fix empty .debug_abbrev handling (PR debug/49364)

Jakub, > the label in that case at all. Rainer, does this make SGI tools > happy? to try this, I'll have to do a full rebuild which is quite slow. I'll see during the next bootstrap this weekend. Thanks. Rainer -- --

Re: [PATCH] Handle vectorization of invariant loads (PR46787)

On Wed, 29 Jun 2011, Richard Guenther wrote: > > The following patch makes us handle invariant loads during vectorization. > Dependence analysis currently isn't clever enough to disambiguate them > thus we insert versioning-for-alias checks. For the testcase hoisting > the load is still always p

Re: [Patch] Fix objc/48109

On 06/30/2011 03:56 AM, Richard Guenther wrote: >> > It is only used by darwin at present - so nothing is split out for any >> > other >> > target (the default hook is simply 'false'). > Yes, I've seen that. Still a hook like this should be generally useful, > and you still process through some p

Re: [PATCH] Sign extend before converting constants to GMP values.

On Thu, 30 Jun 2011, Sebastian Pop wrote: > On Thu, Jun 30, 2011 at 09:57, Sebastian Pop wrote: > >> Canonically extended according to TYPE_UNSIGNED I mean.  So what you > >> do is always create signed mpzs - that should simply work without > >> doing anything to the double-int.  Thus, why not do

Re: PATCH [8/n]: Prepare x32: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

On 06/30/2011 10:42 AM, H.J. Lu wrote: Register may be saved/restored either by address or value. My patch doesn't change the reg field. The other way will be #ifdef USE_UNWIND_WORD _Unwind_Word reg[DWARF_FRAME_REGISTERS+1]; #else void *reg[DWARF_FRAME_REGISTERS+1]; #endif We need i

[PATCH] Address lowering [3/3] Test suite additions and changes

Here is the third patch related to address lowering. It contains the changes to the test suite: 13 new tests, and 18 tests that needed to be modified. The latter occur because of specific patterns expected in tree dumps; for example, a field reference x.f might now be expected to be a MEM_REF in

Re: [PATCH] Sign extend before converting constants to GMP values.

On Thu, 30 Jun 2011, Sebastian Pop wrote: > On Thu, Jun 30, 2011 at 09:46, Richard Guenther wrote: > > On Thu, 30 Jun 2011, Sebastian Pop wrote: > > > >> On Thu, Jun 30, 2011 at 05:19, Richard Guenther wrote: > >> > That looks odd.  So you given -1U as input you sign-extend that to -1 > >> > >>

Re: [PATCH] Sign extend before converting constants to GMP values.

On Thu, Jun 30, 2011 at 09:57, Sebastian Pop wrote: >> Canonically extended according to TYPE_UNSIGNED I mean.  So what you >> do is always create signed mpzs - that should simply work without >> doing anything to the double-int.  Thus, why not do >> >> static inline void >> tree_int_to_gmp (tree

Re: [trans-mem] Beginning of refactoring

On 06/30/2011 04:15 AM, Torvald Riegel wrote: > On Wed, 2011-06-29 at 16:01 -0700, Richard Henderson wrote: >> On 05/25/2011 02:10 PM, Torvald Riegel wrote: >>> Patch 2: _ITM_dropReferences is not sufficiently defined in the ABI. It >>> seems to target some form of open nesting for txnal wrappers,

Re: [PATCH] Sign extend before converting constants to GMP values.

On Thu, Jun 30, 2011 at 09:46, Richard Guenther wrote: > On Thu, 30 Jun 2011, Sebastian Pop wrote: > >> On Thu, Jun 30, 2011 at 05:19, Richard Guenther wrote: >> > That looks odd.  So you given -1U as input you sign-extend that to -1 >> >> correct >> >> > and then set the mpz to -1ULLL. >> >> and

Re: [PATCH 4/6] Fix computation of precision.

On 06/30/2011 09:50 AM, Sebastian Pop wrote: On Wed, Jun 29, 2011 at 18:16, Tobias Grosser wrote: why do we continue to call low 'low' and up 'up', if we actually just have two values v1 and v2 where we do not know which one is larger? I think this wrong and probably comes because we pass the l

Re: [PATCH 4/6] Fix computation of precision.

On Wed, Jun 29, 2011 at 18:16, Tobias Grosser wrote: > why do we continue to call low 'low' and up 'up', if we actually just have > two values v1 and v2 where we do not know which one is larger? I think this > wrong and probably comes because we pass the lower loop bound to val_one and > the upper

[PATCH] Address lowering [2/3] Constrain forward propagator

Here is the second patch related to address lowering. It looks for a specific pattern where replacement of a reg in a mem rtx is not profitable, and constrains it from occurring. Thanks, Bill 2011-06-30 Bill Schmidt * fwprop.c (fwprop_bb_aux_d): New struct. (record_mem_plus_r

Re: [PATCH] Sign extend before converting constants to GMP values.

On Thu, 30 Jun 2011, Sebastian Pop wrote: > On Thu, Jun 30, 2011 at 05:19, Richard Guenther wrote: > > That looks odd.  So you given -1U as input you sign-extend that to -1 > > correct > > > and then set the mpz to -1ULLL. > > and then it sets the mpz to signed -1, given that I'm passing false

Re: [PATCH] Sign extend before converting constants to GMP values.

On Thu, Jun 30, 2011 at 05:19, Richard Guenther wrote: > That looks odd.  So you given -1U as input you sign-extend that to -1 correct > and then set the mpz to -1ULLL. and then it sets the mpz to signed -1, given that I'm passing false to UNS: /* Sets RESULT to VAL, taken unsigned if UNS is t

Re: PATCH [8/n]: Prepare x32: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

On Thu, Jun 30, 2011 at 7:08 AM, Jason Merrill wrote: > On 06/28/2011 02:53 PM, H.J. Lu wrote: >> >> This updated patch.  It allows multiple unwind contexts.  It replaces >> >> char by_value[DWARF_FRAME_REGISTERS+1]; >> >> with >> >> _Unwind_Word value[DWARF_FRAME_REGISTERS+1]; >> >> The code is c

Re: PATCH [8/n]: Prepare x32: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

On 06/28/2011 02:53 PM, H.J. Lu wrote: This updated patch. It allows multiple unwind contexts. It replaces char by_value[DWARF_FRAME_REGISTERS+1]; with _Unwind_Word value[DWARF_FRAME_REGISTERS+1]; The code is cleaner than conditionally replacing void *reg[DWARF_FRAME_REGISTERS+1]; with _

[PATCH] Enable IPA-PTA for ltrans stage

This patch makes IPA-PTA work during the ltrans stage of LTO. It basically makes sure to honor the accessed-from-other-ltrans-unit flags for variables and functions where appropriate. With this patch, IPA-PTA survives LTO bootstrap (with -fipa-pta hard-wired to 1 in common.opt). Before the patch

Re: PATCH] PR 49580

Zdenek Dvorak wrote on 30/06/2011 15:21:43: > From: Zdenek Dvorak > To: Razya Ladelsky/Haifa/IBM@IBMIL > Cc: gcc-patches@gcc.gnu.org, Richard Guenther > Date: 30/06/2011 15:21 > Subject: Re: PATCH] PR 49580 > > Hi, > > > This patch fixes the build failure of gcc spec2006 benchmark. > > The c

Re: Ping #1: [Patch, AVR]: Fix PR34734

Weddington, Eric wrote: >> -Original Message- >>> You can backport it if you want. >>> >>> I'm usually didn't backport such patches. >>> >>> Denis. >> Ok, maybe Eric or Anatoly have some preference for 4.5/4.6. > > I would like it for the 4.6 series, if possible. I think that the > next

Re: [PATCH] ARM fixed-point support [4/6]: allow overriding of fixed-point helper function names

On 05/13/11 15:58, Julian Brown wrote: > This patch allows the decoration of helper function names for > fixed-point support to be changed from the default "__foo" to a > different scheme, as required by the ARM EABI (these are > "vendor-specific" helpers, so should go in a particular namespace). >

Re: [PATCH] ARM fixed-point support [6/6]: target-specific parts

On 26/05/11 17:45, Julian Brown wrote: > On Fri, 13 May 2011 17:31:18 +0100 > Julian Brown wrote: > >> On Fri, 13 May 2011 14:54:47 + (UTC) >> "Joseph S. Myers" wrote: >> >>> On Fri, 13 May 2011, Julian Brown wrote: >>> although Joseph's comments have (hopefully) all been addressed. >>>

Re: [PATCH, PR 49094] Refrain from creating misaligned accesses in SRA

Hi, I had to add a test that the analyzed expression is not an SSA name. This has been approved by Rchi on IRC yesterday. Thus, I have committed the following as revision 175703 after successful run of c and c++ testsuite on sparc64 (and a bootstrap and test with another patch on x86_64-linux).

Re: [PATCH] ARM fixed-point support [5.5/6]: argument & return padding for libcalls

On 26/05/11 17:56, Julian Brown wrote: > This patch allows padding to be specified per-target for libcalls. This > hasn't been traditionally important, because libcalls haven't accepted > quantities which might need padding, but that's no longer true with the > new(-ish) fixed-point support helper

Re: [PATCH] ARM fixed-point support [4/6]: allow overriding of fixed-point helper function names

On 13/05/11 14:58, Julian Brown wrote: > This patch allows the decoration of helper function names for > fixed-point support to be changed from the default "__foo" to a > different scheme, as required by the ARM EABI (these are > "vendor-specific" helpers, so should go in a particular namespace). >

Re: [PATCH] ARM fixed-point support [3/6]: missing FIXED_CST in tree-ssa-sccvn.c:copy_reference_ops_from_ref

On 13/05/11 14:57, Julian Brown wrote: > This patch adds FIXED_CST to copy_reference_ops_from_ref in > tree-ssa-sccvn.c. Some test code (using fixed-point constants) failed > for me without it. This looks obvious to me, unless there's some > particular reason fixed-point constants need to be handle

Re: [PATCH] ARM fixed-point support [2/6]: assembler format for fixed-point constants

On 13/05/11 14:57, Julian Brown wrote: > This patch changes the format used to print fixed-point constants from > hex to decimal. For instance a negative "short fract" fixed-point > constant may be represented as a signed HOST_WIDE_INT from -1 to -128, > which would currently be printed as e.g. 0xf

Re: [pph] Fixing string streaming functions and their comments (issue4654076)

On 2011/06/30 01:37:59, Gabriel Charette wrote: 2011-06-29 Gabriel Charette * pph-streamer.h (struct pph_stream): Fix comment of data_in field. (pph_out_string_with_length): lto_output_string_with_length now handles NULL strings, call it

Re: [PATCH] ARM fixed-point support [1/6]: comparison optab biasing

On 13/05/11 14:57, Julian Brown wrote: > As per Joseph's suggestion here: > > http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00846.html > > I am re-posting machine-independent parts of my ARM fixed-point support > patches, with explanations as to why each part is necessary. > > This is the first:

Re: [PATCH, ARM] Avoid element-order-dependent operations for quad-word vectors in big-endian mode for NEON

On 12/01/11 17:38, Julian Brown wrote: > On Thu, 9 Dec 2010 14:56:39 + > Julian Brown wrote: > >> On Thu, 9 Dec 2010 14:40:59 + (UTC) >> "Joseph S. Myers" wrote: >> >>> On Thu, 9 Dec 2010, Julian Brown wrote: >>> Unfortunately for C only, since building C++ was broken at the ti

[Patch, AVR, longlong.h]: Implement __clrsbhi2.

The new count leading signs is not implemented for HI: builtin-bitops-1.c:(.text+0xd1e): undefined reference to `__clrsbhi2' FAIL: gcc.c-torture/execute/builtin-bitops-1.c compilation, -O0 Extended clz/ctz support in longlong.h With the patch testcase passes. Johann libgcc/ * config/a

Re: PATCH] PR 49580

Hi, > This patch fixes the build failure of gcc spec2006 benchmark. > The change is in gimple_duplicate_sese_tail(), where we need to subtract > 1 from the loop's number of iterations. > The stmt created to change the rhs of the loop's condition, used to be > inserted right after the defining s

Re: [PATCH 00/12] Eliminate the text-based dwarf2out interface

On 06/29/11 23:49, Richard Henderson wrote: > In preparation for the changes required to support shrink-wrapping, > it makes things Much Easier if we no longer have to handle seeing > new unwind states being introduced during pass_final, while emitting > the actual assembly. Thus we want to conver

Re: PATCH] PR 49580

On Thu, Jun 30, 2011 at 2:00 PM, Razya Ladelsky wrote: > Hi, > > This patch fixes the build failure of gcc spec2006 benchmark. > The change is in  gimple_duplicate_sese_tail(), where we need to subtract > 1 from the loop's number of iterations. > The stmt created to change the rhs of the loop's co

PATCH] PR 49580

Hi, This patch fixes the build failure of gcc spec2006 benchmark. The change is in gimple_duplicate_sese_tail(), where we need to subtract 1 from the loop's number of iterations. The stmt created to change the rhs of the loop's condition, used to be inserted right after the defining stmt of the

Re: [patch tree-optimization]: Do bitwise operator optimizations for X op !X patterns

On Wed, Jun 29, 2011 at 3:00 PM, Kai Tietz wrote: > - Original Message - > From: "Kai Tietz" > To: "Richard Guenther" > Cc: gcc-patches@gcc.gnu.org > Sent: Wednesday, June 29, 2011 1:33:30 PM > Subject: Re: [patch tree-optimization]: Do bitwise operator optimizations for > X op !X patte

Re: [trans-mem] Beginning of refactoring

On Wed, 2011-06-29 at 16:01 -0700, Richard Henderson wrote: > On 05/25/2011 02:10 PM, Torvald Riegel wrote: > > Patch 2: _ITM_dropReferences is not sufficiently defined in the ABI. It > > seems to target some form of open nesting for txnal wrappers, but the > > prose in the ABI specification is unc

New German PO file for 'gcc' (version 4.6.0)

Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the German team of translators. The file is available at: http://translationproject.org/latest/gcc/de.po (This file, 'gcc-4.6.0.de.po', has just

Re: [Patch] Fix objc/48109

On Thu, Jun 30, 2011 at 12:50 PM, Iain Sandoe wrote: > > On 30 Jun 2011, at 11:27, Richard Guenther wrote: > >> On Wed, Jun 29, 2011 at 7:47 PM, Iain Sandoe >> wrote: >>> >>> The bug arises because of the use, by the ObjC FE, of two old target >>> macros >>> that emit efficient representations of

Re: [Patch] Fix objc/48109

On 30 Jun 2011, at 11:27, Richard Guenther wrote: On Wed, Jun 29, 2011 at 7:47 PM, Iain Sandoe wrote: The bug arises because of the use, by the ObjC FE, of two old target macros that emit efficient representations of class definitions and references. This 'works fine' (however wrong it m

[committed] Fix BUILT_IN_ASSUME_ALIGNED handling in find_func_aliases_for_builtin_call

Hi! __builtin_assume_aligned here needs to be handled just like a lhs = dest assignment, while for memcpy/etc. resp. memset/etc. it was doing some further processing, undesirable for this builtin. Fixed thusly, tested on x86_64-linux, acked by Richi on IRC, committed to trunk. 2011-06-30 Jakub J

Re: [PATCH] Add support on powerpc to change CASE_VALUES_THRESHOLD

On Thu, Jun 30, 2011 at 12:34 AM, Michael Meissner wrote: > On the powerpc, switch statements can be expensive, and we would like to be > able to tune the threshold of when the compiler generates if statements > vs. using a table jump operation (and different processors within the powerpc > have d

Re: [Patch] Fix objc/48109

On Wed, Jun 29, 2011 at 7:47 PM, Iain Sandoe wrote: > The bug arises because of the use, by the ObjC FE, of two old target macros > that emit efficient representations of class definitions and references. > > This 'works fine' (however wrong it might be conceptually), until LTO is > engaged, where

Re: [PATCH] Sign extend before converting constants to GMP values.

On Wed, 29 Jun 2011, Sebastian Pop wrote: > Hi, > > This patch fixes PR47653 by sign extending the double int constants > before converting them to a GMP value. There still are some places > where we should not sign extend the values converted: upper bounds of > unsigned types should for example

Re: [Design notes, RFC] Address-lowering prototype design (PR46556)

On Wed, Jun 29, 2011 at 8:18 PM, William J. Schmidt wrote: > On Tue, 2011-06-14 at 15:39 +0200, Richard Guenther wrote: >> On Fri, Jun 10, 2011 at 5:11 PM, William J. Schmidt >> wrote: >> > On Tue, 2011-06-07 at 16:49 +0200, Richard Guenther wrote: >> >> On Tue, Jun 7, 2011 at 4:14 PM, William J.

Re: [PATCH] Reintroduce repeat field for ctors (PR fortran/49540)

On 06/29/2011 11:35 PM, Jakub Jelinek wrote: Bootstrapped/regtested on x86_64-linux and i686-linux. Ok for trunk and after a while for 4.6 too? OK - thanks for the quick patch! @@ -4608,29 +4608,56 @@ gfc_conv_array_initializer (tree type, g index = gfc_conv_mpz_to_tree (c->off

[Patch, AVR, 4.6+trunk]: PR44643 addendum

avr_insert_attributes uses TREE_READONLY on get readonlyness of node. That does not work for C++ arrays: it gives false error "variable must be const in order to be put into read-only section by means of '__attribute__((progmem))'". This patch peels arrays and uses TYPE_READONLY. I did not open

  1   2   >