[PATCH] D25480: __builtin_fpclassify missing one int parameter

2016-10-14 Thread David Sheinkman via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL284277: __builtin_fpclassify missing one int parameter (authored by davidsh). Changed prior to commit: https://reviews.llvm.org/D25480?vs=74377=74740#toc Repository: rL LLVM

r284277 - __builtin_fpclassify missing one int parameter

2016-10-14 Thread David Sheinkman via cfe-commits
Author: davidsh Date: Fri Oct 14 15:43:37 2016 New Revision: 284277 URL: http://llvm.org/viewvc/llvm-project?rev=284277=rev Log: __builtin_fpclassify missing one int parameter Patch by Tania Albarghouthi. Differential Revision: https://reviews.llvm.org/D25480 Modified:

r283460 - [OpenMP] Check if the template specialization is mappable instead of specialized template Differential Revision: https://reviews.llvm.org/D25252

2016-10-06 Thread David Sheinkman via cfe-commits
Author: davidsh Date: Thu Oct 6 10:47:36 2016 New Revision: 283460 URL: http://llvm.org/viewvc/llvm-project?rev=283460=rev Log: [OpenMP] Check if the template specialization is mappable instead of specialized template Differential Revision: https://reviews.llvm.org/D25252 Modified:

[PATCH] D25252: [OpenMP] Check if the template specialization is mappable instead of specialized template

2016-10-05 Thread David Sheinkman via cfe-commits
davidsh created this revision. davidsh added reviewers: carlo.bertolli, arpith-jacob, kkwli0, sfantao, ABataev. davidsh added a subscriber: cfe-commits. Clang segfaults on this case instead of diagnosing because it looks at the specialized template: template struct MyClass; template<> struct

[PATCH] D24615: [OpenMP] clang doesnt diagnose if there is a lexical block around a for stmt for OpenMP loops. It is technically not allowed in the OpenMP standard

2016-10-04 Thread David Sheinkman via cfe-commits
davidsh updated this revision to Diff 73520. davidsh added a comment. Adding a parameter to IgnoreContainers instead of copying the logic. https://reviews.llvm.org/D24615 Files: include/clang/AST/Stmt.h lib/AST/Stmt.cpp lib/Sema/SemaOpenMP.cpp test/OpenMP/for_loop_messages.cpp Index:

r283223 - [OpenMP] fix segfault when a variable referenced in reduction clause is a reference parameter\nDifferential Revision: http://reviews.llvm.org/D24524

2016-10-04 Thread David Sheinkman via cfe-commits
Author: davidsh Date: Tue Oct 4 09:41:36 2016 New Revision: 283223 URL: http://llvm.org/viewvc/llvm-project?rev=283223=rev Log: [OpenMP] fix segfault when a variable referenced in reduction clause is a reference parameter\nDifferential Revision: http://reviews.llvm.org/D24524 Modified:

[PATCH] D24615: [OpenMP] clang doesnt diagnose if there is a lexical block around a for stmt for OpenMP loops. It is technically not allowed in the OpenMP standard

2016-09-15 Thread David Sheinkman via cfe-commits
davidsh created this revision. davidsh added reviewers: carlo.bertolli, arpith-jacob, kkwli0, sfantao, ABataev. davidsh added a subscriber: cfe-commits. #pragma omp for { for(...) } This is technically not allowed by the standard, gcc doesnt allow such code. https://reviews.llvm.org/D24615