<div dir="ltr">I posted some questions to the phab site about the design of the hasher.  I suggest you try your hasher on clang/test/Modules/odr_hash.h with the flag -DFIRST.  I put many of the issues I encountered into there.<div><br></div><div>I'm still iffy on how basing all the hashers on *Collectors.td would work.  Is there enough flexibility for changes each hasher needs that it wouldn't result in a lot of custom code anyways?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 21, 2018 at 4:04 AM, Christian Dietrich <span dir="ltr"><<a href="mailto:dietrich@sra.uni-hannover.de" target="_blank">dietrich@sra.uni-hannover.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey!<br>
<br>
I wanted to ping on this patch[1] as it still lacks some reviewers. I<br>
rebased the patch against the current HEAD, checked compilation, run our<br>
testsuite and the ASTTest unit test.<br>
<br>
If there are further questions or open points to discuss, I would love<br>
to participate in that discussion. Thank you all for your efforts on<br>
this patch.<br>
<br>
chris<br>
<br>
<br>
[1] <a href="https://reviews.llvm.org/D40731" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D40731</a><br>
<div class="HOEnZb"><div class="h5">--<br>
Christian Dietrich, M.Sc. (Scientific Staff)<br>
Institute for Systems Engineering (Systems and Computerarchitecture)<br>
Leibniz Universität Hannover<br>
Appelstraße 4<br>
30167 Hannover, Germany<br>
<br>
Tel:    <a href="tel:%2B49%20511%20762-19737" value="+4951176219737">+49 511 762-19737</a><br>
Fax:    <a href="tel:%2B49%20511%20762-19733" value="+4951176219733">+49 511 762-19733</a><br>
eMail:  <a href="mailto:dietrich@sra.uni-hannover.de">dietrich@sra.uni-<wbr>hannover.de</a><br>
WWW:    <a href="https://www.sra.uni-hannover.de" rel="noreferrer" target="_blank">https://www.sra.uni-<wbr>hannover.de</a><br>
</div></div></blockquote></div><br></div>