
<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Not commenting on the DebugInfo caching part. But as a general comment, if there was a performance hit, you may want to<div>provide performance numbers before and after this patch, say on Cocoa.h, and sample of other more modern framework</div><div>files which use if private ivars (and of course while generating debug info).</div><div><br></div><div>- Fariborz</div><div><br><div><div>On Feb 26, 2013, at 6:08 PM, Adrian Prantl <<a href="mailto:aprantl@apple.com">aprantl@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;">Hi CFEers,<br><br>After re-thinking it, here is an updated version of this patch that does not completely disable caching for incomplete interfaces. This should minimize the performance hit of the previous version.<br>Basically I am now storing ObjCInterface-Types in a separate cache, together with a “checksum” (really the number of ivars). If we look up the type again, I see if the checksum changed and otherwise just return the type from the cache.<br><br>-- Adrian<br><br><span><0001-Ensure-that-DIType-is-regenerated-after-we-visited-a.patch></span>_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</div></blockquote></div><br></div></body></html>