Re: [PATCH] Fix PR89150, GC of tree-form bitmaps

2019-02-08 Thread Michael Ploujnikov
On 2019-02-07 3:09 p.m., Jakub Jelinek wrote: > On Thu, Feb 07, 2019 at 03:04:21PM -0500, Michael Ploujnikov wrote: >> 2019-02-07 Michael Ploujnikov >> >> PR middle-end/89150 >> * bitmap.c (test_bitmap_tree_marking): New test. >> (NOT

Re: [PATCH] Fix PR89150, GC of tree-form bitmaps

2019-02-07 Thread Michael Ploujnikov
ree patch, the issue is only latent on trunk > (together with another unfixed pch issue of bitmaps). My apologies for not noticing this thread/PR earlier. I originally discovered this issue while working on an experimental patch, but since then I've also come up with a test case for this speci

[PATCH] String contents hash map key example

2019-01-18 Thread Michael Ploujnikov
of security). - Michael From 3433efe4ac558de05410a9b185f4ff0a01e7e5df Mon Sep 17 00:00:00 2001 From: Michael Ploujnikov Date: Fri, 11 Jan 2019 09:22:14 -0500 Subject: [PATCH] Document how to hash based on key string contents. gcc: 2019-01-18 Michael Ploujnikov * hash-map-tests.c

[PING 1] Testcase for PR 88297 and minor fixes

2018-12-12 Thread Michael Ploujnikov
On 2018-12-06 3:13 p.m., Michael Ploujnikov wrote: > Thanks to Martin we now have a test that exercises (cp) cloning > machinery during the WPA stage of LTO. > > Also, during debugging I found that print_all_lattices would trigger > an assert if I tried to

[PATCH] Testcase for PR 88297 and minor fixes

2018-12-06 Thread Michael Ploujnikov
as a bonus. Bootstrapping (--with-build-config=bootstrap-lto) and regression testing on x86_64. Ok for trunk after tests pass? - Michael From f8f59d44141726e688cde077aabb5f2ce0bf53e0 Mon Sep 17 00:00:00 2001 From: Michael Ploujnikov Date: Thu, 6 Dec 2018 13:36:51 -0500 Subject: [PATCH 1/3] Skip

Re: [PING v2][PATCH] Make function clone name numbering independent.

2018-12-04 Thread Michael Ploujnikov
On 2018-12-04 7:48 a.m., Martin Jambor wrote: > Hi, > > On Tue, Sep 04 2018, Michael Ploujnikov wrote: >> >> I've tried building with numbered_clone_function_name replaced by >> suffixed_function_name and with --enable-offload-targets=hsa and >> didn't see any

Re: [PATCH v3] Make function clone name numbering independent.

2018-12-03 Thread Michael Ploujnikov
On 2018-12-03 12:00 p.m., Michael Ploujnikov wrote: > On 2018-12-01 11:29 a.m., H.J. Lu wrote: >> This caused: >> >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88297 >> > > Sorry about that. Looks like I should have been testing with > --with-build-config=boo

Re: [PATCH v3] Make function clone name numbering independent.

2018-12-03 Thread Michael Ploujnikov
On 2018-12-01 11:29 a.m., H.J. Lu wrote: > This caused: > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88297 > Sorry about that. Looks like I should have been testing with --with-build-config=bootstrap-lto rather than just --enable-bootstrap. The quick fix would be to undo the patch to

Re: [PATCH v3] Make function clone name numbering independent.

2018-11-30 Thread Michael Ploujnikov
OK with those changes. > > Thanks, > Richard. Thank you for the review. I will commit as soon as my last test run finishes. - Michael From ac1f1579d37804c97833d460ec6cd5b87d6184c7 Mon Sep 17 00:00:00 2001 From: Michael Ploujnikov Date: Thu, 1 Nov 2018 12:57:30 -0400 Subject: [PATCH 1/3]

Re: [PATCH v3] Make function clone name numbering independent.

2018-11-29 Thread Michael Ploujnikov
On 2018-11-28 5:49 p.m., Segher Boessenkool wrote: > Hi! > > On Wed, Nov 28, 2018 at 04:09:14PM -0500, Michael Ploujnikov wrote: >> I've also included a small change to rs6000 which I'm pretty sure is >> safe, but I have no way of testing. > > Do you have an acco

[PATCH v3] Make function clone name numbering independent.

2018-11-28 Thread Michael Ploujnikov
for trunk? - Michael From 40cb5c888522d69bc42791f0c884dcb9e29eff37 Mon Sep 17 00:00:00 2001 From: Michael Ploujnikov Date: Thu, 1 Nov 2018 12:57:30 -0400 Subject: [PATCH 1/4] Make function assembly more independent. This is achieved by having clone_function_name assign unique clone numbers for each

Re: [doc PATCH] Fix weakref description.

