Re: [PATCH] Refactor optimize isl

On 09/11/2015 07:07 PM, Aditya Kumar wrote: Updated patch with corrections: Refactor graphite-optimize-isl.c. Renamed function name, variable names etc., and indented the source according to gcc style guidelines. Modified comments accordingly. No functional change intended. Looks reasonable.

[gomp4] Update fortran tests.

Hi! Attached is a patch to update two (2) tests that were failing because the test cases were incorrect. Committed after regtesting on x86_64 and powerpc64le. Thanks! Jim Index: lib-12.f90 === --- lib-12.f90 (revision 227667) +++

Re: [PATCH] Remove dead code from graphite-optimize-isl.c

On 09/11/2015 07:29 PM, Aditya Kumar wrote: The variable `static bool enable_polly_vector' is always assigned to false. This results in dead code in optimize-isl.c. Removing the dead code. No functional change intended. Fine with me as well. This code is used in Polly to enable outer loop

Re: Use resolution info to get rid of weak symbols

On Sun, May 18, 2014 at 12:38 PM, Jan Hubicka wrote: > Hi, > this patch makes GCC to use resolution info to turn COMDAT and WEAK > symbols into regular symbols based on feedback given by linker plugin. > If resolution says that given symbol is prevailing, it is possible > to turn

RE: [PATCH, MIPS] Frame header optimization for MIPS O32 ABI

On Fri, 2015-09-04 at 01:40 -0700, Matthew Fortune wrote: > A few comments below. I found some of the comments a bit hard to parse but > have > not attempted any rewording. I'd like Catherine to comment too as I have > barely > any experience at the gimple level to know if this accounts for any

libbacktrace patch committed: Update dependencies

Although libbacktrace uses automake, it can't use automatic dependency tracking, because it breaks when using bootstrap-lean (PR 54732). The dependencies for sort.lo and stest.lo were never added to the list. Also, the dependencies for backtrace.lo were not updated for my recent change to that

Re: Ping^2 Re: Pass -foffload targets from driver to libgomp at link time

On Fri, 11 Sep 2015, Jakub Jelinek wrote: > On Fri, Sep 11, 2015 at 03:26:04PM +, Joseph Myers wrote: > > On Fri, 11 Sep 2015, Bernd Schmidt wrote: > > > > > I expect you know best what to do in the OpenACC testsuite driver, but you > > > might want to run the libgomp.exp parts by Jakub. If

Re: [PATCH][AArch64] Use preferred aliases for CSNEG, CSINC, CSINV

On 11/09/15 16:31, James Greenhalgh wrote: On Tue, Sep 01, 2015 at 11:08:10AM +0100, Kyrill Tkachov wrote: Hi all, The ARMv8-A reference manual says: "CNEG , , is equivalent to CSNEG , , , invert() and is the preferred disassembly when Rn == Rm && cond != '111x'." That is, when the two

Re: [patch] libstdc++/67173 Fix filesystem::canonical for Solaris 10.

On 09/11/2015 08:21 AM, Jonathan Wakely wrote: Solaris 10 doesn't follow POSIX in accepting a null pointer as the second argument to realpath(), so allocate a buffer for it. FWIW, the NULL requirement is new in Issue 7. In Issue 6, the behavior is implementation-defined, and before then it was

Re: [PATCH] vectorizing conditional expressions (PR tree-optimization/65947)

>>> Saying that all reductions have equivalent performance is unlikely to be >>> true for many platforms. On PowerPC, for example, a PLUS reduction has >>> very different cost from a MAX reduction. If the model isn't >>> fine-grained enough, let's please be aggressive about fixing it. I'm >>>

[gcc-5-branch][PATCH][AARCH64]Fix for branch offsets over 1 MiB

Conditional branches have a maximum range of [-1048576, 1048572]. Any destination further away can not be reached by these. To be able to have conditional branches in very large functions, we invert the condition and change the destination to jump over an unconditional branch to the original,

[PATCH] Refactor optimize isl

Updated patch with corrections: Refactor graphite-optimize-isl.c. Renamed function name, variable names etc., and indented the source according to gcc style guidelines. Modified comments accordingly. No functional change intended. Passes regtest and bootstap on x86_64. gcc/ChangeLog:

[PATCH] Remove dead code from graphite-optimize-isl.c

The variable `static bool enable_polly_vector' is always assigned to false. This results in dead code in optimize-isl.c. Removing the dead code. No functional change intended. Passes bootstrap and regtest. gcc/ChangeLog: 2015-09-11 Aditya Kumar *

