Re: powerpc64le configure fix

2013-05-09 Thread Alan Modra
Another tweak for little-endian powerpc. Committed revision 198734. * configure.ac (HAVE_AS_TLS): Enable tests for powerpcle and powerpc64le. * configure: Regenerate. Index: gcc/configure.ac === --- gcc/confi

Re: new port: msp430-elf

2013-05-09 Thread Richard Sandiford
DJ Delorie writes: >> It looks like the patch also treats PSImode->SImode as a truncation though. > > I'd have to check specifically for partial_int_mode, since there's > nothing in PARTIAL_INT_MODE() to set the precision, unless there's > some other way to set PSImode's bits *and* reliably use th

Re: [Patch, Aarch64] [RFC] : Macros for profile code generation to enable gprof support

2013-05-09 Thread Marcus Shawcroft
On 9 May 2013 07:48, Venkataramanan Kumar wrote: > Hi Maintainers, > > The attach patch contains the following for aarch64 backend to enable > gprof support. > > 1. Changes to "aarch64_return_addr" to get return address from a stack frame. > 2. Defines macros associated with generating code for pr

[PATCH] PR32219, weak hidden reference segfault

2013-05-09 Thread Chung-Lin Tang
Hi, with reference to the old dicussion on PR 32219: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32219 It seems that a patch was submitted to put the DECL_WEAK check before the visibility check, but that patch was never approved or applied, due to concerns in the wording of surrounding comments: h

Re: [PATCH] Allow nested use of attributes in MD-files

2013-05-09 Thread Michael Zolotukhin
> Sweet, I like it… Thanks! Who could approve it for commit if it's ok? Best regards, Michael On 28 April 2013 23:43, Mike Stump wrote: > On Apr 26, 2013, at 8:17 AM, Michael Zolotukhin > wrote: >> This patch allows to use attributes inside other attributes in MD-files. > > Sweet, I like it…

Re: [PATCH, PR 57084] Create real cgraph node during late intraprocedural devirtualization

2013-05-09 Thread Martin Jambor
On Tue, May 07, 2013 at 11:43:33PM +0200, Jan Hubicka wrote: > > Hi, > > > > the problem in PR 57084 is that late PRE devirtualization creates a > > direct call to a decl fro which we only have an inlined call graph > > mode in the given partition. I tried to find a most universal place > > where

Re: [PATCH, PR 57084] Create real cgraph node during late intraprocedural devirtualization

2013-05-09 Thread Martin Jambor
On Thu, May 09, 2013 at 03:52:20PM +0200, Martin Jambor wrote: > On Tue, May 07, 2013 at 11:43:33PM +0200, Jan Hubicka wrote: > > > Hi, > > > > > > the problem in PR 57084 is that late PRE devirtualization creates a > > > direct call to a decl fro which we only have an inlined call graph > > > mod

Re: [patch i386 windows]: Fix for PR/56975

2013-05-09 Thread Richard Henderson
On 05/08/2013 11:41 AM, Kai Tietz wrote: > 2013-05-08 Kai Tietz > > PR target/56975 > * config/i386/cygming.h (TARGET_PECOFF): Define as true. > * config/i386/i386.h (TARGET_PECOFF): Define by default > as false. > (PIC_OFFSET_TABLE_REGNUM): Use TARGET_PECOFF. > * config

Re: [PATCH] PR55033: Fix

2013-05-09 Thread Richard Henderson
On 02/22/2013 02:51 AM, Alan Modra wrote: > Richard, can I bother you with a review request? > http://gcc.gnu.org/ml/gcc-patches/2013-02/msg00544.html Sorry for the truly excessive delay here. This patch is ok for mainline and 4.8.1. r~

PATCH to implement C++14 VLA semantics

2013-05-09 Thread Jason Merrill
At the last C++ standards meeting, we agreed to add VLAs to the language. But they're significantly different from GNU/C99 VLAs: you can't form a pointer to a VLA, or take its sizeof, or really anything other than directly use it. We also need to throw an exception if we try to create one wit

