[patch tree-ssa-math-opts]: Improve bswap detection

Hello, this patch improves the bswap (32/64) detection and fixes an issue about shifted bit values out of type-size precision. ChangeLog 2011-06-27 Kai Tietz * tree-ssa-math-opts.c (do_shift_rotate): Zero bits out of type precision after operation. (find_bswap): Take

Re: [PATCH (0/7)] Improve use of Widening Multiplies

On 25/06/11 15:12, Bernd Schmidt wrote: That all sounds good, but missing from this list is something that occurs on many CPUs - widening from the high part of a register. The current machinery only recognizes lowxlow widening multiplication, but hardware often exists for highxlow and highxhigh.

Re: [Patch, AVR]: Fix PR46779

Denis Chertykov wrote: > 2011/6/26 Georg-Johann Lay : >> Denis Chertykov schrieb: >>> 2011/6/24 Richard Henderson : >>> On 06/23/2011 01:15 PM, Denis Chertykov wrote: >> textdata bss dec hex filename >> 10032 25 0 100572749 bld-avr-orig/gcc/z.o >>

Re: varpool alias reorg

On Thu, 23 Jun 2011, Jan Hubicka wrote: > > On Sat, Jun 18, 2011 at 7:19 AM, H.J. Lu wrote: > > > On Sat, Jun 18, 2011 at 1:32 AM, Jan Hubicka wrote: > > >> Hi, > > >> this patch makes symetric changes to varpool as did the prevoius series > > >> to cgraph. > > >> Basically the aliases are now

Re: varpool alias reorg

> > There are two problems here > > 1) We do not stream builtin decls and merge them outside lto-symtab (by > > just > > streaming references to builtins with their asm names). There is at > > least > > one extra PR related to this and on my TODO is to simply remove the > > code. >

Re: PR tree-optimize/49373 (IPA-PTA regression)

On Sat, 25 Jun 2011, Jan Hubicka wrote: > Hi, > just for those who are interested, this is quick&dirty patch adding another > pass of local optimization passes at WPA time. I've added early inliner and > IPA-SRA because I was curious how much of optimization oppurtunities we are > missing by limi

Re: [patch tree-ssa-math-opts]: Improve bswap detection

On Mon, Jun 27, 2011 at 10:06 AM, Kai Tietz wrote: > Hello, > > this patch improves the bswap (32/64) detection and fixes an issue about > shifted bit values out of type-size precision. > > ChangeLog > > 2011-06-27  Kai Tietz   > >        * tree-ssa-math-opts.c (do_shift_rotate): Zero bits >    

Re: [patch tree-ssa-math-opts]: Improve bswap detection

2011/6/27 Richard Guenther : > On Mon, Jun 27, 2011 at 10:06 AM, Kai Tietz wrote: >> Hello, >> >> this patch improves the bswap (32/64) detection and fixes an issue about >> shifted bit values out of type-size precision. >> >> ChangeLog >> >> 2011-06-27  Kai Tietz   >> >>        * tree-ssa-math-o

Re: [patch tree-ssa-math-opts]: Improve bswap detection

On Mon, Jun 27, 2011 at 11:50 AM, Kai Tietz wrote: > 2011/6/27 Richard Guenther : >> On Mon, Jun 27, 2011 at 10:06 AM, Kai Tietz wrote: >>> Hello, >>> >>> this patch improves the bswap (32/64) detection and fixes an issue about >>> shifted bit values out of type-size precision. >>> >>> ChangeLog

Re: [Patch, AVR]: Fix PR46779

2011/6/27 Georg-Johann Lay : > Denis Chertykov wrote: >> 2011/6/26 Georg-Johann Lay : >>> Denis Chertykov schrieb: 2011/6/24 Richard Henderson : > On 06/23/2011 01:15 PM, Denis Chertykov wrote: > >>>  text    data     bss     dec     hex filename >>> 10032      25       0

Re: [patch tree-ssa-forwprop]: Improve binary and/or/xor folding

On Wed, Jun 22, 2011 at 3:09 PM, Kai Tietz wrote: > Hello, > > This patch improves via type-sinking folding of binary and, or, and > xor operations. > First we do sinking also for compatible types with same precision, as > those don't need to be preserved for these operations. > Additional try to

Re: Ping: The TI C6X port

On 06/14/11 12:40, Bernd Schmidt wrote: > Ping^5 for the C6X port. >> regrename across basic block boundaries: >> http://gcc.gnu.org/ml/gcc-patches/2011-05/msg02193.html http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01595.html >> >>> 6/11: REG_WORDS_BIG_ENDIAN >>> http://gcc.gnu.org/ml/gcc-patches/2

Re: PR tree-optimize/49373 (IPA-PTA regression)