[testsuite] Link gcc.dg/pie-link.c with -pie

While starting to develop the patch for Solaris PIE support, the gcc.dg/pie-link.c test succeeded at a point when it shouldn't have, i.e. before I had PIC crt files. In its current form, the test doesn't test what it's supposed to test, namely successfully linking position independent

Re: [testsuite] Link gcc.dg/pie-link.c with -pie

On Fri, Sep 11, 2015 at 11:08:03AM +0200, Rainer Orth wrote: > While starting to develop the patch for Solaris PIE support, > the gcc.dg/pie-link.c test succeeded at a point when it shouldn't have, > i.e. before I had PIC crt files. In its current form, the test doesn't > test what it's supposed

Re: [PATCH 2/2] shrink-wrap: Rewrite try_shrink_wrapping

On 09/10/2015 05:14 PM, Segher Boessenkool wrote: This patch rewrites the shrink-wrapping algorithm, allowing non-linear pieces of CFG to be duplicated for use without prologue instead of just linear pieces. An example would be good for this kind of patch, also in the comments. -

Re: [Patch] Teach RTL ifcvt to handle multiple simple set instructions

On Fri, Sep 11, 2015 at 10:04:12AM +0100, Ramana Radhakrishnan wrote: > On Fri, Sep 11, 2015 at 10:53:13AM +0200, Bernd Schmidt wrote: > > On 09/10/2015 11:11 PM, Jeff Law wrote: > > >I think that's probably what James is most interested in getting some > > >ideas around -- the cost model. > > > >

Re: [libgo] Use stat_atim.go on Solaris 12+

Ian Lance Taylor writes: > On Wed, Aug 26, 2015 at 4:14 AM, Rainer Orth > wrote: >> Solaris 12 changes the stat_[amc]tim members of struct stat from >> timestruc_t to timespec_t for XPG7 compatiblity, thus breaking the libgo >> build. The

[patch] libstdc++/65092 Allocator-extended constructors for container adaptors.

These should have been added as part of finishing C++11 support but I missed them. Now we have even more constructors to slow down compilation, yay allocators. Tested powerpc64le-linux, commtted to trunk. commit 0a8e02d699f4fe30e44678eb7cbd08bb6c6aed97 Author: Jonathan Wakely

Re: [RFC AArch64][PR 63304] Handle literal pools for functions > 1 MiB in size.

On Thu, Aug 27, 2015 at 03:07:30PM +0100, Marcus Shawcroft wrote: > On 27 July 2015 at 15:33, Ramana Radhakrishnan > wrote: > > > Ramana Radhakrishnan > > > > PR target/63304 > > * config/aarch64/aarch64.c

Re: [Patch] Teach RTL ifcvt to handle multiple simple set instructions

On 10/09/15 22:11, Jeff Law wrote: On 09/10/2015 12:23 PM, Bernd Schmidt wrote: > No testcase provided, as currently I don't know of targets with a high > enough branch cost to actually trigger the optimisation. Hmm, so the code would not actually be used right now? In that case I'll

Re: [Patch] Teach RTL ifcvt to handle multiple simple set instructions

On 09/10/2015 11:11 PM, Jeff Law wrote: I think that's probably what James is most interested in getting some ideas around -- the cost model. I think the fundamental problem is BRANCH_COST isn't actually relative to anything other than the default value of "1". It doesn't directly correspond

Re: [Patch] Teach RTL ifcvt to handle multiple simple set instructions

