Re: [PATCH] Harmonize headers between both dg-extract-results scripts

2023-09-17 Thread Paul Iannetta via Gcc-patches
On Thu, Sep 14, 2023 at 04:24:33PM +0200, Paul Iannetta wrote: > Hi, > > This is a small patch so that both dg-extract-results.py and > dg-extract-results.sh share the same header. In particular, it fixes > the fact that the regexp r'^Test Run By (\S+) on (.*)$' was never > matched in the python

[PATCH] Harmonize headers between both dg-extract-results scripts

2023-09-14 Thread Paul Iannetta via Gcc-patches
Hi, This is a small patch so that both dg-extract-results.py and dg-extract-results.sh share the same header. In particular, it fixes the fact that the regexp r'^Test Run By (\S+) on (.*)$' was never matched in the python file. Thanks, -- Paul Kalray Author: Paul Iannetta Date: Thu Sep 14 1

Re: [PATCH v4] c++: parser - Support for target address spaces in C++

2022-11-10 Thread Paul Iannetta via Gcc-patches
Hi, It took a bit of time to rework the rough corners. I tried to be mirror as much as possible the C front-end, especially when it comes to implicit conversions, and warnings; and expose the same hooks as the C front-end does. The C++ front-end produces as much warnings that the C front-end, ho

Re: [PATCH v3] c++: parser - Support for target address spaces in C++

2022-11-10 Thread Paul Iannetta via Gcc-patches
On Thu, Nov 03, 2022 at 02:38:39PM +0100, Georg-Johann Lay wrote: > > [PATCH v3] c++: parser - Support for target address spaces in C++ > > First of all, it is great news that GCC is going to implement named address > spaces for C++. > > I have some questions: > > 1. How is name-mangling going t

Re: [PATCH v3] c++: parser - Support for target address spaces in C++

2022-10-26 Thread Paul Iannetta via Gcc-patches
On Wed, Oct 19, 2022 at 02:55:21PM -0400, Jason Merrill wrote: > On 10/18/22 13:01, Paul Iannetta wrote: > > Thank you very much for the detailed review. > > > > On Tue, Oct 18, 2022 at 10:24:23AM -0400, Jason Merrill wrote: > > > On 10/18/22 03:37, Paul Iannetta wrote: > > > > On Fri, Oct 14, 202

Re: Announcement: Porting the Docs to Sphinx - 9. November 2022

2022-10-19 Thread Paul Iannetta via Gcc-patches
On Wed, Oct 19, 2022 at 09:24:06AM +0200, Martin Liška wrote: > On 10/17/22 16:16, Paul Iannetta wrote: > > Hi Martin, > > > > Thank you very much for porting the documentation to Sphinx, it is > > very convenient to use, especially the menu on the left and the > > search bar. > > Thanks, I also

Re: [PATCH v3] c++: parser - Support for target address spaces in C++

2022-10-18 Thread Paul Iannetta via Gcc-patches
Thank you very much for the detailed review. On Tue, Oct 18, 2022 at 10:24:23AM -0400, Jason Merrill wrote: > On 10/18/22 03:37, Paul Iannetta wrote: > > On Fri, Oct 14, 2022 at 11:19:50AM -0400, Jason Merrill wrote: > > > On 10/13/22 17:57, Paul Iannetta wrote: > > > > On Thu, Oct 13, 2022 at 03:

Re: [RFC] Add support for vectors in comparisons (like the C++ frontend does)

2022-10-18 Thread Paul Iannetta via Gcc-patches
On Mon, Oct 17, 2022 at 09:22:23AM +0200, Richard Biener wrote: > On Fri, Oct 14, 2022 at 4:18 PM Paul Iannetta via Gcc-patches > wrote: > > > > On Wed, Oct 12, 2022 at 01:18:19AM +0200, Paul Iannetta wrote: > > > On Mon, Oct 10, 2022 at 11:07:06PM +, Joseph Myer

Re: [PATCH v3] c++: parser - Support for target address spaces in C++

2022-10-18 Thread Paul Iannetta via Gcc-patches
On Fri, Oct 14, 2022 at 11:19:50AM -0400, Jason Merrill wrote: > On 10/13/22 17:57, Paul Iannetta wrote: > > On Thu, Oct 13, 2022 at 03:41:16PM -0400, Jason Merrill wrote: > > > On 10/13/22 12:02, Paul Iannetta wrote: > > > > On Thu, Oct 13, 2022 at 11:47:42AM -0400, Jason Merrill wrote: > > > > >

Re: Announcement: Porting the Docs to Sphinx - 9. November 2022

2022-10-17 Thread Paul Iannetta via Gcc-patches
Hi Martin, Thank you very much for porting the documentation to Sphinx, it is very convenient to use, especially the menu on the left and the search bar. However, I also regularly browse and search the documentation through info, especially when I want to use regexps to search or need to include

Re: [RFC] Add support for vectors in comparisons (like the C++ frontend does)

2022-10-14 Thread Paul Iannetta via Gcc-patches
On Wed, Oct 12, 2022 at 01:18:19AM +0200, Paul Iannetta wrote: > On Mon, Oct 10, 2022 at 11:07:06PM +, Joseph Myers wrote: > > On Mon, 10 Oct 2022, Paul Iannetta via Gcc-patches wrote: > > > > > I have a patch to bring this feature to the C front-end as well, and >

Re: [PATCH v2] c++: parser - Support for target address spaces in C++

2022-10-13 Thread Paul Iannetta via Gcc-patches
On Thu, Oct 13, 2022 at 03:41:16PM -0400, Jason Merrill wrote: > On 10/13/22 12:02, Paul Iannetta wrote: > > On Thu, Oct 13, 2022 at 11:47:42AM -0400, Jason Merrill wrote: > > > On 10/13/22 11:23, Paul Iannetta wrote: > > > > On Thu, Oct 13, 2022 at 11:02:24AM -0400, Jason Merrill wrote: > > > > >

