r299919 - Add more examples to clang-format configuration

Author: sylvestre Date: Tue Apr 11 02:07:05 2017 New Revision: 299919 URL: http://llvm.org/viewvc/llvm-project?rev=299919&view=rev Log: Add more examples to clang-format configuration Reviewers: djasper Reviewed By: djasper Subscribers: Eugene.Zelenko, klimek, cfe-commits Differential Revision

[clang-tools-extra] r299920 - Add more examples to clang tidy checkers

Author: sylvestre Date: Tue Apr 11 02:10:48 2017 New Revision: 299920 URL: http://llvm.org/viewvc/llvm-project?rev=299920&view=rev Log: Add more examples to clang tidy checkers Reviewers: alexfh Reviewed By: alexfh Subscribers: cfe-commits Tags: #clang-tools-extra Differential Revision: https

r299921 - [lsan] Enable LSan on arm Linux, clang part

Author: chefmax Date: Tue Apr 11 02:22:11 2017 New Revision: 299921 URL: http://llvm.org/viewvc/llvm-project?rev=299921&view=rev Log: [lsan] Enable LSan on arm Linux, clang part This is a compiler part of https://reviews.llvm.org/D29586. Enable LSan on arm Linux. Differential Revision: https://

[PATCH] D31760: [lsan] Enable LSan on arm Linux, clang part

This revision was automatically updated to reflect the committed changes. Closed by commit rL299921: [lsan] Enable LSan on arm Linux, clang part (authored by chefmax). Changed prior to commit: https://reviews.llvm.org/D31760?vs=94708&id=94787#toc Repository: rL LLVM https://reviews.llvm.org

[PATCH] D31853: [clangd] Implement item kind for completion results

krasimir accepted this revision. krasimir added a comment. This revision is now accepted and ready to land. Looks good! Thank you! https://reviews.llvm.org/D31853 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/

[PATCH] D31860: Add more examples to clang tidy checkers

alexfh added a comment. Another couple of post-commit comments. Comment at: docs/clang-tidy/checks/misc-unused-parameters.rst:15 + + void a(int /*i*/) {} + nit: two spaces before the comment. Comment at: docs/clang-tidy/checks/readability-u

[PATCH] D31887: [clangd] Add documentation page

xazax.hun added a comment. Maybe this is a good time to decide ClangD, Clangd, or clangd is the correct capitalization. Repository: rL LLVM https://reviews.llvm.org/D31887 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm

[PATCH] D31862: docs/clang-tidy/tools/dump_check_docs.py: Remove deprecated script

alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. LG https://reviews.llvm.org/D31862 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-com

[PATCH] D31417: [OpenMP] Add support for omp simd pragmas without runtime

huntergr added inline comments. Comment at: lib/Parse/ParseOpenMP.cpp:174 + case OMPD_target_teams_distribute_simd: +DKind = OMPD_simd; +break; ABataev wrote: > huntergr wrote: > > rengolin wrote: > > > I'd like @ABataev to confirm this is th

r299927 - [clang-format] Handle NSString literals by merging tokens.

Author: alexfh Date: Tue Apr 11 04:55:00 2017 New Revision: 299927 URL: http://llvm.org/viewvc/llvm-project?rev=299927&view=rev Log: [clang-format] Handle NSString literals by merging tokens. Summary: This fixes a few outstanding bugs: * incorrect breaking of NSString literals containing double

[PATCH] D31706: [clang-format] Handle NSString literals by merging tokens.

This revision was automatically updated to reflect the committed changes. Closed by commit rL299927: [clang-format] Handle NSString literals by merging tokens. (authored by alexfh). Changed prior to commit: https://reviews.llvm.org/D31706?vs=94218&id=94801#toc Repository: rL LLVM https://re

r299930 - Warn about unused static file scope function template declarations.

Author: vvassilev Date: Tue Apr 11 05:13:54 2017 New Revision: 299930 URL: http://llvm.org/viewvc/llvm-project?rev=299930&view=rev Log: Warn about unused static file scope function template declarations. Reviewed by Richard Smith (D29877)! Modified: cfe/trunk/lib/Sema/Sema.cpp cfe/trunk

[PATCH] D29877: Warn about unused static file scope function template declarations.

v.g.vassilev closed this revision. v.g.vassilev marked 5 inline comments as done. v.g.vassilev added a comment. Landed in r299930. https://reviews.llvm.org/D29877 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/

