[PATCH] D89105: [X86] Support Intel avxvnni

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 22 02:14:46 PDT 2020


RKSimon added inline comments.
Herald added a subscriber: dexonsmith.


================
Comment at: clang/lib/Headers/avxvnniintrin.h:31
+
+#include "commonvnniintrin.h"
+
----------------
LiuChen3 wrote:
> LiuChen3 wrote:
> > RKSimon wrote:
> > > Is having a commonvnniintrin.h header the approach gcc/icc are taking?
> > GCC doesn't have commonvnniintrin.h headr, they only have avxvnniintrin.h. For ICC, I will check with this and give you answer later.
> GCC and ICC doesn't break intrinsics into multiple headers.
Is there anyway that we can avoid this as well then? I get worried when header layouts start to diverge...


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89105/new/

https://reviews.llvm.org/D89105



More information about the llvm-commits mailing list