<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 8, 2016 at 10:08 AM, Artem Belevich via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tra added inline comments.<br>
<br>
================<br>
Comment at: include/clang/Driver/Action.h:36<br>
@@ -35,1 +35,3 @@<br>
+///<br>
+/// Actions are usually owned by a Compilation.<br>
 class Action {<br>
----------------<br>
There's no API to pass ownership to Compilation explicitly, so the only way for an Action to be owned by Compilation is to create it with MakeAction.<br>
<br>
Perhaps "Actions created with MakeAction<>() are owned by Compilation"<br>
<br>
BTW, should we (can we?) make MakeAction<>() the only way to create actions?<br></blockquote><div><br></div><div>I suspect not - from an API design perspective it might be nice to be able to test Actions in isolation without the Compilation (I assume this isn't being done, because we don't unit test all that much - and I don't know the APIs well enough to know if such testing makes sense (maybe Action and Compilation are too tied together to do that separation anyway))</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D15911" rel="noreferrer" target="_blank">http://reviews.llvm.org/D15911</a><br>
<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div></div>