Re: [PATCH v4] Missed function specialization + partial devirtualization

2019-09-26 Thread Martin Liška
On 9/26/19 7:23 AM, luoxhu wrote: > Thanks Martin, > > > On 2019/9/25 18:57, Martin Liška wrote: >> On 9/25/19 5:45 AM, luoxhu wrote: >>> Hi, >>> >>> Sorry for replying so late due to cauldron conference and other LTO issues >>> I was working on. >> >> Hello. >> >> That's fine, we still have plen

Re: [PATCH v4] Missed function specialization + partial devirtualization

2019-09-25 Thread luoxhu
Thanks Martin, On 2019/9/25 18:57, Martin Liška wrote: On 9/25/19 5:45 AM, luoxhu wrote: Hi, Sorry for replying so late due to cauldron conference and other LTO issues I was working on. Hello. That's fine, we still have plenty of time for patch review. Not fixed issues which I reported in

Re: [PATCH v4] Missed function specialization + partial devirtualization

2019-09-25 Thread Martin Liška
On 9/25/19 5:45 AM, luoxhu wrote: > Hi, > > Sorry for replying so late due to cauldron conference and other LTO issues > I was working on. Hello. That's fine, we still have plenty of time for patch review. Not fixed issues which I reported in v3 (and still valid in v4): - please come up with in

[PATCH v4] Missed function specialization + partial devirtualization

2019-09-24 Thread luoxhu
Hi, Sorry for replying so late due to cauldron conference and other LTO issues I was working on. v4 Changes: 1. Rebase to trunk. 2. Remove num_of_ics and use vector's length to avoid redundancy. 3. Update the code in ipa-profile.c to improve review feasibility. 4. Add function has_indirect_ca