[PING^2] C++ patches ping

Hi, Could I please have feedback on the following patches? PR c++/109918: Unexpected -Woverloaded-virtual with virtual conversion operators => https://gcc.gnu.org/pipermail/gcc-patches/2024-October/665650.html PR c++/114292: ICE with a generic (templated) lambda capturing a constant for VLA

C++ patches ping

Hi, I’d like to ping the following C++ patches: PR c++/109918: Unexpected -Woverloaded-virtual with virtual conversion operators => https://gcc.gnu.org/pipermail/gcc-patches/2024-October/665650.html PR c++/114619:ICE with -fno-elide-constructors in C++14 mode for non-constant initializer in

Re: C/C++ frontend patches ping

On Fri, Mar 15, 2024 at 6:20 AM Andi Kleen wrote: > > Andrew Pinski writes: > > > On Thu, Mar 14, 2024 at 9:36 PM Andi Kleen wrote: > >> > >> > >> musttail support for C/C++ > >> > >> https://gcc.gnu.org/pipermail/gcc-patches/2024-January/643867.html > >> > >> > >> Support constexpr for asm stat

Re: C/C++ frontend patches ping

Andrew Pinski writes: > On Thu, Mar 14, 2024 at 9:36 PM Andi Kleen wrote: >> >> >> musttail support for C/C++ >> >> https://gcc.gnu.org/pipermail/gcc-patches/2024-January/643867.html >> >> >> Support constexpr for asm statements in C++ >> >> https://gcc.gnu.org/pipermail/gcc-patches/2024-January

Re: C/C++ frontend patches ping

On Thu, Mar 14, 2024 at 9:36 PM Andi Kleen wrote: > > > musttail support for C/C++ > > https://gcc.gnu.org/pipermail/gcc-patches/2024-January/643867.html > > > Support constexpr for asm statements in C++ > > https://gcc.gnu.org/pipermail/gcc-patches/2024-January/643933.html Both of these were po

C/C++ frontend patches ping

musttail support for C/C++ https://gcc.gnu.org/pipermail/gcc-patches/2024-January/643867.html Support constexpr for asm statements in C++ https://gcc.gnu.org/pipermail/gcc-patches/2024-January/643933.html

C++ and DWARF patches ping

