Re: Patch ping^2

2024-02-26 Thread Jan Hubicka
> Hi! > > I'd like to ping 2 patches: > > https://gcc.gnu.org/pipermail/gcc-patches/2024-January/644580.html > > > PR113617 P1 - Handle private COMDAT function symbol reference in

Re: Patch ping^2

2024-02-26 Thread Uros Bizjak
On Mon, Feb 26, 2024 at 10:33 AM Jakub Jelinek wrote: > > Hi! > > I'd like to ping 2 patches: > https://gcc.gnu.org/pipermail/gcc-patches/2024-February/645326.html > i386: Enable _BitInt support on ia32 > > all the FAILs mentioned in that mail have been fixed by now. LGTM, based on HJ's advice.

Re: [PATCH PING 2 (tree)] c++: source position of lambda captures [PR84471]

2022-12-19 Thread Jason Merrill via Gcc-patches
On 12/2/22 10:45, Jason Merrill wrote: Tested x86_64-pc-linux-gnu, OK for trunk? -- 8< -- If the DECL_VALUE_EXPR of a VAR_DECL has EXPR_LOCATION set, then any use of that variable looks like it has that location, which leads to the debugger jumping back and forth for both lambdas and structured

Re: [PATCH] [PING^2] Asan changes for RISC-V.

2020-11-11 Thread Jim Wilson
Original message here https://gcc.gnu.org/pipermail/gcc-patches/2020-October/557406.html This has non-RISC-V changes, so I need a global reviewer to look at it. Jim On Wed, Nov 4, 2020 at 12:10 PM Jim Wilson wrote: > > > On Wed, Oct 28, 2020 at 4:59 PM Jim Wilson wrote: > >> We have only risc

Re: [PATCH ping 2] ppc64 check for incompatible setting of minimal-toc

2020-06-17 Thread Douglas B Rupp
Greetings again, Could you please look at this patch when convenient? --Doug On 6/1/20 10:13 AM, Douglas B Rupp wrote: Greetings, Curious if you've had a chance to look at this patch yet? --Doug On 5/18/20 4:02 PM, Douglas B Rupp wrote: Greetings, The attached patch is proposed for rs6000

Re: Patch ping^2

2020-03-17 Thread Richard Biener
On Tue, 17 Mar 2020, Jakub Jelinek wrote: > Hi! > > I'd like to ping this patch again: OK. Thanks, Richard. > On Tue, Mar 10, 2020 at 01:28:19PM +0100, Jakub Jelinek wrote: > > I'd like to ping the > > https://gcc.gnu.org/legacy-ml/gcc-patches/2020-03/msg00154.html > > P1 PR94015 > > patch,

Re: Patch ping^2

2017-11-15 Thread Jim Wilson
On 11/14/2017 08:29 AM, Jakub Jelinek wrote: On Mon, Nov 06, 2017 at 05:22:36PM +0100, Jakub Jelinek wrote: I'd like to ping the: http://gcc.gnu.org/ml/gcc-patches/2017-10/msg01895.html PR debug/82718 Fix DWARF5 .debug_loclist handling with hot/cold partitioning patch. Thanks Ping^

Re: Patch ping^2

2017-04-18 Thread Jeff Law
On 04/18/2017 07:55 AM, Jakub Jelinek wrote: I'd like to ping following patch: PR debug/80263 http://gcc.gnu.org/ml/gcc-patches/2017-04/msg4.html avoid emitting sizetype artificial name into debug info OK. jeff

Re: Patch ping^2

2017-02-15 Thread Jakub Jelinek
On Tue, Feb 14, 2017 at 10:05:04AM -0500, Nathan Sidwell wrote: > On 02/13/2017 10:46 AM, Jakub Jelinek wrote: > > Hi! > > > > I'd like to ping a couple of patches: > > > - C++ P1 PR79288 - wrong default TLS model for __thread static data members > > http://gcc.gnu.org/ml/gcc-patches/2017-01/ms

Re: Patch ping^2

