[PATCH] D65300: [clang] [CodeGen] clang-misexpect prototype for compiler warnings

paulkirth updated this revision to Diff 213379. paulkirth edited the summary of this revision. paulkirth added a comment. Update threshold values to match those assigned when lowering __builtin_expect intrinsic. I've modified the branch probability to match the probability assigned in

r367885 - [AST] Fix RecursiveASTVisitorTest multiline string literal. NFC

Author: dmgreen Date: Mon Aug 5 09:27:36 2019 New Revision: 367885 URL: http://llvm.org/viewvc/llvm-project?rev=367885=rev Log: [AST] Fix RecursiveASTVisitorTest multiline string literal. NFC Some compiler, notably older gccs (< 8) can have trouble with multiline raw string literals inside

[PATCH] D65754: Fix toHalfOpenFileRange assertion fail

SureYeaah created this revision. SureYeaah added reviewers: sammccall, kadircet. Herald added subscribers: cfe-commits, arphaman, jkorous, ilya-biryukov. Herald added a project: clang. - Removed assumption that begin and end need to be in same file id. - Fixes the crash at

[PATCH] D65753: Builtins: Add some v2f16 variants

arsenm created this revision. arsenm added reviewers: rjmccall, Anastasia, yaxunl, rsmith. Herald added a subscriber: wdng. arsenm added a parent revision: D65597: WIP: Builtins: Start adding half versions of math builtins. This will avoid hacks in the OpenCL builtin library.

[PATCH] D65752: [Sema] Refactor LookupVisibleDecls. NFC

ilya-biryukov added a comment. This is just a proposal, there are probably other ways to reach better readability, e.g. group some of those parameters into a struct. But let me know what you think, happy to refactor in a slightly different manner or simply drop this revision. Repository: rG

Re: r367829 - [CrossTU][NFCI] Refactor loadExternalAST function

The problem is probably this part from the diff: - unsigned NumASTLoaded{0u}; + + /// The number successfully loaded ASTs. Used to indicate, and - with the + /// appropriate threshold value - limit the memory usage of the + /// CrossTranslationUnitContext. + unsigned NumASTLoaded; i.e.

[PATCH] D65752: [Sema] Refactor LookupVisibleDecls. NFC

ilya-biryukov updated this revision to Diff 213371. ilya-biryukov added a comment. - Remove accidental change from revision Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65752/new/ https://reviews.llvm.org/D65752 Files:

r367875 - Undo what looks like an unintentional change in r367829

Author: nico Date: Mon Aug 5 08:23:10 2019 New Revision: 367875 URL: http://llvm.org/viewvc/llvm-project?rev=367875=rev Log: Undo what looks like an unintentional change in r367829 The MSan bot was (rightfully) complaining that NumASTLoaded was unitialized, so put the initialization removed in

[PATCH] D65752: [Sema] Refactor LookupVisibleDecls. NFC

ilya-biryukov created this revision. ilya-biryukov added a reviewer: sammccall. Herald added subscribers: kadircet, arphaman, jkorous. Herald added a project: clang. We accumulated some configuration parameters for LookupVisibleDecls that are being passed unchanged to recursive calls, e.g.

[PATCH] D65738: [clangd] Added a TextMate theme parser that updates when the current theme changes.

hokein added a comment. Haven't looked at the patch in details. Looks like the patch is doing different things, and the test just covers a small set of the code. 1. find and parse the theme definition files `json` ; 2. define related data structures to save the TM scopes and do the

[PATCH] D65510: [clangd] Fix implicit template instatiations appearing as topLevelDecls.

ilya-biryukov added inline comments. Comment at: clang-tools-extra/clangd/ClangdUnit.cpp:68 + if (const auto *TD = dyn_cast(D)) +return TD->getTemplateSpecializationKind() == TSK_ImplicitInstantiation; + return false; ilya-biryukov wrote: > jvikstrom

[PATCH] D65510: [clangd] Fix implicit template instatiations appearing as topLevelDecls.

ilya-biryukov added inline comments. Comment at: clang-tools-extra/clangd/ClangdUnit.cpp:68 + if (const auto *TD = dyn_cast(D)) +return TD->getTemplateSpecializationKind() == TSK_ImplicitInstantiation; + return false; jvikstrom wrote: > ilya-biryukov

[PATCH] D65043: [Format] Add C++20 standard to style options

