<div dir="ltr">Seems reasonable to me. It just seemed odd that we were changing. Header compatibility isn't necessarily a good reason, but being able to keep source compatibility if possible is nice.<div><br></div><div>Thanks!</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Dec 10, 2015 at 1:41 PM Craig Topper via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-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">The stance I had always taken was that if clang and gcc both have a builtin for something we match them. If clang doesn't need a builtin and gcc does then we just don't match.<span></span><br><br>On Thursday, December 10, 2015, Elena Demikhovsky via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">delena added a subscriber: delena.<br>
delena added a comment.<br>
<br>
Hi Eric,<br>
<br>
We did not complete the SKX intrinsics yet. We are adding them to the backend and to the clang. And we still have a lot of work to do.<br>
The names and parameters should be the same in GCC and in Clang headers.<br>
The current patch is just a minor fix.<br>
Could you, please, explain your point?<br>
<br>
- Elena<br>
<br>
From: Eric Christopher [mailto:<a>echristo@gmail.com</a>]<br>
Sent: Thursday, December 10, 2015 20:23<br>
To: reviews+<a href="http://reviews.llvm.org/D15328+public+c78cb0b2afbde1c2@reviews.llvm.org" target="_blank">http://reviews.llvm.org/D15328+public+c78cb0b2afbde1c2@reviews.llvm.org</a>; Demikhovsky, Elena <<a>elena.demikhovsky@intel.com</a>><br>
Subject: Re: [PATCH] <a href="http://reviews.llvm.org/D15328" target="_blank">http://reviews.llvm.org/D15328</a>: [x86][avx512] more changes in intrinsics to be align with gcc format<br>
<br>
Why again? We're already not compatible because we don't handle, nor do we want to, other builtins.<br>
<br>
-eric<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D15328" target="_blank">http://reviews.llvm.org/D15328</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a>llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote><br><br>-- <br>~Craig<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>