2017-02-14 Thread Nathan Sidwell
On 02/13/2017 10:46 AM, Jakub Jelinek wrote: Hi! I'd like to ping a couple of patches: - C++ P1 PR79288 - wrong default TLS model for __thread static data members http://gcc.gnu.org/ml/gcc-patches/2017-01/msg02349.html This is ok, but don't you think the changelog is misleading? In your

Re: Patch ping^2

2017-02-13 Thread Nathan Sidwell
On 02/13/2017 12:28 PM, Jakub Jelinek wrote: The reason for that VOID_TYPE_P stuff in COND_EXPR handling is not that stabilize_expr doesn't like it, but to avoid warning twice. If I replace all 3 VOID_TYPE_P tests (with the patch) in cp_build_modify_expr with 0 && VOID_TYPE_P, the above testcas

Re: Patch ping^2

2017-02-13 Thread Jakub Jelinek
On Mon, Feb 13, 2017 at 05:44:54PM +0100, Jakub Jelinek wrote: > On Mon, Feb 13, 2017 at 11:41:48AM -0500, Nathan Sidwell wrote: > > On 02/13/2017 10:46 AM, Jakub Jelinek wrote: > > > Hi! > > > > > > I'd like to ping a couple of patches: > > > > > > - C++ P1 PR79232 - ICEs and wrong-code with COM

Re: Patch ping^2

2017-02-13 Thread Nathan Sidwell
On 02/13/2017 10:46 AM, Jakub Jelinek wrote: Hi! I'd like to ping a couple of patches: - C++ P1 PR79232 - ICEs and wrong-code with COMPOUND_EXPR on lhs of assignment http://gcc.gnu.org/ml/gcc-patches/2017-01/msg02341.html What puzzles me about (and may be an existing orthogonal issue), is t

Re: Patch ping^2

2017-02-13 Thread Jakub Jelinek
On Mon, Feb 13, 2017 at 11:41:48AM -0500, Nathan Sidwell wrote: > On 02/13/2017 10:46 AM, Jakub Jelinek wrote: > > Hi! > > > > I'd like to ping a couple of patches: > > > > - C++ P1 PR79232 - ICEs and wrong-code with COMPOUND_EXPR on lhs of > > assignment > > http://gcc.gnu.org/ml/gcc-patches/

Re: [PATCH][PING^2] Skip preprocessor directives in mklog

2015-05-12 Thread Jeff Law
On 05/12/2015 09:19 AM, Yury Gribov wrote: On 04/30/2015 12:03 PM, Yury Gribov wrote: On 04/21/2015 02:26 PM, Yury Gribov wrote: Hi all, Contrib/mklog is currently faked by preprocessor directives inside functions to produce invalid ChangeLog. The attached patch fixes this. Tested with my lo

Re: [PATCH][PING^2] Skip preprocessor directives in mklog

2015-05-12 Thread Jeff Law
On 05/12/2015 10:14 AM, Yury Gribov wrote: On 05/12/2015 06:57 PM, Trevor Saunders wrote: On Tue, May 12, 2015 at 06:33:28PM +0300, Yury Gribov wrote: On 05/12/2015 06:23 PM, Diego Novillo wrote: The patch looks fine to me. I'm not really involved in GCC development anymore. I would suggest t

Re: [PATCH][PING^2] Skip preprocessor directives in mklog

2015-05-12 Thread Yury Gribov
On 05/12/2015 06:57 PM, Trevor Saunders wrote: On Tue, May 12, 2015 at 06:33:28PM +0300, Yury Gribov wrote: On 05/12/2015 06:23 PM, Diego Novillo wrote: The patch looks fine to me. I'm not really involved in GCC development anymore. I would suggest that this script should be maintained by whoe

Re: [PATCH][PING^2] Skip preprocessor directives in mklog

2015-05-12 Thread Diego Novillo
On Tue, May 12, 2015 at 11:50 AM, Tom de Vries wrote: > I'm not a good choice to be the maintainer of a perl script. I'm all kinds of sorry about the original choice of scripting language. I'd just spend a couple of hours re-writing it in python.