On Fri, Sep 11, 2015 at 10:53:13AM +0200, Bernd Schmidt wrote: > On 09/10/2015 11:11 PM, Jeff Law wrote: > >I think that's probably what James is most interested in getting some > >ideas around -- the cost model. > > > >I think the fundamental problem is BRANCH_COST isn't actually relative > >to

Re: [testsuite] Link gcc.dg/pie-link.c with -pie

Jakub Jelinek writes: > On Fri, Sep 11, 2015 at 11:08:03AM +0200, Rainer Orth wrote: >> While starting to develop the patch for Solaris PIE support, >> the gcc.dg/pie-link.c test succeeded at a point when it shouldn't have, >> i.e. before I had PIC crt files. In its current

Re: [RFC AArch64][PR 63304] Handle literal pools for functions > 1 MiB in size.

On 11/09/15 09:48, Ramana Radhakrishnan wrote: > On Thu, Aug 27, 2015 at 03:07:30PM +0100, Marcus Shawcroft wrote: >> On 27 July 2015 at 15:33, Ramana Radhakrishnan >> wrote: >> >>> Ramana Radhakrishnan >>> >>> PR

Re: [PATCH][RTL-ifcvt] PR rtl-optimization/67465: Handle pairs of complex+simple blocks and empty blocks more gracefully

Kyrill Tkachov writes: > On 10/09/15 12:43, Rainer Orth wrote: >> Hi Kyrill, >> >>> Rainer, could you please check that this patch still fixes the SPARC >>> regressions? >> unfortunately, it breaks sparc-sun-solaris2.10 bootstrap: compiling >> stage2 libiberty/regex.c

Re: [PATCH] [PATCH][ARM] Fix pr63210.c testcase.

Hi, Committed fsf-trunk r227677, fsf-5 r227678. Kind regards, Alex

Re: [PATCH] PR67401: Fix wrong code generated by expand_atomic_compare_and_swap

On 09/11/2015 01:21 AM, John David Anglin wrote: As noted in the PR, expand_atomic_compare_and_swap can generate wrong code when libcalls are emitted for the sync_compare_and_swap and the result comparison test. This is fixed by emitting a move insn to copy the result rtx of the

Re: [PATCH] vectorizing conditional expressions (PR tree-optimization/65947)

Hi Bill, I’d be a bit worried about asking the backend for the cost of a COND_REDUCTION, as that will rely on the backend understanding the implementation the vectorizer is using - every time the vectorizer changed, the backends would need to be updated too. I’m hoping soon to get together a

Re: [PATCH 2/2] shrink-wrap: Rewrite try_shrink_wrapping

Segher Boessenkool writes: > On Thu, Sep 10, 2015 at 08:14:29AM -0700, Segher Boessenkool wrote: >> This patch rewrites the shrink-wrapping algorithm, allowing non-linear >> pieces of CFG to be duplicated for use without prologue instead of just >> linear pieces. > >> Bootstrapped and regression

RE: [PATCH] Remove dead code from graphite-optimize-isl.c

For the record, the patch LGTM. Aditya and I have discussed about this in the morning. Thanks, Sebastian -Original Message- From: Aditya Kumar [mailto:aditya...@samsung.com] Sent: Friday, September 11, 2015 12:30 PM To: gcc-patches@gcc.gnu.org Cc: tob...@grosser.es;

Re: [C++ Patch] PR 51911 V2 ("G++ accepts new auto { list }")

On 09/11/2015 03:11 PM, Paolo Carlini wrote: this is a slightly reworked (simplified) version of a patch I sent a while ago. The issue is that we are not enforcing at all 5.3.4/2 in the parser, thus we end up rejecting the first test below with a misleading error message talking about

Re: [PATCH] Convert SPARC to LRA

On 09/11/2015 12:43 PM, David Miller wrote: > From: David Miller > Date: Tue, 08 Sep 2015 21:41:15 -0700 (PDT) > >> I'm therefore reasonably confident in these changes, but I will >> not apply them just yet to give the other sparc maintainers some >> time to review and give

Merge from trunk to gccgo branch

I merged trunk revision 227689 to the gccgo branch. Ian

