Re: [PATCH] D19835: Tolerate incorrect return type for 'isinf' and 'isnan' in tests.

2016-05-27 Thread Eric Fiselier via cfe-commits
EricWF accepted this revision. EricWF added a reviewer: EricWF. EricWF added a comment. This revision is now accepted and ready to land. Accepting to get the test suite passing on newer linux distributions again. I tried modifying the tests to test the GLIBC version but IMO it was more complexit

[libcxx] r271060 - Tolerate incorrect return type for 'isinf' and 'isnan' in tests.

2016-05-27 Thread Eric Fiselier via cfe-commits
Author: ericwf Date: Fri May 27 17:19:53 2016 New Revision: 271060 URL: http://llvm.org/viewvc/llvm-project?rev=271060&view=rev Log: Tolerate incorrect return type for 'isinf' and 'isnan' in tests. Summary: GLIBC recently removed the incorrect `int isinf(double)` and `int isnan(double)` overload

Re: [PATCH] D20389: NVPTX: Add supported CL features

2016-05-27 Thread Jan Vesely via cfe-commits
jvesely updated the summary for this revision. jvesely added a subscriber: cfe-commits. jvesely updated this revision to Diff 58849. jvesely added a comment. Report only clinfo exported extensions. Repository: rL LLVM http://reviews.llvm.org/D20389 Files: lib/Basic/Targets.cpp test/Misc/

r271063 - docs: Clarify that LTO visibility is based on source-level symbol visibility.

2016-05-27 Thread Peter Collingbourne via cfe-commits
Author: pcc Date: Fri May 27 17:31:30 2016 New Revision: 271063 URL: http://llvm.org/viewvc/llvm-project?rev=271063&view=rev Log: docs: Clarify that LTO visibility is based on source-level symbol visibility. Modified: cfe/trunk/docs/LTOVisibility.rst Modified: cfe/trunk/docs/LTOVisibility.rs

Re: [PATCH] D20388: AMDGPU: Fix supported CL features

2016-05-27 Thread Jan Vesely via cfe-commits
jvesely marked an inline comment as done. jvesely added a comment. Repository: rL LLVM http://reviews.llvm.org/D20388 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D20388: AMDGPU: Fix supported CL features

2016-05-27 Thread Jan Vesely via cfe-commits
jvesely updated the summary for this revision. jvesely added a subscriber: cfe-commits. jvesely updated this revision to Diff 58850. jvesely marked an inline comment as done. jvesely added a comment. report only exported extensions. int64 atomics, fp16, and 3d image writes are actually not curren

[libcxx] r271065 - Guard testing of tuple extensions to make tests portable

2016-05-27 Thread Eric Fiselier via cfe-commits
Author: ericwf Date: Fri May 27 17:41:45 2016 New Revision: 271065 URL: http://llvm.org/viewvc/llvm-project?rev=271065&view=rev Log: Guard testing of tuple extensions to make tests portable Modified: libcxx/trunk/test/std/utilities/tuple/tuple.tuple/tuple.cnstr/UTypes.pass.cpp libcxx/tr

Re: [PATCH] D20388: AMDGPU: Fix supported CL features

2016-05-27 Thread Matt Arsenault via cfe-commits
arsenm added inline comments. Comment at: test/Misc/r600.languageOptsOpenCL.cl:4 @@ +3,3 @@ +// RUN: %clang_cc1 -x cl -cl-std=CL %s -verify -triple r600-unknown-unknown -target-cpu cypress +// RUN: %clang_cc1 -x cl -cl-std=CL %s -verify -triple r600-unknown-unknown -target-cpu t

[libcxx] r271070 - Fix PR27875. Parse LIBCXX_CXX_ABI_LIBRARY_PATH as a path not a string

2016-05-27 Thread Eric Fiselier via cfe-commits
Author: ericwf Date: Fri May 27 18:05:37 2016 New Revision: 271070 URL: http://llvm.org/viewvc/llvm-project?rev=271070&view=rev Log: Fix PR27875. Parse LIBCXX_CXX_ABI_LIBRARY_PATH as a path not a string Modified: libcxx/trunk/cmake/Modules/HandleLibCXXABI.cmake Modified: libcxx/trunk/cmake/M

[libcxx] r271073 - Fix PR27751. Add proper dependancies for install-libcxx-headers rule

2016-05-27 Thread Eric Fiselier via cfe-commits
Author: ericwf Date: Fri May 27 18:33:10 2016 New Revision: 271073 URL: http://llvm.org/viewvc/llvm-project?rev=271073&view=rev Log: Fix PR27751. Add proper dependancies for install-libcxx-headers rule Modified: libcxx/trunk/include/CMakeLists.txt Modified: libcxx/trunk/include/CMakeLists.tx