Re: [PATCH v2] c++: parser - Support for target address spaces in C++

2022-10-13 Thread Paul Iannetta via Gcc-patches
On Thu, Oct 13, 2022 at 11:47:42AM -0400, Jason Merrill wrote: > On 10/13/22 11:23, Paul Iannetta wrote: > > On Thu, Oct 13, 2022 at 11:02:24AM -0400, Jason Merrill wrote: > > > On 10/12/22 20:52, Paul Iannetta wrote: > > > > On Tue, Oct 11, 2022 at 09:49:43PM -0400, Jason Merrill wrote: > > > > >

Re: [PATCH v2] c++: parser - Support for target address spaces in C++

2022-10-13 Thread Paul Iannetta via Gcc-patches
On Thu, Oct 13, 2022 at 11:02:24AM -0400, Jason Merrill wrote: > On 10/12/22 20:52, Paul Iannetta wrote: > > On Tue, Oct 11, 2022 at 09:49:43PM -0400, Jason Merrill wrote: > > > > > > It surprises that this is the only place we complain about an object with > > > an > > > address-space qualifier.

Re: [PATCH v2] c++: parser - Support for target address spaces in C++

2022-10-13 Thread Paul Iannetta via Gcc-patches
On Thu, Oct 13, 2022 at 07:46:46AM +0200, Jakub Jelinek wrote: > On Thu, Oct 13, 2022 at 02:52:59AM +0200, Paul Iannetta via Gcc-patches wrote: > > + if (type != error_mark_node > > + && !ADDR_SPACE_GENERIC_P

Re: [PATCH v2] c++: parser - Support for target address spaces in C++

2022-10-12 Thread Paul Iannetta via Gcc-patches
On Tue, Oct 11, 2022 at 09:49:43PM -0400, Jason Merrill wrote: > > It surprises that this is the only place we complain about an object with an > address-space qualifier. Shouldn't we also complain about e.g. automatic > variables/parameters or non-static data members with address-space qualified

Re: [RFC] Add support for vectors in comparisons (like the C++ frontend does)

2022-10-11 Thread Paul Iannetta via Gcc-patches
On Mon, Oct 10, 2022 at 11:07:06PM +, Joseph Myers wrote: > On Mon, 10 Oct 2022, Paul Iannetta via Gcc-patches wrote: > > > I have a patch to bring this feature to the C front-end as well, and > > would like to hear your opinion on it, especially since it may affect > >

Re: [PATCH v2] c++: parser - Support for target address spaces in C++

2022-10-11 Thread Paul Iannetta via Gcc-patches
Thank you very much for the comments. On Mon, Oct 10, 2022 at 03:20:13PM -0400, Jason Merrill wrote: > On 10/9/22 12:12, Paul Iannetta wrote: > > That's a nice feature! LLVM is using AS to mangle the > > address-name qualified types but that would have required an update of > > libiberty's demangl

Re: [RFC] Add support for vectors in comparisons (like the C++ frontend does)

2022-10-10 Thread Paul Iannetta via Gcc-patches
Hi Ian, Observations are also very welcomed, thank you! On Mon, Oct 10, 2022 at 03:37:24PM +0100, Iain Sandoe wrote: > Hi Paul, > > Not a review of the patch - but a couple of observations. > > > On 10 Oct 2022, at 15:11, Paul Iannetta via Gcc-patches > > wrote: >

[RFC] Add support for vectors in comparisons (like the C++ frontend does)

2022-10-10 Thread Paul Iannetta via Gcc-patches
Hi, I am trying to bridge the gap between the extensions supported by the C and C++ front-ends. When it comes to vector extensions, C++ supports vectors in comparisons and within conditional expressions whereas the C front-end does not. I have a patch to bring this feature to the C front-end as

Re: [PATCH] c++: parser - Support for target address spaces in C++

2022-10-09 Thread Paul Iannetta via Gcc-patches
Hi, On Thu, Oct 06, 2022 at 01:34:40PM -0400, Jason Merrill wrote: [snip] > > Hmm? We mangle __restrict: > > void f(int *__restrict *p) { } // _Z1fPrPi > Indeed, I have overlooked that point. Thank you for pointing it out. > but cv-qualifiers (including restrict) are dropped on the top-leve

[RFC] c++: parser - Support for target address spaces in C++

2022-10-06 Thread Paul Iannetta via Gcc-patches
Hi, Presently, GCC supports target address spaces as a GNU extension (cf. `info -n "(gcc)Named Address Spaces"`). This is however supported only by the C frontend, which is a bit sad, since all the GIMPLE machinery is readily available and, moreover, LLVM supports this GNU extension both for C an

Re: [PING^1][PATCH] postscan_insn hook not called after input_asm

2022-04-07 Thread Paul Iannetta via Gcc-patches
Hi, I would like to draw your attention on this patch, it's a pretty tiny patch which does not affect many targets. - Original Message - From: "gcc-patches" To: "gcc-patches" Sent: Tuesday, March 29, 2022 4:10:16 PM Subject: [PATCH] postscan_insn hook not called after input_asm Hi, W

[PATCH] postscan_insn hook not called after input_asm

2022-03-29 Thread Paul Iannetta via Gcc-patches
Hi, While working on the Kalray port of gcc, I noticed that the hook TARGET_ASM_FINAL_POSTSCAN_INSN is not called after emitting an instruction coming from a basic asm block. Here is a patch which fixes this behavior. The following check: ``` $ find gcc/config/ -type f -exec grep "#define TARG