RE: [PATCH] D41583: [x86][icelake][vaes]

I see. agreed. Will upload shortly From: Craig Topper [mailto:craig.top...@gmail.com] Sent: Tuesday, December 26, 2017 19:52 To: reviews+d41583+public+5c6eea8282599...@reviews.llvm.org Cc: cfe-commits@lists.llvm.org; Tayree, Coby ; mgo...@gentoo.org Subject: Re: [PATCH]

[PATCH] D26587: [X86][AVX512][InlineASM][MS][clang] (I|G)CC Memory adjustments compatibility

coby updated this revision to Diff 78681. Repository: rL LLVM https://reviews.llvm.org/D26587 Files: test/CodeGen/ms-inline-asm-avx512-memory-adjustments.c Index: test/CodeGen/ms-inline-asm-avx512-memory-adjustments.c === ---

[PATCH] D26258: [AVX512][clang] Amending vpmultishiftqb

coby abandoned this revision. coby added a comment. deprecated. Repository: rL LLVM https://reviews.llvm.org/D26258 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D26587: [X86][AVX512][InlineASM][MS][clang] (I|G)CC Memory adjustments compatibility

coby created this revision. coby added reviewers: m_zuckerman, rnk, myatsina. coby added a subscriber: cfe-commits. coby set the repository for this revision to rL LLVM. (I|G)CC will adjust a missing size qualifier on an indirect memory reference according to a certain logic, which is presented

[PATCH] D25760: [AVX512][Clang] Adding missing instructions' variations

coby abandoned this revision. coby added a comment. Amending vpmultishiftqb is currently being maintained at the following patches: https://reviews.llvm.org/D26258 https://reviews.llvm.org/D26257 All other changes are discarded Repository: rL LLVM https://reviews.llvm.org/D25760

[PATCH] D26258: [AVX512][llvm] Amending vpmultishiftqb

coby created this revision. coby added reviewers: craig.topper, m_zuckerman, igorb, delena, AsafBadouh. coby added a subscriber: cfe-commits. coby set the repository for this revision to rL LLVM. The 'vpmultishiftqb' instruction was implemented falsely, this patch amend it. This is the clang

[PATCH] D25760: [AVX512][Clang] Adding missing instructions' variations

coby created this revision. coby added reviewers: m_zuckerman, igorb, AsafBadouh, craig.topper. coby added a subscriber: cfe-commits. coby set the repository for this revision to rL LLVM. The following patch introduces the following alternations: 1. correcting the 'vpmulshiftqb' instruction,

[PATCH] D25717: [x86][inline-asm][clang][fixup] accept 'v' constraint

coby created this revision. coby added a reviewer: m_zuckerman. coby added a subscriber: cfe-commits. coby set the repository for this revision to rL LLVM. In respect to https://reviews.llvm.org/D25004 (reverted) Repository: rL LLVM https://reviews.llvm.org/D25717 Files:

[PATCH] D25004: [x86][inline-asm][clang] accept 'v' constraint

coby closed this revision. coby added a comment. Commitd to revision 283716 https://reviews.llvm.org/D25004 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D25004: [x86][inline-asm][clang] accept 'v' constraint

coby removed rL LLVM as the repository for this revision. coby updated this revision to Diff 74060. https://reviews.llvm.org/D25004 Files: lib/Basic/Targets.cpp test/CodeGen/x86-inline-asm-v-constraint.c Index: lib/Basic/Targets.cpp

Re: [PATCH] D25004: [x86][inline-asm][clang] accept 'v' constraint

coby updated this revision to Diff 72961. coby marked an inline comment as done. coby added a comment. Addressing comments: Added a check for KNL as well Repository: rL LLVM https://reviews.llvm.org/D25004 Files: x86-inline-asm-v-constraint.c Index: x86-inline-asm-v-constraint.c

[PATCH] D25004: [x86][inline-asm][clang] accept 'v' constraint

coby created this revision. coby added reviewers: echristo, delena. coby added a subscriber: cfe-commits. coby set the repository for this revision to rL LLVM. Herald added a subscriber: mehdi_amini. 1. 'v' constraint for (x86) non-avx arch imitates the already implemented 'x' constraint, i.e.