2018-11-13 Thread Michael Ploujnikov
On 2018-11-12 12:50 p.m., Michael Ploujnikov wrote: > On 2018-11-02 1:59 p.m., Michael Ploujnikov wrote: >> I came across this typo and also added a similar ld invocation for >> illustration purposes as mentioned by Jakub on irc. >> > > After talking to Jakub abou

Re: [doc PATCH] Fix weakref description.

2018-11-12 Thread Michael Ploujnikov
On 2018-11-02 1:59 p.m., Michael Ploujnikov wrote: > I came across this typo and also added a similar ld invocation for > illustration purposes as mentioned by Jakub on irc. > After talking to Jakub about it, I went with different terminology. - Mic

[doc PATCH] Fix weakref description.

2018-11-02 Thread Michael Ploujnikov
I came across this typo and also added a similar ld invocation for illustration purposes as mentioned by Jakub on irc. From 2df4903f04fbe68e9e6a1ae0eea460e7592a8512 Mon Sep 17 00:00:00 2001 From: Michael Ploujnikov Date: Fri, 2 Nov 2018 13:40:50 -0400 Subject: [PATCH] Fix weakref description

Re: [PATCH v4] Avoid unnecessarily numbering cloned symbols.

2018-10-29 Thread Michael Ploujnikov
On 2018-10-29 6:49 a.m., Martin Liška wrote: > On 10/29/18 9:40 AM, Martin Liška wrote: >> On 10/27/18 6:15 PM, Michael Ploujnikov wrote: >>> Hi, >>> >>> On 2018-10-26 10:25 a.m., Jan Hubicka wrote: >>>>> From aea94273e7a477a03d1ee10a5d9043d6d1

Re: [PATCH v4] Avoid unnecessarily numbering cloned symbols.

2018-10-27 Thread Michael Ploujnikov
Hi, On 2018-10-26 10:25 a.m., Jan Hubicka wrote: >> From aea94273e7a477a03d1ee10a5d9043d6d13b8e8d Mon Sep 17 00:00:00 2001 >> From: Michael Ploujnikov >> Date: Thu, 25 Oct 2018 13:16:36 -0400 >> Subject: [PATCH] Avoid unnecessarily numbering cloned symbols. >> >

Re: [PATCH v4] Avoid unnecessarily numbering cloned symbols.

2018-10-26 Thread Michael Ploujnikov
Hi Martin, Thanks for the review. On 2018-10-26 03:51 AM, Martin Liška wrote: > On 10/26/18 12:59 AM, Michael Ploujnikov wrote: >> I've taken the advice from a discussion on IRC and re-wrote the patch >> with more uniform function names and using overloading. >> >

[PATCH v4] Avoid unnecessarily numbering cloned symbols.

2018-10-25 Thread Michael Ploujnikov
to be like that unless ASM_FORMAT_PRIVATE_NAME making the number optional. From 0df79d0ac6a9891b344f988c7157a54cebbc1cb8 Mon Sep 17 00:00:00 2001 From: Michael Ploujnikov Date: Thu, 25 Oct 2018 13:16:36 -0400 Subject: [PATCH] Avoid unnecessarily numbering cloned symbols. gcc/ChangeLog: 2018-10-25

[PATCH v3] Avoid unnecessarily numbered clone symbols

2018-10-23 Thread Michael Ploujnikov
On 2018-10-21 09:14 PM, Michael Ploujnikov wrote: > Continuing from https://gcc.gnu.org/ml/gcc-patches/2018-10/msg01258.html > > Fixed up the code after the change to concat suggested by Bernhard > Reutner. > > Outstanding question still remains: > > To wr

[PATCH v2] Avoid unnecessarily numbered clone symbols

2018-10-21 Thread Michael Ploujnikov
prefix by mistake? - Michael From 74435e1d8c5984eaee766d7940eeffbe565fcc2e Mon Sep 17 00:00:00 2001 From: Michael Ploujnikov Date: Tue, 7 Aug 2018 20:36:53 -0400 Subject: [PATCH 1/4] Rename clone_function_name_1 and clone_function_name to clarify usage. gcc: 2018-10-19 Michael Ploujnikov

Re: Avoid unnecessarily numbered clone symbols

2018-10-20 Thread Michael Ploujnikov
On 2018-10-20 07:39 AM, Bernhard Reutner-Fischer wrote: > On 20 October 2018 00:26:15 CEST, Michael Ploujnikov > wrote: >> While working on >> https://gcc.gnu.org/ml/gcc-patches/2018-09/msg00228.html I've >> accumulated a few easy patches. > > > +/* Return

Avoid unnecessarily numbered clone symbols

2018-10-19 Thread Michael Ploujnikov
that they don't need to be numbered. Regards, - Michael From 0bbcf3b8c20498f4d861e088ff7ab38e2a43800b Mon Sep 17 00:00:00 2001 From: Michael Ploujnikov Date: Tue, 7 Aug 2018 20:36:53 -0400 Subject: [PATCH 1/4] Rename clone_function_name_1 and clone_function_name to clarify usage. gcc: 2018-10-19

