[PATCH] D39817: [CUDA] Fix std::min on device side to return the min, not the max.

2017-11-10 Thread Justin Lebar via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL317961: [CUDA] Fix std::min on device side to return the min, not the max. (authored by jlebar). Changed prior to commit: https://reviews.llvm.org/D39817?vs=122151&id=122563#toc Repository: rL LLVM

[PATCH] D39818: [CUDA] [test-suite] Test std::min and std::max with C++11.

2017-11-10 Thread Justin Lebar via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL317962: [CUDA] [test-suite] Test std::min and std::max with C++11. (authored by jlebar). Repository: rL LLVM https://reviews.llvm.org/D39818 Files: test-suite/trunk/External/CUDA/algorithm.cu Inde

[PATCH] D39050: Add index-while-building support to Clang

2017-11-10 Thread Argyrios Kyrtzidis via Phabricator via cfe-commits
akyrtzi added a comment. Hey Eric, In https://reviews.llvm.org/D39050#921748, @ioeric wrote: > >> - I think the implementation of the index output logic (e.g. > >> `IndexUnitWriter` and bit format file) can be abstracted away (and split > >> into separate patches) so that you can unit-test the

[PATCH] D39706: [refactor][extract] Initial implementation of variable captures

2017-11-10 Thread Alex Lorenz via Phabricator via cfe-commits
arphaman updated this revision to Diff 122565. arphaman marked 3 inline comments as done. arphaman added a comment. Address review comments Repository: rL LLVM https://reviews.llvm.org/D39706 Files: lib/Tooling/Refactoring/CMakeLists.txt lib/Tooling/Refactoring/Extract/CaptureVariables.c

[PATCH] D39706: [refactor][extract] Initial implementation of variable captures

2017-11-10 Thread Alex Lorenz via Phabricator via cfe-commits
arphaman added inline comments. Comment at: lib/Tooling/Refactoring/Extract/CaptureVariables.h:36 + explicit CapturedExtractedEntity(const VarDecl *VD) + : Kind(CapturedVarDecl), VD(VD) {} + ioeric wrote: > arphaman wrote: > > ioeric wrote: > > > Maybe a `F

r317960 - Add CLANG_DEFAULT_OBJCOPY to allow Clang to use llvm-objcopy for dwarf fission

2017-11-10 Thread Jake Ehrlich via cfe-commits
Author: jakehehrlich Date: Fri Nov 10 17:15:41 2017 New Revision: 317960 URL: http://llvm.org/viewvc/llvm-project?rev=317960&view=rev Log: Add CLANG_DEFAULT_OBJCOPY to allow Clang to use llvm-objcopy for dwarf fission llvm-objcopy is getting to where it can be used in non-trivial ways (such as fo

[PATCH] D39768: [coroutines] Promote cleanup.dest.slot allocas to registers to avoid storing it in the coroutine frame

2017-11-10 Thread Eric Niebler via Phabricator via cfe-commits
eric_niebler accepted this revision. eric_niebler added a comment. This revision is now accepted and ready to land. Looks ok to me. https://reviews.llvm.org/D39768 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin

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

2017-11-10 Thread Craig Topper via Phabricator via cfe-commits
craig.topper accepted this revision. craig.topper added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D38737 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/li

[PATCH] D39936: [OpenCL] Add extensions cl_intel_subgroups and cl_intel_subgroups_short

2017-11-10 Thread Alexey Sotkin via Phabricator via cfe-commits
AlexeySotkin created this revision. https://reviews.llvm.org/D39936 Files: include/clang/Basic/OpenCLExtensions.def lib/Headers/opencl-c.h test/SemaOpenCL/extension-version.cl Index: test/SemaOpenCL/extension-version.cl === --

<    1   2