<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 26, 2015 at 2:51 PM, Chad Rosier <span dir="ltr"><<a href="mailto:mcrosier@codeaurora.org" target="_blank">mcrosier@codeaurora.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":hkk" class="a3s" style="overflow:hidden">+  /// \returns True if the intrinsic is a supported memory intrinsic.  Info<br>
+  /// will contain additional information - whether the intrinsic may write<br>
+  /// or read to memory, volatility and the pointer.  Info is undefined<br>
+  /// if false is returned.<br>
+  virtual bool getTgtMemIntrinsic(IntrinsicInst *Inst,<br>
+                                  MemIntrinsicInfo &Info) const;</div></blockquote></div><br>Please don't abbreviate this 'Tgt'. Just say "Target" if it is necessary.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Also, could you instead have a value of the enumerator which is == 0 be returned? That will avoid a (fairly ugly) output parameter and make the calling code still clean as you can just test for 'if (!foo)'.</div></div>