<div dir="ltr">Yeah, i honestly don't know.<div>It's a need way to avoid constructing a real hash, my next question would be "is it actually demonstrably slower to compute a hash for real"?<br><br></div><div>Also, since the stuff is constant anyway, can't you just compute it at init time anyway?<br>It's not like you have to compute it repeatedly.</div><div><br><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 25, 2016 at 1:15 PM, Adrian Prantl <span dir="ltr"><<a href="mailto:aprantl@apple.com" target="_blank">aprantl@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">aprantl added inline comments.<br>
<span class=""><br>
================<br>
Comment at: lib/CodeGen/LiveDebugValues.cpp:103<br>
@@ +102,3 @@<br>
+      } RegisterLoc;<br>
+      uint64_t Hash;<br>
+    } Loc;<br>
----------------<br>
</span><span class="">dberlin wrote:<br>
> Errr, do you really never use the hash and the registerloc at the same time?<br>
><br>
</span>The hash is meant to be a convenient alternative view of the the two uint32_t as a single uint64_t. Now that I think of it, I wonder whether that is guaranteed on all imaginable architectures.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D20178" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20178</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>