Hi! I'd like to ping a couple of patches: - PR77886 http://gcc.gnu.org/ml/gcc-patches/2016-10/msg00524.html - fix -Wimplicit-fallthrough handling in templates - DWARF http://gcc.gnu.org/ml/gcc-patches/2016-10/msg01183.html - DW_AT_inline for C++17 inline variables - DWARF5 http://gcc.gnu.or

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 02/25/2016 04:51 PM, Jakub Jelinek wrote: Do you have some short Ada testcase where the DW_OP_call4 referring to DW_TAG_dwarf_procedure is supposed to be emitted? I believe you must be getting there the .Ldebug_info0+0 invalid reference in the DW_OP_call4 operand. Sure! Here’s one: $ gcc -

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 02/25/2016 10:48 AM, Jakub Jelinek wrote: The first one just fixes what I mainly care about, the committed patch assumed that DW_TAG_dwarf_procedure is always only created for the Ada variable sized structures or whatever it was meant for, which is not the case, and thus if we emit DW_TAG_dwar

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On Thu, Feb 25, 2016 at 11:35:07AM +0100, Pierre-Marie de Rodat wrote: > On 02/25/2016 10:48 AM, Jakub Jelinek wrote: > >Unfortunately, this broke the DW_OP_GNU_implicit_pointer support, on vast > >majority of binaries and libraries gcc now emits invalid DWARF (which both > >gdb and dwz complain ab

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

> I agree that catching this in scan-assembler test is hard, but guality test > would catch this. It is true that some guality tests (mostly the ones that > test behaviour of optimized code, which differs a lot between different > architectures) have known FAILs (or known XFAILs), because the targ

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On Thu, Feb 25, 2016 at 11:35:07AM +0100, Pierre-Marie de Rodat wrote: > As I said at the end of a message in another thread > (https://gcc.gnu.org/ml/gcc-patches/2016-01/msg01078.html), I always feel > uncomfortable writing brittle dg-scan testcases, hence the current lack of > testcases for those

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 02/25/2016 10:48 AM, Jakub Jelinek wrote: Unfortunately, this broke the DW_OP_GNU_implicit_pointer support, on vast majority of binaries and libraries gcc now emits invalid DWARF (which both gdb and dwz complain about and dwz refuses to optimize because of that). Arg, sorry about this! I'm

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On Wed, Dec 16, 2015 at 09:53:37AM +0100, Pierre-Marie de Rodat wrote: > On 12/11/2015 09:25 PM, Jason Merrill wrote: > >Hmm, can we generate the DWARF procedures during finalize_size_functions > >to avoid the need for preserve_body? > > Good idea, thank you! Here’s the updated patch (bootstrapped

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 01/03/2016 12:45 AM, Eric Botcazou wrote: On powerpc64: FAIL: gnat.dg/specs/debug1.ads scan-assembler-times DW_AT_artificial 18 $ grep -c DW_AT_artificial debug1.s 17 Test removed. Thank you both! -- Pierre-Marie de Rodat

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

> On powerpc64: > > FAIL: gnat.dg/specs/debug1.ads scan-assembler-times DW_AT_artificial 18 > > $ grep -c DW_AT_artificial debug1.s > 17 Test removed. -- Eric Botcazou

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

Eric Botcazou writes: >> I'm also seeing >> +FAIL: gnat.dg/specs/debug1.ads scan-assembler-times DW_AT_artificial 17 >> and from what I can see in gcc-testresults, I'm not alone. > > Minor oversight, adjusted like so: > > * gnat.dg/specs/debug1.ads: Bump final count to 18. On powerpc64: F

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 12/21/2015 04:39 PM, Jason Merrill wrote: OK, thanks. Committed. Thank you again! -- Pierre-Marie de Rodat

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

OK, thanks. Jason

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 12/18/2015 09:23 PM, Pierre-Marie de Rodat wrote: On 12/18/2015 06:56 PM, Jason Merrill wrote: These broke a lot of tests in the GDB C++ testsuite. Specifically, the commit DWARF: handle variable-length records and variant parts Arg, sad to hear that! I did testing at some point with

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 12/18/2015 06:56 PM, Jason Merrill wrote: These broke a lot of tests in the GDB C++ testsuite. Specifically, the commit DWARF: handle variable-length records and variant parts Arg, sad to hear that! I did testing at some point with the GDB testsuite… I’ll investigate on Monday, thank

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

> I'm also seeing > +FAIL: gnat.dg/specs/debug1.ads scan-assembler-times DW_AT_artificial 17 > and from what I can see in gcc-testresults, I'm not alone. Minor oversight, adjusted like so: * gnat.dg/specs/debug1.ads: Bump final count to 18. -- Eric Botcazou Index: gnat.dg/specs/debug1.a

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On Fri, Dec 18, 2015 at 12:56:07PM -0500, Jason Merrill wrote: > On 12/17/2015 09:10 AM, Pierre-Marie de Rodat wrote: > >On 12/16/2015 10:30 PM, Jason Merrill wrote: > >>OK with those changes. > > > >All changes done, and all patches pushed. Thank you very much!! > > These broke a lot of tests in

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 12/17/2015 09:10 AM, Pierre-Marie de Rodat wrote: On 12/16/2015 10:30 PM, Jason Merrill wrote: OK with those changes. All changes done, and all patches pushed. Thank you very much!! These broke a lot of tests in the GDB C++ testsuite. Specifically, the commit DWARF: handle variabl

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 12/16/2015 10:30 PM, Jason Merrill wrote: OK with those changes. All changes done, and all patches pushed. Thank you very much!! -- Pierre-Marie de Rodat

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 12/16/2015 03:53 AM, Pierre-Marie de Rodat wrote: + /* Called from finalize_size_functions for functions whose body is needed to + generate complete debug info. For instance, functions used to compute the + size of variable-length structures. */ + void (* function_body) (tree decl)

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 12/11/2015 09:25 PM, Jason Merrill wrote: Hmm, can we generate the DWARF procedures during finalize_size_functions to avoid the need for preserve_body? Good idea, thank you! Here’s the updated patch (bootstrapped and regtested on x86_64-linux, as usual). -- Pierre-Marie de Rodat >From 385

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 11/26/2015 07:34 AM, Pierre-Marie de Rodat wrote: On 11/25/2015 07:35 PM, Jason Merrill wrote: Actually, even though my patches introduce DWARF procedures for only one case (size functions from stor-layout.c), they don’t necessarily come from code generation (GENERIC): they are just a way to

Re: [PATCHES, PING*2] Enhance standard DWARF for Ada

Ping for the patches submitted in and for the 2/8 update submitted in . Thank you in advance! -- Pierre-Marie de Rodat

[PATCHES, PING] Enhance standard DWARF for Ada

On 11/26/2015 01:34 PM, Pierre-Marie de Rodat wrote: Done! (I repalced the dwarf_proc_decl_table hash table with a dwarf_proc_stack_usage_map hash_map) Here's an update for the only affected patch. Regtested again on x86_64-linux. Ping for the patches submitted in

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 11/25/2015 07:35 PM, Jason Merrill wrote: Actually, even though my patches introduce DWARF procedures for only one case (size functions from stor-layout.c), they don’t necessarily come from code generation (GENERIC): they are just a way to factorize common DWARF operations. Thinking more abou

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 11/24/2015 04:17 AM, Pierre-Marie de Rodat wrote: On 11/23/2015 10:08 PM, Jason Merrill wrote: On 11/23/2015 08:53 AM, Pierre-Marie de Rodat wrote: Do you think the other patches could make it before the branch? (if they could, I will rebase+retest them as quick as possible). Probably, ye

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 11/23/2015 10:08 PM, Jason Merrill wrote: On 11/23/2015 08:53 AM, Pierre-Marie de Rodat wrote: Do you think the other patches could make it before the branch? (if they could, I will rebase+retest them as quick as possible). Probably, yes. I can't find the DW_AT_static_link patch, though;

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 11/23/2015 08:53 AM, Pierre-Marie de Rodat wrote: Do you think the other patches could make it before the branch? (if they could, I will rebase+retest them as quick as possible). Probably, yes. I can't find the DW_AT_static_link patch, though; it doesn't seem to have been attached to you

Re: [PATCHES, PING*5] Enhance standard DWARF for Ada

On 10/20/2015 04:13 PM, Pierre-Marie de Rodat wrote: Sorry about the slow review on these patches. In future please feel free to ping me as often as once a week. + /* DWARF operations all work on signed integers. Note that this will no longer be the case in DWARF 5, where stack elements

[PATCHES, PING*4] Enhance standard DWARF for Ada

On 07/16/2015 10:34 AM, Pierre-Marie de Rodat wrote: This patch series aims at enhancing GCC to emit standard DWARF in place of the current GNAT encodings (non-standard DWARF) for a set of "basic" types: dynamic arrays, variable-length records, variant parts, etc. Ping for the patch series:

[PATCHES, PING*3] Enhance standard DWARF for Ada

Hi, On 07/16/2015 10:34 AM, Pierre-Marie de Rodat wrote: This patch series aims at enhancing GCC to emit standard DWARF in place of the current GNAT encodings (non-standard DWARF) for a set of "basic" types: dynamic arrays, variable-length records, variant parts, etc. Ping for the patch series

Re: [PATCHES, PING*2] Enhance standard DWARF for Ada

On 07/31/2015 12:54 PM, Pierre-Marie de Rodat wrote: On 07/16/2015 10:34 AM, Pierre-Marie de Rodat wrote: This patch series aims at enhancing GCC to emit standard DWARF in place of the current GNAT encodings (non-standard DWARF) for a set of "basic" types: dynamic arrays, variable-length records

[PATCHES, PING*2] Enhance standard DWARF for Ada

On 07/16/2015 10:34 AM, Pierre-Marie de Rodat wrote: This patch series aims at enhancing GCC to emit standard DWARF in place of the current GNAT encodings (non-standard DWARF) for a set of "basic" types: dynamic arrays, variable-length records, variant parts, etc. Ping for the patch series:

[PATCHES, PING] Enhance standard DWARF for Ada

On 07/16/2015 10:34 AM, Pierre-Marie de Rodat wrote: This patch series aims at enhancing GCC to emit standard DWARF in place of the current GNAT encodings (non-standard DWARF) for a set of "basic" types: dynamic arrays, variable-length records, variant parts, etc. Ping for the patch series:

Patches ping

Hello, [PATCH, SMS] Prevent the creation of reg-moves for definitions with MODE_CC http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01459.html [PATCH SMS 2/2, RFC] Register pressure estimation for the partial schedule http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01330.html Thanks, Revital

Re: Patches ping

Hi, Thanks for the review! > > >Changelog: > > > >       (sms_schedule_by_order): Update call to get_sched_window. > >       all set_must_precede_follow. >     ^^^ >     call Done. > >+/* Set bitmaps TMP_FOLLOW and TMP_PRECEDE to MUST_FOLLOW and MUST_PRECEDE > >+   respectively only if

Re: Patches ping

> [PATCH, SMS] Fix calculation of issue_rate > http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01344.html This is ok (with the updated Changelog). Alternatively, we can have a local variable for holding the issue_rate. Ayal. 2011/7/20 Revital Eres : > Hello, > > [PATCH, SMS 3/4] Optimize stage coun

Re: Patches ping

>[PATCH, SMS 4/4] Misc. fixes http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01342.html Sure, this is fine. (Sorry for all the previous '?'s..). Thanks, Ayal. 2011/7/20 Revital Eres > > Hello, > > [PATCH, SMS 3/4] Optimize stage count > http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01341.html > > [P

Re: Patches ping

> [PATCH, SMS 3/4] Optimize stage count > http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01341.html This patch for minimizing the stage count (which also refactors and cleans up the code) is approved. Have some minor comments below, followed by some thoughts for possible follow-up improvements. Than

Patches ping

Hello, [PATCH, SMS 3/4] Optimize stage count http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01341.html [PATCH, SMS 4/4] Misc. fixes http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01342.html [PATCH, SMS] Fix calculation of issue_rate http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01344.html Thanks, Re

Patches ping

Hello, [PATCH, SMS 1/4] Fix calculation of row_rest_count http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01339.html [PATCH, SMS 2/4] Move the creation of anti-dep edge http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01340.html [PATCH, SMS 3/4] Optimize stage count http://gcc.gnu.org/ml/gcc-patches/20

Patches ping

Hello, http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00350.html - Support closing_branch_deps http://gcc.gnu.org/ml/gcc-patches/2011-04/msg01309.html - Support instructions with REG_INC_NOTE http://gcc.gnu.org/ml/gcc-patches/2011-04/msg01294.html - Avoid considering debug_insn when calculating SC