Re: [PING v2][PATCH] Make function clone name numbering independent.

2018-09-04 Thread Michael Ploujnikov
Hi Martin, On 2018-09-03 06:01 AM, Martin Jambor wrote: > Hi, > > On Fri, Aug 31 2018, Michael Ploujnikov wrote: >> I've done some more digging into the current uses of >> numbered_clone_function_name and checked if any tests fail if I change >> it to suffixed_f

Re: [PING v2][PATCH] Make function clone name numbering independent.

2018-09-04 Thread Michael Ploujnikov
Hi Martin, Richard, Thanks for your responses. On 2018-09-03 09:15 AM, Martin Jambor wrote: > Hi, > > On Mon, Sep 03 2018, Richard Biener wrote: >> On Mon, Sep 3, 2018 at 12:02 PM Martin Jambor wrote: >>> >>> Hi, >>> >>> On Fri, Aug 31 2018,

Re: [PING v2][PATCH] Make function clone name numbering independent.

2018-08-31 Thread Michael Ploujnikov
On 2018-08-13 07:58 PM, Michael Ploujnikov wrote: > Ping and I've updated the patch since last time as follows: > > - unittest scans assembly rather than the constprop dump because its > forward changed > - unittests should handle different hosts where any of >

[PING][PATCH] Make function clone name numbering independent.

2018-08-13 Thread Michael Ploujnikov
bootstrapped and regtested. - Michael On 2018-08-02 03:05 PM, Michael Ploujnikov wrote: > On 2018-08-01 06:37 AM, Richard Biener wrote: >> On Tue, Jul 31, 2018 at 7:40 PM Michael Ploujnikov >> wrote: >>> >>> On 2018-07-26 01:27 PM, Michael Ploujnikov wrote: >>>&g

Re: [PATCH] Make function clone name numbering independent.

2018-08-02 Thread Michael Ploujnikov
On 2018-08-01 06:37 AM, Richard Biener wrote: > On Tue, Jul 31, 2018 at 7:40 PM Michael Ploujnikov > wrote: >> >> On 2018-07-26 01:27 PM, Michael Ploujnikov wrote: >>> On 2018-07-24 09:57 AM, Michael Ploujnikov wrote: >>>> On 2018-07-20 06:05 AM, Richar

Re: [PATCH] Make function clone name numbering independent.

2018-07-31 Thread Michael Ploujnikov
On 2018-07-26 01:27 PM, Michael Ploujnikov wrote: > On 2018-07-24 09:57 AM, Michael Ploujnikov wrote: >> On 2018-07-20 06:05 AM, Richard Biener wrote: >>>> /* Return a new assembler name for a clone with SUFFIX of a decl named >>>> NAME. *

Re: [PATCH] Make function clone name numbering independent.

2018-07-26 Thread Michael Ploujnikov
On 2018-07-24 09:57 AM, Michael Ploujnikov wrote: > On 2018-07-20 06:05 AM, Richard Biener wrote: >>> /* Return a new assembler name for a clone with SUFFIX of a decl named >>> NAME. */ >>> @@ -521,14 +521,13 @@ tree >>> clone_function_name_

Re: [PATCH] Make function clone name numbering independent.

2018-07-24 Thread Michael Ploujnikov
On 2018-07-20 06:05 AM, Richard Biener wrote: > On Fri, Jul 20, 2018 at 4:48 AM Michael Ploujnikov > wrote: >> >> On 2018-07-17 04:25 PM, Michael Ploujnikov wrote: >>> On 2018-07-17 06:02 AM, Richard Biener wrote: >>>> On Tue, Jul 17, 2018 at 8:10

Re: [PATCH] Make function clone name numbering independent.

2018-07-19 Thread Michael Ploujnikov
On 2018-07-17 04:25 PM, Michael Ploujnikov wrote: > On 2018-07-17 06:02 AM, Richard Biener wrote: >> On Tue, Jul 17, 2018 at 8:10 AM Bernhard Reutner-Fischer >> wrote: >>> >>> On 16 July 2018 21:38:36 CEST, Michael Ploujnikov >>> wrote: >>&

[PATCH] Make function clone name numbering independent.

2018-07-17 Thread Michael Ploujnikov
On 2018-07-17 06:02 AM, Richard Biener wrote: > On Tue, Jul 17, 2018 at 8:10 AM Bernhard Reutner-Fischer > wrote: >> >> On 16 July 2018 21:38:36 CEST, Michael Ploujnikov >> wrote: >>> Hi, >>> >> >>> +clone_fn_ids = hash_map::crea

[PATCH] Make function clone name numbering independent.

2018-07-16 Thread Michael Ploujnikov
impact as the only user of the clone names that I found (https://gcc.gnu.org/ml/gcc/2013-03/msg00268.html) doesn't actually care about the specific numeric values. Thanks - Michael gcc: 2018-07-16 Michael Ploujnikov Make function clone name numbering independent