RKSimon added a comment.
Close this? It appears to have been committed, including the auto upgrade tests
requested by Craig.
https://reviews.llvm.org/D20359
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailm
RKSimon added a comment.
Abandon this patch? We replaced the x86 vector non-temporal store builtins with
__builtin_nontemporal_store directly in the headers.
https://reviews.llvm.org/D20358
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
ht
Author: rksimon
Date: Fri Nov 11 08:38:34 2016
New Revision: 286595
URL: http://llvm.org/viewvc/llvm-project?rev=286595&view=rev
Log:
[X86] Merge (near) duplicate scalar non-temporal store code. NFCI.
Modified:
cfe/trunk/lib/CodeGen/CGBuiltin.cpp
Modified: cfe/trunk/lib/CodeGen/CGBuiltin.cpp
Author: rksimon
Date: Thu Nov 10 07:54:39 2016
New Revision: 286449
URL: http://llvm.org/viewvc/llvm-project?rev=286449&view=rev
Log:
Fix -Wdocumentation warning
Modified:
clang-tools-extra/trunk/clang-move/ClangMove.h
Modified: clang-tools-extra/trunk/clang-move/ClangMove.h
URL:
http://llv
Author: rksimon
Date: Fri Oct 28 05:09:35 2016
New Revision: 285390
URL: http://llvm.org/viewvc/llvm-project?rev=285390&view=rev
Log:
Fix MSVC "not all control paths return a value" warning
Add unreachable after enum switch statement
Modified:
cfe/trunk/lib/Driver/Action.cpp
Modified: cfe/t
You’re not kidding, its incredibly annoying. I can add a comment explaining why
its there if you wish?
Simon
>> On 27 Oct 2016, at 12:51, Simon Pilgrim via cfe-commits
>> wrote:
>>
>> Author: rksimon
>> Date: Thu Oct 27 05:51:29 2016
>> New Revision: 2852
Author: rksimon
Date: Thu Oct 27 05:51:29 2016
New Revision: 285281
URL: http://llvm.org/viewvc/llvm-project?rev=285281&view=rev
Log:
Fix MSVC warning about missing 'this' from lambda default capture mode
Modified:
cfe/trunk/lib/Lex/PPLexerChange.cpp
Modified: cfe/trunk/lib/Lex/PPLexerChange
Author: rksimon
Date: Tue Oct 25 07:59:15 2016
New Revision: 285067
URL: http://llvm.org/viewvc/llvm-project?rev=285067&view=rev
Log:
Fix MSVC unused variable warning.
LLVM_ATTRIBUTE_UNUSED doesn't work for non-gcc style compilers.
Modified:
cfe/trunk/lib/StaticAnalyzer/Checkers/StdLibraryFu
Author: rksimon
Date: Tue Oct 18 08:15:31 2016
New Revision: 284476
URL: http://llvm.org/viewvc/llvm-project?rev=284476&view=rev
Log:
Fix signed/unsigned comparison warnings
Modified:
clang-tools-extra/trunk/unittests/clang-tidy/NamespaceAliaserTest.cpp
clang-tools-extra/trunk/unittests/c
Author: rksimon
Date: Fri Oct 7 08:25:41 2016
New Revision: 283549
URL: http://llvm.org/viewvc/llvm-project?rev=283549&view=rev
Log:
Wdocumentation fix
Modified:
cfe/trunk/include/clang/Sema/Sema.h
Modified: cfe/trunk/include/clang/Sema/Sema.h
URL:
http://llvm.org/viewvc/llvm-project/cfe/t
Author: rksimon
Date: Mon Oct 3 07:37:08 2016
New Revision: 283106
URL: http://llvm.org/viewvc/llvm-project?rev=283106&view=rev
Log:
Wdocumentation fix
Modified:
cfe/trunk/lib/Analysis/CloneDetection.cpp
Modified: cfe/trunk/lib/Analysis/CloneDetection.cpp
URL:
http://llvm.org/viewvc/llvm-p
Author: rksimon
Date: Fri Sep 30 09:25:09 2016
New Revision: 282858
URL: http://llvm.org/viewvc/llvm-project?rev=282858&view=rev
Log:
Strip trailing whitespace (NFCI)
Modified:
cfe/trunk/lib/Sema/SemaExprCXX.cpp
Modified: cfe/trunk/lib/Sema/SemaExprCXX.cpp
URL:
http://llvm.org/viewvc/llvm-p
Author: rksimon
Date: Fri Sep 30 09:18:06 2016
New Revision: 282857
URL: http://llvm.org/viewvc/llvm-project?rev=282857&view=rev
Log:
Fix int <= bool comparison warning on MSVC
Modified:
cfe/trunk/lib/Sema/SemaExprCXX.cpp
Modified: cfe/trunk/lib/Sema/SemaExprCXX.cpp
URL:
http://llvm.org/vie
RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D24825
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe
RKSimon added a subscriber: RKSimon.
Comment at: lib/Headers/avx512fintrin.h:9124
@@ +9123,3 @@
+{
+ return (__m128) __builtin_ia32_movss_mask ((__v4sf) __A, (__v4sf) __B,
+ (__v4sf) __W,
delena wrote:
> please try the following:
> if (__U)
> retu
Author: rksimon
Date: Thu Sep 8 06:03:41 2016
New Revision: 280921
URL: http://llvm.org/viewvc/llvm-project?rev=280921&view=rev
Log:
Moved unreachable to appease msvc, gcc and clang
Modified:
cfe/trunk/lib/CodeGen/CGVTables.cpp
Modified: cfe/trunk/lib/CodeGen/CGVTables.cpp
URL:
http://llvm
Author: rksimon
Date: Thu Sep 8 04:59:58 2016
New Revision: 280917
URL: http://llvm.org/viewvc/llvm-project?rev=280917&view=rev
Log:
Fixed a 'not all control paths return a value' warning on MSVC builds
Modified:
cfe/trunk/lib/CodeGen/CGVTables.cpp
Modified: cfe/trunk/lib/CodeGen/CGVTables.
Author: rksimon
Date: Sat Aug 20 15:21:27 2016
New Revision: 279382
URL: http://llvm.org/viewvc/llvm-project?rev=279382&view=rev
Log:
Wdocumentation fix
Modified:
cfe/trunk/lib/Analysis/CloneDetection.cpp
Modified: cfe/trunk/lib/Analysis/CloneDetection.cpp
URL:
http://llvm.org/viewvc/llvm-p
Author: rksimon
Date: Fri Aug 12 06:43:57 2016
New Revision: 278503
URL: http://llvm.org/viewvc/llvm-project?rev=278503&view=rev
Log:
Fix Wdocumentation unknown parameter warning
Modified:
cfe/trunk/lib/Sema/SemaTemplateDeduction.cpp
Modified: cfe/trunk/lib/Sema/SemaTemplateDeduction.cpp
URL
Author: rksimon
Date: Wed Aug 10 04:59:49 2016
New Revision: 278208
URL: http://llvm.org/viewvc/llvm-project?rev=278208&view=rev
Log:
[X86][AVX] Ensure we only match against 1-byte alignment
Modified:
cfe/trunk/test/CodeGen/avx-builtins.c
Modified: cfe/trunk/test/CodeGen/avx-builtins.c
URL:
Author: rksimon
Date: Tue Aug 9 05:02:11 2016
New Revision: 278111
URL: http://llvm.org/viewvc/llvm-project?rev=278111&view=rev
Log:
Fix Wdocumentation unknown parameter warning
Modified:
clang-tools-extra/trunk/include-fixer/IncludeFixer.h
Modified: clang-tools-extra/trunk/include-fixer/In
Author: rksimon
Date: Wed Jul 27 11:41:56 2016
New Revision: 276889
URL: http://llvm.org/viewvc/llvm-project?rev=276889&view=rev
Log:
Fix unnecessary default switch warning
Modified:
cfe/trunk/lib/Sema/CodeCompleteConsumer.cpp
Modified: cfe/trunk/lib/Sema/CodeCompleteConsumer.cpp
URL:
http:
Author: rksimon
Date: Fri Jul 22 08:58:56 2016
New Revision: 276417
URL: http://llvm.org/viewvc/llvm-project?rev=276417&view=rev
Log:
[X86][AVX] Added support for lowering to VBROADCASTF128/VBROADCASTI128 with
generic IR
As discussed on D22460, I've updated the vbroadcastf128 pd256/ps256 builtin
This revision was automatically updated to reflect the committed changes.
Closed by commit rL276102: [X86][SSE] Reimplement SSE fp2si conversion
intrinsics instead of using… (authored by RKSimon).
Changed prior to commit:
https://reviews.llvm.org/D22105?vs=64534&id=64653#toc
Repository:
rL L
Author: rksimon
Date: Wed Jul 20 05:18:01 2016
New Revision: 276102
URL: http://llvm.org/viewvc/llvm-project?rev=276102&view=rev
Log:
[X86][SSE] Reimplement SSE fp2si conversion intrinsics instead of using generic
IR
D20859 and D20860 attempted to replace the SSE (V)CVTTPS2DQ and VCVTTPD2DQ
tru
RKSimon updated this revision to Diff 64534.
RKSimon added a comment.
Removed sitofp conversion changes
Repository:
rL LLVM
https://reviews.llvm.org/D22105
Files:
include/clang/Basic/BuiltinsX86.def
lib/Headers/avxintrin.h
lib/Headers/emmintrin.h
lib/Headers/xmmintrin.h
test/CodeGe
RKSimon added a comment.
In https://reviews.llvm.org/D22105#488566, @eli.friedman wrote:
> The x86-specific operation is affected by the rounding mode... but so is a C
> cast. This is specified by Annex F in the C standard.
>
> Of course, you're going to end up with undefined behavior if you ac
RKSimon added a comment.
In https://reviews.llvm.org/D22105#488513, @eli.friedman wrote:
> I don't think we need to use x86-specific operations for sitofp-like
> conversions; the C cast is equivalent given that a 32 or 64-bit integer is
> always in within the range of a 32-bit float.
I think
Author: rksimon
Date: Thu Jul 7 17:32:26 2016
New Revision: 274799
URL: http://llvm.org/viewvc/llvm-project?rev=274799&view=rev
Log:
Update switch statement to match coding standards.
Modified:
cfe/trunk/lib/Basic/Targets.cpp
Modified: cfe/trunk/lib/Basic/Targets.cpp
URL:
http://llvm.org/v
RKSimon created this revision.
RKSimon added reviewers: eli.friedman, mkuper, craig.topper, spatel, andreadb.
RKSimon added a subscriber: cfe-commits.
RKSimon set the repository for this revision to rL LLVM.
D20859 and D20860 attempted to replace the SSE (V)CVTTPS2DQ and VCVTTPD2DQ
truncating con
Author: rksimon
Date: Thu Jul 7 06:24:38 2016
New Revision: 274748
URL: http://llvm.org/viewvc/llvm-project?rev=274748&view=rev
Log:
Fix "not all control paths return a value" warning on MSVC
This time without causing a 'all enums handled' warning on other compilers.
Modified:
cfe/trunk/lib
Author: rksimon
Date: Thu Jul 7 06:12:02 2016
New Revision: 274746
URL: http://llvm.org/viewvc/llvm-project?rev=274746&view=rev
Log:
Fix "not all control paths return a value" warning on MSVC
Modified:
cfe/trunk/lib/Basic/Targets.cpp
Modified: cfe/trunk/lib/Basic/Targets.cpp
URL:
http://ll
Author: rksimon
Date: Tue Jul 5 09:49:31 2016
New Revision: 274554
URL: http://llvm.org/viewvc/llvm-project?rev=274554&view=rev
Log:
[X86][AVX512] Remove vector BROADCAST builtins.
Modified:
cfe/trunk/include/clang/Basic/BuiltinsX86.def
Modified: cfe/trunk/include/clang/Basic/BuiltinsX86.de
Author: rksimon
Date: Tue Jul 5 07:59:33 2016
New Revision: 274544
URL: http://llvm.org/viewvc/llvm-project?rev=274544&view=rev
Log:
[X86][AVX512] Converted the VBROADCAST intrinsics to generic IR
Modified:
cfe/trunk/lib/Headers/avx512bwintrin.h
cfe/trunk/lib/Headers/avx512fintrin.h
Author: rksimon
Date: Mon Jul 4 16:30:47 2016
New Revision: 274523
URL: http://llvm.org/viewvc/llvm-project?rev=274523&view=rev
Log:
[X86][AVX512] Converted the VSHUFPD intrinsics to generic IR
Modified:
cfe/trunk/lib/Headers/avx512fintrin.h
cfe/trunk/lib/Headers/avx512vlintrin.h
cfe
Author: rksimon
Date: Mon Jul 4 08:34:44 2016
New Revision: 274502
URL: http://llvm.org/viewvc/llvm-project?rev=274502&view=rev
Log:
[X86][AVX512] Converted the VPERMPD/VPERMQ intrinsics to generic IR
Modified:
cfe/trunk/lib/Headers/avx512fintrin.h
cfe/trunk/lib/Headers/avx512vlintrin.h
Author: rksimon
Date: Mon Jul 4 06:06:15 2016
New Revision: 274492
URL: http://llvm.org/viewvc/llvm-project?rev=274492&view=rev
Log:
[X86][AVX512] Converted the VPERMILPD/VPERMILPS intrinsics to generic IR
Modified:
cfe/trunk/lib/Headers/avx512fintrin.h
cfe/trunk/test/CodeGen/avx512f-bui
Author: rksimon
Date: Sat Jul 2 12:16:25 2016
New Revision: 274442
URL: http://llvm.org/viewvc/llvm-project?rev=274442&view=rev
Log:
[X86][AVX512] Converted the MOVDDUP/MOVSLDUP/MOVSHDUP masked intrinsics to
generic IR
llvm companion patch imminent
Modified:
cfe/trunk/include/clang/Basic/B
Author: rksimon
Date: Wed Jun 29 09:04:08 2016
New Revision: 274126
URL: http://llvm.org/viewvc/llvm-project?rev=274126&view=rev
Log:
[X86][SSE2] Updated tests to match
llvm\test\CodeGen\X86\sse2-intrinsics-fast-isel-x86_64.ll
Modified:
cfe/trunk/test/CodeGen/sse2-builtins.c
Modified: cfe/t
RKSimon accepted this revision.
RKSimon added a reviewer: RKSimon.
RKSimon added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rL LLVM
http://reviews.llvm.org/D21504
___
cfe-commits mailing list
cfe-commits@lists.ll
RKSimon added inline comments.
Comment at: tools/clang/test/CodeGen/sse2-builtins.c:1526
@@ +1525,3 @@
+ // CHECK-LABEL: test_mm_loadu_si64
+ // CHECK: load i64, i64* %__u
+ // CHECK: insertelement <2 x i64> undef, i64 %{{.*}}, i32 0
Please can add the alignmen
RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.
LGTM - the compile warning is clear and it could be a problem if we allow
undefined values through on pre-AVX targets.
The only other thing we could do is handle these in CGBuiltin and 'acce
RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.
LGTM
Comment at: lib/Headers/bmiintrin.h:284
@@ -283,2 +283,3 @@
///bits in the operand.
+
static __inline__ unsigned int __RELAXED_FN_ATTRS
Why the n
RKSimon added a subscriber: RKSimon.
Comment at: tools/clang/test/CodeGen/sse2-builtins.c:1527
@@ +1526,3 @@
+ // CHECK: load i64, i64* %__u
+ // CHECK: insertelement <2 x i64> undef, i64 %4, i32 0
+ // CHECK: insertelement <2 x i64> %{{.*}}, i64 0, i32 1
Repla
Author: rksimon
Date: Sat Jun 18 13:20:14 2016
New Revision: 273090
URL: http://llvm.org/viewvc/llvm-project?rev=273090&view=rev
Log:
[X86][XOP] Refreshed builtin tests ready for creation of llvm fast-isel tests
Modified:
cfe/trunk/test/CodeGen/xop-builtins.c
Modified: cfe/trunk/test/CodeGen
Author: rksimon
Date: Sat Jun 18 12:09:40 2016
New Revision: 273086
URL: http://llvm.org/viewvc/llvm-project?rev=273086&view=rev
Log:
[X86][TBM] Refreshed builtin tests ready for creation of llvm fast-isel tests
Modified:
cfe/trunk/test/CodeGen/tbm-builtins.c
Modified: cfe/trunk/test/CodeGen
Author: rksimon
Date: Fri Jun 17 09:28:16 2016
New Revision: 273003
URL: http://llvm.org/viewvc/llvm-project?rev=273003&view=rev
Log:
[X86][SSE4A] Use native IR for mask movntsd/movntss intrinsics.
Depends on llvm side commit r273002.
Modified:
cfe/trunk/lib/CodeGen/CGBuiltin.cpp
cfe/tru
RKSimon added a comment.
It seems like part of the need for this is because the _mm_cmp_ps style
intrinsics are defined as macros (to get around the problem of trying to use an
immediate as an argument):
#define _mm_cmp_ps(a, b, c) __extension__ ({ \
(__m128)__builtin_ia32_cmpps((__v4sf)(
RKSimon added a subscriber: RKSimon.
RKSimon added a reviewer: RKSimon.
Comment at: lib/Headers/bmiintrin.h:296
@@ -290,1 +295,3 @@
+}
+
#ifdef __x86_64__
Why not just #define to __tzcnt_u32 like the (many) other duplicate tzcnt
intrinsics we have:
```
#define
RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.
LGTM
http://reviews.llvm.org/D21268
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-
RKSimon added a comment.
http://reviews.llvm.org/D21272 has now been committed, which I think removes
the need for this patch. http://reviews.llvm.org/D20359 is still needed (with
the additional tests requested by Craig).
http://reviews.llvm.org/D20358
__
Author: rksimon
Date: Mon Jun 13 05:05:19 2016
New Revision: 272541
URL: http://llvm.org/viewvc/llvm-project?rev=272541&view=rev
Log:
Fix unused variable warning
Modified:
cfe/trunk/lib/CodeGen/CGBuiltin.cpp
Modified: cfe/trunk/lib/CodeGen/CGBuiltin.cpp
URL:
http://llvm.org/viewvc/llvm-proj
Author: rksimon
Date: Mon Jun 13 04:57:52 2016
New Revision: 272540
URL: http://llvm.org/viewvc/llvm-project?rev=272540&view=rev
Log:
[Clang][X86] Convert non-temporal store builtins to generic
__builtin_nontemporal_store in headers
We can now use __builtin_nontemporal_store instead of target sp
This revision was automatically updated to reflect the committed changes.
Closed by commit rL272540: [Clang][X86] Convert non-temporal store builtins to
generic… (authored by RKSimon).
Changed prior to commit:
http://reviews.llvm.org/D21272?vs=60481&id=60499#toc
Repository:
rL LLVM
http://r
RKSimon added a comment.
In http://reviews.llvm.org/D20358#446241, @RKSimon wrote:
> In http://reviews.llvm.org/D20358#446220, @ab wrote:
>
> > In http://reviews.llvm.org/D20358#446218, @ab wrote:
> >
> > > In http://reviews.llvm.org/D20358#446210, @RKSimon wrote:
> > >
> > > > Is there any reaso
RKSimon created this revision.
RKSimon added reviewers: craig.topper, ab, spatel, andreadb.
RKSimon added a subscriber: cfe-commits.
RKSimon set the repository for this revision to rL LLVM.
As discussed on D20358, we can now use __builtin_nontemporal_store instead of
target specific builtins for
RKSimon added a comment.
Eeep that's certainly a lot more work than just adding a few extra cases!
Please add a TODO explaining what we need to do?
If there is a problem with the header documentation please can you raise a
bugzilla and CC Katya Romanova.
http://reviews.llvm.org/D21268
RKSimon added a comment.
Is there any reason that we shouldn't include the avxintrin.h
__builtin_ia32_cmppd/__builtin_ia32_cmpps/__builtin_ia32_cmppd256/__builtin_ia32_cmpps256
packed intrinsics in this CGBuiltin.cpp patch? Since we're heading towards
nixing them anyhow.
http://reviews.llvm.o
Author: rksimon
Date: Sat Jun 11 17:40:01 2016
New Revision: 272490
URL: http://llvm.org/viewvc/llvm-project?rev=272490&view=rev
Log:
[X86][BMI] Improved bmi intrinsics checks
Ready for matching with llvm/test/CodeGen/X86/bmi-intrinsics-fast-isel.ll (to
be added shortly)
Modified:
cfe/trun
RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.
LGTM - add test_mm512_setzero_pd() as well if you can.
Comment at: test/CodeGen/avx512f-builtins.c:7291
@@ +7290,3 @@
+
+__m512i test_mm512_setzero_ps()
+{
RKSimon added a comment.
Can you add tests for the existing _mm512_setzero_* intrinsics as well please?
http://reviews.llvm.org/D20871
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
RKSimon added a comment.
In http://reviews.llvm.org/D20358#446220, @ab wrote:
> In http://reviews.llvm.org/D20358#446218, @ab wrote:
>
> > In http://reviews.llvm.org/D20358#446210, @RKSimon wrote:
> >
> > > Is there any reason why we can't just get rid of all the SSE movnt
> > > builtins and use
RKSimon added a subscriber: RKSimon.
RKSimon added a comment.
Is there any reason why we can't just get rid of all the SSE movnt builtins and
use __builtin_nontemporal_store instead (http://reviews.llvm.org/D12313)?
http://reviews.llvm.org/D20358
_
RKSimon added a subscriber: RKSimon.
RKSimon added a comment.
Tests?
http://reviews.llvm.org/D20871
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: rksimon
Date: Wed Jun 1 16:46:51 2016
New Revision: 271436
URL: http://llvm.org/viewvc/llvm-project?rev=271436&view=rev
Log:
[X86][SSE] Replace (V)CVTTPS2DQ and VCVTTPD2DQ truncating (round to zero)
f32/f64 to i32 with generic IR (clang)
The 'cvtt' truncation (round to zero) conversions
This revision was automatically updated to reflect the committed changes.
Closed by commit rL271436: [X86][SSE] Replace (V)CVTTPS2DQ and VCVTTPD2DQ
truncating (round to zero)… (authored by RKSimon).
Changed prior to commit:
http://reviews.llvm.org/D20859?vs=59204&id=59284#toc
Repository:
rL
RKSimon created this revision.
RKSimon added reviewers: ab, mkuper, craig.topper, spatel, andreadb.
RKSimon added a subscriber: cfe-commits.
RKSimon set the repository for this revision to rL LLVM.
The 'cvtt' truncation (round to zero) conversions can be safely represented as
generic __builtin_co
Author: rksimon
Date: Mon May 30 14:20:55 2016
New Revision: 271227
URL: http://llvm.org/viewvc/llvm-project?rev=271227&view=rev
Log:
[X86] Ensure load/store tests unaligned pointers really are align 1
Modified:
cfe/trunk/test/CodeGen/avx-builtins.c
cfe/trunk/test/CodeGen/sse-builtins.c
Author: rksimon
Date: Mon May 30 12:58:38 2016
New Revision: 271219
URL: http://llvm.org/viewvc/llvm-project?rev=271219&view=rev
Log:
[X86][SSE] Added missing tests (merge failure)
Differential Revision: http://reviews.llvm.org/D20617
Modified:
cfe/trunk/test/CodeGen/sse-builtins.c
Modified
This revision was automatically updated to reflect the committed changes.
Closed by commit rL271218: [X86][SSE] _mm_store1_ps/_mm_store1_pd should
require an aligned pointer (authored by RKSimon).
Changed prior to commit:
http://reviews.llvm.org/D20617?vs=58397&id=58979#toc
Repository:
rL LL
Author: rksimon
Date: Mon May 30 12:55:25 2016
New Revision: 271218
URL: http://llvm.org/viewvc/llvm-project?rev=271218&view=rev
Log:
[X86][SSE] _mm_store1_ps/_mm_store1_pd should require an aligned pointer
According to the gcc headers, intel intrinsics docs and msdn codegen the
_mm_store1_pd (a
Author: rksimon
Date: Sun May 29 13:49:08 2016
New Revision: 271187
URL: http://llvm.org/viewvc/llvm-project?rev=271187&view=rev
Log:
[X86][SSE] Make unsigned integer vector types generally available
As discussed on http://reviews.llvm.org/D20684, move the unsigned integer
vector types used for
RKSimon added a subscriber: RKSimon.
RKSimon added a comment.
Should AVX512 store support (non-temporal or otherwise) be added to
X86FastISel::X86FastEmitStore ?
http://reviews.llvm.org/D20359
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
This revision was automatically updated to reflect the committed changes.
Closed by commit rL271106: [X86][SSE] Replace VPMOVSX and (V)PMOVZX integer
extension intrinsics with… (authored by RKSimon).
Changed prior to commit:
http://reviews.llvm.org/D20684?vs=58626&id=58884#toc
Repository:
rL
Author: rksimon
Date: Sat May 28 03:12:45 2016
New Revision: 271106
URL: http://llvm.org/viewvc/llvm-project?rev=271106&view=rev
Log:
[X86][SSE] Replace VPMOVSX and (V)PMOVZX integer extension intrinsics with
generic IR (clang)
The VPMOVSX and (V)PMOVZX sign/zero extension intrinsics can be safe
RKSimon added a comment.
In http://reviews.llvm.org/D20684#442514, @ab wrote:
> I'd add the unsigned typedefs with their signed counterparts; no reason not
> to.
> With that, LGTM.
Thanks, I'll do that as a follow up commit.
Repository:
rL LLVM
http://reviews.llvm.org/D20684
_
RKSimon added inline comments.
Comment at: lib/Headers/emmintrin.h:598
@@ -594,3 +597,3 @@
static __inline__ void __DEFAULT_FN_ATTRS
-_mm_store_pd(double *__dp, __m128d __a)
+_mm_store_pd1(double *__dp, __m128d __a)
{
majnemer wrote:
> You could use `__attribute
RKSimon created this revision.
RKSimon added reviewers: mkuper, craig.topper, spatel, andreadb.
RKSimon added a subscriber: cfe-commits.
RKSimon set the repository for this revision to rL LLVM.
The VPMOVSX and (V)PMOVZX sign/zero extension intrinsics can be safely
represented as generic __builtin
Author: rksimon
Date: Thu May 26 05:20:25 2016
New Revision: 270836
URL: http://llvm.org/viewvc/llvm-project?rev=270836&view=rev
Log:
[X86][F16C] Improved f16c intrinsics checks
Added checks for upper elements being zero'd in scalar conversions
Modified:
cfe/trunk/test/CodeGen/f16c-builtins
Author: rksimon
Date: Thu May 26 04:56:50 2016
New Revision: 270833
URL: http://llvm.org/viewvc/llvm-project?rev=270833&view=rev
Log:
[X86][AVX2] Improved checks for float/double mask generation for non-masked
gathers
Modified:
cfe/trunk/test/CodeGen/avx2-builtins.c
Modified: cfe/trunk/test
RKSimon added a comment.
In http://reviews.llvm.org/D20617#439200, @craig.topper wrote:
> Can you double check gcc's xmmintrin.h again. I'm pretty sure _mm_store1_ps
> is calling _mm_storeu_ps.
Yes you're right - for gcc _mm_store1_pd is aligned (and there is a comment
saying it must be), but
Author: rksimon
Date: Wed May 25 10:10:49 2016
New Revision: 270708
URL: http://llvm.org/viewvc/llvm-project?rev=270708&view=rev
Log:
[X86][AVX2] Full set of AVX2 intrinsics tests
llvm/test/CodeGen/X86/avx2-intrinsics-fast-isel.ll will be synced to this
Modified:
cfe/trunk/test/CodeGen/avx2-
RKSimon created this revision.
RKSimon added reviewers: craig.topper, spatel, andreadb.
RKSimon added a subscriber: cfe-commits.
RKSimon set the repository for this revision to rL LLVM.
According to the gcc headers, intel intrinsics docs and msdn codegen the
_mm_store1_ps/_mm_store1_pd (and their
Author: rksimon
Date: Wed May 25 04:20:08 2016
New Revision: 270679
URL: http://llvm.org/viewvc/llvm-project?rev=270679&view=rev
Log:
[X86][SSE] Updated _mm_store_ps1 test to match _mm_store1_ps
Modified:
cfe/trunk/test/CodeGen/sse-builtins.c
Modified: cfe/trunk/test/CodeGen/sse-builtins.c
U
This revision was automatically updated to reflect the committed changes.
Closed by commit rL270499: [X86][SSE] Replace lossless i32/f32 to f64
conversion intrinsics with generic IR (authored by RKSimon).
Changed prior to commit:
http://reviews.llvm.org/D20528?vs=58146&id=58160#toc
Repository:
Author: rksimon
Date: Mon May 23 17:13:02 2016
New Revision: 270499
URL: http://llvm.org/viewvc/llvm-project?rev=270499&view=rev
Log:
[X86][SSE] Replace lossless i32/f32 to f64 conversion intrinsics with generic IR
Both the (V)CVTDQ2PD(Y) (i32 to f64) and (V)CVTPS2PD(Y) (f32 to f64) conversion
i
RKSimon added a comment.
In http://reviews.llvm.org/D20528#437165, @mkuper wrote:
> Presumably, the fast-isel lowering of the IR pattern is already correct, and
> in any case, it isn't affected by this patch.
> I just want to make sure we don't regress the optimized DAG codegen - that
> is, it
RKSimon added a comment.
In http://reviews.llvm.org/D20528#437117, @mkuper wrote:
> Sorry, I didn't intend to imply the rest of the llvm work is necessary for
> this to go in. Just that I'd be happier with this patch knowing that we have
> a regression test for doing the (shuffle + fpext, say)
RKSimon updated this revision to Diff 58146.
Repository:
rL LLVM
http://reviews.llvm.org/D20528
Files:
include/clang/Basic/BuiltinsX86.def
lib/Headers/avxintrin.h
lib/Headers/emmintrin.h
test/CodeGen/avx-builtins.c
test/CodeGen/builtins-x86.c
test/CodeGen/sse2-builtins.c
test/Cod
RKSimon added a comment.
In http://reviews.llvm.org/D20528#436893, @mkuper wrote:
> This looks right, but we may lose some end-to-end tests, since right now we
> have a clang-level test that checks the builtin is lowered to the intrinsic,
> and (hopefully) a CG-level test that the intrinsic is
RKSimon created this revision.
RKSimon added reviewers: qcolombet, craig.topper, mkuper, andreadb, spatel.
RKSimon added a subscriber: cfe-commits.
RKSimon set the repository for this revision to rL LLVM.
Both the (V)CVTDQ2PD(Y) (i32 to f64) and (V)CVTPS2PD(Y) (f32 to f64) conversion
instructions
This revision was automatically updated to reflect the committed changes.
Closed by commit rL270330: [X86][AVX] Ensure zero-extension of
_mm256_extract_epi8 and _mm256_extract_epi16 (authored by RKSimon).
Changed prior to commit:
http://reviews.llvm.org/D20468?vs=57927&id=58045#toc
Repository:
Author: rksimon
Date: Sat May 21 16:14:35 2016
New Revision: 270330
URL: http://llvm.org/viewvc/llvm-project?rev=270330&view=rev
Log:
[X86][AVX] Ensure zero-extension of _mm256_extract_epi8 and _mm256_extract_epi16
Ensure _mm256_extract_epi8 and _mm256_extract_epi16 zero extend their i8/i16
resu
RKSimon added a comment.
In http://reviews.llvm.org/D20468#435522, @mkuper wrote:
> Could you point me to where in the documentation it says they must be
> zero-extended?
> The Intel intrinsics guide actually has them with shorter return types:
>
> __int8 _mm256_extract_epi8 (__m256i a, const
Author: rksimon
Date: Fri May 20 10:49:17 2016
New Revision: 270227
URL: http://llvm.org/viewvc/llvm-project?rev=270227&view=rev
Log:
[X86][AVX] Added _mm256_testc_si256/_mm256_testnzc_si256/_mm256_testz_si256
tests
Modified:
cfe/trunk/test/CodeGen/avx-builtins.c
Modified: cfe/trunk/test/Co
RKSimon created this revision.
RKSimon added reviewers: mkuper, craig.topper, kromanova, spatel.
RKSimon added a subscriber: cfe-commits.
RKSimon set the repository for this revision to rL LLVM.
Ensure _mm256_extract_epi8 and _mm256_extract_epi16 zero extend their i8/i16
result to i32. This match
Author: rksimon
Date: Fri May 20 07:57:21 2016
New Revision: 270212
URL: http://llvm.org/viewvc/llvm-project?rev=270212&view=rev
Log:
[X86][AVX] Added _mm256_extract_epi64 test
Modified:
cfe/trunk/test/CodeGen/avx-builtins.c
Modified: cfe/trunk/test/CodeGen/avx-builtins.c
URL:
http://llvm.o
Author: rksimon
Date: Fri May 20 07:41:02 2016
New Revision: 270210
URL: http://llvm.org/viewvc/llvm-project?rev=270210&view=rev
Log:
[X86][AVX] Full set of AVX intrinsics tests
llvm/test/CodeGen/X86/avx-intrinsics-fast-isel.ll will be synced to this
Modified:
cfe/trunk/test/CodeGen/avx-buil
Author: rksimon
Date: Thu May 19 12:11:31 2016
New Revision: 270083
URL: http://llvm.org/viewvc/llvm-project?rev=270083&view=rev
Log:
[X86][SSE] Sync with llvm/test/CodeGen/X86/sse-intrinsics-fast-isel.ll
sse-builtins.c now just covers SSE1 intrinsics
Modified:
cfe/trunk/test/CodeGen/sse-bui
Author: rksimon
Date: Thu May 19 11:48:59 2016
New Revision: 270079
URL: http://llvm.org/viewvc/llvm-project?rev=270079&view=rev
Log:
[X86][SSE2] Fixed shuffle of results in _mm_cmpnge_sd/_mm_cmpngt_sd tests
Modified:
cfe/trunk/test/CodeGen/sse2-builtins.c
Modified: cfe/trunk/test/CodeGen/ss
901 - 1000 of 1068 matches
Mail list logo
