RE: Another build!=host fix

2013-12-17 Thread Bernd Edlinger
Hi, On Tue, 17 Dec 2013 09:57:10, Mike Strump wrote: > > On Dec 17, 2013, at 5:47 AM, Bernd Edlinger wrote: >> Ok for trunk? > > ENOPATCH? Ooops -- thanks for catching this. Again, this time with patch: there is a small problem with SSIZE_MAX, because it is not always defined, especially not i

Re: PR middle-end/35535 part I

2013-12-17 Thread Jeff Law
On 12/17/13 23:53, Tobias Burnus wrote: Am 17.12.2013 21:56, schrieb Jeff Law: * tree-vrp.c (extract_range_from_unary_expr_1): Add OBJ_TYPE_REF s/Add/Handle. Please add the PR marker as well. OK with that trivial nit. And the proper PR. I don't think that INVALID C++ PR is the PR you wa

Re: PR middle-end/35535 part I

2013-12-17 Thread Tobias Burnus
Am 17.12.2013 21:56, schrieb Jeff Law: * tree-vrp.c (extract_range_from_unary_expr_1): Add OBJ_TYPE_REF s/Add/Handle. Please add the PR marker as well. OK with that trivial nit. And the proper PR. I don't think that INVALID C++ PR is the PR you want to refer to. Tobias

Re: [PING]: [GOMP4] [PATCH] SIMD-Enabled Functions (formerly Elemental functions) for C

