<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span style="font-size:12.8px;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">I'm not sure if there is a suitable place for that function. This is needed in  "ObjectFileMachO" and two dynamic loader plugins.</span></blockquote><div><br></div><div>Then your factory idea may be the next best thing. While we're at it, maybe we can remove the UUID::SetBytes() from the public interface, and make the UUID an immutable value type:</div><div><br></div><div>Ex. instead of:</div><div><br></div><div>UUID uuid;</div><div>...</div><div>uuid.SetBytes(...)</div><div><br></div><div>We'd have:</div><div><br></div><div>UUID uuid;</div><div><br></div><div>uuid = UUID(...); </div><div>// or </div><div>uuid = { ... };</div><div>// or</div><div>uuid = UUID::factory(...);</div><div><br></div><div>What do you think?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 22, 2018 at 12:29 PM, Pavel Labath via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">labath added a comment.<br>
<span class=""><br>
In <a href="https://reviews.llvm.org/D48479#1141067" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D48479#1141067</a>, @lemo wrote:<br>
<br>
>   One solution might be to encapsulate the MachO convention in the MachO<br>
><br>
> code: check in there (maybe through a helper function) if the UUID is<br>
>  "000...0" and map it to the empty UUID in that case. The UUID interface<br>
>  would not have to know/care about this convention. Would this work?<br>
<br>
<br>
</span>I'm not sure if there is a suitable place for that function. This is needed in  "ObjectFileMachO" and two dynamic loader plugins.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D48479" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D48479</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>