<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Feb 10, 2014 at 7:44 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@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 class="HOEnZb"><div class="h5">On 7 February 2014 18:06, Manman Ren <<a href="mailto:manman.ren@gmail.com">manman.ren@gmail.com</a>> wrote:<br>

><br>
> Hi all,<br>
><br>
> This function adds an extra path argument to lto_module_create_from_memory.<br>
> The path argument will be passed to makeBuffer to make sure the MemoryBuffer<br>
> has a name and the created module has a module identifier. Modules created<br>
> by lto_module_create_from_memory will have empty module identifiers.<br>
><br>
> This is mainly for emitting warning messages from the linker. When we emit<br>
> warning message on one module, we can then use its module identifier.<br>
><br>
> LTO_API_VERSION is bumped to 9.<br>
<br>
<br>
</div></div>+  /// makeBuffer -<br>
<br>
While at it, please remove the function name from the comment.<br>
<br>
-                                  std::string &errMsg);<br>
+                                  std::string &errMsg, const char *path = "");<br>
<br>
The C++ files should probably use a StringRef.<br></blockquote><div><br></div><div>Thanks Rafael for reviewing the patch. Committed at r201114 with fixes addressing the above.</div><div><br></div><div>Manman</div><div> </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
LGTM with that.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div></div>