RE: r274110 - [AVX512] Zero extend cmp intrinsic return value.

2016-07-05 Thread Demikhovsky, Elena via cfe-commits
Most of AVX-512 intrinsics were not supported in 3.8 version and added between 3.8 and 3.9. In this case, when a BE intrinsic was introduced between releases, it can be removed, right? Anyway, we’d better keep the BE intrinsic inside than maintain it in the autoupgrade. > Also should we change

RE: r274110 - [AVX512] Zero extend cmp intrinsic return value.

2016-07-04 Thread Demikhovsky, Elena via cfe-commits
I think that AVX-512 intrinsic handled in Clang should be removed from AutoUpgrade. If we decide to replace a call to llvm-backend intrinsic with generating LLVM IR directly from clang, we can remove llvm-backend intrinsic at all. Otherwise we’ll need to duplicate support in FE and BE for all AVX

RE: r262817 - [CLANG][AVX512][BUILTIN] Adding vpmultishiftqb{128|256|512}

2016-04-12 Thread Demikhovsky, Elena via cfe-commits
Yes, we’ll try make the patches smaller and avoid such mistakes. - Elena From: Chandler Carruth [mailto:chandl...@google.com] Sent: Tuesday, April 12, 2016 20:25 To: Zuckerman, Michael ; cfe-commits@lists.llvm.org; Demikhovsky, Elena Subject: Re: r262817 - [CLANG][AVX512][BUILTIN] Add