<div dir="ltr">Hi Renato,<div><br></div><div>Ideally, shouldn't the clang tests be checking that the LLVM target parsing library is called with the correct arguments? then separate tests inside LLVM check that the target parser works correctly?</div><div><br></div><div>As it stands, it seems like a very deliberate layering violation that could really do with chopping up.</div><div><br></div><div>James</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, 16 Nov 2015 at 12:19 Renato Golin via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">rengolin added a comment.<br>
<br>
In <a href="http://reviews.llvm.org/D14570#288132" rel="noreferrer" target="_blank">http://reviews.llvm.org/D14570#288132</a>, @compnerd wrote:<br>
<br>
> Wow, this is tricky: the code change is in LLVM, and test change in clang :(.  However, this does seem to preserve the features.<br>
<br>
<br>
The problem is that Clang is the most important user of this library, which also has other users in LLVM. We just need to make sure that Clang is doing the right thing, and that the intentions are recorded on the right places. In this case, it's Clang. :)<br>
<br>
<br>
<a href="http://reviews.llvm.org/D14570" rel="noreferrer" target="_blank">http://reviews.llvm.org/D14570</a><br>
<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</blockquote></div>