Re: [PATCH] D13861: [x86] fix wrong maskload/store intrinsic definitions in avxintrin.h (follow up of D13776).

2015-10-20 Thread Andrea Di Biagio via cfe-commits
andreadb added a comment. In http://reviews.llvm.org/D13861#270811, @delena wrote: > LGTM Thanks Elena! Committed revision250816. Repository: rL LLVM http://reviews.llvm.org/D13861 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http:/

Re: [PATCH] D13861: [x86] fix wrong maskload/store intrinsic definitions in avxintrin.h (follow up of D13776).

2015-10-20 Thread Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL250816: [x86] Fix maskload/store intrinsic definitions in avxintrin.h (authored by adibiagio). Changed prior to commit: http://reviews.llvm.org/D13861?vs=37741&id=37852#toc Repository: rL LLVM http:

Re: [PATCH] D13861: [x86] fix wrong maskload/store intrinsic definitions in avxintrin.h (follow up of D13776).

2015-10-19 Thread Elena Demikhovsky via cfe-commits
delena added a comment. LGTM http://reviews.llvm.org/D13861 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D13861: [x86] fix wrong maskload/store intrinsic definitions in avxintrin.h (follow up of D13776).

2015-10-19 Thread Andrea Di Biagio via cfe-commits
andreadb created this revision. andreadb added reviewers: bruno, mkuper, delena, qcolombet. andreadb added a subscriber: cfe-commits. Hi, This patch is a follow up of D13776. According to the Intel documentation, the mask operand of a maskload and maskstore intrinsics is always a vector of packe