[PATCH] D31887: [clangd] Add documentation page

kromanenkov added inline comments. Comment at: docs/clangd.rst:10 + +:program:`clangd` is an implementation of the `Languge Server Protocol `_ leveraging Clang. +Clangd's goal is to provide language "smartness" features like

[PATCH] D31413: [libc++] Use __attribute__((init_priority(101))) to ensure streams get initialized early

hfinkel added a comment. In https://reviews.llvm.org/D31413#712013, @aaron.ballman wrote: > I'm not certain of a good way to test it, but I have a question about the > value you picked for `init_priority`. My understanding of the values starting > from 101 is that 1-100 are reserved for impleme

[PATCH] D31887: [clangd] Add documentation page

JonasToth added inline comments. Comment at: docs/clangd.rst:16 + +Clangd is not meant to be used by C/C++ developpers directly but rather from a client implementing the protocol. +A client would be typically implemented in an IDE or an editor. kromanenkov wrote

[PATCH] D24892: [clang-tidy] Add option "LiteralInitializers" to cppcoreguidelines-pro-type-member-init

malcolm.parsons added a comment. The `modernize-use-default-member-init` check now has an option with the same effect, but it is called `UseAssignment`. We should use consistent option names. Is there any way for multiple checks to share an option? https://reviews.llvm.org/D24892 ___

[PATCH] D31853: [clangd] Implement item kind for completion results

krasimir added a comment. Do you still need someone to commit this? https://reviews.llvm.org/D31853 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D31153: Add the ability to use the children() range API in a const-correct manner

aaron.ballman updated this revision to Diff 94809. aaron.ballman added a comment. Addressing David's review feedback. https://reviews.llvm.org/D31153 Files: include/clang/AST/Expr.h include/clang/AST/StmtIterator.h lib/AST/Expr.cpp Index: lib/AST/Expr.cpp

Re: [PATCH] D31853: [clangd] Implement item kind for completion results

Thanks for the review! Yes, unfortunately I still need someone to commit it. On Apr 11, 2017 15:21, "Krasimir Georgiev via Phabricator" < revi...@reviews.llvm.org> wrote: krasimir added a comment. Do you still need someone to commit this? https://reviews.llvm.org/D31853 __

[PATCH] D22638: Module: add debug_type to dump debugging messages related to modules being out of date

v.g.vassilev added a comment. How do we proceed with this. Shall we close it? https://reviews.llvm.org/D22638 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D29877: Warn about unused static file scope function template declarations.

chapuni added a comment. This brought warnings in llvm tree. Could you take a look, please? http://bb.pgr.jp/builders/clang-3stage-x86_64-linux/builds/14936 (w/o asserts) llvm/include/llvm/ADT/PointerUnion.h:193:13: warning: unused function 'operator==' [-Wunused-function] llvm/include/llvm

[clang-tools-extra] r299935 - [clangd] Implement item kind for completion results

Author: krasimir Date: Tue Apr 11 08:27:15 2017 New Revision: 299935 URL: http://llvm.org/viewvc/llvm-project?rev=299935&view=rev Log: [clangd] Implement item kind for completion results Summary: The patch implements the conversion method from CXCursorKind to clangd::CompletionItemKind. Contribu

[PATCH] D31853: [clangd] Implement item kind for completion results

This revision was automatically updated to reflect the committed changes. Closed by commit rL299935: [clangd] Implement item kind for completion results (authored by krasimir). Changed prior to commit: https://reviews.llvm.org/D31853?vs=94720&id=94811#toc Repository: rL LLVM https://reviews

[PATCH] D29877: Warn about unused static file scope function template declarations.

v.g.vassilev added a comment. Thanks for pinging us. Yeah, I knew them but I was hesitant whether I should fix them or they should be fixed by the code owners. Shall I commit the fixes? https://reviews.llvm.org/D29877 ___ cfe-commits mailing list c

[PATCH] D29877: Warn about unused static file scope function template declarations.

chapuni added a comment. I think yes, you may do. Warnings fix may be trivial as far as they are NFC. It'd be fine that clang tree should be free from warnings with just-built clang. https://reviews.llvm.org/D29877 ___ cfe-commits mailing list cfe-

[PATCH] D29877: Warn about unused static file scope function template declarations.

