<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 8, 2016 at 10:55 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">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></span><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></div></blockquote><div><br>(but if we wanted to, we could, by making each Action subclass's ctor private and friending Compilation from them - might be a bit repetitious though)<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div><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" target="_blank">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></span></div><br></div></div>
</blockquote></div><br></div></div>