Re: [PATCH] Convert SPARC to LRA

From: David Miller Date: Tue, 08 Sep 2015 21:41:15 -0700 (PDT) > I'm therefore reasonably confident in these changes, but I will > not apply them just yet to give the other sparc maintainers some > time to review and give feedback. Richard, Eric, any objections?

[PATCH] v2 shrink-wrap: Rewrite

[ v2: patch adjusted after Bernd's comments ] This patch rewrites the shrink-wrapping algorithm, allowing non-linear pieces of CFG to be duplicated for use without prologue instead of just linear pieces. On PowerPC, this enables shrink-wrapping of about 2%-3% more functions. I expected more, but

[gomp4] PTX partition discovery cleanup

This preliminary patch changes PTX's loop discovery to use a recursive DFS walk, rather than a worklist. The significant changes, from the POC of the next patch I'll be committing are: 1) always insert a 'fork' insn, which keeps the entry to a partitioned region as a possible

[gomp4] parallel reduction nested inside data regions

This patch corrects the way that build_outer_var_ref deals with data mappings in acc parallel and kernels when they are nested in some other construct (i.e. acc data). This issue can be reproduced with acc parallel reduction nested nested inside a acc data region. I've applied this fix to

[C++ Patch] PR 51911 V2 ("G++ accepts new auto { list }")

Hi, this is a slightly reworked (simplified) version of a patch I sent a while ago. The issue is that we are not enforcing at all 5.3.4/2 in the parser, thus we end up rejecting the first test below with a misleading error message talking about list-initialization (and a wrong location),

RE: [PATCH] Refactor optimize isl

-Original Message- From: Tobias Grosser [mailto:tob...@grosser.es] Sent: Friday, September 11, 2015 1:16 PM To: Aditya Kumar; gcc-patches@gcc.gnu.org Cc: richard.guent...@gmail.com; s@samsung.com; seb...@gmail.com Subject: Re: [PATCH] Refactor optimize isl On 09/11/2015 07:07 PM,

Go patch committed: Fix possible out of bounds memcmp

This patch by Chris Manghane fixes the Go frontend to avoid a possible out of bounds memcmp when looking for a go:nointerface comment. This fixes https://golang.org/issue/11577 . Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian Index:

Re: [Patch] Teach RTL ifcvt to handle multiple simple set instructions

On 09/11/2015 02:49 AM, Kyrill Tkachov wrote: On 10/09/15 22:11, Jeff Law wrote: On 09/10/2015 12:23 PM, Bernd Schmidt wrote: > No testcase provided, as currently I don't know of targets with a high > enough branch cost to actually trigger the optimisation. Hmm, so the code would not

[gomp4] SESE region neutering

One optimization one can apply to the worker & vector neutering that the PTX backend does, is to neuter Single-Entry, Single-Exit regions, rather than individual BBs. An SESE region in a region bounded by a single entry BB and a single exit BB. (A single BB is a trivial SESE region).

Re: [PATCH] v2 shrink-wrap: Rewrite

On 09/11/2015 02:49 PM, Segher Boessenkool wrote: On PowerPC, this enables shrink-wrapping of about 2%-3% more functions. I expected more, but in most cases this would help we cannot yet shrink- wrap because there are non-volatile registers used, often in the first block already. I looked into

[PATCH] PR28901 -Wunused-variable ignores unused const initialised variables

12 years ago it was decided that -Wunused-variable shouldn't warn about static const variables because some code used const static char rcsid[] strings which were never used but wanted in the code anyway. But as the bug points out this hides some real bugs. These days the usage of rcsids is not

Re: [PATCH] v2 shrink-wrap: Rewrite

On 09/11/2015 05:40 PM, Segher Boessenkool wrote: Another issue I saw for PowerPC that prevented shrink-wrapping was the >case where non-volatile CR's are used somewhere in the function. This >causes a 'mfcr' to be generated in the prolog to save the non-volatile >CR's, which currently lists all

Re: [PATCH] PR28901 -Wunused-variable ignores unused const initialised variables