2013-12-17 Thread Jakub Jelinek
On Tue, Dec 17, 2013 at 11:38:48PM +, Iyer, Balaji V wrote: > > What I meant is > > if (((mask >> PRAGMA_CILK_CLAUSE_VECTORLENGTH) & 1) != 0) > > is_cilk_simd_fn = true; > > (note, for 32-bit HWI targets, omp_clause_mask is a class and not all > > arithmetic is actually supported on it, s

New Brazilian Portuguese PO file for 'cpplib' (version 4.8.0)

2013-12-17 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'cpplib' has been submitted by the Brazilian Portuguese team of translators. The file is available at: http://translationproject.org/latest/cpplib/pt_BR.po (This file, 'cppl

Contents of PO file 'cpplib-4.8.0.pt_BR.po'

2013-12-17 Thread Translation Project Robot
cpplib-4.8.0.pt_BR.po.gz Description: Binary data The Translation Project robot, in the name of your translation coordinator.

RE: [PING]: [GOMP4] [PATCH] SIMD-Enabled Functions (formerly Elemental functions) for C

2013-12-17 Thread Iyer, Balaji V
> -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Jakub Jelinek > Sent: Tuesday, December 17, 2013 4:26 PM > To: Iyer, Balaji V > Cc: Joseph S. Myers; Aldy Hernandez (al...@redhat.com); 'gcc- > patc...@gcc.gnu.org' > Subject:

Re: Two build != host fixes

2013-12-17 Thread Alan Modra
On Tue, Dec 17, 2013 at 01:14:23PM +0100, Bernd Edlinger wrote: > the reason for this is overwriting GMPINC for the auto-build generation, > because > many test scripts include which fails now completely (it is not > installed, > I have it in-tree). Yes, I understand the reason why your setup i

[Google][gcc-4_8] Completed backport of r202818 from trunk to google/gcc-4_8 as r206071.

2013-12-17 Thread Paul Pluzhnikov
Greetings, I've finished backport of r202818 from trunk to google/gcc-4_8 as r206071. Thanks, -- Paul Pluzhnikov Index: libstdc++-v3/include/debug/array === --- libstdc++-v3/include/debug/array(revision 206038) +++ libstdc++-v3/

Re: [patch] Fix PR debug/59418

2013-12-17 Thread Jakub Jelinek
On Sun, Dec 15, 2013 at 06:53:07PM +0100, Eric Botcazou wrote: > 2013-12-15 Eric Botcazou > > PR debug/59418 > * dwarf2cfi.c (dwarf2out_frame_debug_cfa_offset): Fix comment and clean > up implementation. > (dwarf2out_frame_debug_cfa_restore): Handle TARGET_DWARF_REGISTE

Re: [PATCH] Fix PR58944

2013-12-17 Thread H.J. Lu
On Tue, Dec 17, 2013 at 10:29 AM, Sriraman Tallam wrote: > On Fri, Dec 13, 2013 at 5:06 AM, H.J. Lu wrote: >> On Mon, Dec 2, 2013 at 6:46 PM, Sriraman Tallam wrote: >>> On Thu, Nov 28, 2013 at 9:36 PM, Bernd Edlinger >>> wrote: Hi, On Wed, 27 Nov 2013 19:49:39, Uros Bizjak wrote:

[committed] Fix MASK_{LOAD,STORE} caused ICE (PR tree-optimization/59523)

2013-12-17 Thread Jakub Jelinek
Hi! I forgot to update_stmt stmts I've changed. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, committed to trunk as obvious. 2013-12-17 Jakub Jelinek PR tree-optimization/59523 * tree-vectorizer.c (fold_loop_vectorized_call): Call update_stmt on

Re: GOMP_target: alignment (was: [gomp4] #pragma omp target* fixes)

2013-12-17 Thread Thomas Schwinge
Hi! On Mon, 16 Dec 2013 17:58:26 +0100, Jakub Jelinek wrote: > I'd indeed prefer if you just used one > array, it can be say just uchar array of twice the width, with even chars > for alignment and odd for kinds (or vice versa), compared to two arrays > it is tiny bit cheaper at the caller side I

[patch] fix .DOT file generation for IPA passes

2013-12-17 Thread Aldy Hernandez
I was trying to generate a graph file with -fdump-ipa-tmipa-blocks-details-vops-graph, but the .dot file was corrupted. It looks like the header bits printed in start_graph_dump() are not dumped because we are predicating the calls to clean_graph_dump_file->start_graph_dump by: &&

Re: [Patch, microblaze]: Fix ICE with mhard-float

2013-12-17 Thread Spenser Gilliland
Hi, Just wanted to say that this patch works for me. Thanks, Spenser

Re: [PING]: [GOMP4] [PATCH] SIMD-Enabled Functions (formerly Elemental functions) for C

2013-12-17 Thread Jakub Jelinek
On Tue, Dec 17, 2013 at 09:13:05PM +, Iyer, Balaji V wrote: > @@ -10418,6 +10528,12 @@ >step = c_parser_expression (parser).value; >mark_exp_read (step); >step = c_fully_fold (step, false, NULL); > + if (is_cilk_simd_fn && TREE_CODE (step) == PARM_DECL) > + { >

Go patch committed: Use backend interface for runtime errors

2013-12-17 Thread Ian Lance Taylor
This patch from Chris Manghane uses the backend interface to call the runtime error function. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 4108b7cd8ca9 go/expressions.cc --- a/go/expressions.cc Mon Dec 16 11:55:05 2013 -0800 +++ b/go/expressi

PR middle-end/35535 part II

2013-12-17 Thread Jan Hubicka
Hi, this patch contines on tract of making us to handle OBJ_TYPE_REF in expressions more gracefully. The change in gimple_fold_stmt_to_constant_1 makes it to skip the wrapper, so valuelize functions of passes never see it, and in addition OBJ_TYPE_REF is stripped once argument becomes constant. I

RE: [PING]: [GOMP4] [PATCH] SIMD-Enabled Functions (formerly Elemental functions) for C

2013-12-17 Thread Iyer, Balaji V
Hi Jakub, Attached, please find a fixed patch. I have fixed all the issues you have mentioned below. I have also answered your questions below. Is this OK for trunk/branch? Here are the ChangeLog entries: gcc/ChangeLog 2013-12-17 Balaji V. Iyer * omp-low.c (simd_clone_clauses

Re: [PING]: [GOMP4] [PATCH] SIMD-Enabled Functions (formerly Elemental functions) for C

2013-12-17 Thread Jakub Jelinek
Hi! On Tue, Dec 17, 2013 at 05:23:43PM +, Iyer, Balaji V wrote: > +/* Returns name of the next clause in Cilk Plus SIMD-enabled function's > + attribute. > + If the clause is not recognized PRAGMA_OMP_CLAUSE_NONE is returned and > + the token is not consumed. Otherwise appropriate pragm

Re: [PATCH] Fix PR58944

2013-12-17 Thread Sriraman Tallam
On Fri, Dec 13, 2013 at 5:06 AM, H.J. Lu wrote: > On Mon, Dec 2, 2013 at 6:46 PM, Sriraman Tallam wrote: >> On Thu, Nov 28, 2013 at 9:36 PM, Bernd Edlinger >> wrote: >>> Hi, >>> >>> On Wed, 27 Nov 2013 19:49:39, Uros Bizjak wrote: On Mon, Nov 25, 2013 at 10:08 PM, Sriraman Tallam >>>

Re: GOMP_target: alignment (was: [gomp4] #pragma omp target* fixes)

2013-12-17 Thread Jakub Jelinek
On Tue, Dec 17, 2013 at 08:21:57PM +0100, Thomas Schwinge wrote: > On Mon, 16 Dec 2013 17:58:26 +0100, Jakub Jelinek wrote: > > I'd indeed prefer if you just used one > > array, it can be say just uchar array of twice the width, with even chars > > for alignment and odd for kinds (or vice versa),

Re: [PATCH] Builtins handling in IVOPT

2013-12-17 Thread Wei Mi
Ping. Thanks, Wei. On Mon, Dec 9, 2013 at 9:54 PM, Wei Mi wrote: > Ping. > > Thanks, > wei. > > On Sat, Nov 23, 2013 at 10:46 AM, Wei Mi wrote: >> bootstrap and regression of the updated patch pass. >> >> On Sat, Nov 23, 2013 at 12:05 AM, Wei Mi wrote: >>> On Thu, Nov 21, 2013 at 12:19 AM, Zde

Re: [Patch, i386] PR 59422 - Support more targets for function multi versioning

2013-12-17 Thread Allan Sandfeld Jensen
On Tuesday 17 December 2013, Allan Sandfeld Jensen wrote: > On Monday 16 December 2013, Uros Bizjak wrote: > > On Mon, Dec 16, 2013 at 10:34 AM, Uros Bizjak wrote: > > > On Sun, Dec 15, 2013 at 7:54 PM, Allan Sandfeld Jensen > > > > > > wrote: > > >> Hi again > > >> > > >> On Wednesday 11 Decem

Re: PR middle-end/35535 part II

2013-12-17 Thread Jeff Law
On 12/17/13 13:45, Jan Hubicka wrote: Hi, this patch contines on tract of making us to handle OBJ_TYPE_REF in expressions more gracefully. The change in gimple_fold_stmt_to_constant_1 makes it to skip the wrapper, so valuelize functions of passes never see it, and in addition OBJ_TYPE_REF is str

Re: PR middle-end/35535 part I

2013-12-17 Thread Jeff Law
On 12/17/13 11:00, Jan Hubicka wrote: Here we should devirtualize since B's foo is dominating target and we correctly do so. We however do more, we trace the code into: for (i = 0; i < 1; i++) { if (i%7==0) { ap = new A(); ap->foo(); }

[Patch, microblaze]: Add __builtin_trap instruction pattern

2013-12-17 Thread Spenser Gilliland
Hi, Just wanted to make a note that I have tested this patch and it works for me. Thanks, Spenser

Re: [Patch] Fix PR 59527 (assert in cfg fixup with function splitting)

2013-12-17 Thread Steven Bosscher
On Tue, Dec 17, 2013 at 5:38 PM, Teresa Johnson wrote: > PR gcov-profile/59527 > * cfgrtl.c (fixup_reorder_chain): Handle a region-crossing > branch, which can't be eliminated. This is OK. Thanks! Ciao! Steven

Re: [PATCH] Add __int128 test to ubsan

2013-12-17 Thread Marek Polacek
On Tue, Dec 17, 2013 at 06:50:24PM +0100, Jakub Jelinek wrote: > Isn't this undefined behavior in C? I mean, shouldn't you > shift up (unsigned __int128) 1 and only cast to (__int128) at the end? Oh my, how could I. Yeah, (__int128) 1 << 127 is UB. Fixed below, ok now? 2013-12-17 Marek Polace

Re: [PATCH] Fix PR 58867: asan and ubsan tests not run for installed testing

2013-12-17 Thread Jakub Jelinek
On Tue, Oct 29, 2013 at 07:56:32AM -0700, Andrew Pinski wrote: > * lib/ubsan-dg.exp (check_effective_target_fundefined_sanitizer): New > function. > (ubsan_init): Save off ALWAYS_CXXFLAGS. > (ubsan_finish): Restore ALWAYS_CXXFLAGS correctly. > * lib/asan-dg.exp (check_effective_target_faddress_san

Re: [PATCH] Add __int128 test to ubsan

2013-12-17 Thread Jakub Jelinek
On Tue, Dec 17, 2013 at 07:16:18PM +0100, Marek Polacek wrote: > On Tue, Dec 17, 2013 at 06:50:24PM +0100, Jakub Jelinek wrote: > > Isn't this undefined behavior in C? I mean, shouldn't you > > shift up (unsigned __int128) 1 and only cast to (__int128) at the end? > > Oh my, how could I. > > Yea

Re: [patch] libgomp test fixes for FreeBSD

2013-12-17 Thread Andreas Tobler
On 17.12.13 18:54, Jakub Jelinek wrote: > On Tue, Dec 17, 2013 at 06:42:22PM +0100, Andreas Tobler wrote: >> 2013-12-17 Andreas Tobler >> >> * testsuite/libgomp.c/affinity-1.c: Remove alloca.h inlcude. > > In this case please also change the alloca (...) use in the testcase > to __builtin_

Re: [PATCH] Support addsub/subadd as non-isomorphic operations for SLP vectorizer.

2013-12-17 Thread Cong Hou
Ping? thanks, Cong On Mon, Dec 2, 2013 at 5:02 PM, Cong Hou wrote: > Any comment on this patch? > > > thanks, > Cong > > > On Fri, Nov 22, 2013 at 11:40 AM, Cong Hou wrote: >> On Fri, Nov 22, 2013 at 3:57 AM, Marc Glisse wrote: >>> On Thu, 21 Nov 2013, Cong Hou wrote: >>> On Thu, Nov 21

Re: Another build!=host fix

2013-12-17 Thread Mike Stump
On Dec 17, 2013, at 5:47 AM, Bernd Edlinger wrote: > Ok for trunk? ENOPATCH?

Re: [PATCH] Introducing SAD (Sum of Absolute Differences) operation to GCC vectorizer.

2013-12-17 Thread Cong Hou
Ping? thanks, Cong On Mon, Dec 2, 2013 at 5:06 PM, Cong Hou wrote: > Hi Richard > > Could you please take a look at this patch and see if it is ready for > the trunk? The patch is pasted as a text file here again. > > Thank you very much! > > > Cong > > > On Mon, Nov 11, 2013 at 11:25 AM, Cong

PR middle-end/35535 part I

2013-12-17 Thread Jan Hubicka
Hi, PR 35545 has trivial testcase of feedback directed devirtualization: int main() { int i; A* ap = 0; for (i = 0; i < 1; i++) { if (i%7==0) ap = new A(); else ap = new B(); ap->foo(); Here we should devirtualize since B's foo is dominating targe

Re: [RFC] libgcov.c re-factoring and offline profile-tool

2013-12-17 Thread Xinliang David Li
On Tue, Dec 17, 2013 at 7:48 AM, Teresa Johnson wrote: > On Mon, Dec 16, 2013 at 2:48 PM, Xinliang David Li wrote: >> Ok -- gcov_write_counter and gcov_write_tag_length are qualified as >> low level primitives for basic gcov format and probably should be kept >> in gcov-io.c. >> >> gcov_rewrite i

Re: [patch] libgomp test fixes for FreeBSD

2013-12-17 Thread Jakub Jelinek
On Tue, Dec 17, 2013 at 06:42:22PM +0100, Andreas Tobler wrote: > 2013-12-17 Andreas Tobler > > * testsuite/libgomp.c/affinity-1.c: Remove alloca.h inlcude. In this case please also change the alloca (...) use in the testcase to __builtin_alloca (...) and update the ChangeLog entry. Ok w

Re: [PATCH] Add __int128 test to ubsan

2013-12-17 Thread Jakub Jelinek
On Tue, Dec 17, 2013 at 06:17:01PM +0100, Marek Polacek wrote: > Regtested on x86_64-linux with -m32/-m64. > > Ok? > > 2013-12-17 Marek Polacek > > testsuite/ > * c-c++-common/ubsan/overflow-int128.c: New test. > > --- gcc/testsuite/c-c++-common/ubsan/overflow-int128.c.mp 2013-12-1

[patch] libgomp test fixes for FreeBSD

2013-12-17 Thread Andreas Tobler
Hello, the below patch fixes three unresolved and a FAIL test case in libgomp. The FAIL test case can be solved with removing the include of the alloca.h header which is not present on FreeBSD. On Linux this header will be pulled in via stdlib.h if we define _GNU_SOURCE. And this we do. Tested o

[committed] Add new testcase (PR ipa/pr58290)

2013-12-17 Thread Jakub Jelinek
Hi! Richard fixed this PR recently by adding a fixup_cfg pass again right after IPA passes, I'm just including a testcase from this PR, verified on x86_64-linux and verified it fails again if I comment out the fixup_cfg pass from passes.def. Committed as obvious to trunk. 2013-12-17 Jakub Jelin

RE: [PING]: [GOMP4] [PATCH] SIMD-Enabled Functions (formerly Elemental functions) for C

2013-12-17 Thread Iyer, Balaji V
Hi Jakub, Please see attached patch and my answers to your questions below. Aldy, I have made a couple changes to #pragma simd routines, can you please give me your blessing on those? Thanks, Balaji V. Iyer. > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [ma

[PATCH] Add __int128 test to ubsan

2013-12-17 Thread Marek Polacek
Regtested on x86_64-linux with -m32/-m64. Ok? 2013-12-17 Marek Polacek testsuite/ * c-c++-common/ubsan/overflow-int128.c: New test. --- gcc/testsuite/c-c++-common/ubsan/overflow-int128.c.mp 2013-12-17 16:54:28.123468111 +0100 +++ gcc/testsuite/c-c++-common/ubsan/overflow-int12

[Patch] Fix PR 59527 (assert in cfg fixup with function splitting)

2013-12-17 Thread Teresa Johnson
Add handling to fixup_reorder_chain for a region crossing branch, which cannot be optimized away (since it is needed to cross the region boundary). In the case when there is no fallthru for a conditional jump the comments indicate that this can happen if the conditional jump has side effects and ca

Re: [Patch,testsuite] Fix testcases that use bind_pic_locally

2013-12-17 Thread Rainer Orth
Hi Vidya, > bind_pic_locally is broken for targets that doesn't pass -fPIC/-fpic by > default [1][2]. > > One of the suggestions was to have a effective target check called > bind_pic_locally_ok which checks if bind_pic_locally will work and have it > included in all the tests that uses bind_pic_l

Re: [RFC] libgcov.c re-factoring and offline profile-tool

2013-12-17 Thread Teresa Johnson
On Mon, Dec 16, 2013 at 2:48 PM, Xinliang David Li wrote: > Ok -- gcov_write_counter and gcov_write_tag_length are qualified as > low level primitives for basic gcov format and probably should be kept > in gcov-io.c. > > gcov_rewrite is petty much libgcov runtime implementation details so I > thin

[committed] Fix up retval1.f90 testcase (PR testsuite/59534)

2013-12-17 Thread Jakub Jelinek
Hi! Because e5 and f5 share space (like in C union), it is undesirable to use non-shortcircuited comparisons, because e.g. on alpha we can end up with denormal exception. Committed thusly: 2013-12-17 Jakub Jelinek PR testsuite/59534 * testsuite/libgomp.fortran/retval1.f90 (e5

Re: [PATCH i386] Enable -freorder-blocks-and-partition

2013-12-17 Thread Teresa Johnson
Thanks for the data. A few questions: - Do you have the raw data used to generate your pdfs available? Since you gave me the binaries, if I have the data in terms of exactly what addresses are being plotted I can correlate with the specific cold functions via nm. Once I know what cold functions ar

Re: Fix devirt2.C testcase

2013-12-17 Thread Jan Hubicka
> On Tue, Dec 17, 2013 at 3:04 AM, Jan Hubicka wrote: > > Hi, > > I forgot the following change in my tree. It fixes type consistency sanity > > check in get_polymorphic_call_info. With the change to gimple-fold it is > > now needed to devrirtualize devirt2.C. (previously the bug went latent sin

RE: [PING]: [GOMP4] [PATCH] SIMD-Enabled Functions (formerly Elemental functions) for C

2013-12-17 Thread Iyer, Balaji V
> -Original Message- > From: Jakub Jelinek [mailto:ja...@redhat.com] > Sent: Tuesday, December 17, 2013 1:18 AM > To: Iyer, Balaji V > Cc: Joseph S. Myers; Aldy Hernandez; 'gcc-patches@gcc.gnu.org' > Subject: Re: [PING]: [GOMP4] [PATCH] SIMD-Enabled Functions (formerly > Elemental functio

Re: Fix devirt2.C testcase

2013-12-17 Thread H.J. Lu
On Tue, Dec 17, 2013 at 3:04 AM, Jan Hubicka wrote: > Hi, > I forgot the following change in my tree. It fixes type consistency sanity > check in get_polymorphic_call_info. With the change to gimple-fold it is > now needed to devrirtualize devirt2.C. (previously the bug went latent since > the o

Re: [PATCH] Masked load/store vectorization (take 7)

2013-12-17 Thread H.J. Lu
On Mon, Dec 9, 2013 at 12:27 PM, Jakub Jelinek wrote: > Hi! > > On Fri, Dec 06, 2013 at 01:49:50PM +0100, Richard Biener wrote: >> >basic_block bb = ifc_bbs[i]; >> >gimple_seq stmts; >> > >> > - if (!is_predicated (bb)) >> > + if (!is_predicated (bb) >> > + || dominat

[Patch,testsuite] Fix testcases that use bind_pic_locally

2013-12-17 Thread Vidya Praveen
Hello, bind_pic_locally is broken for targets that doesn't pass -fPIC/-fpic by default [1][2]. One of the suggestions was to have a effective target check called bind_pic_locally_ok which checks if bind_pic_locally will work and have it included in all the tests that uses bind_pic_locally in dg-a

Re: patch for elimination to SP when it is changed in RTL (PR57293)

2013-12-17 Thread Yvan Roux
On 17 December 2013 00:03, Vladimir Makarov wrote: > On 12/13/2013, 8:07 AM, Yvan Roux wrote: >> >> Thanks for your help Vlad. Another bad news about this PR fix, is >> that it has resurrected the thumb_movhi_clobber bug (PR 58785) but in >> a different manner as the original failing testcase sti

Another build!=host fix

2013-12-17 Thread Bernd Edlinger
Hi, there is a small problem with SSIZE_MAX, because it is not always defined, especially not in gcc/glimits.h, which seems to be the fall-back if the target fails to have a working limits.h. When I create a cross-compiler for --target=arm-linux-gnueabihf, the working limits.h is overwritten by f

Re: [Patch,avr]: Fix wrong warning PR59396

2013-12-17 Thread Georg-Johann Lay
Am 12/05/2013 04:09 PM, schrieb Richard Biener: On Thu, Dec 5, 2013 at 3:53 PM, Georg-Johann Lay wrote: This is a fix of a wrong warning for a bas ISR name. The assumption was that if DECL_ASSEMBLER_NAME is set, it would always starts with a *. This is not the case for LTO compiler where the

RE: Two build != host fixes

2013-12-17 Thread Bernd Edlinger
Hi Alan, just for the records, this is how my cross-build fails: ../gcc-4.9-20131215/configure --prefix=/home/ed/gnu/x/arm-linux-gnueabihf-cross --host=arm-linux-gnueabihf --target=arm-linux-gnueabihf --enable-languages=c,c++ --with-arch=armv7-a --with-tune=cortex-a9 --with-fpu=vfpv3-d16 --wi

Minor cleanup in expmed.c

2013-12-17 Thread Eric Botcazou
This fixes a few glitches introduced by the recent changes to the file. Tested on x86-64/Linux, applied on the mainline as obvious. 2013-12-17 Eric Botcazou * expmed.c (lowpart_bit_field_p): Fix comment. (store_bit_field_using_insv): Fix formatting. (store_bit_field):

Re: [PATCH][ARM] Wire up scheduling for Cortex-A12

2013-12-17 Thread Richard Earnshaw
On 17/12/13 10:51, Kyrill Tkachov wrote: > Hi all, > > This patch wires up the Cortex-A12 instruction scheduling to use the > Cortex-A15 > pipeline description and sets the issue rate for it to 2 in arm_issue_rate. > > This patch depends on James' recent rework of the tuning parameters posted a

Re: [ARM 5/5 big.LITTLE] Add support for -mcpu=cortex-a57.cortex-a53

2013-12-17 Thread Richard Earnshaw
On 17/12/13 10:40, James Greenhalgh wrote: > > Hi, > > This patch wires up -mcpu=cortex-a57.cortex-a53 as an option to > -mcpu. > > Bootstrapped in series, and sanity checked. > > OK? > > Thanks, > James > > --- > 2013-12-17 James Greenhalgh > > * config/arm/arm-cores.def (cortex-a5

Re: [ARM 4/5 big.LITTLE] Add support for -mcpu=cortex-a57

2013-12-17 Thread Richard Earnshaw
On 17/12/13 10:40, James Greenhalgh wrote: > > Hi, > > This patch wires up -mcpu=cortex-a57 as an option to > -mcpu. As we don't yet have a scheduling model for Cortex-A57 > available, for now we use the scheduling description for another > "big" core, the Cortex-A15. > > Bootstrapped in series

Re: [ARM 3/5 big.LITTLE] Add support for -mcpu=cortex-a15.cortex-a7

2013-12-17 Thread Richard Earnshaw
On 17/12/13 10:40, James Greenhalgh wrote: > > Hi, > > This patch wires up -mcpu=cortex-a15.cortex-a7 as an option to > -mcpu. > > Bootstrapped in series, with --with-cpu=cortex-a15.cortex-a7. > > OK? > > Thanks, > James > > --- > 2013-12-17 James Greenhalgh > > * config/arm/arm-cor

Re: [ARM 2/5 big.LITTLE] Allow tuning parameters without unique tuning targets.

2013-12-17 Thread Richard Earnshaw
On 17/12/13 10:40, James Greenhalgh wrote: > > Hi, > > A limitation in the ARM backend is that each core added to arm-cores.def > must provide a unique identifier to be used for tuning. This restricts > us when we want to share the same identifier between a number of cores. > > The machinery her

Re: [ARM 1/5 big.LITTLE] Add driver support for rewriting -mcpu names

2013-12-17 Thread Richard Earnshaw
On 17/12/13 10:40, James Greenhalgh wrote: > > Hi, > > This patch adds machinery to the driver to ensure that big.LITTLE > style tuning names are rewritten before they are passed to the > assembler. This reduces the coupling needed between GCC versions > and assembler versions. > > The rule is s

[RS6000] bswapdi2 pattern, reload and lra

2013-12-17 Thread Alan Modra
This patch is aimed at fixing test failures introduced by my 2013-12-07 change to bswapdi2_32bit: FAIL: gcc.target/powerpc/pr53199.c scan-assembler-times lwbrx 6 FAIL: gcc.target/powerpc/pr53199.c scan-assembler-times stwbrx 6 The 2013-12-07 change was necessary to make -m32 -mlra generate good co

[RFC][gomp4] Offloading patches (3/3): Add invocation of target compiler

2013-12-17 Thread Michael V. Zolotukhin
Hi everybody, Here is a patch 3/3: Add invocation of target compiler. With this patch lto-wrapper performs invocation of target compilers and embeds the resultant target images into the host binary. The targets and the corresponding compilers are supposed to be specified in a special environment

[RFC][gomp4] Offloading patches (2/3): Add tables generation

2013-12-17 Thread Michael V. Zolotukhin
Hi everybody, Here is a patch 2/3: Add tables generation. This patch is just a slightly modified patch sent a couple of weeks ago. When compiling with '-fopenmp' compiler generates a special symbol, containing addresses and sizes of globals/omp_fn-functions, and places it into a special section.

[RFC][gomp4] Offloading patches (1/3): Add '-fopenmp_target' option

2013-12-17 Thread Michael V. Zolotukhin
Hi everybody, Here is a set of patches implementing one more piece of offloading support in GCC. These three patches allow to build a host binary with target image and all tables embedded. Along with patches for libgomp and libgomp plugin, which hopefully will be sent soon, that gives a function

Re: [PING]: [GOMP4] [PATCH] SIMD-Enabled Functions (formerly Elemental functions) for C

2013-12-17 Thread Thomas Schwinge
Hi! On Tue, 17 Dec 2013 11:27:51 +0100, Jakub Jelinek wrote: > On Tue, Dec 17, 2013 at 11:03:12AM +0100, Thomas Schwinge wrote: > > My understanding/reasoning is that PRAGMA_OMP_* just literally represents > > a parser token of a pragma line (see the one-to-one translation in > > c-parser.c:c_par

Fix devirt2.C testcase

2013-12-17 Thread Jan Hubicka
Hi, I forgot the following change in my tree. It fixes type consistency sanity check in get_polymorphic_call_info. With the change to gimple-fold it is now needed to devrirtualize devirt2.C. (previously the bug went latent since the old code handled the testcase) I am re-testing x86_64-linux and

[PATCH][ARM] Wire up scheduling for Cortex-A12

2013-12-17 Thread Kyrill Tkachov
Hi all, This patch wires up the Cortex-A12 instruction scheduling to use the Cortex-A15 pipeline description and sets the issue rate for it to 2 in arm_issue_rate. This patch depends on James' recent rework of the tuning parameters posted at http://gcc.gnu.org/ml/gcc-patches/2013-12/msg01477.h

[ARM 1/5 big.LITTLE] Add driver support for rewriting -mcpu names

2013-12-17 Thread James Greenhalgh
Hi, This patch adds machinery to the driver to ensure that big.LITTLE style tuning names are rewritten before they are passed to the assembler. This reduces the coupling needed between GCC versions and assembler versions. The rule is simple, we truncate the CPU name at the first '.' character we

[ARM 2/5 big.LITTLE] Allow tuning parameters without unique tuning targets.

2013-12-17 Thread James Greenhalgh
Hi, A limitation in the ARM backend is that each core added to arm-cores.def must provide a unique identifier to be used for tuning. This restricts us when we want to share the same identifier between a number of cores. The machinery here is a bit messy, and we don't really make it any nicer in

[ARM 5/5 big.LITTLE] Add support for -mcpu=cortex-a57.cortex-a53

2013-12-17 Thread James Greenhalgh
Hi, This patch wires up -mcpu=cortex-a57.cortex-a53 as an option to -mcpu. Bootstrapped in series, and sanity checked. OK? Thanks, James --- 2013-12-17 James Greenhalgh * config/arm/arm-cores.def (cortex-a57.cortex-a53): New. * doc/invoke.texi: Document -mcpu=cortex-a57.co

[ARM 3/5 big.LITTLE] Add support for -mcpu=cortex-a15.cortex-a7

2013-12-17 Thread James Greenhalgh
Hi, This patch wires up -mcpu=cortex-a15.cortex-a7 as an option to -mcpu. Bootstrapped in series, with --with-cpu=cortex-a15.cortex-a7. OK? Thanks, James --- 2013-12-17 James Greenhalgh * config/arm/arm-cores.def (cortex-a15.cortex-a7): New. * doc/invoke.texi: Document -mc

[ARM 4/5 big.LITTLE] Add support for -mcpu=cortex-a57

2013-12-17 Thread James Greenhalgh
Hi, This patch wires up -mcpu=cortex-a57 as an option to -mcpu. As we don't yet have a scheduling model for Cortex-A57 available, for now we use the scheduling description for another "big" core, the Cortex-A15. Bootstrapped in series and sanity checked. OK? Thanks, James --- 2013-12-17 Jame

[Patch ARM] Add big.LITTLE tuning options

2013-12-17 Thread James Greenhalgh
Hi, This patch series adds machinery and functionality to enable tuning for big.LITTLE systems when compiling for the ARM target. We take the convention for names to -mcpu that for some big.LITTLE system where the big core is 'x' and the little core is 'y', the -mcpu name will be x.y In order to

Re: [PING]: [GOMP4] [PATCH] SIMD-Enabled Functions (formerly Elemental functions) for C

2013-12-17 Thread Jakub Jelinek
On Tue, Dec 17, 2013 at 11:03:12AM +0100, Thomas Schwinge wrote: > > > Also, If I created CILK_CLAUSE_* variants, I have to re-create another > > > function similar to c_parser_omp_all_clauses, whose workings will be > > > identical to the c_parser_omp_all_clauses. Is that OK with you? > > > > N

Re: [Patch, i386] PR 59422 - Support more targets for function multi versioning

2013-12-17 Thread Allan Sandfeld Jensen
On Monday 16 December 2013, Uros Bizjak wrote: > On Mon, Dec 16, 2013 at 10:34 AM, Uros Bizjak wrote: > > On Sun, Dec 15, 2013 at 7:54 PM, Allan Sandfeld Jensen > > > > wrote: > >> Hi again > >> > >> On Wednesday 11 December 2013, Uros Bizjak wrote: > >>> Hello! > >>> > >>> > PR gcc/59422 > >>

Re: [PING]: [GOMP4] [PATCH] SIMD-Enabled Functions (formerly Elemental functions) for C

2013-12-17 Thread Thomas Schwinge
Hi! For reference, here's my rationale for OpenACC on this topic: On Tue, 17 Dec 2013 07:17:31 +0100, Jakub Jelinek wrote: > On Tue, Dec 17, 2013 at 03:51:14AM +, Iyer, Balaji V wrote: > > Hi Jakub, > > I will work on this, but I need a couple clarifications about some of > > your co

Re: [PATCH] Fix convert_mode

2013-12-17 Thread Richard Biener
Jakub Jelinek wrote: >On Mon, Dec 16, 2013 at 11:12:40PM +0100, Jakub Jelinek wrote: >> When testing the patch the overflow-2.c testcase didn't exist yet, >> nor was ubsan on -m32 actually ever reporting overflows on the DImode >> multiplication (it simply expanded it as normal DImode multiplicati

Re: [patch] fix fortran regressions on FreeBSD10.0/11.0

2013-12-17 Thread Janne Blomqvist
On Tue, Dec 17, 2013 at 8:51 AM, Jakub Jelinek wrote: > On Tue, Dec 17, 2013 at 06:19:43AM +0100, Andreas Tobler wrote: >> The below patch allows me to get back to normal, means zero unexpected >> fails, on FreeBSD. The patch has been tested on Linux/x86 as well, no >> regressions. > > On Linux, m