<div dir="ltr">As you are one of the spir v authors I feel you can make this call. :) I doubt it'll run into anything Anastasia is doing either as she's focused on opencl. <br>
</div><span>
</span><br><div class="gmail_quote"><div dir="ltr">On Fri, Apr 22, 2016, 2:54 AM Neil Henning via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@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">Hey all,<br>
<br>
We've just bumped our clang revision internally and found the following<br>
regression with SVN 264241 (differential <a href="http://reviews.llvm.org/D17596" rel="noreferrer" target="_blank">http://reviews.llvm.org/D17596</a>).<br>
<br>
The line that is causing us umbrage is in<br>
clang/lib/CodeGen/TargetInfo.cpp:7393:<br>
<br>
assert(CGM.getLangOpts().OpenCL && "SPIR is only for OpenCL");<br>
<br>
We compile from non-OpenCL sources targeting SPIR/SPIR64, and then link<br>
these with OpenCL kernels. Because of the assert, we can no longer<br>
compile the non-OpenCL source files.<br>
<br>
Would anyone greatly miss us removing this assert? Locally everything<br>
works as expected when I do this.<br>
<br>
-Neil.<br>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
</blockquote></div>