C++ PATCH for c++/57222 (bogus error with alias-template)

2013-05-09 Thread Jason Merrill
lookup_template_class wasn't dealing properly with getting a TEMPLATE_DECL that represents a template template parameter. Tested x86_64-pc-linux-gnu, applying to trunk and 4.8. commit 7eed04dd8adeea6654e9275666586e52aef0b629 Author: Jason Merrill Date: Thu May 9 09:37:53 2013 -0400 PR c

Re: PATCH to implement C++14 VLA semantics

2013-05-09 Thread Gabriel Dos Reis
On Thu, May 9, 2013 at 11:41 AM, Jason Merrill wrote: > At the last C++ standards meeting, we agreed to add VLAs to the language. > But they're significantly different from GNU/C99 VLAs: you can't form a > pointer to a VLA, or take its sizeof, or really anything other than directly > use it. We a

Re: new port: msp430-elf

2013-05-09 Thread Mike Stump
On May 9, 2013, at 2:06 AM, Richard Sandiford wrote: >> Hmmm... psi/si in memory are tieable, psi/si in registers >> aren't... what's the right thing to do here? > > I think we assume everything is tieable in memory. MODES_TIEABLE_P says > whether we should assume that a pseudo register in one

Re: RFC: PATCH to avoid linking multiple front ends at once with parallel make

2013-05-09 Thread Mike Stump
On May 8, 2013, at 8:59 PM, Jason Merrill wrote: > Thanks for the suggestion. Here's a revised patch: Nice. I too see a whole machine style lock up on linux (openSUSE 11.4) and my machine has 12G and 8*2 cores. I just ignore it and wait patiently for the storm to pass, but, it is annoying.

Re: powerpc64le configure fix

2013-05-09 Thread Edmar Wienskoski
Alan, The first hunk will match powerpc64 as well. Edmar On Thu, May 9, 2013 at 2:27 AM, Alan Modra wrote: > Another tweak for little-endian powerpc. Committed revision 198734. > > * configure.ac (HAVE_AS_TLS): Enable tests for powerpcle and > powerpc64le. > * configure:

[GOOGLE] Check conditions before calling varpool_node

2013-05-09 Thread Carrot Wei
This patch fixed google bug entry 6124850. The usage of varpool_node has some restrictions on the corresponding var decl. In LIPO mode function notice_global_symbol may call varpool_node with a decl that doesn't satisfy these restrictions since the function notice_global_symbol can be directly or

[PATCH] Rotate pattern recognition in forwprop (PR tree-optimization/57157)

2013-05-09 Thread Jakub Jelinek
Hi! The bit_rotate: code in fold-const.c handles only a subset of rotates, and e.g. with C++ FE when sizeof isn't folded early fold_binary often even isn't called with something that could recognize the rotate pattern, even if bit_rotate: has been extended. Furthermore, not all the rotate pattern

[PATCH] Improve rotation by mode bitsize - 1

2013-05-09 Thread Jakub Jelinek
Hi! This is something I've noticed while working on the rotate recognizer patch I've just posted. We emit say roll %eax instead of roll $1, %eax because the former is shorter, but emit roll $31, %eax instead of the equivalent, but shorter rorl %eax The following patch let us optimize even

Re: [PATCH] Improve rotation by mode bitsize - 1

2013-05-09 Thread Steven Bosscher
On Thu, May 9, 2013 at 8:45 PM, Jakub Jelinek wrote: > Hi! > > This is something I've noticed while working on the rotate recognizer > patch I've just posted. We emit say > roll %eax > instead of > roll $1, %eax > because the former is shorter, but emit > roll $31, %eax > instead of the equi

Re: [PATCH] Improve rotation by mode bitsize - 1