> On Sat, 25 Jun 2011, Jan Hubicka wrote: > > > Hi, > > just for those who are interested, this is quick&dirty patch adding another > > pass of local optimization passes at WPA time. I've added early inliner and > > IPA-SRA because I was curious how much of optimization oppurtunities we are > > m

Re: [PATCH] __builtin_assume_aligned

On Fri, Jun 24, 2011 at 4:22 PM, Jakub Jelinek wrote: > Hi! > > This patch introduces a new extension, to hint the compiler > that a pointer is guaranteed to be somehow aligned (or misaligned). > It is designed as a pass-thru builtin which just returns its first > argument, so that it is more obvi

Re: PR tree-optimize/49373 (IPA-PTA regression)

On Mon, 27 Jun 2011, Jan Hubicka wrote: > > On Sat, 25 Jun 2011, Jan Hubicka wrote: > > > > > Hi, > > > just for those who are interested, this is quick&dirty patch adding > > > another > > > pass of local optimization passes at WPA time. I've added early inliner > > > and > > > IPA-SRA becaus

[PATCH] Fix PR49536

This fixes PR49536 by making sure to use only non-aggregate component types when building vector types. If we try to vectorize a load of an aggregate with SImode use a vector with a component type corresponding with that mode, not the aggregate type. This way we'll still vectorize memory copy lo

Re: [patch tree-ssa-forwprop]: Improve binary and/or/xor folding

2011/6/27 Richard Guenther : > On Wed, Jun 22, 2011 at 3:09 PM, Kai Tietz wrote: >> Hello, >> >> This patch improves via type-sinking folding of binary and, or, and >> xor operations. >> First we do sinking also for compatible types with same precision, as >> those don't need to be preserved for t

Re: [RFC] Fix unwind info for sparc -mflat

> I have 4 regressions in the C testsuite with -mflat on SPARC/Solaris: And 2 regressions in 64-bit mode: FAIL: gcc.c-torture/execute/nestfunc-6.c execution, -O1 [...] FAIL: gcc.dg/torture/stackalign/nested-6.c -O1 execution test [...] Another latent problem exposed by the change. Tested o

Unreviewed build/libgcc, mudflap patches

The following three patches have remained unreviewed for a week: [build] Move unwinder to toplevel libgcc http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01452.html This needs at least a build and/or libgcc maintainer, though a global reviewer might be helpful. I'm not Cc'ing the va

Patch ping #3

Extend TYPE_DECL_IS_STUB trick (1 line): http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00683.html Emit DW_AT_artificial for types: http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00700.html Fix PR lto/48492 (bis) (1 line): http://gcc.gnu.org/ml/gcc-patches/2011-04/msg01461.html Thanks in advan

Re: Unreviewed build/libgcc, mudflap patches

Hi - On Mon, Jun 27, 2011 at 01:32:30PM +0200, Rainer Orth wrote: > [...] > Support libmudflap on Solaris (PR libmudflap/38738) > http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01446.html Looks good to me, thanks for taking a look at the area. - FChE

Re: Unreviewed build/libgcc, mudflap patches

Hi Frank, > On Mon, Jun 27, 2011 at 01:32:30PM +0200, Rainer Orth wrote: >> [...] >> Support libmudflap on Solaris (PR libmudflap/38738) >> http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01446.html > > Looks good to me, thanks for taking a look at the area. Could you also comment on th

[wwwdocs] Updated GCC 4.6 status on the index pages.

Applied. Gerald Index: index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v retrieving revision 1.801 diff -u -3 -p -r1.801 index.html --- index.html 9 May 2011 00:31:43 - 1.801 +++ index.html 27 Jun 2011 12:06:1

[alpha/vms]: committed: always generate .end directive

Hi, -finhibit-size-directive prevented the generation of .end directive on alpha-vms. This doesn't make sense as .end does almost nothing and will be required by newly version of gas. Committed on trunk. Tristan. 2011-06-27 Tristan Gingold * config/alpha/alpha.c (alpha_end_functio

[MELT] Separation between build and install

Hello, While packaging GCC MELT as a plugin for Mandriva, I noticed that we could do a slight improvement to the current build script to ease a bit packaging, by adding a clear separation between the building steps and installation steps. This allows to "cleanly" exploit the build script in both %

[PATCH] MELT: Separate build and install steps

In order to ease a bit packaging, this commit add two switches to the current build infrastructure of "GCC MELT as plugin": - "-b" which runs the script to build GCC MELT - "-i" which runs the script to install GCC MELT Both are mutually exclusive. --- ChangeLog.MELT |4 c

[vms] Rewrite CRTL mapping

