
<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On Apr 5, 2011, at 1:44 PM, John Thompson wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div>

<div> </div></div>

<div>Argyrios,</div>

<div> </div>

<div>

<div>

<blockquote style="border-left-color: rgb(204, 204, 204); border-left-width: 1px; border-left-style: solid; margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0.8ex; padding-left: 1ex; position: static; z-index: auto; " class="gmail_quote">

<div>I think we should move away from the llvm::Registry mechanism. How about the exported function returns a PluginASTAction*, we store all PluginASTAction* pointers that the plugins return and iterate over them when invoking plugin actions.</div>

</blockquote></div></div>

<div> </div>

<div>Sounds good to me.  But if not, at least fix the Registry mechanism by adding the needed accessors, which curiously it doesn't have, which is what I tried to do in my patch.</div></blockquote><div><br></div><div>Well your patch pokes at and exposes, IMO, implementation details of Registry; it doesn't seem the right way to go about it..</div><br><blockquote type="cite">

<div> </div>

<div>-John<br><br>-- <br>John Thompson<br><a href="mailto:John.Thompson.JTSoftware@gmail.com" target="_blank">John.Thompson.JTSoftware@gmail.com</a><br><br></div>

_______________________________________________<br>cfe-dev mailing list<br><a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev<br></blockquote></div><br></body></html>