Re: [PATCH][PING^2] Skip preprocessor directives in mklog

2015-05-12 Thread Trevor Saunders
On Tue, May 12, 2015 at 06:33:28PM +0300, Yury Gribov wrote: > On 05/12/2015 06:23 PM, Diego Novillo wrote: > >The patch looks fine to me. > > > >I'm not really involved in GCC development anymore. I would suggest > >that this script should be maintained by whoever's been hacking on it > >the most.

Re: [PATCH][PING^2] Skip preprocessor directives in mklog

2015-05-12 Thread Tom de Vries
On 12-05-15 17:33, Yury Gribov wrote: On 05/12/2015 06:23 PM, Diego Novillo wrote: The patch looks fine to me. I'm not really involved in GCC development anymore. I would suggest that this script should be maintained by whoever's been hacking on it the most. It's a simple script, so it shouldn'

Re: [PATCH][PING^2] Skip preprocessor directives in mklog

2015-05-12 Thread Yury Gribov
On 05/12/2015 06:23 PM, Diego Novillo wrote: The patch looks fine to me. I'm not really involved in GCC development anymore. I would suggest that this script should be maintained by whoever's been hacking on it the most. It's a simple script, so it shouldn't be hard to find a new maintainer for

Re: [PATCH][PING^2] Skip preprocessor directives in mklog

2015-05-12 Thread Diego Novillo
The patch looks fine to me. I'm not really involved in GCC development anymore. I would suggest that this script should be maintained by whoever's been hacking on it the most. It's a simple script, so it shouldn't be hard to find a new maintainer for it. Diegop. On Tue, May 12, 2015 at 11:19 AM

Re: [PATCH] [PING^2] Fix for PR libstdc++/60758

2014-05-20 Thread Alexey Merzlyakov
On 20.05.2014 17:16, Richard Earnshaw wrote: On 20/05/14 14:12, Ramana Radhakrishnan wrote: The following PR is opened for this problem: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61223 Thumb1 failure was also detected and reported in pr60758. I've proposed a thumb1 bugfix there. Regtest for

Re: [PATCH] [PING^2] Fix for PR libstdc++/60758

2014-05-20 Thread Richard Earnshaw
On 20/05/14 14:12, Ramana Radhakrishnan wrote: > >> >> The following PR is opened for this problem: >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61223 >> >> Thumb1 failure was also detected and reported in pr60758. >> I've proposed a thumb1 bugfix there. Regtest for the fix currently is in >> p

Re: [PATCH] [PING^2] Fix for PR libstdc++/60758

2014-05-20 Thread Ramana Radhakrishnan
The following PR is opened for this problem: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61223 Thumb1 failure was also detected and reported in pr60758. I've proposed a thumb1 bugfix there. Regtest for the fix currently is in progress. Patches must be proposed on gcc-patches and / or in thi

Re: [PATCH] [PING^2] Fix for PR libstdc++/60758

2014-05-20 Thread Alexey Merzlyakov
On 20.05.2014 16:25, Richard Earnshaw wrote: On 16/05/14 14:56, Alexey Merzlyakov wrote: On 07.05.2014 13:28, Ramana Radhakrishnan wrote: On 05/07/14 09:19, Yury Gribov wrote: Original Message Subject: [PING] [PATCH] Fix for PR libstdc++/60758 Date: Thu, 17 Apr 2014 17:48:12

Re: [PATCH] [PING^2] Fix for PR libstdc++/60758

2014-05-20 Thread Richard Earnshaw
On 16/05/14 14:56, Alexey Merzlyakov wrote: > On 07.05.2014 13:28, Ramana Radhakrishnan wrote: >> On 05/07/14 09:19, Yury Gribov wrote: >>> Original Message >>> Subject: [PING] [PATCH] Fix for PR libstdc++/60758 >>> Date: Thu, 17 Apr 2014 17:48:12 +0400 >>> From: Alexey Merzlyakov