Hi, one of the particularities of VMS is the use of prefix for the C runtime (C-RTL). For example, the function fputs() is implemented by the routine decc$fputs. The map is now written in a text file (vms-crtlmap.map) which is translate during build by a script. There is now only one map file

Re: [patch tree-ssa-forwprop]: Improve binary and/or/xor folding

On Mon, Jun 27, 2011 at 12:58 PM, Kai Tietz wrote: > 2011/6/27 Richard Guenther : >> On Wed, Jun 22, 2011 at 3:09 PM, Kai Tietz wrote: >>> Hello, >>> >>> This patch improves via type-sinking folding of binary and, or, and >>> xor operations. >>> First we do sinking also for compatible types with

Re: Fix ACATS c94007a on 4.6 branch for SPARC64/Solaris

> 2011-06-21 Eric Botcazou > > * reorg.c (fill_simple_delay_slots): Use stop_search_p to stop the > forward scan as well. Applied on the mainline and 4.6 branch after re-bootstrapping on SPARC/Solaris. -- Eric Botcazou

Re: PR 49169: testing the alignment of a function

On Fri, Jun 24, 2011 at 8:50 AM, Richard Guenther wrote: > On Fri, Jun 24, 2011 at 4:42 PM, Richard Sandiford > wrote: >> This patch fixes PR 49169, where GCC is incorrectly optimising away >> a test for whether a function is Thumb rather than ARM.  The patch >> was posted by Richard in the PR: >

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

On Sun, 26 Jun 2011, Martin Jambor wrote: > Hi, > > under some circumstances involving user specified alignment and/or > packed attributes, SRA can create a misaligned MEM_REF. As the > testcase demonstrates, it is not enough to not consider variables with > these type attributes, mainly because

Re: varpool alias reorg

On Fri, 24 Jun 2011, Jan Hubicka wrote: > Hi, > this is yet another variant of the fix. This time we stream builtins decls as > usually, but at fixup time we copy the assembler names (if set) into the > builtin decls used by folders. Not sure if it is any better than breaking > memops-asm, but I

[patch tree-optimization]: Try to sink type-casts for binary and/or/xor operations

Hello, this patch sink type conversions in forward-propagate for the following patterns: - ((type) X) op ((type) Y): If X and Y have compatible types. - ((type) X) op CST: If the conversion of (type) ((type-x) CST) == CST and X has integral type. - CST op ((type) X): If the conversion of (type)

Re: Patch ping #3

On 06/27/2011 07:47 AM, Eric Botcazou wrote: Extend TYPE_DECL_IS_STUB trick (1 line): http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00683.html OK. Emit DW_AT_artificial for types: http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00700.html OK. Fix PR lto/48492 (bis) (1 line): http://g

Re: [patch tree-optimization]: Try to sink type-casts for binary and/or/xor operations

On Mon, Jun 27, 2011 at 3:46 PM, Kai Tietz wrote: > Hello, > > this patch sink type conversions in forward-propagate for the following > patterns: > - ((type) X) op ((type) Y): If X and Y have compatible types. > - ((type) X) op CST: If the conversion of (type) ((type-x) CST) == CST and X > has

[PATCH] Fixup current_pass after running IPA transforms

Running IPA transforms will mess up current_pass (and reset it to NULL), ICEing later for IPA PTA. I also don't see any code that would avoid doing IPA transforms twice in case we put another small IPA pass in the late IPA pass queue. Honza, I guess we need to add some PROP_ipa_transforms_applie

Re: [pph] Stream chain of struct fields (issue4631072)

On Fri, Jun 24, 2011 at 19:13, Gabriel Charette wrote: > We were only streaming the first field of every struct. Struct fields have a > chain link to the next field, thus we need to stream the DECL_CHAIN of every > field as well recursively. > > I limited this to VAR_DECL and FUNCTION_DECL for n

Re: Unreviewed build/libgcc, mudflap patches

Hi - > >> http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01446.html > > Looks good to me, thanks for taking a look at the area. > > Could you also comment on the open questions in the submission so > mudflap support on Solaris can be improved further? Sure. Transcribing: > One generic is