arphaman added a comment. The operators in PointerUnion seem useful. I don't think they should be removed even if they're not used in tree, since they might be used by some out-of-tree code. https://reviews.llvm.org/D29877 ___ cfe-commits mailing

[PATCH] D29877: Warn about unused static file scope function template declarations.

v.g.vassilev added a comment. @arphaman, it seems they are marked as static and this seemed like a bug to @rsmith. https://reviews.llvm.org/D29877 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinf

[PATCH] D29877: Warn about unused static file scope function template declarations.

arphaman added a comment. Ah, I see. I guess if `static` is removed the warning will go away. Thanks! https://reviews.llvm.org/D29877 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D31757: [clang-tidy] Add a clang-tidy check for possible inefficient vector operations

aaron.ballman added inline comments. Comment at: clang-tidy/performance/InefficientVectorOperationCheck.cpp:53-54 +PushBackCall)), + hasParent(compoundStmt(unless(has(ReserveCall)), + has(VectorVarDefStmt +

[PATCH] D30691: [analyzer] Support for naive cross translational unit analysis

xazax.hun updated this revision to Diff 94814. xazax.hun edited the summary of this revision. xazax.hun added a comment. - Removed a clang tool, replaced with python tool functionality. https://reviews.llvm.org/D30691 Files: include/clang/AST/ASTContext.h include/clang/AST/Decl.h include/

[libcxx] r299941 - Mark P0599 as complete. It was implemented in r298573

Author: marshall Date: Tue Apr 11 09:04:03 2017 New Revision: 299941 URL: http://llvm.org/viewvc/llvm-project?rev=299941&view=rev Log: Mark P0599 as complete. It was implemented in r298573 Modified: libcxx/trunk/www/cxx1z_status.html Modified: libcxx/trunk/www/cxx1z_status.html URL: http://

[libcxx] r299942 - [libc++] Fix unknown pragma warning on MSVC

Author: bcraig Date: Tue Apr 11 09:06:39 2017 New Revision: 299942 URL: http://llvm.org/viewvc/llvm-project?rev=299942&view=rev Log: [libc++] Fix unknown pragma warning on MSVC Modified: libcxx/trunk/include/limits Modified: libcxx/trunk/include/limits URL: http://llvm.org/viewvc/llvm-proje

[PATCH] D31404: [OpenCL] Allow alloca return non-zero private pointer

yaxunl marked 2 inline comments as done. yaxunl added a comment. Any further comments? Thanks. https://reviews.llvm.org/D31404 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D31887: [clangd] Add documentation page

malaperle-ericsson updated this revision to Diff 94815. malaperle-ericsson marked 6 inline comments as done. malaperle-ericsson added a comment. Fixed typos and other comments. https://reviews.llvm.org/D31887 Files: docs/clangd.rst docs/index.rst Index: docs/index.rst =

[PATCH] D31887: [clangd] Add documentation page

malaperle-ericsson added a comment. Sorry about all the typos. I have now installed a spell checked in VS Code :) https://reviews.llvm.org/D31887 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/

[PATCH] D29905: [OpenMP] Pass argument to device kernel by reference when map is used.

gtbercea marked an inline comment as done. gtbercea added inline comments. Comment at: lib/Sema/SemaOpenMP.cpp:358-360 + /// Do the check specified in \a Check to all component lists at a given level + /// and return true if any issue is found. + bool checkMappableExprCompone

Re: [PATCH] D29877: Warn about unused static file scope function template declarations.

On Tue, Apr 11, 2017 at 6:59 AM, Alex Lorenz via Phabricator via cfe-commits wrote: > arphaman added a comment. > > Ah, I see. I guess if `static` is removed the warning will go away. Thanks! > > https://reviews.llvm.org/D29877 > > I would have appreciated a heads-up here, since libc++ no longer

[PATCH] D31652: [clang-format] Recognize Java logical shift assignment operator

bradfier added a comment. Friendly ping. Repository: rL LLVM https://reviews.llvm.org/D31652 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D31757: [clang-tidy] Add a clang-tidy check for possible inefficient vector operations

hokein added inline comments. Comment at: clang-tidy/performance/InefficientVectorOperationCheck.cpp:53-54 +PushBackCall)), + hasParent(compoundStmt(unless(has(ReserveCall)), + has(VectorVarDefStmt + .b

[PATCH] D29877: Warn about unused static file scope function template declarations.

mclow.lists added a comment. The following idiom for detecting member typedefs now throws an warning: struct __two {char __lx; char __lxx;}; namespace __has_pointer_type_imp { template __two __test(...); template char __test(typename _Up::pointer* = 0); } template

[PATCH] D29877: Warn about unused static file scope function template declarations.

v.g.vassilev added a comment. @chapuni, the LLVM warnings should be fixed in r299947. @mclow.lists, sorry for not giving heads up :( I am working on the false positive that you reported. https://reviews.llvm.org/D29877 ___ cfe-commits mailing list

r299950 - [Parser][ObjC++] Improve diagnostics and recovery when C++ keywords are used

Author: arphaman Date: Tue Apr 11 10:01:53 2017 New Revision: 299950 URL: http://llvm.org/viewvc/llvm-project?rev=299950&view=rev Log: [Parser][ObjC++] Improve diagnostics and recovery when C++ keywords are used as identifiers in Objective-C++ This commit improves the 'expected identifier' errors

[PATCH] D26503: [Parser][ObjC] Improve diagnostics and recovery when C++ keywords are used as identifiers in Objective-C++

This revision was automatically updated to reflect the committed changes. Closed by commit rL299950: [Parser][ObjC++] Improve diagnostics and recovery when C++ keywords are used (authored by arphaman). Changed prior to commit: https://reviews.llvm.org/D26503?vs=78218&id=94823#toc Repository:

[PATCH] D29877: Warn about unused static file scope function template declarations.

v.g.vassilev added a comment. @mclow.lists, hm... it seems that I cannot reproduce it. It'd really help if you could paste a standalone reproducer. https://reviews.llvm.org/D29877 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists

[PATCH] D29660: [OpenMP] Add flag for overwriting default PTX version for OpenMP targets

gtbercea updated this revision to Diff 94824. gtbercea added a comment. Integrate review. Repository: rL LLVM https://reviews.llvm.org/D29660 Files: include/clang/Driver/Options.td lib/Driver/ToolChains/Cuda.cpp test/Driver/openmp-offload.c Index: test/Driver/openmp-offload.c ===

[PATCH] D31652: [clang-format] Recognize Java logical shift assignment operator

thakis accepted this revision. thakis added a comment. This revision is now accepted and ready to land. Looks good. Do you have commit access? Repository: rL LLVM https://reviews.llvm.org/D31652 ___ cfe-commits mailing list cfe-commits@lists.llvm

[PATCH] D31757: [clang-tidy] Add a clang-tidy check for possible inefficient vector operations

aaron.ballman added inline comments. Comment at: clang-tidy/performance/InefficientVectorOperationCheck.cpp:53-54 +PushBackCall)), + hasParent(compoundStmt(unless(has(ReserveCall)), + has(VectorVarDefStmt +

[PATCH] D31652: [clang-format] Recognize Java logical shift assignment operator

bradfier added a comment. In https://reviews.llvm.org/D31652#723664, @thakis wrote: > Looks good. Do you have commit access? Thanks! And no I don't have commit access. Repository: rL LLVM https://reviews.llvm.org/D31652 ___ cfe-commits mailing

[PATCH] D29905: [OpenMP] Pass argument to device kernel by reference when map is used.

ABataev added inline comments. Comment at: lib/Sema/SemaOpenMP.cpp:358-360 + /// Do the check specified in \a Check to all component lists at a given level + /// and return true if any issue is found. + bool checkMappableExprComponentListsForDeclAtLevel( gtbe

[PATCH] D29660: [OpenMP] Add flag for overwriting default PTX version for OpenMP targets

ABataev added inline comments. Comment at: lib/Driver/ToolChains/Cuda.cpp:499 +options::OPT_fopenmp_ptx_EQ); +CC1Args.back() = (!PtxVersion.empty()) ? PtxVersion.data() : "+ptx42"; + } else No, use `CC1Args.push_back()` here, or `CC1Args.emplace_back

[PATCH] D29877: Warn about unused static file scope function template declarations.

mclow.lists added a comment. Complete reproducer: // Tested with with: clang++ -std=c++14 -Wunused-function UnusedFVassily.cpp // // UnusedFVassily.cpp:8:39: warning: unused function '__test' [-Wunused-function] // template static __two __test(...); //

[PATCH] D31417: [OpenMP] Add support for omp simd pragmas without runtime

ABataev added inline comments. Comment at: lib/Parse/ParseOpenMP.cpp:174 + case OMPD_target_teams_distribute_simd: +DKind = OMPD_simd; +break; huntergr wrote: > ABataev wrote: > > huntergr wrote: > > > rengolin wrote: > > > > I'd like @ABatae

r299951 - Fix PR13910: Don't warn that __builtin_unreachable() is unreachable

Author: arphaman Date: Tue Apr 11 10:36:06 2017 New Revision: 299951 URL: http://llvm.org/viewvc/llvm-project?rev=299951&view=rev Log: Fix PR13910: Don't warn that __builtin_unreachable() is unreachable Differential Revision: https://reviews.llvm.org/D25321 Modified: cfe/trunk/lib/Analysis/R

[PATCH] D25321: Fix PR13910: Don't warn that __builtin_unreachable() is unreachable

This revision was automatically updated to reflect the committed changes. Closed by commit rL299951: Fix PR13910: Don't warn that __builtin_unreachable() is unreachable (authored by arphaman). Changed prior to commit: https://reviews.llvm.org/D25321?vs=73898&id=94834#toc Repository: rL LLVM

r299952 - [clang-format] Recognize Java logical shift assignment operator

Author: nico Date: Tue Apr 11 10:50:04 2017 New Revision: 299952 URL: http://llvm.org/viewvc/llvm-project?rev=299952&view=rev Log: [clang-format] Recognize Java logical shift assignment operator At present, clang-format mangles Java containing logical right shift operators ('>>>=' or '>>>'), spl

[PATCH] D31652: [clang-format] Recognize Java logical shift assignment operator

thakis closed this revision. thakis added a comment. Landed in r299952: http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-20170410/189882.html Repository: rL LLVM https://reviews.llvm.org/D31652 ___ cfe-commits mailing list cfe-commits@lis

[PATCH] D31404: [OpenCL] Allow alloca return non-zero private pointer

Anastasia accepted this revision. Anastasia added a comment. LGTM! Thanks! https://reviews.llvm.org/D31404 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r299956 - Revert temporarily D29877 "Warn about unused static file scope function template declarations."

Author: vvassilev Date: Tue Apr 11 11:05:23 2017 New Revision: 299956 URL: http://llvm.org/viewvc/llvm-project?rev=299956&view=rev Log: Revert temporarily D29877 "Warn about unused static file scope function template declarations." We need to address cases (breaking libc++) such as template st

[PATCH] D30863: [clang-format] make docs/tools/{dump_format_style.py, dump_ast_matchers.py} flake8 compliant

sylvestre.ledru added a comment. Manuel, is that ok with you? https://reviews.llvm.org/D30863 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D29660: [OpenMP] Add flag for overwriting default PTX version for OpenMP targets

gtbercea updated this revision to Diff 94841. gtbercea added a comment. Refactor. Repository: rL LLVM https://reviews.llvm.org/D29660 Files: include/clang/Driver/Options.td lib/Driver/ToolChains/Cuda.cpp test/Driver/openmp-offload.c Index: test/Driver/openmp-offload.c ===

[PATCH] D29877: Warn about unused static file scope function template declarations.

v.g.vassilev added a comment. Reverted in r299956, due to failures like: template static int __test(...); template auto v = __test<_Tp>(0); https://reviews.llvm.org/D29877 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lis

[PATCH] D29660: [OpenMP] Add flag for overwriting default PTX version for OpenMP targets

gtbercea updated this revision to Diff 94842. gtbercea added a comment. Run clang format. Repository: rL LLVM https://reviews.llvm.org/D29660 Files: include/clang/Driver/Options.td lib/Driver/ToolChains/Cuda.cpp test/Driver/openmp-offload.c Index: test/Driver/openmp-offload.c ===

[PATCH] D31167: Use FPContractModeKind universally

yaxunl added inline comments. Comment at: cfe/trunk/include/clang/Basic/LangOptions.def:220 +/// \brief FP_CONTRACT mode (on/off/fast). +ENUM_LANGOPT(DefaultFPContractMode, FPContractModeKind, 2, FPC_Off, "FP contraction type") LANGOPT(NoBitFieldTypeAlign , 1, 0, "bit-field typ

[clang-tools-extra] r299961 - Add the definition of P in the clang tidy example

Author: sylvestre Date: Tue Apr 11 11:28:15 2017 New Revision: 299961 URL: http://llvm.org/viewvc/llvm-project?rev=299961&view=rev Log: Add the definition of P in the clang tidy example Modified: clang-tools-extra/trunk/docs/clang-tidy/checks/readability-uniqueptr-delete-release.rst Modifie

[PATCH] D31860: Add more examples to clang tidy checkers

sylvestre.ledru added inline comments. Comment at: docs/clang-tidy/checks/misc-unused-parameters.rst:15 + + void a(int /*i*/) {} + alexfh wrote: > nit: two spaces before the comment. I believe it is the case ? I stole this from the unit test //test/clang-tidy/

[PATCH] D31153: Add the ability to use the children() range API in a const-correct manner

dblaikie added inline comments. Comment at: include/clang/AST/Expr.h:4025 child_range children() { +const_child_range CCR = const_cast(this)->children(); +return child_range(cast_away_const(CCR.begin()), If this is adding const, can you use a weaker ca

[PATCH] D31167: Use FPContractModeKind universally

hfinkel added inline comments. Comment at: cfe/trunk/include/clang/Basic/LangOptions.def:220 +/// \brief FP_CONTRACT mode (on/off/fast). +ENUM_LANGOPT(DefaultFPContractMode, FPContractModeKind, 2, FPC_Off, "FP contraction type") LANGOPT(NoBitFieldTypeAlign , 1, 0, "bit-field ty

[PATCH] D31167: Use FPContractModeKind universally

yaxunl added inline comments. Comment at: cfe/trunk/include/clang/Basic/LangOptions.def:220 +/// \brief FP_CONTRACT mode (on/off/fast). +ENUM_LANGOPT(DefaultFPContractMode, FPContractModeKind, 2, FPC_Off, "FP contraction type") LANGOPT(NoBitFieldTypeAlign , 1, 0, "bit-field typ

r299962 - [ASTPrinter] Print nested name specifiers for out-of-line functions

Author: arphaman Date: Tue Apr 11 11:46:03 2017 New Revision: 299962 URL: http://llvm.org/viewvc/llvm-project?rev=299962&view=rev Log: [ASTPrinter] Print nested name specifiers for out-of-line functions rdar://31501863 Added: cfe/trunk/test/Misc/ast-print-out-of-line-func.cpp Modified: c

[PATCH] D31167: Use FPContractModeKind universally

hfinkel added inline comments. Comment at: cfe/trunk/include/clang/Basic/LangOptions.def:220 +/// \brief FP_CONTRACT mode (on/off/fast). +ENUM_LANGOPT(DefaultFPContractMode, FPContractModeKind, 2, FPC_Off, "FP contraction type") LANGOPT(NoBitFieldTypeAlign , 1, 0, "bit-field ty

[PATCH] D31167: Use FPContractModeKind universally

anemet added inline comments. Comment at: cfe/trunk/include/clang/Basic/LangOptions.def:220 +/// \brief FP_CONTRACT mode (on/off/fast). +ENUM_LANGOPT(DefaultFPContractMode, FPContractModeKind, 2, FPC_Off, "FP contraction type") LANGOPT(NoBitFieldTypeAlign , 1, 0, "bit-field typ

[libcxx] r299963 - Implement LWG#2873: 'Add noexcept to several shared_ptr related functions' This issue missed a couple, so I added those as well (see LWG#2942)

Author: marshall Date: Tue Apr 11 12:08:53 2017 New Revision: 299963 URL: http://llvm.org/viewvc/llvm-project?rev=299963&view=rev Log: Implement LWG#2873: 'Add noexcept to several shared_ptr related functions' This issue missed a couple, so I added those as well (see LWG#2942) Modified: libc

[PATCH] D31153: Add the ability to use the children() range API in a const-correct manner

aaron.ballman added inline comments. Comment at: include/clang/AST/Expr.h:4025 child_range children() { +const_child_range CCR = const_cast(this)->children(); +return child_range(cast_away_const(CCR.begin()), dblaikie wrote: > If this is adding const,

[PATCH] D31153: Add the ability to use the children() range API in a const-correct manner

aaron.ballman updated this revision to Diff 94852. aaron.ballman added a comment. Addressing review comments. https://reviews.llvm.org/D31153 Files: include/clang/AST/Expr.h include/clang/AST/StmtIterator.h lib/AST/Expr.cpp Index: lib/AST/Expr.cpp

r299965 - [OpenCL] Map default address space to alloca address space

Author: yaxunl Date: Tue Apr 11 12:24:23 2017 New Revision: 299965 URL: http://llvm.org/viewvc/llvm-project?rev=299965&view=rev Log: [OpenCL] Map default address space to alloca address space For OpenCL, the private address space qualifier is 0 in AST. Before this change, 0 address space qualifi

[PATCH] D31404: [OpenCL] Allow alloca return non-zero private pointer

This revision was automatically updated to reflect the committed changes. Closed by commit rL299965: [OpenCL] Map default address space to alloca address space (authored by yaxunl). Changed prior to commit: https://reviews.llvm.org/D31404?vs=94399&id=94855#toc Repository: rL LLVM https://re

[PATCH] D31646: [coroutines] Build GRO declaration and return GRO statement

GorNishanov added a comment. Gentle ping. When all outstanding CRs are committed we will get 90% of working corouitnes in Clang https://reviews.llvm.org/D31646 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bi

[PATCH] D31153: Add the ability to use the children() range API in a const-correct manner

dblaikie accepted this revision. dblaikie added a comment. This revision is now accepted and ready to land. Looks good to me - thanks! Comment at: include/clang/AST/StmtIterator.h:137 + inline friend StmtIterator + cast_away_const(const struct ConstStmtIterator &RHS); }; ---

Re: [PATCH] D31153: Add the ability to use the children() range API in a const-correct manner

On Tue, Apr 11, 2017 at 10:27 AM Aaron Ballman via Phabricator < revi...@reviews.llvm.org> wrote: > aaron.ballman added inline comments. > > > > Comment at: include/clang/AST/Expr.h:4025 >child_range children() { > +const_child_range CCR = const_cast *>(this)->children();

[PATCH] D31153: Add the ability to use the children() range API in a const-correct manner

aaron.ballman added inline comments. Comment at: include/clang/AST/StmtIterator.h:137 + inline friend StmtIterator + cast_away_const(const struct ConstStmtIterator &RHS); }; dblaikie wrote: > the "struct" here is a bit atypical/should be removed The forward re

Re: [PATCH] D31153: Add the ability to use the children() range API in a const-correct manner

Ah - perhaps it'd be worth having a standalone forward declaration for clarity? I don't think I've seen this construct anywhere else in LLVM? But I could be wrong. On Tue, Apr 11, 2017 at 10:51 AM Aaron Ballman via Phabricator < revi...@reviews.llvm.org> wrote: > aaron.ballman added inline commen

[PATCH] D29660: [OpenMP] Add flag for overwriting default PTX version for OpenMP targets

ABataev accepted this revision. ABataev added a comment. This revision is now accepted and ready to land. LG Repository: rL LLVM https://reviews.llvm.org/D29660 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin

[PATCH] D31781: [Modules] Allow local submodule visibility without c++

rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Can you also add a basic test that this works in C? Thanks! https://reviews.llvm.org/D31781 ___ cfe-commits mailing list cfe-commits@lists.llvm.o

[PATCH] D31757: [clang-tidy] Add a clang-tidy check for possible inefficient vector operations

hokein added inline comments. Comment at: clang-tidy/performance/InefficientVectorOperationCheck.cpp:53-54 +PushBackCall)), + hasParent(compoundStmt(unless(has(ReserveCall)), + has(VectorVarDefStmt + .b

[PATCH] D27604: [Driver] Add compiler option to generate a reproducer

rsmith accepted this revision. rsmith added a comment. LGTM with one change. Comment at: include/clang/Basic/DiagnosticDriverKinds.td:95 def err_drv_force_crash : Error< - "failing because environment variable '%0' is set">; + "failing because %select{environment variable|op

[PATCH] D29905: [OpenMP] Pass argument to device kernel by reference when map is used.

gtbercea updated this revision to Diff 94867. gtbercea added a comment. Refactor code. Repository: rL LLVM https://reviews.llvm.org/D29905 Files: lib/Sema/SemaOpenMP.cpp test/OpenMP/target_map_codegen.cpp Index: test/OpenMP/target_map_codegen.cpp ===

r299976 - [clang-import-test] Lookup inside contexts

Author: spyffe Date: Tue Apr 11 14:33:35 2017 New Revision: 299976 URL: http://llvm.org/viewvc/llvm-project?rev=299976&view=rev Log: [clang-import-test] Lookup inside contexts clang-import-test has until now been only able to report top-level Decls. This is clearly insufficient; we should be able

[PATCH] D30435: [clang-import-test] Lookup inside entities

This revision was automatically updated to reflect the committed changes. Closed by commit rL299976: [clang-import-test] Lookup inside contexts (authored by spyffe). Changed prior to commit: https://reviews.llvm.org/D30435?vs=93929&id=94875#toc Repository: rL LLVM https://reviews.llvm.org/D

[PATCH] D31956: Implement (part of) LWG2857: `{variant, optional, any}::emplace` should return the constructed value

mclow.lists created this revision. Implement the `optional` and `any` part of this issue. This changes the return type of these functions. https://reviews.llvm.org/D31956 Files: include/any include/optional test/std/utilities/any/any.class/any.modifiers/emplace.pass.cpp test/std/utili

r299977 - [ExternalASTMerger] Fix the MSVC build

Author: spyffe Date: Tue Apr 11 14:50:37 2017 New Revision: 299977 URL: http://llvm.org/viewvc/llvm-project?rev=299977&view=rev Log: [ExternalASTMerger] Fix the MSVC build Modified: cfe/trunk/lib/AST/ExternalASTMerger.cpp Modified: cfe/trunk/lib/AST/ExternalASTMerger.cpp URL: http://llvm.or

[PATCH] D31692: [coroutines] Wrap the body of the coroutine in try-catch

GorNishanov added inline comments. Comment at: lib/Sema/SemaCoroutine.cpp:977 + // Since the body of the coroutine will be wrapped in try-catch, it will + // be incompativle with SEH __try if present in a function. + if (!S.getLangOpts().Borland && Fn.FirstSEHTryLoc.isValid()

Re: [PATCH] D29877: Warn about unused static file scope function template declarations.

On 11 April 2017 at 08:35, Marshall Clow via Phabricator via cfe-commits < cfe-commits@lists.llvm.org> wrote: > mclow.lists added a comment. > > Complete reproducer: > > // Tested with with: clang++ -std=c++14 -Wunused-function > UnusedFVassily.cpp > // > // UnusedFVassily.cpp:8:39: warning: unus

[PATCH] D30837: [libcxx] Support for shared_ptr

erik.pilkington added a comment. Ping! https://reviews.llvm.org/D30837 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r299981 - Add const children() accessors to match the existing non-const children() accessors.

Author: aaronballman Date: Tue Apr 11 15:21:30 2017 New Revision: 299981 URL: http://llvm.org/viewvc/llvm-project?rev=299981&view=rev Log: Add const children() accessors to match the existing non-const children() accessors. Modified: cfe/trunk/include/clang/AST/Expr.h cfe/trunk/include/c

[PATCH] D31153: Add the ability to use the children() range API in a const-correct manner

aaron.ballman closed this revision. aaron.ballman added a comment. Commit in r299981. https://reviews.llvm.org/D31153 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D31440: PR32382: Adapt to LLVM changes in DIExpression.

echristo requested changes to this revision. echristo added a comment. This revision now requires changes to proceed. Sounds like Dave is asking for changes so I'll put it down as Request Changes to get it out of my queue. :) https://reviews.llvm.org/D31440 __

r299982 - Modular Codegen: Add/use a bit in serialized function definitions to track whether they are the subject of modular codegen

Author: dblaikie Date: Tue Apr 11 15:46:34 2017 New Revision: 299982 URL: http://llvm.org/viewvc/llvm-project?rev=299982&view=rev Log: Modular Codegen: Add/use a bit in serialized function definitions to track whether they are the subject of modular codegen Some decls are created not where they

[PATCH] D29901: Modular Codegen: Add/use a bit in serialized function definitions to track whether they are the subject of modular codegen

This revision was automatically updated to reflect the committed changes. Closed by commit rL299982: Modular Codegen: Add/use a bit in serialized function definitions to track… (authored by dblaikie). Changed prior to commit: https://reviews.llvm.org/D29901?vs=91193&id=94886#toc Repository:

[PATCH] D31757: [clang-tidy] Add a clang-tidy check for possible inefficient vector operations

aaron.ballman added inline comments. Comment at: clang-tidy/performance/InefficientVectorOperationCheck.cpp:53-54 +PushBackCall)), + hasParent(compoundStmt(unless(has(ReserveCall)), + has(VectorVarDefStmt +

  1   2   >