Re: [PATCH 1/N] Clean-up usage of ipa_fn_summary and ipa_call_summary summaries.

2018-06-19 Thread Jan Hubicka
> > Hmm, I do not think we should try to merge -O0 functions :) > > But again, if you have testcase that triggers this path, we could > > handle it later. > > This is valid: > > ./xgcc -B. /home/marxin/Programming/gcc/gcc/testsuite/g++.dg/ipa/pr64146.C > -fipa-icf > during IPA pass: icf > /home/

Re: [PATCH 1/N] Clean-up usage of ipa_fn_summary and ipa_call_summary summaries.

2018-06-19 Thread Martin Liška
On 06/19/2018 03:37 PM, Jan Hubicka wrote: >> Hello. >> >> ICEs on: >> >> $ cat ice.ii >> #pragma GCC optimize 0 >> int main() {} >> >> $ ./xgcc -B. -O2 ice.ii >> during IPA pass: cp >> ice.ii:2:13: internal compiler error: Segmentation fault >> int main() {} >> ^ >> 0xdbb6bf crash_si

Re: [PATCH 1/N] Clean-up usage of ipa_fn_summary and ipa_call_summary summaries.

2018-06-19 Thread Jan Hubicka
> Hello. > > ICEs on: > > $ cat ice.ii > #pragma GCC optimize 0 > int main() {} > > $ ./xgcc -B. -O2 ice.ii > during IPA pass: cp > ice.ii:2:13: internal compiler error: Segmentation fault > int main() {} > ^ > 0xdbb6bf crash_signal > /home/marxin/Programming/gcc/gcc/toplev.c

Re: [PATCH 1/N] Clean-up usage of ipa_fn_summary and ipa_call_summary summaries.

2018-06-19 Thread Martin Liška
On 06/15/2018 05:05 PM, Jan Hubicka wrote: >> Hi. >> >> This is first part of IPA summary conversion clean-up. It removes >> ::get_create and >> uses ::get for ipa-inline related symbol (and call) summaries. I'm planning >> to >> investigate also other summaries. >> >> Patch can bootstrap on x86_

Re: [PATCH 1/N] Clean-up usage of ipa_fn_summary and ipa_call_summary summaries.

2018-06-15 Thread Martin Jambor
Hi, On Fri, Jun 15 2018, Martin Liška wrote: > Hi. > > This is first part of IPA summary conversion clean-up. It removes > ::get_create and > uses ::get for ipa-inline related symbol (and call) summaries. I'm planning to > investigate also other summaries. > > Patch can bootstrap on x86_64-linux-

Re: [PATCH 1/N] Clean-up usage of ipa_fn_summary and ipa_call_summary summaries.

2018-06-15 Thread Jan Hubicka
> Hi. > > This is first part of IPA summary conversion clean-up. It removes > ::get_create and > uses ::get for ipa-inline related symbol (and call) summaries. I'm planning to > investigate also other summaries. > > Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > > Read

[PATCH 1/N] Clean-up usage of ipa_fn_summary and ipa_call_summary summaries.

2018-06-15 Thread Martin Liška
Hi. This is first part of IPA summary conversion clean-up. It removes ::get_create and uses ::get for ipa-inline related symbol (and call) summaries. I'm planning to investigate also other summaries. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Ready to be installed? M