On 09/12/2015 12:12 AM, Mark Wielaard wrote: 12 years ago it was decided that -Wunused-variable shouldn't warn about static const variables because some code used const static char rcsid[] strings which were never used but wanted in the code anyway. But as the bug points out this hides some real

Re: [PATCH] v2 shrink-wrap: Rewrite

On Fri, Sep 11, 2015 at 05:06:47PM -0500, Pat Haugen wrote: > On 09/11/2015 02:49 PM, Segher Boessenkool wrote: > >On PowerPC, this enables shrink-wrapping of about 2%-3% more functions. > >I expected more, but in most cases this would help we cannot yet shrink- > >wrap because there are

[PATCH] Another small cleanup to the const_and_copies stack

While working on the next significant hunk of changes towards 47679, I saw a few more obvious cleanups that ought to happen. First, we have a policy for data members that they ought to be prefixed with m_ to denote the extra overhead in accessing those members. The data member in the

Re: [PATCH][PR67476] Add param parloops-schedule

On Fri, Sep 11, 2015 at 12:55:00PM +0200, Tom de Vries wrote: > Hi, > > this patch adds a param parloops-schedule=<0-4>, which sets the omp schedule > for loops paralellized by parloops. > > The <0-4> maps onto . > > Bootstrapped and reg-tested on x86_64. >

[patch] libstdc++/58265 Implement N4258 noexcept for std::basic_string.

This updates the non-COW basic_string to meet the noexcept requirements in the current draft, which required finishing the allocator propagation support. Tested powerpc64le-linux, normal and debug. Committed to trunk. commit 909cf7116d75d42920e4b4f89ee8ddf04c843a9c Author: Jonathan Wakely

Re: [PATCH 04/22] Reimplement diagnostic_show_locus, introducing rich_location classes

