[PATCH] D41168: [X86] Lowering X86 avx512 sqrt intrinsics to IR

2017-12-27 Thread Uriel Korach via Phabricator via cfe-commits
uriel.k updated this revision to Diff 128232. uriel.k retitled this revision from "[X86][avx512] Lowering X86 avx512 sqrt intrinsics to IR" to "[X86] Lowering X86 avx512 sqrt intrinsics to IR". uriel.k edited the summary of this revision. uriel.k added a comment. now takes care of sqrt with

[PATCH] D41168: [X86][avx512] Lowering X86 avx512 sqrt intrinsics to IR

2017-12-13 Thread Uriel Korach via Phabricator via cfe-commits
uriel.k created this revision. uriel.k added reviewers: craig.topper, igorb. seperates the non-round version from the round version of sqrt builtins and catching them in CGBuiltin.cpp to replace builtin with IR. https://reviews.llvm.org/D41168 Files: include/clang/Basic/BuiltinsX86.def

[PATCH] D38737: [X86] test/testn intrinsics lowering to IR. clang side

2017-11-13 Thread Uriel Korach via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL318035: [X86] test/testn intrinsics lowering to IR. clang side (authored by uriel.k). Changed prior to commit: https://reviews.llvm.org/D38737?vs=122082=122639#toc Repository: rL LLVM

[PATCH] D38737: [X86] test/testn intrinsics lowering to IR. clang side

2017-11-08 Thread Uriel Korach via Phabricator via cfe-commits
uriel.k added inline comments. Comment at: lib/Headers/avx512vlbwintrin.h:2425 + return _mm_mask_cmpneq_epi8_mask (__U, _mm_and_si128 (__A, __B), + _mm_setzero_hi()); } uriel.k wrote: > For strange reason this line moved and is

[PATCH] D38737: [X86] test/testn intrinsics lowering to IR. clang side

2017-11-08 Thread Uriel Korach via Phabricator via cfe-commits
uriel.k added inline comments. Comment at: lib/Headers/avx512vlbwintrin.h:2425 + return _mm_mask_cmpneq_epi8_mask (__U, _mm_and_si128 (__A, __B), + _mm_setzero_hi()); } For strange reason this line moved and is not aligned with

[PATCH] D38737: [X86] test/testn intrinsics lowering to IR. clang side

2017-11-08 Thread Uriel Korach via Phabricator via cfe-commits
uriel.k updated this revision to Diff 122082. uriel.k marked 2 inline comments as done. https://reviews.llvm.org/D38737 Files: include/clang/Basic/BuiltinsX86.def lib/Headers/avx512bwintrin.h lib/Headers/avx512fintrin.h lib/Headers/avx512vlbwintrin.h lib/Headers/avx512vlintrin.h

[PATCH] D38737: [X86] test/testn intrinsics lowering to IR. clang side

2017-11-06 Thread Uriel Korach via Phabricator via cfe-commits
uriel.k updated this revision to Diff 121716. https://reviews.llvm.org/D38737 Files: include/clang/Basic/BuiltinsX86.def lib/Headers/avx512bwintrin.h lib/Headers/avx512fintrin.h lib/Headers/avx512vlbwintrin.h lib/Headers/avx512vlintrin.h test/CodeGen/avx512bw-builtins.c

[PATCH] D38737: [X86] test/testn intrinsics lowering to IR. clang side

2017-11-06 Thread Uriel Korach via Phabricator via cfe-commits
uriel.k updated this revision to Diff 121699. uriel.k marked an inline comment as done. https://reviews.llvm.org/D38737 Files: include/clang/Basic/BuiltinsX86.def lib/Headers/avx512bwintrin.h lib/Headers/avx512fintrin.h lib/Headers/avx512vlbwintrin.h lib/Headers/avx512vlintrin.h

[PATCH] D38737: [X86] test/testn intrinsics lowering to IR. clang side

2017-10-10 Thread Uriel Korach via Phabricator via cfe-commits
uriel.k created this revision. Changed Header files of the intrinsics fro lowering test and testn intrinsics to IR code. Removed test and testn builtins from clang this is the clang side, the llvm side revision is D38736 . https://reviews.llvm.org/D38737

[PATCH] D37694: [X86] [PATCH] [intrinsics] Lowering X86 ABS intrinsics to IR. (clang)

2017-09-13 Thread Uriel Korach via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL313133: [X86] [PATCH] [intrinsics] Lowering X86 ABS intrinsics to IR. (clang) (authored by uriel.k). Changed prior to commit: https://reviews.llvm.org/D37694?vs=114613=115001#toc Repository: rL LLVM

[PATCH] D37694: [X86] [PATCH] [intrinsics] Lowering X86 ABS intrinsics to IR. (clang)

2017-09-12 Thread Uriel Korach via Phabricator via cfe-commits
uriel.k added a comment. If I remove the builtins from the .def file this will not work, as I didn't replace the builtin call in the Header files but implemented the lowering by adding cases to the CGBuiltin.cpp and adding the IR by hand. https://reviews.llvm.org/D37694

[PATCH] D37694: [X86] [PATCH] [intrinsics] Lowering X86 ABS intrinsics to IR. (clang)

2017-09-11 Thread Uriel Korach via Phabricator via cfe-commits
uriel.k created this revision. This patch, together with a matching llvm patch (https://reviews.llvm.org/D37693), implements the lowering of X86 ABS intrinsics to IR. https://reviews.llvm.org/D37694 Files: lib/CodeGen/CGBuiltin.cpp test/CodeGen/avx2-builtins.c