2013-05-09 Thread Jakub Jelinek
On Thu, May 09, 2013 at 08:50:59PM +0200, Steven Bosscher wrote: > On Thu, May 9, 2013 at 8:45 PM, Jakub Jelinek wrote: > > This is something I've noticed while working on the rotate recognizer > > patch I've just posted. We emit say > > roll %eax > > instead of > > roll $1, %eax > > because t

Re: LRA patch for powerpc

2013-05-09 Thread Vladimir Makarov
On 05/07/2013 03:32 PM, Steven Bosscher wrote: On Tue, May 7, 2013 at 9:18 PM, Vladimir Makarov wrote: The following patch contains some work for porting LRA to PPC. The patch adds code analogous to recent changes made for reload.c::find_reloads (it permits to fix a lot testsuite failures fo

[PATCH 6/n, i386]: Add zero-extended vec_extract patterns

2013-05-09 Thread Uros Bizjak
Hello! movd from xmm register to integer register also zero-extends the value in integer register. Also, the patch adds a couple of missing alternatives to *vec_extractv4si, we can "extract" the values using 128bit SSE shifts. 2013-05-09 Uros Bizjak * config/i386/sse.md (*vec_extractv4si_

Re: [PATCH] Rotate pattern recognition in forwprop (PR tree-optimization/57157)

2013-05-09 Thread Marc Glisse
On Thu, 9 May 2013, Jakub Jelinek wrote: Hi! The bit_rotate: code in fold-const.c handles only a subset of rotates, and e.g. with C++ FE when sizeof isn't folded early fold_binary often even isn't called with something that could recognize the rotate pattern, even if bit_rotate: has been extend

Re: [PATCH 6/n, i386]: Add zero-extended vec_extract patterns

2013-05-09 Thread Uros Bizjak
On Thu, May 9, 2013 at 9:43 PM, Uros Bizjak wrote: > movd from xmm register to integer register also zero-extends the value > in integer register. Also, the patch adds a couple of missing > alternatives to *vec_extractv4si, we can "extract" the values using > 128bit SSE shifts. > > 2013-05-09 Ur

Re: [PATCH] Improve rotation by mode bitsize - 1

2013-05-09 Thread Uros Bizjak
On Thu, May 9, 2013 at 8:45 PM, Jakub Jelinek wrote: > This is something I've noticed while working on the rotate recognizer > patch I've just posted. We emit say > roll %eax > instead of > roll $1, %eax > because the former is shorter, but emit > roll $31, %eax > instead of the equivalent

Re: [GOOGLE] Check conditions before calling varpool_node

2013-05-09 Thread Xinliang David Li
This is not correct. current_module_id is used only in FE parsing. The real question is why the decl is created, neither static nor external? David On Thu, May 9, 2013 at 11:39 AM, Carrot Wei wrote: > This patch fixed google bug entry 6124850. > > The usage of varpool_node has some restrictions

[PATCH, v3]: Add CXXABI_1.3.8 to util/testsuite_abi.cc to fix libstdc++-abi/abi_check FAIL

2013-05-09 Thread Uros Bizjak
Hello! We have to handle CXXABI_1.3.8 in util/testsuite_abi.cc. (Also, reorder entries a bit for better grouping). 2013-05-09 Uros Bizjak * testsuite/util/testsuite_abi.cc (check_version): Add CXXABI_1.3.8. Testing on x86_64-pc-linux-gnu in progress. OK for mainline if it passes? Ur

Re: [PATCH] PR32219, weak hidden reference segfault

2013-05-09 Thread Bernhard Reutner-Fischer
On Thu, May 09, 2013 at 05:52:26PM +0800, Chung-Lin Tang wrote: >2013-05-09 Chung-Lin Tang > > PR target/32219 > * rtlanal.c (nonzero_address_p): Robustify checking by look >recursively into PIC constant offsets and (CONST (UNSPEC ...)) > expressions. >Index: rtlanal.

Re: [PATCH] Improve rotation by mode bitsize - 1

2013-05-09 Thread Jakub Jelinek
On Thu, May 09, 2013 at 09:47:49PM +0200, Uros Bizjak wrote: > On Thu, May 9, 2013 at 8:45 PM, Jakub Jelinek wrote: > > 2013-05-09 Jakub Jelinek > > > > * config/i386/i386.md (rotateinv): New code attr. > > (*3_1, *si3_1_zext, > > *qi3_1_slp): Emit rorl %eax instead of >

Committed: fix epiphany ICE building _gcov.o

2013-05-09 Thread Joern Rennecke
This patch is required to build epiphany libgcc with the default options (-O2 -g). The scheduler moved instructions around in the prologue of gcov_exit so that the setting of the frame pointer ended up between the loading of a temp register with a stack adjust offset and the stack adjust instruc

[Patch][google/gcc-4_8] Backport trunk@198547 for pr target/56732

2013-05-09 Thread 沈涵
Hi, I'm to backport trunk patch @198547 for pr target/56732 to google branch google/gcc-4_8. This patch fixes arm ICE. Ok for google/gcc-4_8? [patch attached] H. r198547.patch Description: Binary data

Re: GCC does not support *mmintrin.h with function specific opts

2013-05-09 Thread Sriraman Tallam
cc:Diego On Tue, May 7, 2013 at 2:41 PM, Sriraman Tallam wrote: > Ping. > > On Thu, May 2, 2013 at 3:51 PM, Sriraman Tallam wrote: >> Ping. >> >> On Mon, Apr 29, 2013 at 10:47 AM, Sriraman Tallam >> wrote: >>> On Thu, Apr 25, 2013 at 12:41 PM, Joseph S. Myers >>> wrote: On Tue, 16 Apr 20

Re: [PATCH] Generate a label for the split cold function while using -freorder-blocks-and-partition

2013-05-09 Thread Sriraman Tallam
cc:Diego On Tue, May 7, 2013 at 2:41 PM, Sriraman Tallam wrote: > Ping. > > On Thu, Apr 25, 2013 at 4:50 PM, Sriraman Tallam wrote: >> Attaching an updated patch. >> >> Thanks >> Sri >> >> On Thu, Apr 25, 2013 at 4:42 PM, Sriraman Tallam wrote: >>> On Tue, Apr 23, 2013 at 9:59 PM, Jakub Jelinek

Re: Fix PR 53743 and other -freorder-blocks-and-partition failures (issue6823047)

2013-05-09 Thread Diego Novillo
On 2013-05-08 01:13 , Teresa Johnson wrote: Somehow Rietveld didn't upload the patch properly. I've attached the patch to this email instead. Here is the description: Rietveld has turned out to be far less useful that I had hoped. If you are running ubuntu precise, the upload script is having

Re: Fix PR 53743 and other -freorder-blocks-and-partition failures (issue6823047)

2013-05-09 Thread Steven Bosscher
On Thu, May 9, 2013 at 11:42 PM, Diego Novillo wrote: > On 2013-05-08 01:13 , Teresa Johnson wrote: >> -static void >> +void >> emit_barrier_after_bb (basic_block bb) >> { >>rtx barrier = emit_barrier_after (BB_END (bb)); >> - BB_FOOTER (bb) = unlink_insn_chain (barrier, barrier); >> + if (

Re: Fix PR 53743 and other -freorder-blocks-and-partition failures (issue6823047)

2013-05-09 Thread Xinliang David Li
On Thu, May 9, 2013 at 3:40 PM, Steven Bosscher wrote: > On Thu, May 9, 2013 at 11:42 PM, Diego Novillo wrote: >> On 2013-05-08 01:13 , Teresa Johnson wrote: >>> -static void >>> +void >>> emit_barrier_after_bb (basic_block bb) >>> { >>>rtx barrier = emit_barrier_after (BB_END (bb)); >>> -

Re: [PATCH, v3]: Add CXXABI_1.3.8 to util/testsuite_abi.cc to fix libstdc++-abi/abi_check FAIL

2013-05-09 Thread Paolo Carlini
On 05/09/2013 10:05 PM, Uros Bizjak wrote: Hello! We have to handle CXXABI_1.3.8 in util/testsuite_abi.cc. (Also, reorder entries a bit for better grouping). 2013-05-09 Uros Bizjak * testsuite/util/testsuite_abi.cc (check_version): Add CXXABI_1.3.8. Testing on x86_64-pc-linux-gnu

Re: [GOOGLE] Check conditions before calling varpool_node

2013-05-09 Thread Carrot Wei
On Thu, May 9, 2013 at 12:53 PM, Xinliang David Li wrote: > This is not correct. current_module_id is used only in FE parsing. > Suppose the var decl has correct flags and varpool_node can accept it, a new varpool_node will be created for it, the module_id for the new node is set to current_module

Re: [GOOGLE] Check conditions before calling varpool_node

2013-05-09 Thread Xinliang David Li
On Thu, May 9, 2013 at 4:46 PM, Carrot Wei wrote: > On Thu, May 9, 2013 at 12:53 PM, Xinliang David Li wrote: >> This is not correct. current_module_id is used only in FE parsing. >> > Suppose the var decl has correct flags and varpool_node can accept it, > a new varpool_node will be created for

Re: powerpc64le configure fix

2013-05-09 Thread Alan Modra
On Thu, May 09, 2013 at 01:33:59PM -0500, Edmar Wienskoski wrote: > On Thu, May 9, 2013 at 2:27 AM, Alan Modra wrote: > > Another tweak for little-endian powerpc. Committed revision 198734. > > > > * configure.ac (HAVE_AS_TLS): Enable tests for powerpcle and > > powerpc64le. > >

Re: powerpc64le configure fix

2013-05-09 Thread David Edelsohn
On Thu, May 9, 2013 at 8:19 PM, Alan Modra wrote: > On Thu, May 09, 2013 at 01:33:59PM -0500, Edmar Wienskoski wrote: >> On Thu, May 9, 2013 at 2:27 AM, Alan Modra wrote: >> > Another tweak for little-endian powerpc. Committed revision 198734. >> > >> > * configure.ac (HAVE_AS_TLS): Enab

Re: [PATCH] PR55033: Fix

2013-05-09 Thread Alan Modra
On Thu, May 09, 2013 at 08:47:19AM -0700, Richard Henderson wrote: > On 02/22/2013 02:51 AM, Alan Modra wrote: > > Richard, can I bother you with a review request? > > http://gcc.gnu.org/ml/gcc-patches/2013-02/msg00544.html > > Sorry for the truly excessive delay here. > > This patch is ok for ma

[PATCH] Dynamic dispatch of multiversioned functions and CPU mocks for code coverage.

2013-05-09 Thread Sriraman Tallam
Hi, This patch is an enhancement to the Function Multiversioning feature. This patch achieves two things: * Primarily, this patch makes it easy to test for code coverage of multiversioned functions. * Secondary, It makes function multiversioning work when there is no ifunc support. Sin

Re: Fix PR 53743 and other -freorder-blocks-and-partition failures (issue6823047)

2013-05-09 Thread Jeff Law
On 05/07/13 23:13, Teresa Johnson wrote: -- Revised patch that fixes failures encountered when enabling -freorder-blocks-and-partition, including the failure reported in PR 53743. This includes new verification code to ensure no cold blocks dominate hot blocks contributed by

[PATCH, PR preprocessor/42014] Added LAST_SOURCE_COLUMN in while loop

2013-05-09 Thread Shakthi Kannan
Hi, The attached patch adds LAST_SOURCE_COLUMN to pp_verbatim function in the while loop present in diagnostic_report_current_module(). This makes the output consistent for any error parsing program as stated in the bug. 2013-05-10 Shakthi Kannan PR preprocessor/42014 * gcc/diagnostic.c