Re: [PATCH PING^2] Simplify a VEC_SELECT fed by its own inverse

2014-05-19 Thread Jeff Law
On 05/19/14 07:10, Bill Schmidt wrote: Hi, I'd like to once again ping this patch from 2014-04-22: http://gcc.gnu.org/ml/gcc-patches/2014-04/msg01319.html OK for the trunk. Thanks for your patience. jeff

Re: [PATCH] [PING^2] Fix for PR libstdc++/60758

2014-05-16 Thread Alexey Merzlyakov
On 07.05.2014 13:28, Ramana Radhakrishnan wrote: On 05/07/14 09:19, Yury Gribov wrote: Original Message Subject: [PING] [PATCH] Fix for PR libstdc++/60758 Date: Thu, 17 Apr 2014 17:48:12 +0400 From: Alexey Merzlyakov To: Ramana Radhakrishnan CC: gcc-patches@gcc.gnu.org , Viac

Re: [PATCH] [PING^2] Fix for PR libstdc++/60758

2014-05-07 Thread Ramana Radhakrishnan
On 05/07/14 09:19, Yury Gribov wrote: Original Message Subject: [PING] [PATCH] Fix for PR libstdc++/60758 Date: Thu, 17 Apr 2014 17:48:12 +0400 From: Alexey Merzlyakov To: Ramana Radhakrishnan CC: gcc-patches@gcc.gnu.org , Viacheslav Garbuzov , Yury Gribov Hi, This fixes in

Re: [PATCH] [PING^2] Fix for PR libstdc++/60758

2014-05-07 Thread Paolo Carlini
Hi, On 05/07/2014 10:19 AM, Yury Gribov wrote: Original Message Subject: [PING] [PATCH] Fix for PR libstdc++/60758 Date: Thu, 17 Apr 2014 17:48:12 +0400 From: Alexey Merzlyakov To: Ramana Radhakrishnan CC: gcc-patches@gcc.gnu.org , Viacheslav Garbuzov , Yury Gribov Hi, T

Re: [PATCH][PING^2] Fix for PR59600 (prohibit inlining if no_sanitize_address)

2014-02-04 Thread Yury Gribov
Richard wrote: What about updated patch? Ok if it passes bootstrap and regtesting. It does, commited in r207497. -Y

Re: [PATCH][PING^2] Fix for PR59600 (prohibit inlining if no_sanitize_address)

2014-02-04 Thread Richard Biener
On Tue, Feb 4, 2014 at 3:32 PM, Yury Gribov wrote: > Richard wrote: >> >> I think you can't rely on pointer equivalence of the lookup_attribute >> result so you want instead of > > > Thanks, makes sense. > > >> Please also name CIF_OPTION_MISMATCH as CIF_ATTRIBUTE_MISMATCH >> and say "function att

Re: [PATCH][PING^2] Fix for PR59600 (prohibit inlining if no_sanitize_address)

2014-02-04 Thread Yury Gribov
Richard wrote: I think you can't rely on pointer equivalence of the lookup_attribute result so you want instead of Thanks, makes sense. Please also name CIF_OPTION_MISMATCH as CIF_ATTRIBUTE_MISMATCH and say "function attribute mismatch" in the description. Done. What about updated patch?

Re: [PATCH][PING^2] Fix for PR59600 (prohibit inlining if no_sanitize_address)

2014-02-04 Thread Richard Biener
On Tue, Feb 4, 2014 at 4:39 AM, Yury Gribov wrote: > Original Message > Subject: [PATCH][PING] Fix for PR59600 (prohibit inlining if > no_sanitize_address) > Date: Tue, 28 Jan 2014 09:13:10 +0400 > From: Yury Gribov > To: GCC Patches > > Original Message > Sub

Re: patch ping^2: PR 51516 (TM clone aliases)

