<p dir="ltr">Actually ccing them. <br>
</p>
<br><div class="gmail_quote">On Mon, Jun 1, 2015, 5:34 AM Hal Finkel <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">[+Richard, John]<br>
<br>
Hi Justin,<br>
<br>
I agree that we really should so *something* in this space; cc'ing Richard and John to get some feedback from the Clang folks before we move forward here.<br>
<br>
 -Hal<br>
<br>
----- Original Message -----<br>
> From: "Justin Bogner" <<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>><br>
> To: "llvm-commits" <<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a>><br>
> Sent: Friday, May 29, 2015 7:15:45 PM<br>
> Subject: [PATCH] Teach intrinsics to report their required target features<br>
><br>
> This implements a way to look up the required target feature for a<br>
> particular target-specific intrinsic so that frontends can detect and<br>
> report unsupported target intrinsics, rather than emitting them and<br>
> then<br>
> crashing later in SelectionDAG.<br>
><br>
> I'll follow this up with a change in clang that calls into this when<br>
> lowering intrinsics in CGBuiltin and checks if a required feature is<br>
> available using clang::TargetInfo::hasFeature.<br>
><br>
> This ends up being a very simple and unobtrusive way to do this, and<br>
> gives nice error messages instead of crashing and burning when<br>
> somebody<br>
> uses (for example) the wrong GCC builtin for their specific target.<br>
><br>
> The first patch is the tablegen change and the update to the<br>
> intrinsics<br>
> API, the other two are mostly-complete updates to the X86 and ARM<br>
> intrinsics tablegen files to show what this looks like. Assuming<br>
> everyone's happy with the approach, I'll clean those up and audit<br>
> them<br>
> for correctness before they go in.<br>
><br>
> Okay to go ahead with this?<br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
><br>
<br>
--<br>
Hal Finkel<br>
Assistant Computational Scientist<br>
Leadership Computing Facility<br>
Argonne National Laboratory<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>