Re: [Patch, Fortran] PR 86830: [8/9 Regression] Contiguous array pointer function result not recognized as contiguous

2018-09-10 Thread Janus Weil
Am Mo., 10. Sep. 2018 um 08:54 Uhr schrieb Paul Richard Thomas : > > Hi Janus, > > That's fine - OK for both branches. Committed to trunk as r264201. Will do 8-branch soon. Thanks for the review, Janus > On 9 September 2018 at 21:34, Janus Weil wrote: > > Hi all, > > > > the attached patch fix

Re: [PATCH, OpenACC] C++ reference mapping (PR middle-end/86336)

2018-09-10 Thread Julian Brown
On Mon, 10 Sep 2018 22:22:15 +0100 Jason Merrill wrote: > On Mon, Sep 10, 2018 at 7:07 PM, Julian Brown > wrote: > > I think it's more accurate to say that OpenACC says nothing about > > C++ references at all, nor about how unadorned pointers are mapped > > in copy/copyin/copyout clauses. So arg

Re: [PATCH, OpenACC] C++ reference mapping (PR middle-end/86336)

2018-09-10 Thread Jakub Jelinek
On Mon, Sep 10, 2018 at 10:22:15PM +0100, Jason Merrill wrote: > > As noted in a previous email, PGI seems to treat pointers to > > aggregates specially, mapping them as ptr[0:1], but it's unclear if the > > same is true for pointers to scalars with their compiler. Neither > > behaviour seems to be

Re: [wwwdocs] projects/cfo.html -- simplify

2018-09-10 Thread Gerald Pfeifer
On Sun, 2 Sep 2018, Gerald Pfeifer wrote: > On Sun, 2 Sep 2018, Gerald Pfeifer wrote: >> This uses a new CSS class *once* instead of attributing most elements >> in the table with align="right" and also brings us closer to HTML 5 >> compliance. > And this also takes care of align="center" by using

Re: [Patch, Fortran, F03] PR 85395: private clause contained in derived type acquires spurious scope

2018-09-10 Thread Janus Weil
Am Mo., 10. Sep. 2018 um 08:55 Uhr schrieb Paul Richard Thomas : > > Hi Janus, > > That's OK for trunk and, I would suggest 8-branch. Thanks, Paul. Committed to trunk as r264196. Will backport to 8-branch in a week or so. Cheers, Janus > On 9 September 2018 at 17:31, Janus Weil wrote: > > Hi

Re: [PATCH, OpenACC] C++ reference mapping (PR middle-end/86336)

2018-09-10 Thread Jason Merrill
On Mon, Sep 10, 2018 at 7:07 PM, Julian Brown wrote: > On Mon, 10 Sep 2018 10:52:47 -0700 > Cesar Philippidis wrote: > >> On 09/10/2018 10:37 AM, Jason Merrill wrote: >> > On Mon, Sep 10, 2018 at 4:05 AM, Julian Brown >> > wrote: >> >> This patch (by Cesar) changes the way C++ references are map

Re: Keep std::deque algos specializations in Debug mode

2018-09-10 Thread François Dumont
One more update, running more tests show that template alias was badly defined when users are using directly . So I enventually defined it in std::__detail namespace where it is easier to use _GLIBCXX_STD_C. Too bad that _Deque_iterator didn't got defined in __detail namespace so that we could

[PATCH, i386]: Rewrite x87 trigonometric patterns

2018-09-10 Thread Uros Bizjak
Hello! Attached patch removes a bunch of sincos -> sin, cos splitters. These are not necessary anymore since middle end handles the transformation in a generic way. Additionally, the patch removes unnecessary mixed-mode patters (SFmode/DFmode input operand and XFmode outputs). These are not needed

[wwwdocs] Use plain

2018-09-10 Thread Gerald Pfeifer
Per the HTML 5 validator used by w3.org. Applied. Gerald Index: projects/cxx0x.html === RCS file: /cvs/gcc/wwwdocs/htdocs/projects/cxx0x.html,v retrieving revision 1.76 diff -u -r1.76 cxx0x.html --- projects/cxx0x.html 1 Sep 2018 23

[wwwdocs] Move gcc-3.4/criteria.html to HTML 5