[libcxx] r271074 - Add missing include to test_iterators.h

2016-05-27 Thread Eric Fiselier via cfe-commits
Author: ericwf Date: Fri May 27 18:43:29 2016 New Revision: 271074 URL: http://llvm.org/viewvc/llvm-project?rev=271074&view=rev Log: Add missing include to test_iterators.h Modified: libcxx/trunk/test/support/test_iterators.h Modified: libcxx/trunk/test/support/test_iterators.h URL: http://

Re: [PATCH] D20640: [AMDGPU] Set default dwarf version to 2

2016-05-27 Thread Tom Stellard via cfe-commits
tstellarAMD accepted this revision. tstellarAMD added a comment. This revision is now accepted and ready to land. LGTM. http://reviews.llvm.org/D20640 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/list

Re: [PATCH] D20614: Remove trailing spaces in x86 intrinsic headers

2016-05-27 Thread Katya Romanova via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL271077: Clean up: remove trailing spaces in x86 intrinsic headers. (authored by kromanova). Changed prior to commit: http://reviews.llvm.org/D20614?vs=58472&id=58869#toc Repository: rL LLVM http://r

r271077 - Clean up: remove trailing spaces in x86 intrinsic headers.

2016-05-27 Thread Ekaterina Romanova via cfe-commits
Author: kromanova Date: Fri May 27 19:18:59 2016 New Revision: 271077 URL: http://llvm.org/viewvc/llvm-project?rev=271077&view=rev Log: Clean up: remove trailing spaces in x86 intrinsic headers. Differential Revision: http://reviews.llvm.org/D20614 Modified: cfe/trunk/lib/Headers/__wmmintri

r271080 - AMDGPU: Add fract builtin

2016-05-27 Thread Matt Arsenault via cfe-commits
Author: arsenm Date: Fri May 27 19:43:27 2016 New Revision: 271080 URL: http://llvm.org/viewvc/llvm-project?rev=271080&view=rev Log: AMDGPU: Add fract builtin Modified: cfe/trunk/include/clang/Basic/BuiltinsAMDGPU.def cfe/trunk/lib/CodeGen/CGBuiltin.cpp cfe/trunk/test/CodeGenOpenCL/bu

[PATCH] D20766: [clang-tidy] Fix script adding new clang-tidy check

2016-05-27 Thread Etienne Bergeron via cfe-commits
etienneb created this revision. etienneb added a reviewer: alexfh. etienneb added a subscriber: cfe-commits. The `getName()` call is useless. It's better to show a better example as tutorial. http://reviews.llvm.org/D20766 Files: clang-tidy/add_new_check.py Index: clang-tidy/add_new_check.py

Re: [PATCH] D20451: [Parser] Fix look ahead after EOF while parsing objc message and lambdas

2016-05-27 Thread Doug Gregor via cfe-commits
doug.gregor accepted this revision. doug.gregor added a comment. This revision is now accepted and ready to land. LGTM, sorry for the delay! http://reviews.llvm.org/D20451 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org

Re: [PATCH] D20730: [libcxx] Prefer UNSUPPORTED over conditional compilation for tests

2016-05-27 Thread Marshall Clow via cfe-commits
mclow.lists accepted this revision. mclow.lists added a reviewer: mclow.lists. mclow.lists added a comment. LGTM. http://reviews.llvm.org/D20730 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/c

[PATCH] Fix stack overflow in MSVC unqualified type lookup logic

2016-05-27 Thread Will Wilson via cfe-commits
Hi Alexey, A customer encountered a stack overflow in the code from r229817. I've created a small repro for testing the issue and a fix. The fix should also better deal with lookup into partially specialized base templates. Test case included and tested against latest trunk. Let me know if it lo

[PATCH] D20767: [ASTMatchers] Add support of hasCondition for SwitchStmt.

2016-05-27 Thread Etienne Bergeron via cfe-commits
etienneb created this revision. etienneb added reviewers: aaron.ballman, sbenza. etienneb added a subscriber: cfe-commits. Herald added a subscriber: klimek. The switch statement could be added to the hasCondition matcher. Example: ``` clang-query> match switchStmt(hasCondition(ignoringImpCasts(d

Re: [PATCH] D20766: [clang-tidy] Fix script adding new clang-tidy check

2016-05-27 Thread Alexander Kornienko via cfe-commits
alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. LG http://reviews.llvm.org/D20766 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm

<    1   2