r330408 - Fix -Wunused-variable warnings after r330377.

2018-04-20 Thread Andrea Di Biagio via cfe-commits
Author: adibiagio Date: Fri Apr 20 02:47:03 2018 New Revision: 330408 URL: http://llvm.org/viewvc/llvm-project?rev=330408&view=rev Log: Fix -Wunused-variable warnings after r330377. Modified: cfe/trunk/lib/Analysis/ConstructionContext.cpp Modified: cfe/trunk/lib/Analysis/ConstructionContext.

r328968 - Fix unused variable warning introduced at revision 328910.

2018-04-02 Thread Andrea Di Biagio via cfe-commits
Author: adibiagio Date: Mon Apr 2 05:04:37 2018 New Revision: 328968 URL: http://llvm.org/viewvc/llvm-project?rev=328968&view=rev Log: Fix unused variable warning introduced at revision 328910. Modified: cfe/trunk/lib/Analysis/LiveVariables.cpp Modified: cfe/trunk/lib/Analysis/LiveVariables

[PATCH] D26335: [ms] Reinstate https://reviews.llvm.org/D14748 after https://reviews.llvm.org/D20291

2016-11-09 Thread Andrea Di Biagio via cfe-commits
andreadb added inline comments. Comment at: lib/Headers/x86intrin.h:49 +static __inline__ unsigned int __attribute__((__always_inline__, __nodebug__)) +__tzcnt_u32(unsigned int __X) { return __X ? __builtin_ctz(__X) : 32; } +#ifdef __x86_64__ hans wrote: > probin

Re: r276102 - [X86][SSE] Reimplement SSE fp2si conversion intrinsics instead of using generic IR

2016-07-25 Thread Andrea Di Biagio via cfe-commits
Hi Simon (and all), I noticed that this patch changes the definition of intrinsic _mm_cvtsd2_ss in emmintrin.h. Is that intentional? My understanding is that your patch should have only addressed float-to-integer conversions. Was this change to _mm_cvtsd_ss motivated by the fact that (V)CVTSD2SS

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:/

r250816 - [x86] Fix maskload/store intrinsic definitions in avxintrin.h

2015-10-20 Thread Andrea Di Biagio via cfe-commits
Author: adibiagio Date: Tue Oct 20 06:19:54 2015 New Revision: 250816 URL: http://llvm.org/viewvc/llvm-project?rev=250816&view=rev Log: [x86] Fix maskload/store intrinsic definitions in avxintrin.h According to the Intel documentation, the mask operand of a maskload and maskstore intrinsics is al

[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

r249142 - Make test more resilient to FastIsel changes. NFC.

2015-10-02 Thread Andrea Di Biagio via cfe-commits
Author: adibiagio Date: Fri Oct 2 10:10:22 2015 New Revision: 249142 URL: http://llvm.org/viewvc/llvm-project?rev=249142&view=rev Log: Make test more resilient to FastIsel changes. NFC. Currently FastISel doesn't know how to select vector bitcasts. During instruction selection, fast-isel always