<div dir="ltr">Hi,<div>I am working right now on importing based on PGO/FDO data. There is one issue that I found - when we calculate the list of imports, we can't get the ProfileSummaryInfo, which is the best and I </div><div>think only valid way of checking if callsite/callee is hot (isHotCount()). There are 2 solutions that I come up with Teresa and Easwaran:</div><div><br></div><div>1. Add PGO data to summary </div><div>2. Replace CalleeInfo::ProfileCount with enum {None, Cold, Hot} computed during computing summary.</div><div><br></div><div>I like the 2. much more. It will reduce the summary size slightly and I don't think we will need ProfileCount anywhere else.</div><div><br></div><div>The other thing I would like to mention is that I think we should start using the summary versioning and drop support of old version. </div><div>ThinLTO doesn't have enough users right now and parsing many versions of summary will just add additional cost, that will start to grow.</div><div><br></div><div>Piotr</div></div>