modocache updated this revision to Diff 213360. modocache added a comment. Thanks for the reviews, @sammccall, @Quuxplusone, and @MyDeveloperDay. I added C++14 and C++17 options. In an earlier comment I mentioned splitting this work up into a series of commits, but it ended up being a smaller

[PATCH] D65065: [clang-tidy] Possibility of displaying duplicate warnings

bruntib added a comment. Alright, I modified the commit accordingly. Thank you for the suggestions. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65065/new/ https://reviews.llvm.org/D65065 ___

[PATCH] D65065: [clang-tidy] Possibility of displaying duplicate warnings

bruntib updated this revision to Diff 213356. bruntib edited the summary of this revision. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65065/new/ https://reviews.llvm.org/D65065 Files: clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.cpp

[PATCH] D64793: [Driver] Properly use values-X[ca].o, values-xpg[46].o on Solaris

This revision was automatically updated to reflect the committed changes. Closed by commit rL367866: [Driver] Properly use values-X[ca].o, values-xpg[46].o on Solaris (authored by ro, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit:

r367866 - [Driver] Properly use values-X[ca].o, values-xpg[46].o on Solaris

Author: ro Date: Mon Aug 5 07:06:16 2019 New Revision: 367866 URL: http://llvm.org/viewvc/llvm-project?rev=367866=rev Log: [Driver] Properly use values-X[ca].o, values-xpg[46].o on Solaris Builtins-*-sunos :: compiler_rt_logbf_test.c currently FAILs on Solaris, both SPARC and x86, 32 and

Re: r367829 - [CrossTU][NFCI] Refactor loadExternalAST function

The msan bot doesn't like this, it reports an uninitialized read a t clang/lib/CrossTU/CrossTranslationUnit.cpp : http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/34087/steps/check-clang%20msan/logs/stdio Testing: 0 FAIL: Clang ::

[PATCH] D65562: Move LangStandard*, InputKind::Language to Basic

This revision was automatically updated to reflect the committed changes. Closed by commit rL367864: Move LangStandard*, InputKind::Language to Basic (authored by ro, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit:

r367864 - Move LangStandard*, InputKind::Language to Basic

Author: ro Date: Mon Aug 5 06:59:26 2019 New Revision: 367864 URL: http://llvm.org/viewvc/llvm-project?rev=367864=rev Log: Move LangStandard*, InputKind::Language to Basic This patch is a prerequisite for using LangStandard from Driver in https://reviews.llvm.org/D64793. It moves LangStandard*

[PATCH] D65744: [PR42707][OpenCL] Fix addr space deduction for auto

Anastasia created this revision. Anastasia added a reviewer: rjmccall. Herald added subscribers: ebevhan, yaxunl. A regression was introduced in D64400 because auto is using the same logic as templates. However, deduction of addr spaces wasn't working correctly

Re: r367675 - [OpenCL] Allow OpenCL C style vector initialization in C++

On Mon, 5 Aug 2019 at 13:16, Anastasia Stulova wrote: > > Hi Yvan, > > > Sorry for this, it should now be fixed in r367823. Issue fixed, Thanks Anastasia > > Thanks, > > Anastasia > > > > > From: Yvan Roux > Sent: 02 August 2019 14:09 > To: Anastasia Stulova

[PATCH] D65706: [docs] Better documentation for -Weverything

aaron.ballman added inline comments. Comment at: clang/docs/UsersManual.rst:999-1000 +diagnostics contradict each other, users of :option:`-Weverything` therefore +often disable many diagnostics such as :option:`-Wno-c++98-compat` +:option:`-Wno-c++-compat`. +

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

lildmh updated this revision to Diff 213339. lildmh added a comment. Fix declare mapper codegen test when the function argument has name attached. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59474/new/ https://reviews.llvm.org/D59474 Files: include/clang/AST/GlobalDecl.h

[PATCH] D65387: [clangd] Add a callback mechanism for handling responses from client.

This revision was automatically updated to reflect the committed changes. Closed by commit rL367845: [clangd] Add a callback mechanism for handling responses from client. (authored by hokein, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to

[clang-tools-extra] r367845 - [clangd] Add a callback mechanism for handling responses from client.

Author: hokein Date: Mon Aug 5 05:48:09 2019 New Revision: 367845 URL: http://llvm.org/viewvc/llvm-project?rev=367845=rev Log: [clangd] Add a callback mechanism for handling responses from client. Summary: The callback will be invoked in clangd when we receive a reply from the client. This is

[PATCH] D65387: [clangd] Add a callback mechanism for handling responses from client.

sammccall accepted this revision. sammccall marked an inline comment as done. sammccall added inline comments. Comment at: clang-tools-extra/clangd/test/fixits-command.test:206 --- +{"jsonrpc":"2.0","id":0,"result":{"applied":true}} +# CHECK: "id": 4,

[PATCH] D65387: [clangd] Add a callback mechanism for handling responses from client.

hokein added inline comments. Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:205 + // Return a call id of the request. + int bindReply(Callback Reply) { +llvm::Optional>> OldestCB; sammccall wrote: > nit: I think this function could return

[PATCH] D65387: [clangd] Add a callback mechanism for handling responses from client.

hokein updated this revision to Diff 213334. hokein marked 8 inline comments as done. hokein added a comment. address comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65387/new/ https://reviews.llvm.org/D65387 Files:

[PATCH] D62829: [clang-tidy] Check for dynamically initialized statics in headers.

aaron.ballman added inline comments. Comment at: clang-tools-extra/clang-tidy/bugprone/DynamicStaticInitializersCheck.cpp:47 +void DynamicStaticInitializersCheck::registerMatchers(MatchFinder *Finder) { + if (!getLangOpts().CPlusPlus || getLangOpts().ThreadsafeStatics) +

[PATCH] D65573: Add User docs for ASTImporter

martong updated this revision to Diff 213332. martong added a comment. - Add description for `-ast-merge` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65573/new/ https://reviews.llvm.org/D65573 Files: clang/docs/LibASTImporter.rst

r367840 - [clang][NFC] Remove unused private variable 'CI' in CrossTranslationUnit.h

Author: teemperor Date: Mon Aug 5 05:23:39 2019 New Revision: 367840 URL: http://llvm.org/viewvc/llvm-project?rev=367840=rev Log: [clang][NFC] Remove unused private variable 'CI' in CrossTranslationUnit.h It seems because of the recent refactorings this variable has become unused and now we get

[PATCH] D65510: [clangd] Fix implicit template instatiations appearing as topLevelDecls.

jvikstrom updated this revision to Diff 213331. jvikstrom marked an inline comment as done. jvikstrom added a comment. Formatted. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65510/new/ https://reviews.llvm.org/D65510 Files:

[PATCH] D65665: Support for attributes on @class declarations

aaron.ballman added a comment. I don't think I know enough about ObjC semantics to say much about the language design aspects of this patch. Comment at: clang/include/clang/Parse/Parser.h:1496 + DeclGroupPtrTy ParseObjCAtClassDeclaration(SourceLocation atLoc, +

[PATCH] D65735: [AST] Fix RecursiveASTVisitor visiting implicit constructor initializers.

This revision was automatically updated to reflect the committed changes. jvikstrom marked an inline comment as done. Closed by commit rL367839: [AST] Fix RecursiveASTVisitor visiting implicit constructor initializers. (authored by jvikstrom, committed by ). Herald added a project: LLVM. Herald

r367839 - [AST] Fix RecursiveASTVisitor visiting implicit constructor initializers.

Author: jvikstrom Date: Mon Aug 5 05:20:43 2019 New Revision: 367839 URL: http://llvm.org/viewvc/llvm-project?rev=367839=rev Log: [AST] Fix RecursiveASTVisitor visiting implicit constructor initializers. Summary: RecursiveASTVisitor was visiting implcit constructor initializers. This caused

[PATCH] D65387: [clangd] Add a callback mechanism for handling responses from client.

sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:278 ~ReplyOnce() { - if (Server && !Replied) { + if (Server && !Server->isDestructing() &&

[PATCH] D65735: [AST] Fix RecursiveASTVisitor visiting implicit constructor initializers.

hokein accepted this revision. hokein added inline comments. This revision is now accepted and ready to land. Comment at: clang/unittests/Tooling/RecursiveASTVisitorTests/ImplicitCtorInitializer.cpp:26 + VisitedImplicitInitializer = true; +Match("initializer",

[PATCH] D65735: [AST] Fix RecursiveASTVisitor visiting implicit constructor initializers.

jvikstrom updated this revision to Diff 213323. jvikstrom marked 3 inline comments as done. jvikstrom added a comment. Herald added a subscriber: mgorny. Address comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65735/new/

[PATCH] D65735: [AST] Fix RecursiveASTVisitor visiting implicit constructor initializers.

jvikstrom marked an inline comment as done. jvikstrom added inline comments. Comment at: clang/unittests/Tooling/RecursiveASTVisitorTests/ImplicitCtorInitializer.cpp:29 +if (!Init->isWritten()) + VisitedImplicitInitializer = true; +Match("initializer",

[PATCH] D65738: [clangd] Added a TextMate theme parser that updates when the current theme changes.

jvikstrom created this revision. jvikstrom added reviewers: hokein, ilya-biryukov. Herald added subscribers: cfe-commits, kadircet, arphaman, jkorous, MaskRay. Herald added a project: clang. Adds a TextMate parser module to the vscode extension. It watches for changes to the vscode configuration

[PATCH] D63835: [Syntax] Add nodes for most common statements

sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang/include/clang/Tooling/Syntax/Nodes.h:99 /// An abstract node for C++ statements, e.g. 'while', 'if', etc. class Statement : public Tree {

Re: r367675 - [OpenCL] Allow OpenCL C style vector initialization in C++

Hi Yvan, Sorry for this, it should now be fixed in r367823. Thanks, Anastasia From: Yvan Roux Sent: 02 August 2019 14:09 To: Anastasia Stulova Cc: cfe-commits Subject: Re: r367675 - [OpenCL] Allow OpenCL C style vector initialization in C++ Hi

[PATCH] D64753: [CrossTU][NFCI] Refactor loadExternalAST function

This revision was automatically updated to reflect the committed changes. Closed by commit rL367829: [CrossTU][NFCI] Refactor loadExternalAST function (authored by gamesh411, committed by ). Changed prior to commit: https://reviews.llvm.org/D64753?vs=213311=213315#toc Repository: rL LLVM

r367829 - [CrossTU][NFCI] Refactor loadExternalAST function

Author: gamesh411 Date: Mon Aug 5 04:06:41 2019 New Revision: 367829 URL: http://llvm.org/viewvc/llvm-project?rev=367829=rev Log: [CrossTU][NFCI] Refactor loadExternalAST function Summary: Refactor loadExternalAST method of CrossTranslationUnitContext in order to reduce maintenance burden and

[PATCH] D65234: [CodeGen]: don't treat structures returned in registers as memory inputs

glider added a comment. Eli, any other comments? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65234/new/ https://reviews.llvm.org/D65234 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D64753: [CrossTU][NFCI] Refactor loadExternalAST function

gamesh411 updated this revision to Diff 213311. gamesh411 added a comment. - Remove unused member Limit - Rebase to current master Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64753/new/ https://reviews.llvm.org/D64753 Files:

[PATCH] D65589: [clang] Fix mismatched args constructing AddressSpaceAttr.

AntonBikineev added a comment. Ping. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65589/new/ https://reviews.llvm.org/D65589 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D64696: Adds a warning when an inline Doxygen comment has no argument

davezarzycki added a comment. Hello – This change seems to have exposed a bug in -Wdocumentation argument parsing. For example, this warns when it shouldn't(?): /// \c @foobar Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64696/new/

[PATCH] D65635: Sidestep false positive due to a matching git repository name

rogfer01 added a comment. Thanks @efriedma. I will commit this shortly. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65635/new/ https://reviews.llvm.org/D65635 ___ cfe-commits mailing list

[PATCH] D65387: [clangd] Add a callback mechanism for handling responses from client.

hokein updated this revision to Diff 213304. hokein marked 24 inline comments as done. hokein added a comment. address comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65387/new/ https://reviews.llvm.org/D65387 Files:

[PATCH] D65635: Sidestep false positive due to a matching git repository name

This revision was automatically updated to reflect the committed changes. Closed by commit rL367826: Sidestep false positive due to a matching git repository name (authored by rogfer01, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to

[PATCH] D65387: [clangd] Add a callback mechanism for handling responses from client.

hokein added a comment. thanks for the detailed comments! Comment at: clang-tools-extra/clangd/test/request-reply.test:22 +--- +{"jsonrpc":"2.0","id":0,"result":{"applied":false}} +# CHECK: "code": -32001, sammccall wrote: > please use increasing IDs and

r367826 - Sidestep false positive due to a matching git repository name

Author: rogfer01 Date: Mon Aug 5 03:09:06 2019 New Revision: 367826 URL: http://llvm.org/viewvc/llvm-project?rev=367826=rev Log: Sidestep false positive due to a matching git repository name I have failures in this test because the grep @b gets confused by the clang version including a

[PATCH] D65648: [clang-format] Add support to SpacesBeforeTrailingComments to add spaces before Block comments.

MyDeveloperDay requested changes to this revision. MyDeveloperDay added inline comments. This revision now requires changes to proceed. Comment at: lib/Format/TokenAnnotator.cpp:2163 +if (Current->is(TT_BlockComment)){ + std::cout <<

[PATCH] D65286: [OpenCL] Allow OpenCL C style vector initialization in C++

Anastasia added a comment. In D65286#1614239 , @ebrevnov wrote: > Please be aware about build bot failure > (http://lab.llvm.org:8011/builders/clang-cmake-thumbv7-full-sh/builds/2185) > most likely caused by this change. Thanks! I believe it is now

r367823 - [OpenCL] Fix vector literal test broken in rL367675.

Author: stulova Date: Mon Aug 5 02:50:28 2019 New Revision: 367823 URL: http://llvm.org/viewvc/llvm-project?rev=367823=rev Log: [OpenCL] Fix vector literal test broken in rL367675. Avoid checking alignment unnecessary that is not portable among targets. Modified:

[PATCH] D64695: [clang-format] Added new style rule: SortNetBSDIncludes

MyDeveloperDay added a comment. Assuming this works and the other unit tests don't show issues then this LGTM. Please consider running this on your NetBSD code base before committing, if possible please also run on clang code based to ensure existing sorted headers aren't sorted unexpectedly.

[PATCH] D65735: [AST] Fix RecursiveASTVisitor visiting implicit constructor initializers.

ilya-biryukov added a comment. LGTM from my side, a few optional NITs. Feel free to land as soon as @hokein stamps. Comment at: clang/unittests/Tooling/RecursiveASTVisitorTests/ImplicitCtorInitializer.cpp:16 + +// Check to ensure that CXXCtorInitializer is not visited when

[PATCH] D65735: [AST] Fix RecursiveASTVisitor visiting implicit constructor initializers.

hokein added inline comments. Comment at: clang/unittests/Tooling/RecursiveASTVisitorTests/ImplicitCtorInitializer.cpp:2 +//===- unittest/Tooling/RecursiveASTVisitorTests/ImplicitCtorInitializer.cpp +//-===// +// we should make them one line even it exceeds 80

[PATCH] D65735: [AST] Fix RecursiveASTVisitor visiting implicit constructor initializers.

jvikstrom updated this revision to Diff 213294. jvikstrom added a comment. Call the base Traverse.. function in the visitor. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65735/new/ https://reviews.llvm.org/D65735 Files:

[PATCH] D65735: [AST] Fix RecursiveASTVisitor visiting implicit constructor initializers.

jvikstrom updated this revision to Diff 213291. jvikstrom marked 4 inline comments as done. jvikstrom added a comment. Moved test to its own test file (also addressed comments about the test). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D65735: [AST] Fix RecursiveASTVisitor visiting implicit constructor initializers.

ilya-biryukov added a comment. The fix looks good. Comment at: clang/unittests/Tooling/RecursiveASTVisitorTests/InitListExprPreOrder.cpp:17 // syntactic and semantic form. class InitListExprPreOrderVisitor : public ExpectedLocationVisitor { Could you

[PATCH] D65637: [clangd] [WIP] Semantic highlighting prototype for the vscode extension.

hokein added a comment. In D65637#1613692 , @nridge wrote: > Do you plan to support text decoration options other than color, e.g. bold / > underline / italic? I think we'd just support color, and we don't have further plan to support richer

[clang-tools-extra] r367812 - [clangd] Expose -offset-encoding=utf-32, which has been implemented for ages

Author: sammccall Date: Mon Aug 5 01:14:17 2019 New Revision: 367812 URL: http://llvm.org/viewvc/llvm-project?rev=367812=rev Log: [clangd] Expose -offset-encoding=utf-32, which has been implemented for ages Modified: clang-tools-extra/trunk/clangd/tool/ClangdMain.cpp Modified:

[clang-tools-extra] r367811 - [clangd] Fix error message with incorrect TextDocumentcontentChangeEvent.rangeLength

Author: sammccall Date: Mon Aug 5 01:14:13 2019 New Revision: 367811 URL: http://llvm.org/viewvc/llvm-project?rev=367811=rev Log: [clangd] Fix error message with incorrect TextDocumentcontentChangeEvent.rangeLength Modified: clang-tools-extra/trunk/clangd/DraftStore.cpp Modified:

[PATCH] D64696: Adds a warning when an inline Doxygen comment has no argument

This revision was automatically updated to reflect the committed changes. Closed by commit rL367809: Adds a warning when an inline Doxygen comment has no argument (authored by gribozavr, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to

r367809 - Adds a warning when an inline Doxygen comment has no argument

Author: gribozavr Date: Mon Aug 5 01:05:16 2019 New Revision: 367809 URL: http://llvm.org/viewvc/llvm-project?rev=367809=rev Log: Adds a warning when an inline Doxygen comment has no argument Summary: It warns for for comments like /** \pre \em */ where \em has no argument This warning is

[PATCH] D65735: [AST] Fix RecursiveASTVisitor visiting implicit constructor initializers.

hokein added a comment. I think this is a reasonable fix, just a few comments on the test. Comment at: clang/unittests/Tooling/RecursiveASTVisitorTests/InitListExprPreOrder.cpp:26 + bool TraverseConstructorInitializer(CXXCtorInitializer *Init) { +if

Re: [clang-tools-extra] r367687 - [clangd] Fix a crash when presenting values for Hover

Merged to release_90 in r367807. On Fri, Aug 2, 2019 at 5:22 PM Ilya Biryukov via cfe-commits wrote: > > Author: ibiryukov > Date: Fri Aug 2 08:23:04 2019 > New Revision: 367687 > > URL: http://llvm.org/viewvc/llvm-project?rev=367687=rev > Log: > [clangd] Fix a crash when presenting values for

[PATCH] D65655: [clangd] Fix a crash when presenting values for Hover

hans added a comment. In D65655#1612392 , @ilya-biryukov wrote: > @hans, could we merge this commit into the release branch? Sure, merged in r367807. Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65655/new/

Re: r367661 - Don't try emitting dllexported explicitly defaulted non-trivial ctors twice during explicit template instantiation definition (PR42857)

Merged to release_90 in r367805. On Fri, Aug 2, 2019 at 9:50 AM Hans Wennborg via cfe-commits wrote: > > Author: hans > Date: Fri Aug 2 00:51:41 2019 > New Revision: 367661 > > URL: http://llvm.org/viewvc/llvm-project?rev=367661=rev > Log: > Don't try emitting dllexported explicitly defaulted

Re: r367520 - Delay emitting dllexport explicitly defaulted members until the class is fully parsed (PR40006)

Merged to release_90 in r367804. On Thu, Aug 1, 2019 at 10:00 AM Hans Wennborg via cfe-commits wrote: > > Author: hans > Date: Thu Aug 1 01:01:09 2019 > New Revision: 367520 > > URL: http://llvm.org/viewvc/llvm-project?rev=367520=rev > Log: > Delay emitting dllexport explicitly defaulted

[PATCH] D65735: [AST] Fix RecursiveASTVisitor visiting implicit constructor initializers.

jvikstrom created this revision. jvikstrom added reviewers: hokein, ilya-biryukov. Herald added subscribers: cfe-commits, kadircet. Herald added a project: clang. RecursiveASTVisitor was visiting implcit constructor initializers. This caused semantic highlighting in clangd to emit error logs.

[PATCH] D65650: [Analyzer] Iterator Checkers - Fix for Crash on Iterator Differences

baloghadamsoftware closed this revision. baloghadamsoftware added a comment. Closed by commit: [[ https://reviews.llvm.org/rG8557f17d887a | https://reviews.llvm.org/rG8557f17d887a: [Analyzer] Iterator Checkers - Fix for Crash on Iterator Differences ]] Repository: rC Clang CHANGES SINCE

r367802 - [Analyzer] Iterator Checkers - Fix for Crash on Iterator Differences

Author: baloghadamsoftware Date: Sun Aug 4 23:45:41 2019 New Revision: 367802 URL: http://llvm.org/viewvc/llvm-project?rev=367802=rev Log: [Analyzer] Iterator Checkers - Fix for Crash on Iterator Differences Iterators differences were mistakenly handled as random decrements which causes an

<    1   2