<div dir="ltr">Do you intend to use the accessor elsewhere?  For the one use of the accessor in the static helper function, maybe it's better to pass CGM into it and then delete the accessor?<div><br></div><div>Either way the privatizing looks good.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Oct 3, 2013 at 2:26 PM, Mark Lacey <span dir="ltr"><<a href="mailto:mark.lacey@apple.com" target="_blank">mark.lacey@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Add an accessor for the CodeGenModule reference stored in CodeGenTypes.<br>
<br>
No functionality differences.<br>
---<br>
lib/CodeGen/CGCall.cpp     | 2 +-<br>
lib/CodeGen/CodeGenTypes.h | 5 ++---<br>
2 files changed, 3 insertions(+), 4 deletions(-)<br>
<br>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>