Hi, On Thu, 10 Sep 2015, David Malcolm wrote: > +/* FIXME: (dmalcolm) > + This plugin is currently the only user of > + gcc_rich_location::add_range_with_caption > + As such, the symbol is present in libbackend.a, but not in "cc1", > + and running the plugin fails with a linker error:

Re: [PATCH 07/22] Implement token range tracking within libcpp and C/C++ FEs

Hi, On Thu, 10 Sep 2015, David Malcolm wrote: > Does anyone know why this was "carefully packed" and to what extent > this matters? I'm adding an extra 8 bytes to it (or 4 if we eliminate > the existing location_t). As far as I can see, these are > short-lived, and there are only relative few

Re: [PATCH 04/22] Reimplement diagnostic_show_locus, introducing rich_location classes

Hi, On Thu, 10 Sep 2015, David Malcolm wrote: > +/* A range of source locations. > + > + Ranges are half-open: > + m_start is the first location within the range, whereas > + m_finish is the first location *after* the range. I think you eventually decided that they are closed, not

Re: [PATCH][PR67476] Add param parloops-schedule

On 11/09/15 15:28, Tom de Vries wrote: So the definition of param parloop-schedule becomes: ... DEFPARAMENUM PARAM_PARLOOPS_SCHEDULE, "parloops-schedule", "Schedule type of omp schedule for loops parallelized by " "parloops (static, dynamic, guided,

[patch] libstdc++/65142 Check read() result in std::random_device.

We should not silently ignore a failure to read from the random device. Tested powerpc64le-linux, committed to trunk. I'm going to commit this to the gcc-5 branch too. commit 2d2f7012dc3744dafef0de94dd845bd190253dbd Author: Jonathan Wakely Date: Fri Feb 20 17:29:50 2015

Re: [PATCH] vectorizing conditional expressions (PR tree-optimization/65947)

On Fri, Sep 11, 2015 at 2:19 PM, Bill Schmidt wrote: > Hi Alan, > > I probably wasn't clear enough. The implementation in the vectorizer is > fine and I'm not asking that to change per target. What I'm objecting > to is the equivalence between a REDUC_MAX_EXPR and a

[gomp4] Override default target hook

Hi! The attached patch overrides the hook's default action for anchored address usage. Committed after regtest on x86_64 and powerpc64le Thanks! Jim Index: nvptx.c === --- nvptx.c (revision 227683) +++ nvptx.c (working copy) @@

[PATCH][PR67476] Add param parloops-schedule

Hi, this patch adds a param parloops-schedule=<0-4>, which sets the omp schedule for loops paralellized by parloops. The <0-4> maps onto . Bootstrapped and reg-tested on x86_64. OK for trunk? Thanks, - Tom Add param parloops-schedule 2015-09-10 Tom de

Re: [PATCH 2/2] shrink-wrap: Rewrite try_shrink_wrapping

On Fri, Sep 11, 2015 at 10:24:42AM +0100, Jiong Wang wrote: > > Segher Boessenkool writes: > > > On Thu, Sep 10, 2015 at 08:14:29AM -0700, Segher Boessenkool wrote: > >> This patch rewrites the shrink-wrapping algorithm, allowing non-linear > >> pieces of CFG to be duplicated for use without

Re: [Patch, libstdc++] Add specific error message into exceptions

On Mon, Sep 7, 2015 at 4:06 AM, Jonathan Wakely wrote: > On 28/08/15 11:23 -0700, Tim Shen wrote: >> >> On Fri, Aug 28, 2015 at 8:59 AM, Jonathan Wakely >> wrote: >>> >>> There seems to be no need to construct a std::string here, just pass a >>> const

Re: [PATCH][AArch64] Use preferred aliases for CSNEG, CSINC, CSINV

On Tue, Sep 1, 2015 at 6:08 PM, Kyrill Tkachov wrote: > Hi all, > > The ARMv8-A reference manual says: > "CNEG , , > is equivalent to > CSNEG , , , invert() > and is the preferred disassembly when Rn == Rm && cond != '111x'." > > That is, when the two input registers are

Re: [PATCH 2/2] shrink-wrap: Rewrite try_shrink_wrapping

On Fri, Sep 11, 2015 at 11:19:40AM +0200, Bernd Schmidt wrote: > On 09/10/2015 05:14 PM, Segher Boessenkool wrote: > >This patch rewrites the shrink-wrapping algorithm, allowing non-linear > >pieces of CFG to be duplicated for use without prologue instead of just > >linear pieces. > > An example

Re: New power of 2 hash policy

Hi, On Thu, 10 Sep 2015, François Dumont wrote: > Here is a patch to offer an alternative hash policy. This one is > using power of 2 number of buckets allowing a faster modulo operation. > This is obvious when running the performance test that I have adapted to > use this alternative

Re: [PATCH] vectorizing conditional expressions (PR tree-optimization/65947)

Hi Alan, I probably wasn't clear enough. The implementation in the vectorizer is fine and I'm not asking that to change per target. What I'm objecting to is the equivalence between a REDUC_MAX_EXPR and a cost associated with vec_to_scalar. This assumes that the back end will implement a

[patch] libstdc++/64857 Rationalise PCH headers and 17_intro/headers tests.

Ensure that includes everything, and rename the 17_intro/headers/c++200x directory to c++2011. Tested powerpc64le-linux, committed to trunk. commit 3b07b2a428725ba70636e00a1f6ed8a28bb59b44 Author: Jonathan Wakely Date: Thu Jan 29 13:12:22 2015 + Rationalise PCH

Re: [PATCH] Teach genmatch.c to generate single-use restrictions from flags

On 07/08/2015 04:39 PM, Richard Biener wrote: This introduces a :s flag to match expressions which enforces the expression to have a single-use if(!) the simplified expression is larger than one statement. This seems to be missing documentation in match-and-simplify.texi. Bernd

Re: New power of 2 hash policy

On 11/09/15 15:11 +0200, Michael Matz wrote: Hi, On Thu, 10 Sep 2015, François Dumont wrote: Here is a patch to offer an alternative hash policy. This one is using power of 2 number of buckets allowing a faster modulo operation. This is obvious when running the performance test that I

Re: [PATCH][PR67476] Add param parloops-schedule

On 11/09/15 12:57, Jakub Jelinek wrote: On Fri, Sep 11, 2015 at 12:55:00PM +0200, Tom de Vries wrote: >Hi, > >this patch adds a param parloops-schedule=<0-4>, which sets the omp schedule >for loops paralellized by parloops. > >The <0-4> maps onto . >

Re: [PATCH 2/2] shrink-wrap: Rewrite try_shrink_wrapping

On 09/11/2015 02:36 PM, Segher Boessenkool wrote: I'm wondering how your new algorithm prevents the prologue from being placed inside a loop. Can you have a situation where this picks a predecessor that is reachable but not dominated by PRO? It doesn't prevent it! The prologue will not be

Re: New power of 2 hash policy

On 11/09/15 14:18 +0100, Jonathan Wakely wrote: On 11/09/15 15:11 +0200, Michael Matz wrote: Hi, On Thu, 10 Sep 2015, François Dumont wrote: Here is a patch to offer an alternative hash policy. This one is using power of 2 number of buckets allowing a faster modulo operation. This is

[patch] libstdc++/67173 Fix filesystem::canonical for Solaris 10.

Solaris 10 doesn't follow POSIX in accepting a null pointer as the second argument to realpath(), so allocate a buffer for it. Tested x86_64-linux, committed to trunk. commit ed4023452f85f6c745ce473b2503f4e46fb02cd9 Author: Jonathan Wakely Date: Fri Sep 11 15:19:27 2015

Re: [PATCH] vectorizing conditional expressions (PR tree-optimization/65947)

Ramana Radhakrishnan writes: > On Fri, Sep 11, 2015 at 2:19 PM, Bill Schmidt > wrote: >> Hi Alan, >> >> I probably wasn't clear enough. The implementation in the vectorizer is >> fine and I'm not asking that to change per target. What I'm

Re: Ping^2 Re: Pass -foffload targets from driver to libgomp at link time

On 09/11/2015 04:23 PM, Joseph Myers wrote: On Thu, 10 Sep 2015, Bernd Schmidt wrote: On 09/10/2015 03:41 PM, Joseph Myers wrote: Ping^2. This patch is still pending review. No fundamental objections, but I have some questions.

Re: Ping^2 Re: Pass -foffload targets from driver to libgomp at link time

On Thu, 10 Sep 2015, Bernd Schmidt wrote: > On 09/10/2015 03:41 PM, Joseph Myers wrote: > > Ping^2. This patch > > is still > > pending review. > > No fundamental objections, but I have some questions. Cuold you describe > what the

Re: [PATCH 2/2] shrink-wrap: Rewrite try_shrink_wrapping

Segher Boessenkool writes: > On Fri, Sep 11, 2015 at 10:24:42AM +0100, Jiong Wang wrote: >> >> Segher Boessenkool writes: >> >> > On Thu, Sep 10, 2015 at 08:14:29AM -0700, Segher Boessenkool wrote: >> >> This patch rewrites the shrink-wrapping algorithm, allowing non-linear >> >> pieces of CFG

Re: [PATCH] vectorizing conditional expressions (PR tree-optimization/65947)

On Fri, 2015-09-11 at 15:29 +0100, Richard Sandiford wrote: > Ramana Radhakrishnan writes: > > On Fri, Sep 11, 2015 at 2:19 PM, Bill Schmidt > > wrote: > >> Hi Alan, > >> > >> I probably wasn't clear enough. The implementation in the

Re: [PATCH 04/22] Reimplement diagnostic_show_locus, introducing rich_location classes

On Fri, 2015-09-11 at 16:07 +0200, Michael Matz wrote: > Hi, > > On Thu, 10 Sep 2015, David Malcolm wrote: > > > +/* A range of source locations. > > + > > + Ranges are half-open: > > + m_start is the first location within the range, whereas > > + m_finish is the first location *after* the

Re: [PATCH] vectorizing conditional expressions (PR tree-optimization/65947)

Bill Schmidt writes: > On Fri, 2015-09-11 at 15:29 +0100, Richard Sandiford wrote: >> Ramana Radhakrishnan writes: >> > On Fri, Sep 11, 2015 at 2:19 PM, Bill Schmidt >> > wrote: >> >> Hi Alan, >> >> >> >> I

Re: [PATCH] PR67401: Fix wrong code generated by expand_atomic_compare_and_swap

On 2015-09-11 4:15 AM, Bernd Schmidt wrote: On 09/11/2015 01:21 AM, John David Anglin wrote: As noted in the PR, expand_atomic_compare_and_swap can generate wrong code when libcalls are emitted for the sync_compare_and_swap and the result comparison test. This is fixed by emitting a move insn

Re: [PATCH][AArch64] Use preferred aliases for CSNEG, CSINC, CSINV

Ping. https://gcc.gnu.org/ml/gcc-patches/2015-09/msg00020.html Thanks, Kyrill On 01/09/15 11:08, Kyrill Tkachov wrote: Hi all, The ARMv8-A reference manual says: "CNEG , , is equivalent to CSNEG , , , invert() and is the preferred disassembly when Rn == Rm && cond != '111x'." That is, when

Re: Ping^2 Re: Pass -foffload targets from driver to libgomp at link time

On Fri, 11 Sep 2015, Bernd Schmidt wrote: > I expect you know best what to do in the OpenACC testsuite driver, but you > might want to run the libgomp.exp parts by Jakub. If the testsuite parts are > independent of the rest of the patch, please repost them separately. Jakub? The testsuite

Re: [PATCH 21/22] Use Levenshtein distance for various misspellings in C frontend

On 10/09/15 22:28, David Malcolm wrote: There are a couple of FIXMEs here: * where to call levenshtein_distance_unit_tests Should this be part of make check? Perhaps a small program that is compiled and linked with spellcheck.c? This would be possible if spellcheck.c did not depend on tree.h

Re: [PATCH][AArch64] Use preferred aliases for CSNEG, CSINC, CSINV

On Tue, Sep 01, 2015 at 11:08:10AM +0100, Kyrill Tkachov wrote: > Hi all, > > The ARMv8-A reference manual says: > "CNEG , , > is equivalent to > CSNEG , , , invert() > and is the preferred disassembly when Rn == Rm && cond != '111x'." > > That is, when the two input registers are the same we

Re: [PATCH][RTL-ifcvt] PR rtl-optimization/67465: Handle pairs of complex+simple blocks and empty blocks more gracefully

On 11/09/15 09:51, Rainer Orth wrote: Kyrill Tkachov writes: On 10/09/15 12:43, Rainer Orth wrote: Hi Kyrill, Rainer, could you please check that this patch still fixes the SPARC regressions? unfortunately, it breaks sparc-sun-solaris2.10 bootstrap: compiling

Re: Ping^2 Re: Pass -foffload targets from driver to libgomp at link time

On Fri, Sep 11, 2015 at 03:26:04PM +, Joseph Myers wrote: > On Fri, 11 Sep 2015, Bernd Schmidt wrote: > > > I expect you know best what to do in the OpenACC testsuite driver, but you > > might want to run the libgomp.exp parts by Jakub. If the testsuite parts are > > independent of the rest

Re: [PATCH] vectorizing conditional expressions (PR tree-optimization/65947)

On Fri, 2015-09-11 at 16:28 +0100, Richard Sandiford wrote: > Bill Schmidt writes: > > On Fri, 2015-09-11 at 15:29 +0100, Richard Sandiford wrote: > >> Ramana Radhakrishnan writes: > >> > On Fri, Sep 11, 2015 at 2:19 PM, Bill Schmidt > >> >

Re: Openacc launch API

Ping? https://gcc.gnu.org/ml/gcc-patches/2015-08/msg01498.html On 09/07/15 08:48, Nathan Sidwell wrote: On 08/25/15 09:29, Nathan Sidwell wrote: Jakub, This patch changes the launch API for openacc parallels. The current scheme passes the launch dimensions as 3 separate parameters to the