2012-01-09 Thread Richard Henderson
On 01/07/2012 05:47 AM, Aldy Hernandez wrote: > Patrick Marlier > PR middle-end/51516 > * trans-mem.c (get_cg_data): Traverse aliases if requested. > (ipa_tm_scan_calls_block): Update parameters to get_cg_data. > (ipa_tm_note_irrevocable): Same. > (ipa_tm_s

Re: patch ping^2: PR 51516 (TM clone aliases)

2012-01-06 Thread Aldy Hernandez
On 01/05/12 16:05, Richard Henderson wrote: On 01/06/2012 12:53 AM, Aldy Hernandez wrote: http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01088.html It looks like it would be cleaner to put the .alias dereferencing directly into get_cg_data. r~ Arghh, I knew you were going to say that. That

Re: patch ping^2: PR 51516 (TM clone aliases)

2012-01-05 Thread Richard Henderson
On 01/06/2012 12:53 AM, Aldy Hernandez wrote: > http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01088.html It looks like it would be cleaner to put the .alias dereferencing directly into get_cg_data. r~

Re: [PATCH][PING^2] Vectorize conversions directly

2011-11-08 Thread Dmitry Plotnikov
Ping. On 10/28/2011 12:22 PM, Dmitry Plotnikov wrote: Here is the patch updated according to recent comments. 2011-10-28 Dmitry Plotnikov gcc/ * tree-cfg.c (verify_gimple_assign_unary): Allow vector conversions. * optabs.c (supportable_convert_operation): New function. * optabs.h (supportable

Re: Patch ping #2

2011-05-31 Thread Richard Henderson
On 05/31/2011 02:19 AM, Jakub Jelinek wrote: > Hi! > > - http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01182.html > > various debug info improvements (typed DWARF stack etc.) >

Re: Patch ping #2

2011-05-25 Thread Richard Guenther
On Wed, May 25, 2011 at 12:00 AM, Eric Botcazou wrote: >> Yes, I mean when -fstack-usage is enabled.  Thus, make >> -fstack-usage -Wframe-larger-than behave the same as >> -fstack-usage -Wstack-usage.  Or do you say that -Wstack-usage >> doesn't require -fstack-usage? > > Yes, -Wstack-usage doesn'

Re: Patch ping #2

2011-05-24 Thread Eric Botcazou
> Yes, I mean when -fstack-usage is enabled. Thus, make > -fstack-usage -Wframe-larger-than behave the same as > -fstack-usage -Wstack-usage. Or do you say that -Wstack-usage > doesn't require -fstack-usage? Yes, -Wstack-usage doesn't require -fstack-usage. > If it doesn't then I hope -Wstack-u

Re: Patch ping #2

2011-05-23 Thread Richard Guenther
On Mon, May 23, 2011 at 12:15 PM, Eric Botcazou wrote: >> > Fix annoying gcov filename handling (gcov.c, 2 lines): >> >  http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01380.html >> >> Ok. > > Thanks! > >> > Introduce -Wstack-usage: >> >  http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01992.html >> >>

Re: Patch ping #2

2011-05-23 Thread Eric Botcazou
> > Fix annoying gcov filename handling (gcov.c, 2 lines): > >  http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01380.html > > Ok. Thanks! > > Introduce -Wstack-usage: > >  http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01992.html > > I wonder if we can't simply use -Wframe-larger-than but with > the

Re: Patch ping #2

2011-05-23 Thread Richard Guenther
On Mon, May 23, 2011 at 9:59 AM, Eric Botcazou wrote: > Not as many as Jakub, but still. :-) > > > Fix annoying gcov filename handling (gcov.c, 2 lines): >  http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01380.html Ok. > Introduce -Wstack-usage: >  http://gcc.gnu.org/ml/gcc-patches/2011-03/msg0199

Re: Patch ping^2: RFA: Add makefile for cross-configuration torture test

2011-05-03 Thread Alexandre Oliva
On May 2, 2011, Joern Rennecke wrote: > 2010-04-14 Joern Rennecke > * config-list.mk: New file. Ok, thanks! -- Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/ You must be the change you wish to see in the world. -- Gandhi Be Free! -- http://FSFLA.org/ FSF Latin Am