Re: PATCH [10/n]: Prepare x32: PR rtl-optimization/49114: Reload failed to handle (set reg:X (plus:X (subreg:X (reg:Y) 0) (const_int

H.J. Lu wrote: > When reload gets: > > (insn 588 587 589 28 (set (mem:DF (zero_extend:DI (plus:SI (subreg:SI > (reg/v/f:DI 182 [ b ]) 0) > (const_int 8 [0x8]))) [4 MEM[base: b_96(D), index: > D.15020_278, step: 8, offset: 0B]+0 S8 A64]) > (reg:DF 340 [ D.14980 ])) spo

Re: PATCH [11/n]: Prepare x32: PR rtl-optimization/48155: Reload doesn't handle subreg properly

H.J. Lu wrote: > Given input: > > (plus:SI (subreg:SI (plus:DI (reg/f:DI 7 sp) > (const_int 16 [0x10])) 0) > (const_int -1 [0x])) Once again, this seems weird as legitimate address ... If this really can occur validly, there'll probably need to be an insn+splitte

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

On 06/26/2011 05:58 PM, H.J. Lu wrote: The current unwind library scheme provides only one unwind context and is backward compatible with multiple different unwind contexts from multiple unwind libraries: http://gcc.gnu.org/ml/gcc-patches/2006-12/msg01769.html My patch fixes UNITS_PER_WORD > si

Re: RFA; MN10300: Fix AM33 clzsi2 pattern

On 06/25/2011 09:29 PM, Nick Clifton wrote: > * mn10300.md (clzsi2): Use XOR after BSCH to convert bit position > of highest bit set into a count of the high zero bits. Ok. r~

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

On Wed, 22 Jun 2011 17:42:34 +0200, Jason Merrill wrote: > Well, the basic issue is that the "linkage name" is produced by > libiberty/cp-demangle.c and the DW_AT_name is produced by > gcc/cp/error.c, and they don't always agree on the same > pretty-printed representation of a C++ expression. File

Re: [PATCH] Fixup current_pass after running IPA transforms

> > Running IPA transforms will mess up current_pass (and reset it > to NULL), ICEing later for IPA PTA. I also don't see any > code that would avoid doing IPA transforms twice in case > we put another small IPA pass in the late IPA pass queue. Honza, > I guess we need to add some PROP_ipa_trans

PING Re: PATCH: fix collect2 handling of --demangle and --no-demangle

Ping? http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01368.html We had a bug report from a customer that the linker was ignoring the --demangle and --no-demangle options when generating map files. Moreover, it was failing in a host-dependent way; on Windows hosts, it was always emitting demangled

Re: [patch tree-optimization]: Try to sink type-casts for binary and/or/xor operations

2011/6/27 Richard Guenther : > On Mon, Jun 27, 2011 at 3:46 PM, Kai Tietz wrote: >> Hello, >> >> this patch sink type conversions in forward-propagate for the following >> patterns: >> - ((type) X) op ((type) Y): If X and Y have compatible types. >> - ((type) X) op CST: If the conversion of (type

Re: [patch tree-optimization]: Try to sink type-casts for binary and/or/xor operations

- Original Message - From: "Richard Guenther" To: "Kai Tietz" Cc: gcc-patches@gcc.gnu.org Sent: Monday, June 27, 2011 4:08:41 PM Subject: Re: [patch tree-optimization]: Try to sink type-casts for binary and/or/xor operations On Mon, Jun 27, 2011 at 3:46 PM, Kai Tietz wrote: > Hello, >

Re: varpool alias reorg

> On Fri, 24 Jun 2011, Jan Hubicka wrote: > > > Hi, > > this is yet another variant of the fix. This time we stream builtins decls > > as > > usually, but at fixup time we copy the assembler names (if set) into the > > builtin decls used by folders. Not sure if it is any better than breaking >

Re: PATCH [11/n]: Prepare x32: PR rtl-optimization/48155: Reload doesn't handle subreg properly

On Mon, Jun 27, 2011 at 7:52 AM, Ulrich Weigand wrote: > H.J. Lu wrote: > >> Given input: >> >> (plus:SI (subreg:SI (plus:DI (reg/f:DI 7 sp) >>             (const_int 16 [0x10])) 0) >>     (const_int -1 [0x])) > > Once again, this seems weird as legitimate address ...  If this real

Re: varpool alias reorg

Hi, On Mon, 27 Jun 2011, Jan Hubicka wrote: > > I still like to stream unmodified builtins as builtins, as that is > > similar to pre-loading the streamer caches with things like > > void_type_node or sizetype. > > Doing so will need us to solve the other one decl rules probly. I didn't > rea

[RFC] Fix full memory barrier on SPARC-V8

The memory_barrier pattern expands to UNSPEC_MEMBAR on the SPARC and the latter is implemented differently for V8 and V9: (define_insn "*stbar" [(set (match_operand:BLK 0 "" "") (unspec:BLK [(match_dup 0)] UNSPEC_MEMBAR))] "TARGET_V8" "stbar" [(set_attr "type" "multi")]) ;; memba

Re: Simplify Solaris configuration

Eric, >> All bootstraps completed without regressions, so I've installed the >> patch. I'll address eventual issues and further simplifications as a >> followup. > > I cannot bootstrap SPARC64/Solaris anymore though: [...] > from /nile.build/botcazou/gcc-head/src/libgcc/../gcc/libgcc2.c:29: > /us

[pph] Add header files to pph.map (issue4639073)

When I added these tests, I forgot to add them to pph.map, so the header files were being generated into an image, but not used in the second compilation. Most of the failures are in the form of different asm output. There is one new ICE, though. Committed to branch. Diego. * g++.dg/

[pph] New test (issue4629075)

We are very close to having the first simple C++ program working from a PPH image. Adding a runnable test case. This produces assembly miscomparisons and does not link. I'll be working on it this week. Diego. * g++.dg/pph/x1ten-hellos.cc: New. * g++.dg/pph/x1ten-hellos.h: New.

Re: [PATCH] Use TYPE_NEXT_VARIANT instead of TREE_CHAIN as chain_next for types during GC (PR c++/46400)

On Fri, Jun 24, 2011 at 03:21:07PM -0700, Richard Henderson wrote: > On 06/24/2011 07:43 AM, Jakub Jelinek wrote: > > + chain_next ("CODE_CONTAINS_STRUCT (TREE_CODE (&%h.generic), > > TS_TYPE_COMMON) ? ((union lang_tree_node *) TYPE_NEXT_VARIANT > > (&%h.generic)) : CODE_CONTAINS_STRUCT (TR

[Patch, AVR]: AVR backend cleanup: use elfos.h

This patch removes ELF dependencies from avr.h and redundancies from avr.h and avr.c The AVR BE defines many things that are readily available in elfos.h. ELF is the only target supported by avr-gcc. The patch includes elfos.h in config.gcc for all avr* targets together with a new file avr/elf.h.

[committed] Fix promote_debug_loc (PR debug/49544)

Hi! When I've added adding subregs of wider value to cselib_lookup_1 in http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=174508, I've mistakenly only adjusted the assertion, while we need in that case to adjust the second location's setting_insn as well. Fixed thusly, bootstrapped/regtested on x8

Re: [PATCH] Use TYPE_NEXT_VARIANT instead of TREE_CHAIN as chain_next for types during GC (PR c++/46400)

On 06/27/2011 09:42 AM, Jakub Jelinek wrote: > 2011-06-27 Jakub Jelinek > > * c-common.h (c_tree_chain_next): New static inline function. > > * c-decl.c (union lang_tree_node): Use it in chain_next expression. > > * cp-tree.h (union lang_tree_node): Use it in chain_next expr

Re: [PATCH] __builtin_assume_aligned

On Mon, Jun 27, 2011 at 12:17:40PM +0200, Richard Guenther wrote: > Ok if you remove the builtins.c folding and instead verify arguments > from check_builtin_function_arguments. Thanks, here is what I've committed after bootstrapping/regtesting again on x86_64-linux and i686-linux. 2011-06-27 Ja

Re: Unreviewed build/libgcc, mudflap patches

Hi Rainer, On 27 Jun 2011, at 12:32, Rainer Orth wrote: The following three patches have remained unreviewed for a week: [build] Move unwinder to toplevel libgcc http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01452.html This needs at least a build and/or libgcc maintainer, though

Re: [Patch, AVR]: AVR backend cleanup: use elfos.h

2011/6/27 Georg-Johann Lay : > This patch removes ELF dependencies from avr.h and redundancies from > avr.h and avr.c > > The AVR BE defines many things that are readily available in elfos.h. > ELF is the only target supported by avr-gcc. > > The patch includes elfos.h in config.gcc for all avr* ta

Re: [pph] Add header files to pph.map (issue4639073)

Couldn't we have headers look for their corresponding .pph file by default when -fpph-map is on? (especially since pph.map is only temporary for the implementation phase) I would see it like this: if -fpph-map is not NULL: look for mapping in file if not found: look for default mapping (i.

Re: PATCH [11/n]: Prepare x32: PR rtl-optimization/48155: Reload doesn't handle subreg properly

H.J. Lu wrote: > On Mon, Jun 27, 2011 at 7:52 AM, Ulrich Weigand wrote: > > H.J. Lu wrote: > > > >> Given input: > >> > >> (plus:SI (subreg:SI (plus:DI (reg/f:DI 7 sp) > >> =A0 =A0 =A0 =A0 =A0 =A0 (const_int 16 [0x10])) 0) > >> =A0 =A0 (const_int -1 [0x])) > > > > Once again, this

Re: [patch tree-optimization]: Try to sink type-casts for binary and/or/xor operations

Hi, so I modified patch to use int_fits_type_p() for integer CST checking. Well, this approach is - as discussed on IRC suboptimal - as my intial approach was for and-operations with precision type > precision type-x and unsigned type-x for constant values bigger then (type-x)~0. But well, tho

Re: [pph] Add header files to pph.map (issue4639073)

On Mon, Jun 27, 2011 at 13:01, Gabriel Charette wrote: > Couldn't we have headers look for their corresponding .pph file by > default when -fpph-map is on? (especially since pph.map is only > temporary for the implementation phase) The problem is with headers that include other headers. We want

Re: [pph] New test (issue4629075)

Just wondering why you're naming x finishing by an underscore "x_", this is a valid name, but just thinking it's tricky syntax, does this test anything more? or is it just a preference for private members? LGTM On Mon, Jun 27, 2011 at 9:39 AM, Diego Novillo wrote: > We are very close to having t

Re: [pph] New test (issue4629075)

On Mon, Jun 27, 2011 at 13:06, Gabriel Charette wrote: > Just wondering why you're naming x finishing by an underscore "x_", > this is a valid name, but just thinking it's tricky syntax, does this > test anything more? or is it just a preference for private members? No real reason. Just a quick

Re: [PING] [PATCH, libstdc++-v3] Add newlib specific ctype_members.cc

Hi, Ping for: http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00440.html personally, I have only minor comments, like only 2011 as Copyright year for new files, and please also regtest on a gnu-linux system. Before going ahead, however, you should send the patch also to the libstdc++ mailing lis

Re: [pph] Add header files to pph.map (issue4639073)

On Mon, Jun 27, 2011 at 10:06 AM, Diego Novillo wrote: > On Mon, Jun 27, 2011 at 13:01, Gabriel Charette wrote: >> Couldn't we have headers look for their corresponding .pph file by >> default when -fpph-map is on? (especially since pph.map is only >> temporary for the implementation phase) > > T

Re: [patch tree-optimization]: Try to sink type-casts for binary and/or/xor operations

Ups, missed to update patch. Kai - Original Message - From: "Kai Tietz" To: "Richard Guenther" Cc: gcc-patches@gcc.gnu.org Sent: Monday, June 27, 2011 7:04:04 PM Subject: Re: [patch tree-optimization]: Try to sink type-casts for binary and/or/xor operations Hi, so I modified patch to

Re: [pph] Add header files to pph.map (issue4639073)

On Mon, Jun 27, 2011 at 13:13, Gabriel Charette wrote: > On Mon, Jun 27, 2011 at 10:06 AM, Diego Novillo wrote: >> On Mon, Jun 27, 2011 at 13:01, Gabriel Charette wrote: >>> Couldn't we have headers look for their corresponding .pph file by >>> default when -fpph-map is on? (especially since pph

[google] Do not execute pass_warn_function_return after a syntax error (issue4629076)

This patch is the compromise solution for the problem described in http://gcc.gnu.org/ml/gcc/2011-06/msg00213.html. We should not try to analyze the IL in the presence of syntax errors. The full solution is simpler, but it changes diagnostic behaviour too much for the release branch, so this patc

Re: [pph] Add header files to pph.map (issue4639073)

On Mon, Jun 27, 2011 at 10:18 AM, Diego Novillo wrote: > On Mon, Jun 27, 2011 at 13:13, Gabriel Charette wrote: >> On Mon, Jun 27, 2011 at 10:06 AM, Diego Novillo wrote: >>> On Mon, Jun 27, 2011 at 13:01, Gabriel Charette wrote: Couldn't we have headers look for their corresponding .pph fi

{patch tree-ssa-math-opts]: Change searching direction for bswap

Hello, this is the separated patch for issues noticed by doing type-sinking on bitwise-operations. The tests exposed that bswap pattern-matching searches from top to down for each BB. As it replaces the found match for a bswap in tree, but doesn't know about handling its inserted builtin-swap

Re: [Patch, AVR]: AVR backend cleanup: use elfos.h

Denis Chertykov wrote: > 2011/6/27 Georg-Johann Lay : >> This patch removes ELF dependencies from avr.h and redundancies from >> avr.h and avr.c >> >> The AVR BE defines many things that are readily available in elfos.h. >> ELF is the only target supported by avr-gcc. >> >> The patch includes elfos

Re: Unreviewed build/libgcc, mudflap patches

Hi Frank, >> Support libmudflap on Solaris (PR libmudflap/38738) >> http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01446.html > > Looks good to me, thanks for taking a look at the area. I've installed the patch as it, since it at least fixes the libmudflap build on Solaris. Thanks.

Re: [Patch, AVR]: AVR backend cleanup: use elfos.h

On 06/27/11 19:34, Georg-Johann Lay wrote: > Denis Chertykov wrote: >> Approved. >> > http://gcc.gnu.org/ml/gcc-patches/2011-06/msg02011.html > > Is that also for the config.gcc part? > Or does it need review from global reviewer? Denis can approve AVR changes in config.gcc. Bernd

Re: Unreviewed build/libgcc, mudflap patches

Hi Frank, > Sure. Transcribing: > >> One generic issue came up with the port: enabling or disabling >> libmudflap depends on a factor (linker support for --wrap/-z wrap) >> that is best checked dynamically, thus doesn't really seem to belong >> at the toplevel. Is there any precedent for perform

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

On 06/27/2011 11:04 AM, Jan Kratochvil wrote: Had to restrict the patch very much for the template function reference case as otherwise the parameter types are required even in templates there. They should be supressed whenever the function appears in an expression context, either as a pointer

Re: [Patch, AVR]: AVR backend cleanup: use elfos.h

2011/6/27 Georg-Johann Lay : > Denis Chertykov wrote: >> 2011/6/27 Georg-Johann Lay : >>> This patch removes ELF dependencies from avr.h and redundancies from >>> avr.h and avr.c >>> >>> The AVR BE defines many things that are readily available in elfos.h. >>> ELF is the only target supported by av

Re: Unreviewed build/libgcc, mudflap patches

Hi Iain, >> The following three patches have remained unreviewed for a week: >> >> [build] Move unwinder to toplevel libgcc >>http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01452.html >> >> This needs at least a build and/or libgcc maintainer, though a global >> reviewer might be helpfu

Re: Simplify Solaris configuration

> sorry about that: I don't test the 64-bit default configurations so far, > and didn't even have the necessary support libs around to do so. The > problem was that sparc/sol2-64.h was included too late and > DEFAULT_ARCH32_P was wrong for those configurations. > > The following patch should fix t

Re: PATCH [10/n]: Prepare x32: PR rtl-optimization/49114: Reload failed to handle (set reg:X (plus:X (subreg:X (reg:Y) 0) (const_int

On Mon, Jun 27, 2011 at 7:47 AM, Ulrich Weigand wrote: > H.J. Lu wrote: > >> When reload gets: >> >> (insn 588 587 589 28 (set (mem:DF (zero_extend:DI (plus:SI (subreg:SI >> (reg/v/f:DI 182 [ b ]) 0) >>                     (const_int 8 [0x8]))) [4 MEM[base: b_96(D), index: >> D.15020_278, step: 8,

Re: [PATCH][RFC][2/2] Bitfield lowering

On 06/23/2011 02:49 AM, Richard Guenther wrote: > So, do you know of a target that can do insv with a memory > target? I would expect extv on memory to be easily doable. Depends on what you mean by that. INSV is the traditional md pattern that performs unaligned stores. So, many targets can han

Re: PATCH [10/n]: Prepare x32: PR rtl-optimization/49114: Reload failed to handle (set reg:X (plus:X (subreg:X (reg:Y) 0) (const

H.J. Lu wrote: > On Mon, Jun 27, 2011 at 7:47 AM, Ulrich Weigand wrote: > > The actual problem > > here is that this part of gen_reload takes the approach to transform > > > > out <- op0 + op1 > > > > into > > > > out <- op0 > > out <- out + op1 > > > > which is invalid if writing to out clobbe

Re: [RFC, ARM] Convert thumb1 prologue completely to rtl

On 06/24/2011 02:59 AM, Richard Earnshaw wrote: > On 18/06/11 20:02, Richard Henderson wrote: >> I couldn't find anything terribly tricky about the conversion. >> >> The existing push_mult pattern would service thumb1 with just >> a tweak or two to the memory predicate and the length. >> >> The exi

Re: PATCH [10/n]: Prepare x32: PR rtl-optimization/49114: Reload failed to handle (set reg:X (plus:X (subreg:X (reg:Y) 0) (const

On Mon, Jun 27, 2011 at 11:28 AM, Ulrich Weigand wrote: > H.J. Lu wrote: >> On Mon, Jun 27, 2011 at 7:47 AM, Ulrich Weigand wrote: >> > The actual problem >> > here is that this part of gen_reload takes the approach to transform >> > >> >  out <- op0 + op1 >> > >> > into >> > >> >  out <- op0 >>

[pph] Moved token cache streaming to in/out respectively (issue4635073)

Moved all token cache streaming to it's respective in/out streamer. They were non-static exposed methods, but were only used in pph-streamer-in/out.c This is a pure copy/paste patch. I will change the functions names to reflect the pph naming scheme in a subsequent patch. Tested with bootstrap

[patch tree-optimization]: Try to do type sinking on comparisons

Hello, this patch tries to sink conversions for comparisons patterns: a) (type) X cmp (type) Y => x cmp y. b) (type) X cmp CST => x cmp ((type-x) CST). c) CST cmp (type) X => ((type-x) CST) cmp x. This patch just allows type sinking for the case that type-precision of type is wider or equal to t

Re: [pph] New test (issue4629075)

On 6/27/11, Diego Novillo wrote: > On Jun 27, 2011 Gabriel Charette wrote: > > Just wondering why you're naming x finishing by an underscore > > "x_", this is a valid name, but just thinking it's tricky syntax, > > does this test anything more? or is it just a preference for > > private members?

Re: [pph] New test (issue4629075)

On Mon, Jun 27, 2011 at 14:56, Lawrence Crowl wrote: > On 6/27/11, Diego Novillo wrote: >> On Jun 27, 2011 Gabriel Charette wrote: >> > Just wondering why you're naming x finishing by an underscore >> > "x_", this is a valid name, but just thinking it's tricky syntax, >> > does this test anythin

Re: PATCH [10/n]: Prepare x32: PR rtl-optimization/49114: Reload failed to handle (set reg:X (plus:X (subreg:X (reg:Y) 0) (const

H.J. Lu wrote: > Reloads for insn # 588 > Reload 0: reload_in (DI) =3D (reg/v/f:DI 182 [ b ]) > GENERAL_REGS, RELOAD_FOR_OPERAND_ADDRESS (opnum =3D 0) > reload_in_reg: (reg/v/f:DI 182 [ b ]) > reload_reg_rtx: (reg:DI 1 dx) > Reload 1: reload_in (DI) =3D (zero_extend:DI (plu

Re: [pph] Add header files to pph.map (issue4639073)

On 6/27/11, Gabriel Charette wrote: > On Jun 27, 2011 Diego Novillo wrote: > > On Jun 27, 2011 Gabriel Charette wrote: > > > Couldn't we have headers look for their corresponding .pph > > > file by default when -fpph-map is on? (especially since > > > pph.map is only temporary for the implementa

ia64-linux build broken

2011-06-27 Tristan Gingold PR target/44241 * config/vms/vms-protos.h: New file. * config/vms/vms-crtlmap.map: New file. * config/vms/vms.c: New file. * config/vms/make-crtlmap.awk: New file. * config/vms/vms-crtl.h: File removed. * config/

Re: [Patch, AVR]: AVR backend cleanup: use elfos.h

Denis Chertykov schrieb: 2011/6/27 Georg-Johann Lay: http://gcc.gnu.org/ml/gcc-patches/2011-06/msg02011.html [...] Is there a specific reason not to emit .ident? If not, I would remove the #undefs of IDENT_ASM_OP and ASM_OUTPUT_IDENT from avr/elf.h. Sorry, I forgot. Denis. It's just a m

Re: ia64-linux build broken

On 06/27/2011 12:11 PM, Richard Henderson wrote: > gcc -c -DUSE_LIBUNWIND_EXCEPTIONS -g -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE > -W -Wall -Wwrite-strings -Wcast-qual -Wstrict-prototypes -Wmissing-prototypes > -Wmissing-format-attribute -pedantic -Wno-long-long -Wno-variadic-macros > -Wno-overle

Re: [pph] Moved token cache streaming to in/out respectively (issue4635073)

On 2011/06/27 18:51:22, Gabriel Charette wrote: 2011-06-27 Gabriel Charette * pph-streamer-in.c (pth_get_type_from_index): Moved from pph.c. (pth_load_number): Moved from pph.c. (pth_load_token_value): Moved from pph.c. (pth_load_to

Re: Unreviewed build/libgcc, mudflap patches

On Jun 27, 2011, at 9:54 AM, Iain Sandoe wrote: > On 27 Jun 2011, at 12:32, Rainer Orth wrote: > >> The following three patches have remained unreviewed for a week: >> >> [build] Move unwinder to toplevel libgcc >> http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01452.html > As far as Da

Re: ia64-linux build broken

On 06/27/2011 12:23 PM, Richard Henderson wrote: > On 06/27/2011 12:11 PM, Richard Henderson wrote: >> gcc -c -DUSE_LIBUNWIND_EXCEPTIONS -g -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE >> -W -Wall -Wwrite-strings -Wcast-qual -Wstrict-prototypes >> -Wmissing-prototypes -Wmissing-format-attribute -pedan

Fix cgraph_address_taken_from_non_vtable_p

Hi, this patch fixes thinko in cgraph_address_taken_from_non_vtable_p that valks references in NODE instead of references of NODE. It fixes the testcase in http://sourceware.org/bugzilla/show_bug.cgi?id=12942 in non-plugin LTO and Gold, but not with GNU LD that presently handle resolution info inco

  1   2   >