2018-09-10 Thread Gerald Pfeifer
...and remove "DRAFT" from its title on the way, ahem. Applied. Gerald Index: gcc-3.4/criteria.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-3.4/criteria.html,v retrieving revision 1.9 diff -u -r1.9 criteria.html --- gcc-3.4/criter

[PATCH, i386]: Use only memory_operand input operands in x87/SSE constant load splitter

2018-09-10 Thread Uros Bizjak
Hello! Currently, x87/SSE constant load splitter converts memory loads and register copies to supported immediate loads (xorps reg,reg, fld0, fld1, ...). However, it is cheaper to copy the value from register than to rematerialize the constant. Also, the compiler differs between SFmode, DFmode and

[PATCH] [ARC]: core3 features are default for core4

2018-09-10 Thread Vineet Gupta
* config/arc/arc.c: object attributes for core4 not reflected correctly * config/arc/arc.h: Don't restrict DBNZ to core3 (core4 includes core3) Signed-off-by: Vineet Gupta --- gcc/ChangeLog| 7 +++ gcc/config/arc/arc.c | 2 +- gcc/config/arc/arc.h | 2 +- 3 files changed, 9 ins

Re: [PATCH, OpenACC] C++ reference mapping (PR middle-end/86336)

2018-09-10 Thread Julian Brown
On Mon, 10 Sep 2018 10:52:47 -0700 Cesar Philippidis wrote: > On 09/10/2018 10:37 AM, Jason Merrill wrote: > > On Mon, Sep 10, 2018 at 4:05 AM, Julian Brown > > wrote: > >> This patch (by Cesar) changes the way C++ references are mapped in > >> OpenACC regions, fixing an ICE in the non-scalar-

Re: [PATCH] PR86844: Fix for store merging

2018-09-10 Thread Jakub Jelinek
On Mon, Sep 10, 2018 at 04:05:26PM +0200, Andreas Krebbel wrote: > On 20.08.2018 16:30, Jeff Law wrote: > > On 08/18/2018 03:20 AM, Eric Botcazou wrote: > >>> Eric, didn't your patches explicitely handle this case of a non-constant > >>> inbetween? > >> > >> Only if there is no overlap at all, othe

Re: [PATCH, OpenACC] C++ reference mapping (PR middle-end/86336)

