[PATCH] Testcase for PR 88297 and minor fixes

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[PATCH] Testcase for PR 88297 and minor fixes

Michael Ploujnikov
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 call it inside decide_whether_version_node.

Finally I've attached some comment spelling fixes as a bonus.


Bootstrapping (--with-build-config=bootstrap-lto) and regression testing on x86_64.

Ok for trunk after tests pass?


- Michael

0001-Skip-constprop-clones-because-we-don-t-make-lattices.patch (1K) Download Attachment
0002-Testcase-for-cloning-during-LTO-WPA-stage.patch (2K) Download Attachment
0003-Spelling-fixes-in-comments.patch (3K) Download Attachment
signature.asc (499 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

[PING 1] Testcase for PR 88297 and minor fixes

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 call it inside decide_whether_version_node.
>
> Finally I've attached some comment spelling fixes as a bonus.
>
>
> Bootstrapping (--with-build-config=bootstrap-lto) and regression testing on x86_64.
>
> Ok for trunk after tests pass?
>
>
> - Michael
>
Ping?


signature.asc (499 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Testcase for PR 88297 and minor fixes

Jeff Law
In reply to this post by Michael Ploujnikov
On 12/6/18 1:13 PM, 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 call it inside decide_whether_version_node.
>
> Finally I've attached some comment spelling fixes as a bonus.
>
>
> Bootstrapping (--with-build-config=bootstrap-lto) and regression testing on x86_64.
>
> Ok for trunk after tests pass?
Thanks.  I went ahead and installed all 3 patches on the trunk.

jeff