2018-09-10 Thread Cesar Philippidis
On 09/10/2018 10:37 AM, Jason Merrill wrote: > On Mon, Sep 10, 2018 at 4:05 AM, Julian Brown wrote: >> This patch (by Cesar) changes the way C++ references are mapped in >> OpenACC regions, fixing an ICE in the non-scalar-data.C testcase. >> >> Post-patch, references are mapped like this (from the

Re: [PATCH, OpenACC] C++ reference mapping (PR middle-end/86336)

2018-09-10 Thread Jason Merrill
On Mon, Sep 10, 2018 at 4:05 AM, Julian Brown wrote: > This patch (by Cesar) changes the way C++ references are mapped in > OpenACC regions, fixing an ICE in the non-scalar-data.C testcase. > > Post-patch, references are mapped like this (from the omplower dump): > > map(force_present:*x [len: 4])

Re: [PATCH, OpenACC 2.5, libgomp] Add *_async versions of runtime library API functions

2018-09-10 Thread Cesar Philippidis
On 09/10/2018 08:04 AM, Chung-Lin Tang wrote: > GOACC_2.0 { > Index: libgomp/oacc-mem.c > === > --- libgomp/oacc-mem.c(revision 264192) > +++ libgomp/oacc-mem.c(working copy) > @@ -153,8 +153,9 @@ acc_free (void *d) >

Re: [PATCH] Add a dwarf unit type to represent 24 bit values.

2018-09-10 Thread Jason Merrill
On Mon, Sep 10, 2018 at 3:42 PM, John Darrington wrote: > On Mon, Sep 10, 2018 at 03:36:26PM +0100, Jason Merrill wrote: > On Mon, Aug 27, 2018 at 8:20 PM, John Darrington > wrote: > > * include/dwarf2.h (enum dwarf_unit_type) [DE_EH_PE_udata3]: > New member. > > > Wh

[PATCH, OpenACC 2.5, libgomp] Add *_async versions of runtime library API functions

2018-09-10 Thread Chung-Lin Tang
This patch adds *_async versions of several OpenACC runtime library API functions, which is to allow execution of a function asynchronously on particular async stream, an addition to the standard since 2.5. Specifically, these functions: acc_copyin_async acc_copyout_async acc_copyout_finalize_a

Re: [PATCH] Add a dwarf unit type to represent 24 bit values.

2018-09-10 Thread John Darrington
On Mon, Sep 10, 2018 at 03:36:26PM +0100, Jason Merrill wrote: On Mon, Aug 27, 2018 at 8:20 PM, John Darrington wrote: > * include/dwarf2.h (enum dwarf_unit_type) [DE_EH_PE_udata3]: New member. What's the rationale? Do you have a separate patch that uses t

Re: [PATCH] Add a dwarf unit type to represent 24 bit values.

2018-09-10 Thread Jason Merrill
On Mon, Aug 27, 2018 at 8:20 PM, John Darrington wrote: > * include/dwarf2.h (enum dwarf_unit_type) [DE_EH_PE_udata3]: New > member. This is a new macro, not a member of dwarf_unit_type. What's the rationale? Do you have a separate patch that uses this new macro? Jason

Re: [PATCH]: Allow TARGET_SCHED_ADJUST_PRIORITY hook to reduce priority

2018-09-10 Thread John David Anglin
On 2018-09-10 8:35 AM, Andreas Schwab wrote: On Sep 06 2018, Jeff Law wrote: On 09/03/2018 08:32 AM, John David Anglin wrote: The documentation for TARGET_SCHED_ADJUST_PRIORITY indicates that the hook can reduce the priority of INSN to execute it later.  The hppa hook only reduces the priorit

Re: [PATCH] PR86844: Fix for store merging

2018-09-10 Thread Andreas Krebbel
On 20.08.2018 16:30, Jeff Law wrote: > On 08/18/2018 03:20 AM, Eric Botcazou wrote: >>> Eric, didn't your patches explicitely handle this case of a non-constant >>> inbetween? >> >> Only if there is no overlap at all, otherwise you cannot do things simply. >> >>> Can you have a look / review here?

Re: [PATCH][OBVIOUS] Close file on return from verify-intermediate

2018-09-10 Thread Martin Liška
On 09/05/2018 03:29 PM, Joey Ye wrote: > This is a fix to an obvious issue in gcov.exp, where proc verify-intermediate > returns without closing the open file. > > This can be a possible fix to PR85871. gcov-8.C diffs to other gcov testcases > that it invokes verify-intermediate. Not closing an

Do not stream TYPE_VALUES to ltrans units

2018-09-10 Thread Jan Hubicka
Hi, TYPE_VALUES are currently only used to output warnings on ODR mismatched enums. I think those warnings are useful and thus we want to stream them to WPA, but there is no need to stream them further to ltrans units. Bootstrapped/regtested x86_64-linux, OK? * tree-streamer-out.c (write_

Re: [PATCH][4/4][v2] RPO-style value-numbering for FRE/PRE

2018-09-10 Thread Martin Liška
On 09/10/2018 02:19 PM, Gerald Pfeifer wrote: > On Mon, 10 Sep 2018, Martin Liška wrote: >> I can reproduce that locally in a KVM machine running FreeBSD test >> 10.4-RELEASE. I used gcc version 6.4.0 (FreeBSD Ports Collection) to >> build stage1 compiler and I can see Segfaults happening. > > G

Re: [PATCH]: Allow TARGET_SCHED_ADJUST_PRIORITY hook to reduce priority

2018-09-10 Thread Andreas Schwab
On Sep 06 2018, Jeff Law wrote: > On 09/03/2018 08:32 AM, John David Anglin wrote: >> The documentation for TARGET_SCHED_ADJUST_PRIORITY indicates that the >> hook can >> reduce the priority of INSN to execute it later.  The hppa hook only >> reduces the priority >> and it has been this way for y

Re: [PATCH v2] combine: perform jump threading at the end

2018-09-10 Thread Ilya Leoshkevich
> Am 07.09.2018 um 00:39 schrieb Segher Boessenkool > : > > On Thu, Sep 06, 2018 at 12:11:09PM -0600, Jeff Law wrote: >> On 09/05/2018 06:11 AM, Richard Biener wrote: >>> On Wed, Sep 5, 2018 at 2:01 PM Ilya Leoshkevich wrote: + /* Combining insns can change basic blocks in a way that th

Re: [PATCH][4/4][v2] RPO-style value-numbering for FRE/PRE

2018-09-10 Thread Gerald Pfeifer
On Mon, 10 Sep 2018, Martin Liška wrote: > I can reproduce that locally in a KVM machine running FreeBSD test > 10.4-RELEASE. I used gcc version 6.4.0 (FreeBSD Ports Collection) to > build stage1 compiler and I can see Segfaults happening. Great, thanks for helping look into this, Martin! > Iss

Re: [RFH] split {generic,gimple}-match.c files

2018-09-10 Thread Martin Liška
On 09/04/2018 05:07 PM, Martin Liška wrote: > - in order to achieve real speed up we need to split also other generated > (and also dwarf2out.c, i386.c, ..) files: > here I'm most concerned about insn-recog.c, which can't be split the same way > without ending up with a single huge SCC component.

Re: [PATCH][4/4][v2] RPO-style value-numbering for FRE/PRE

2018-09-10 Thread Martin Liška
On 09/05/2018 09:48 AM, Richard Biener wrote: > On Wed, 5 Sep 2018, Gerald Pfeifer wrote: > >> On Tue, 4 Sep 2018, Jeff Law wrote: On the other hand, this ICE has been consistent across a week of daily builds now. >>> An FYI, My i686 builds have been running OK. But given what you've >>

[PATCH v3] combine: perform jump threading at the end

2018-09-10 Thread Ilya Leoshkevich
Consider the following RTL: (code_label 11 10 26 4 2 (nil) [1 uses]) (note 26 11 12 4 [bb 4] NOTE_INSN_BASIC_BLOCK) (insn 12 26 15 4 (set (reg:SI 65) (if_then_else:SI (eq (reg:CCZ 33 %cc) (const_int 0 [0])) (const_int 1 [0x1]) (const_int 0 [0]))) "pr

Re: [PATCH v2] combine: perform jump threading at the end

2018-09-10 Thread Ilya Leoshkevich
> Am 06.09.2018 um 20:11 schrieb Jeff Law : > > On 09/05/2018 06:11 AM, Richard Biener wrote: >> On Wed, Sep 5, 2018 at 2:01 PM Ilya Leoshkevich wrote: >>> >>> gcc/ChangeLog: >>> >>> 2018-09-05 Ilya Leoshkevich >>> >>>PR target/80080 >>>* combine.c (rest_of_handle_combine

Re: C++ PATCH to tidy up build_vtbl_ref

2018-09-10 Thread Jason Merrill
OK. On Mon, Sep 10, 2018 at 12:32 AM, Marek Polacek wrote: > The wrapper for build_vtbl_ref_1 doesn't seem to do anything useful. > > Bootstrapped/regtested on x86_64-linux, ok for trunk? > > 2018-09-09 Marek Polacek > > * class.c (build_vtbl_ref): Remove. > (build_vtbl_ref_1):

[wwwdocs] Add some missing table cells to projects/cxx-status.html

2018-09-10 Thread Gerald Pfeifer
Committed. Gerald Index: projects/cxx-status.html === RCS file: /cvs/gcc/wwwdocs/htdocs/projects/cxx-status.html,v retrieving revision 1.58 diff -u -r1.58 cxx-status.html --- projects/cxx-status.html1 Sep 2018 23:42:09 -

[wwwdocs] projects/h8300-abi.html formatting revamp

2018-09-10 Thread Gerald Pfeifer
Use CSS instead of presentational markup for tables, and properly mark up table headings instead of treating those like regular rows. Committed. Gerald Index: projects/h8300-abi.html === RCS file: /cvs/gcc/wwwdocs/htdocs/projects/h8

Re: [wwwdocs] Revamp formatting markup of bugs/management.html

2018-09-10 Thread Gerald Pfeifer
On Sun, 2 Sep 2018, Gerald Pfeifer wrote: > ...making this page HTML 5 compliant. And with this little follow-up this page solely uses CSS, removing a sole warning the validator still showed after my change last week. Applied. Gerald Index: bugs/management.html =